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

["Thomas Hellström" <thomas.hellstrom@linux.intel.com>]

On Mon, 2025-01-13 at 17:01 +0800, Ming Lei wrote:
> 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@lst.de>
> > > Reviewed-by: John Garry <john.g.garry@oracle.com>
> > > Reported-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> > > Closes:
> > > https://lore.kernel.org/linux-block/ead7c5ce5138912c1f3179d62370b84a64014a38.camel@linux.intel.com/
> > > 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.

I actually reported two new splats in the Closes link.

(The second was found when using the suggested lockdep priming, but
would ofc emerge sooner or later without it). I'm pretty sure
Christoph's series was applied when that patch emerged, but I can retry
if you want.

Thanks,
Thomas



> 
> https://lore.kernel.org/linux-block/20250110054726.1499538-1-hch@lst.de/
> 
> 
> Thanks, 
> Ming
>