Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-27 Thread Eliezer Tamir
On 28/06/2013 03:29, Cody P Schafer wrote: On 06/27/2013 05:25 PM, Cody P Schafer wrote: On 06/24/2013 12:28 AM, Eliezer Tamir wrote: select/poll busy-poll support. ... I'm seeing warnings about using smp_processor_id() while preemptable (log included below) due to this patch. I expect the us

Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-27 Thread Eliezer Tamir
On 28/06/2013 07:43, Andi Kleen wrote: @@ -400,6 +402,8 @@ int do_select(int n, fd_set_bits *fds, struct timespec *end_time) poll_table *wait; int retval, i, timed_out = 0; unsigned long slack = 0; + unsigned int ll_flag = POLL_LL; + u64 ll_time = ll_end_time(

Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-27 Thread Andi Kleen
> @@ -400,6 +402,8 @@ int do_select(int n, fd_set_bits *fds, struct timespec > *end_time) > poll_table *wait; > int retval, i, timed_out = 0; > unsigned long slack = 0; > + unsigned int ll_flag = POLL_LL; > + u64 ll_time = ll_end_time(); So you're adding a sched_clock to

Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-27 Thread Cody P Schafer
On 06/27/2013 05:25 PM, Cody P Schafer wrote: On 06/24/2013 12:28 AM, Eliezer Tamir wrote: select/poll busy-poll support. ... I'm seeing warnings about using smp_processor_id() while preemptable (log included below) due to this patch. I expect the use of ll_end_time() -> sched_clock() here is

Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-27 Thread Cody P Schafer
On 06/24/2013 12:28 AM, Eliezer Tamir wrote: select/poll busy-poll support. ... diff --git a/fs/select.c b/fs/select.c index 8c1c96c..79b876e 100644 --- a/fs/select.c +++ b/fs/select.c @@ -400,6 +402,8 @@ int do_select(int n, fd_set_bits *fds, struct timespec *end_time) poll_table *wai

Re: [PATCH v4 net-next] net: poll/select low latency socket support

2013-06-25 Thread David Miller
From: Eliezer Tamir Date: Mon, 24 Jun 2013 10:28:03 +0300 > select/poll busy-poll support. > > Split sysctl value into two separate ones, one for read and one for poll. > updated Documentation/sysctl/net.txt > > Add a new poll flag POLL_LL. When this flag is set, sock_poll will call > sk_poll_l