[ofw] [RFC] [PATCH 2/3] winmad userspace library

Leonid Keller leonid at mellanox.co.il
Wed Jul 2 10:29:59 PDT 2008


 

> -----Original Message-----
> From: Sean Hefty [mailto:sean.hefty at intel.com] 
> Sent: Wednesday, July 02, 2008 8:23 PM
> To: Leonid Keller; ofw at lists.openfabrics.org
> Subject: RE: [ofw] [RFC] [PATCH 2/3] winmad userspace library
> 
> >Do all the methods of CWMProvider have to be public (WmReadFile, 
> >WmDeviceIoControl)?
> 
> No - I can change that.
> 
> >You use a critical section in all sync calls.
> >Does it mean, that if I have an application with 50 threads, 
> which are 
> >sending MADs synchronously, only one thread will work at a time ?
> 
> Yes.  The alternative is to have a pool of overlap structures 
> and events, or allocate them on a per call basis.  Those are 
> optimizations that could come later, if needed, and the API 
> can support that change.  I just don't think it's worth 
> adding that complexity on an initial release.

Wait-wait-wait. :)
Async calls are not under critical section, so they are running in
parallel now.
And as for sync ones: what's the problem to init a stack event variable
per call ?
You don't need to handle pool of events.
Or I'm missing something ?

>  
> - Sean
> 
> 



More information about the ofw mailing list