On Sat, 12 Sep 2026 00:04:57 +0800, Hao Zhang wrote:
> bio_set_dev() associates the bio with a blkg through bio_associate_blkg().
> If the blkg lookup misses, blkg_tryget_closest() takes q->queue_lock with
> spin_lock_irq() and releases it with spin_unlock_irq(), which
> unconditionally enables local interrupts.
>
> Callers may call bio_set_dev() with interrupts already disabled, e.g.
> dm-thin's pool_map() does so while holding pool->lock taken with
> spin_lock_irq(). The nested spin_unlock_irq() then enables interrupts
> while pool->lock is still held, so an I/O completion softirq can run on
> the same CPU, re-acquire pool->lock (thin_endio(), or overwrite_endio()
> -> complete_mapping_preparation()) and deadlock. lockdep reports this
> as inconsistent SOFTIRQ-ON-W to IN-SOFTIRQ-W usage.
>
> [...]
Applied, thanks!
[1/1] blk-cgroup: save IRQ state in blkg_tryget_closest()
commit: 327c428ba79b13e4cc5253333a9d5c0aa5579bdf
Best regards,
--
Jens Axboe
