[lkml]   [2013]   [Mar]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [RFC PATCH 1/5] net: implement support for low latency socket polling
On 05/03/2013 18:43, Ben Hutchings wrote:
> On Wed, 2013-02-27 at 09:55 -0800, Eliezer Tamir wrote:
> Should the units really be cycles or, say, microseconds? I assume that
> a sysctl setter can do a conversion to cycles so that there's no need to
> multiply every time the value is used. (If the CPU doesn't have
> constant_tsc or equivalent then this conversion doesn't quite work, but
> then low-latency tunng usually includes disabling frequency scaling.)

We are not very sensitive to this setting, anything on the order of your
half round time trip plus a few standard deviations works well.
We are busy waiting, so setting a higher value does not change the
results much.

It does make sense to have this in ms, and it might not matter if the
dynamic cycles mess with the value too much.

BTW on my machines enabling frequency scaling improves performance in
many cases.

> Also, this should be a per-device (or even per-NAPI-context?) setting.

Again, I would expect this to depend more on your workload than on the
NIC, so I would keep this global.
User knobs should be as simple as possible.

>>+int sysctl_net_ll_poll __read_mostly = 150000;
> Nicely tuned for your specific test system, no doubt. :-)

why don't you try this on your NIC and see ;-)

Thanks for the input,

 \ /
  Last update: 2013-03-05 19:01    [W:0.151 / U:0.748 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site