On 05/06/2013 17:17, Eric Dumazet wrote: > On Wed, 2013-06-05 at 06:56 -0700, Eric Dumazet wrote: > >> This looks quite easy, by adding in include/uapi/asm-generic/poll.h >> >> #define POLL_LL 0x8000 >> >> And do the sk_poll_ll() call only if flag is set. >> >> I do not think we have to support select(), as its legacy interface, and >> people wanting ll should really use epoll() or poll(). > > Alternatively, add a per socket flag to enable/disable ll > > This global enable assumes the application owns the host anyway. >
I plan on adding a socket option in the next stage. I'm also testing a patch much like you described with a poll flag. Select/poll set it to indicate that they want to busy poll. Sock_poll sets it to indicate that this socket can (at the moment) busy-poll. If you think the way things are done right now is unacceptable, even as an experimental feature, I would much prefer to drop this patch and have the rest applied rather then bring in new code that is not fully tested at this stage. -Eliezer ------------------------------------------------------------------------------ How ServiceNow helps IT people transform IT departments: 1. A cloud service to automate IT design, transition and operations 2. Dashboards that offer high-level views of enterprise services 3. A single system of record for all IT processes http://p.sf.net/sfu/servicenow-d2d-j _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
