Willem de Bruijn 5f121b9a83 net-rps: fixes for rps flow limit
Caught by sparse:
- __rcu: missing annotation to sd->flow_limit
- __user: direct access in cpumask_scnprintf

Also
- add endline character when printing bitmap if room in buffer
- avoid bucket overflow by reducing FLOW_LIMIT_HISTORY

The last item warrants some explanation. The hashtable buckets are
subject to overflow if FLOW_LIMIT_HISTORY is larger than or equal
to bucket size, since all packets may end up in a single bucket. The
current (rather arbitrary) history value of 256 happens to match the
buffer size (u8).

As a result, with a single flow, the first 128 packets are accepted
(correct), the second 128 packets dropped (correct) and then the
history[] array has filled, so that each subsequent new packet
causes an increment in the bucket for new_flow plus a decrement
for old_flow: a steady state.

This is fine if packets are dropped, as the steady state goes away
as soon as a mix of traffic reappears. But, because the 256th packet
overflowed the bucket to 0: no packets are dropped.

Instead of explicitly adding an overflow check, this patch changes
FLOW_LIMIT_HISTORY to never be able to overflow a single bucket.

Reported-by: Fengguang Wu <fengguang.wu@intel.com>
(first item)

Signed-off-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-13 17:13:05 -07:00
..
2013-05-17 14:31:05 -04:00
2013-05-04 12:31:18 -07:00
2013-03-01 13:39:00 -08:00
2013-01-02 17:36:10 -08:00
2013-05-22 15:08:44 -04:00
2013-05-07 19:46:02 -07:00
2012-12-02 00:05:12 +00:00
2013-04-29 15:40:23 -04:00
2013-03-23 16:11:31 -07:00
2013-03-22 15:18:18 -07:00
2013-05-02 09:03:55 -07:00
2013-04-08 17:39:23 +02:00
2013-03-12 11:30:04 -07:00
2013-05-10 14:37:17 +01:00
2013-05-01 16:36:22 +05:30
2013-03-28 10:10:25 -06:00
2012-12-24 09:36:38 -07:00
2013-02-26 02:46:08 -05:00
2013-05-07 18:38:27 -07:00
2013-03-15 15:09:43 +10:30
2013-04-29 18:28:40 -07:00
2012-11-22 13:43:27 +09:00
2013-05-07 02:16:31 -04:00
2013-02-22 23:31:31 -05:00
2012-11-08 16:58:30 +01:00
2013-05-06 13:07:33 +02:00
2013-01-25 21:03:54 -08:00
2013-06-13 01:23:11 -07:00
2013-01-10 11:44:38 -06:00
2013-04-30 17:04:06 -07:00
2013-04-01 11:04:50 -07:00
2013-05-31 00:48:22 -07:00
2013-04-30 17:04:00 -07:00
2013-03-15 15:09:43 +10:30
2012-12-14 13:06:40 +10:30
2012-12-25 18:45:06 -05:00
2013-06-13 17:13:05 -07:00
2012-12-11 17:22:27 -08:00
2013-05-04 14:47:26 -04:00
2013-05-07 11:22:14 -07:00
2013-04-29 15:54:28 -07:00
2013-05-31 17:19:05 -07:00
2013-03-19 09:47:30 +01:00
2013-02-26 09:34:29 -08:00
2013-04-29 15:54:28 -07:00
2013-04-12 10:26:23 +02:00
2013-05-23 14:23:17 +02:00
2013-04-09 14:13:29 -04:00
2013-02-19 08:43:34 +01:00
2013-03-29 15:31:30 -04:00
2013-04-30 15:50:12 +05:30
2013-01-15 22:43:15 -08:00
2013-03-20 12:10:38 -04:00
2013-03-28 11:10:22 -07:00
2013-04-29 15:54:37 -07:00