diff options
| author | Ranko Zivojnovic <ranko@spidernet.net> | 2007-07-17 03:28:32 +0200 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2007-07-18 10:46:50 +0200 |
| commit | 0929c2dd83317813425b937fbc0041013b8685ff (patch) | |
| tree | 00b7dfc35f18dd059b8ea5a6d0a151754c19a974 /MAINTAINERS | |
| parent | [BNX2]: Update version to 1.6.3. (diff) | |
| download | linux-0929c2dd83317813425b937fbc0041013b8685ff.tar.xz linux-0929c2dd83317813425b937fbc0041013b8685ff.zip | |
[NET]: gen_estimator deadlock fix
-Fixes ABBA deadlock noted by Patrick McHardy <kaber@trash.net>:
> There is at least one ABBA deadlock, est_timer() does:
> read_lock(&est_lock)
> spin_lock(e->stats_lock) (which is dev->queue_lock)
>
> and qdisc_destroy calls htb_destroy under dev->queue_lock, which
> calls htb_destroy_class, then gen_kill_estimator and this
> write_locks est_lock.
To fix the ABBA deadlock the rate estimators are now kept on an rcu list.
-The est_lock changes the use from protecting the list to protecting
the update to the 'bstat' pointer in order to avoid NULL dereferencing.
-The 'interval' member of the gen_estimator structure removed as it is
not needed.
Signed-off-by: Ranko Zivojnovic <ranko@spidernet.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'MAINTAINERS')
0 files changed, 0 insertions, 0 deletions