]> git.baikalelectronics.ru Git - kernel.git/commit
pppoatm: Fix excessive queue bloat
authorDavid Woodhouse <dwmw2@infradead.org>
Sun, 8 Apr 2012 09:55:43 +0000 (09:55 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 13 Apr 2012 17:03:45 +0000 (13:03 -0400)
commitb171d2d947cc583045ec21bf1dc87618f8598789
tree6edff9e75f53762ef0c201a9ae8e871a050dc07a
parentcf06601d4d8bf5217c3c796ec64474093253eb44
pppoatm: Fix excessive queue bloat

We discovered that PPPoATM has an excessively deep transmit queue. A
queue the size of the default socket send buffer (wmem_default) is
maintained between the PPP generic core and the ATM device.

Fix it to queue a maximum of *two* packets. The one the ATM device is
currently working on, and one more for the ATM driver to process
immediately in its TX done interrupt handler. The PPP core is designed
to feed packets to the channel with minimal latency, so that really
ought to be enough to keep the ATM device busy.

While we're at it, fix the fact that we were triggering the wakeup
tasklet on *every* pppoatm_pop() call. The comment saying "this is
inefficient, but doing it right is too hard" turns out to be overly
pessimistic... I think :)

On machines like the Traverse Geos, with a slow Geode CPU and two
high-speed ADSL2+ interfaces, there were reports of extremely high CPU
usage which could partly be attributed to the extra wakeups.

(The wakeup handling could actually be made a whole lot easier if we
 stop checking sk->sk_sndbuf altogether. Given that we now only queue
 *two* packets ever, one wonders what the point is. As it is, you could
 already deadlock the thing by setting the sk_sndbuf to a value lower
 than the MTU of the device, and it'd just block for ever.)

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/atm/pppoatm.c