block: remove unnecessary ioc nested locking
authorJohn Ogness <john.ogness@linutronix.de>
Fri, 19 Jun 2020 15:17:17 +0000 (17:23 +0206)
committerJens Axboe <axboe@kernel.dk>
Thu, 16 Jul 2020 16:22:15 +0000 (10:22 -0600)
commita43f085f8720c4f705b86e7432d19ef955750b36
tree45a1db7f0c6db78ad1432072eb8b467d1768f3bb
parent5b642d8b9f2ef1212f33dfc337816e6b4ab83bb3
block: remove unnecessary ioc nested locking

The legacy CFQ IO scheduler could call put_io_context() in its exit_icq()
elevator callback. This led to a lockdep warning, which was fixed in
commit d8c66c5d5924 ("block: fix lockdep warning on io_context release
put_io_context()") by using a nested subclass for the ioc spinlock.
However, with commit f382fb0bcef4 ("block: remove legacy IO schedulers")
the CFQ IO scheduler no longer exists.

The BFQ IO scheduler also implements the exit_icq() elevator callback but
does not call put_io_context().

The nested subclass for the ioc spinlock is no longer needed. Since it
existed as an exception and no longer applies, remove the nested subclass
usage.

Signed-off-by: John Ogness <john.ogness@linutronix.de>
Reviewed-by: Daniel Wagner <dwagner@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-ioc.c