IB/usnic: Fix potential deadlock
authorParvi Kaustubhi <pkaustub@cisco.com>
Tue, 11 Dec 2018 22:15:42 +0000 (14:15 -0800)
committerJason Gunthorpe <jgg@mellanox.com>
Wed, 19 Dec 2018 23:30:16 +0000 (16:30 -0700)
commit8036e90f92aae2784b855a0007ae2d8154d28b3c
tree9318314659953114da77ef2f3739ef0ff4c7a28b
parent50c582de1d39231af31c881a8444333bff6f11a8
IB/usnic: Fix potential deadlock

Acquiring the rtnl lock while holding usdev_lock could result in a
deadlock.

For example:

usnic_ib_query_port()
| mutex_lock(&us_ibdev->usdev_lock)
 | ib_get_eth_speed()
  | rtnl_lock()

rtnl_lock()
| usnic_ib_netdevice_event()
 | mutex_lock(&us_ibdev->usdev_lock)

This commit moves the usdev_lock acquisition after the rtnl lock has been
released.

This is safe to do because usdev_lock is not protecting anything being
accessed in ib_get_eth_speed(). Hence, the correct order of holding locks
(rtnl -> usdev_lock) is not violated.

Signed-off-by: Parvi Kaustubhi <pkaustub@cisco.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/hw/usnic/usnic_ib_verbs.c