[Openib-windows] [PATCH] fix free builds of iou

Fabian Tillier ftillier at silverstorm.com
Thu Aug 31 10:46:08 PDT 2006


Hi Yossi,

On 8/31/06, Yossi Leybovich <sleybo at mellanox.co.il> wrote:
> Fab
>
> I had typo in the former patch.

Actually, that was my fault, sorry.

> here is a fix

I checked this in, with the changes noted below, as revision 476.

- Fab

> Index: W:/work/clean/core/iou/kernel/iou_driver.c
> ===================================================================
> --- W:/work/clean/core/iou/kernel/iou_driver.c (revision 1640)
> +++ W:/work/clean/core/iou/kernel/iou_driver.c (revision 1641)
> @@ -154,11 +154,17 @@
>  table[1].DefaultLength = sizeof(ULONG);
>  /* Have at it! */
>  status = RtlQueryRegistryValues( RTL_REGISTRY_ABSOLUTE,
> -  param_path.Buffer, table, NULL, NULL );
>
> + param_path.Buffer, table, NULL, NULL );
> + IOU_PRINT( TRACE_LEVEL_ERROR, IOU_DBG_ERROR,
> +   ("debug level %d debug flags 0x%.8x\n",
> +   g_iou_dbg_level,
> +   g_iou_dbg_flags));

I changed the level to information, and the flag to IOU_DBG_DRV as
that seemed more appropriate.  I also moved the print to happen after
the flag fixup below.

> +
> +#if DBG

This needed to be #ifndef EVENT_TRACING

>  if( g_iou_dbg_flags & IOU_DBG_ERR )
>   g_iou_dbg_flags |= CL_DBG_ERROR;
> -
> +#endif
>  cl_free( param_path.Buffer );
>  IOU_EXIT( IOU_DBG_DRV );
>  return status;




More information about the ofw mailing list