[openib-general] Re: [PATCH] Synchronize access to event list in uCM
    Roland Dreier 
    rdreier at cisco.com
       
    Mon Mar  6 16:27:32 PST 2006
    
    
  
 > Fix synchronization around accessing event list.
 > +	down(&file->mutex);
 >  	if (!list_empty(&file->events))
 >  		mask = POLLIN | POLLRDNORM;
 > +	up(&file->mutex);
Actually, could you give a little more detail about why this is
needed, if only for the changelog?  I don't see what putting the mutex
around the list_empty() test buys you -- the list_empty() test already
is pretty atomic I think.
 - R.
    
    
More information about the general
mailing list