[ofw] [PATCH] add missing locks

Alex Naslednikov xalex at mellanox.co.il
Wed Nov 17 01:49:38 PST 2010


Good catch !
Applied at 2997

-----Original Message-----
From: Smith, Stan [mailto:stan.smith at intel.com] 
Sent: Tuesday, November 16, 2010 7:33 PM
To: Alex Naslednikov
Cc: ofw at lists.openfabrics.org
Subject: [PATCH] add missing locks


Hello,
  In order to preserve consistency of the p_port->buf_mgr.recv_pool list, acquire/release recv_lock over the call to __free_received_NBL(), as other calls to __free_received_NBL() do.
Issue is __free_received_NBL() eventually calls __buf_mgr_put_recv().

signed-off-by: stan smith <stan.smith at intel.com>

--- A/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp  Tue Nov 16 09:11:10 2010
+++ B/ulp/ipoib_NDIS6_CM/kernel/ipoib_port.cpp  Tue Nov 16 09:10:59 2010
@@ -2191,7 +2191,9 @@
                                                                                        recv_complete_flags );
                        }
                        else {
+                               cl_spinlock_acquire( &p_port->recv_lock );
                                __free_received_NBL( p_port, p_port->recv_mgr.recv_NBL_array[0] );
+                               cl_spinlock_release( &p_port->recv_lock );
                        }

                } else {



More information about the ofw mailing list