]> git.baikalelectronics.ru Git - kernel.git/commit
ipv6: some ipv6 statistic counters failed to disable bh
authorHannes Frederic Sowa <hannes@stressinduktion.org>
Mon, 31 Mar 2014 18:14:10 +0000 (20:14 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 31 Mar 2014 20:38:12 +0000 (16:38 -0400)
commitb7ded527b3972f123a2035e9e102e59b85e84725
treeb4af50e6130041a6e72c5e97aa8f495e8475b969
parent9d948ef3bdd559818e071ee371c3e8555e1eaab4
ipv6: some ipv6 statistic counters failed to disable bh

After commit 2aafca63a030216 ("ipv6: move DAD and addrconf_verify
processing to workqueue") some counters are now updated in process context
and thus need to disable bh before doing so, otherwise deadlocks can
happen on 32-bit archs. Fabio Estevam noticed this while while mounting
a NFS volume on an ARM board.

As a compensation for missing this I looked after the other *_STATS_BH
and found three other calls which need updating:

1) icmp6_send: ip6_fragment -> icmpv6_send -> icmp6_send (error handling)
2) ip6_push_pending_frames: rawv6_sendmsg -> rawv6_push_pending_frames -> ...
   (only in case of icmp protocol with raw sockets in error handling)
3) ping6_v6_sendmsg (error handling)

Fixes: 2aafca63a030216 ("ipv6: move DAD and addrconf_verify processing to workqueue")
Reported-by: Fabio Estevam <festevam@gmail.com>
Tested-by: Fabio Estevam <fabio.estevam@freescale.com>
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/icmp.c
net/ipv6/ip6_output.c
net/ipv6/mcast.c
net/ipv6/ping.c