]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commit
xen-netfront: fix potential deadlock in xennet_remove()
authorAndrea Righi <andrea.righi@canonical.com>
Fri, 24 Jul 2020 08:59:10 +0000 (10:59 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 25 Jul 2020 00:02:01 +0000 (17:02 -0700)
commitc2c633106453611be07821f53dff9e93a9d1c3f0
tree8061aa72536c8acdca4d693a078a66129de423dd
parentc2b69f24ebd166a13cdc9909b50f33228895998b
xen-netfront: fix potential deadlock in xennet_remove()

There's a potential race in xennet_remove(); this is what the driver is
doing upon unregistering a network device:

  1. state = read bus state
  2. if state is not "Closed":
  3.    request to set state to "Closing"
  4.    wait for state to be set to "Closing"
  5.    request to set state to "Closed"
  6.    wait for state to be set to "Closed"

If the state changes to "Closed" immediately after step 1 we are stuck
forever in step 4, because the state will never go back from "Closed" to
"Closing".

Make sure to check also for state == "Closed" in step 4 to prevent the
deadlock.

Also add a 5 sec timeout any time we wait for the bus state to change,
to avoid getting stuck forever in wait_event().

Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/xen-netfront.c