[ofw] [PATCH] relocate declarations

Smith, Stan stan.smith at intel.com
Mon Jan 3 13:11:30 PST 2011


Alex Naslednikov wrote:
> Please, commit

done in 3054.

>
> -----Original Message-----
> From: Smith, Stan [mailto:stan.smith at intel.com]
> Sent: Tuesday, December 28, 2010 9:08 PM
> To: Alex Naslednikov
> Cc: ofw at lists.openfabrics.org
> Subject: [PATCH] relocate declarations
>
>
> Relocate declarations for consistency and clarity.
> Remove extra space char.
>
> signed-off-by: stan smith <stan.smith at intel.com>
>
> --- A/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp  Tue Dec 28 11:03:16
> 2010 +++ B/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp  Tue Dec 28
> 10:01:08 2010 @@ -1779,7 +1779,7 @@
>          * Dereference the port object since the receive is no longer
>          outstanding. */
>         ipoib_port_deref( p_port, ref_get_recv );
> -       IPOIB_EXIT(  IPOIB_DBG_RECV );
> +       IPOIB_EXIT( IPOIB_DBG_RECV );
>  }
>
>
> @@ -4341,6 +4341,13 @@
>         uint32_t                j;      //Index of DS elements;
>         ULONG                   DataOffset      = 0;
>         UINT                    total_offset;
> +
> +       PSCATTER_GATHER_LIST p_sgl = s_buf->p_sgl;
> +       ipoib_send_desc_t *p_desc = s_buf->p_port->p_desc;
> +
> +       PERF_DECLARE( SendCopy );
> +
> +       IPOIB_ENTER( IPOIB_DBG_SEND );
>
>         /* We calculate the amount of bytes to skip over ETH header
>           in a case of normal send or * LSO header in a case of LSO.
> @@ -4362,12 +4369,6 @@
>                 j = 1; //Skip on the first DS, because it alredy
>         contain IPoIB header }
>
> -       PERF_DECLARE( SendCopy );
> -
> -       IPOIB_ENTER( IPOIB_DBG_SEND );
> -
> -       PSCATTER_GATHER_LIST    p_sgl = s_buf->p_sgl;
> -
>         if( !p_sgl )
>         {
>                 ASSERT( p_sgl );
> @@ -4375,7 +4376,7 @@
>                         ("Failed to get SGL from packet.\n") );
>                 return NDIS_STATUS_FAILURE;
>         }
> -       ipoib_send_desc_t *p_desc = s_buf->p_port->p_desc;
> +
>         /* TODO: Copy only essential data
>            That is, copy only 2 chunks of ETH header if it contained
>            in several first SG elements.




More information about the ofw mailing list