Messages in this thread Patch in this message | | | From | Sebastian Andrzej Siewior <> | Subject | [PATCH v2 2/2] ide: Remove BUG_ON(in_interrupt() || irqs_disabled()) from ide_unregister() | Date | Fri, 20 Nov 2020 10:24:21 +0100 |
| |
Both BUG_ON() were introduced in commit 4015c949fb465 ("[PATCH] update ide core")
when ide_unregister() was extended with semaphore based locking. Both checks won't complain about disabled preemption which is also wrong.
The might_sleep() in today's mutex_lock() will complain about the missuses.
Remove the BUG_ON() statements.
Cc: "David S. Miller" <davem@davemloft.net> Cc: linux-ide@vger.kernel.org Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Acked-by: Jens Axboe <axboe@kernel.dk> --- drivers/ide/ide-probe.c | 3 --- 1 file changed, 3 deletions(-)
diff --git a/drivers/ide/ide-probe.c b/drivers/ide/ide-probe.c index 1c1567bb51942..aefd74c0d8628 100644 --- a/drivers/ide/ide-probe.c +++ b/drivers/ide/ide-probe.c @@ -1539,9 +1539,6 @@ EXPORT_SYMBOL_GPL(ide_port_unregister_devices); static void ide_unregister(ide_hwif_t *hwif) { - BUG_ON(in_interrupt()); - BUG_ON(irqs_disabled()); - mutex_lock(&ide_cfg_mtx); if (hwif->present) { -- 2.29.2
| |