[ofw] FW: [PATCH] remove call to generate port, pass in port arg

Smith, Stan stan.smith at intel.com
Tue Nov 16 09:01:06 PST 2010


Alex,
  Did you have any issues with this patch?

thanks,

stan.


Smith, Stan wrote:
> Hello,
>   In the function ipoib_endpt_queue(), pass in the p_port arg instead
> of calling a function to generate the p_port arg.
>
> signed-off-by: stan smith <stan.smith at intel.com>
>

--- A/ulp/ipoib_NDIS6_CM/kernel/ipoib_endpoint.h        Wed Nov 10 17:06:11 2010
+++ B/ulp/ipoib_NDIS6_CM/kernel/ipoib_endpoint.h        Wed Nov 10 17:02:46 2010
@@ -223,7 +223,8 @@

 NDIS_STATUS
 ipoib_endpt_queue(
-       IN              ipoib_endpt_t* const            p_endpt );
+       IN              ipoib_port_t* const             p_port,
+       IN              ipoib_endpt_t* const            p_endpt );

 struct _ipoib_port *
 ipoib_endpt_parent(

--- A/ulp/ipoib_NDIS6_CM/kernel/ipoib_endpoint.cpp      Wed Nov 10 17:06:48 2010
+++ B/ulp/ipoib_NDIS6_CM/kernel/ipoib_endpoint.cpp      Wed Nov 10 17:04:26 2010
@@ -367,10 +367,10 @@
  */
 NDIS_STATUS
 ipoib_endpt_queue(
+       IN                              ipoib_port_t* const             p_port,
        IN                              ipoib_endpt_t* const            p_endpt )
 {
        ib_api_status_t status;
-       ipoib_port_t    *p_port;
        ib_av_attr_t    av_attr;
        net32_t         flow_lbl;

@@ -400,8 +400,6 @@
        }

        /* This is the first packet for this endpoint. Create the AV. */
-       p_port = __endpt_parent( p_endpt );
-
        cl_memclr( &av_attr, sizeof(ib_av_attr_t) );

        av_attr.port_num = p_port->port_num;

--- A/ipoib_NDIS6_CM/kernel/ipoib_port.cpp      Wed Nov 10 17:05:24 2010
+++ B/ipoib_NDIS6_CM/kernel/ipoib_port.cpp      Wed Nov 10 17:05:07 2010
@@ -6732,7 +6732,7 @@
        cl_obj_unlock( &p_port->obj );

        cl_perf_start( EndptQueue );
-       status = ipoib_endpt_queue( *pp_endpt );
+       status = ipoib_endpt_queue( p_port, *pp_endpt );
        cl_perf_stop( &p_port->p_adapter->perf, EndptQueue );
        if( status != NDIS_STATUS_SUCCESS )
                *pp_endpt = NULL;

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ipoib_endpoint.h-p3
Type: application/octet-stream
Size: 380 bytes
Desc: ipoib_endpoint.h-p3
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20101116/03e5fad3/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ipoib_endpoint.cpp-p3
Type: application/octet-stream
Size: 635 bytes
Desc: ipoib_endpoint.cpp-p3
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20101116/03e5fad3/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ipoib_port.cpp.patch-p3
Type: application/octet-stream
Size: 444 bytes
Desc: ipoib_port.cpp.patch-p3
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20101116/03e5fad3/attachment-0002.obj>


More information about the ofw mailing list