Re: [PATCH V2] block: mark GFP_NOIO around sysfs ->store()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 13, 2025 at 09:50:37AM +0100, Thomas Hellström wrote:
> Hi!
> 
> On Mon, 2025-01-13 at 16:41 +0800, Ming Lei wrote:
> > sysfs ->store is called with queue freezed, meantime we have several
> > ->store() callbacks(update_nr_requests, wbt, scheduler) to allocate
> > memory with GFP_KERNEL which may run into direct reclaim code path,
> > then potential deadlock can be caused.
> > 
> > Fix the issue by marking NOIO around sysfs ->store()
> > 
> > Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> > Reviewed-by: John Garry <john.g.garry@xxxxxxxxxx>
> > Reported-by: Thomas Hellström <thomas.hellstrom@xxxxxxxxxxxxxxx>
> > Closes:
> > https://lore.kernel.org/linux-block/ead7c5ce5138912c1f3179d62370b84a64014a38.camel@xxxxxxxxxxxxxxx/
> > Fixes: bd166ef183c2 ("blk-mq-sched: add framework for MQ capable IO
> > schedulers")
> 
> Does this fix also the #2 lockdep splat in that email?

No.

The #2 splat fix has been merged to for-6.14/block, and this patch only
covers the one reported in the Closes link.

https://lore.kernel.org/linux-block/20250110054726.1499538-1-hch@xxxxxx/


Thanks, 
Ming





[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux