Re: [PATCH RFC v2 01/18] blk-mq: Add a flag for reserved requests

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

 



On 14/06/2022 07:43, Christoph Hellwig wrote:
On Thu, Jun 09, 2022 at 06:29:02PM +0800, John Garry wrote:
Add a flag for reserved requests so that drivers may know this for any
special handling.

The 'reserved' argument in blk_mq_ops.timeout callback could now be
replaced by using this flag.
And we should probably do that ASAP, independent of the rest of this
series.

We only have 2x users of the 'reserved' arg for 11x implementations of blk_mq_ops.timeout:
- mtip32xx.c
- scsi_lib.c

scsi_lib.c is dubious as currently scsi does use reserved commands. So we really only have 1x.

I'd be happy to take any spin-off series to make this one more manageable, but is just removing an arg a strong enough reason for that? That reserved arg is passed around a lot in the blk-mq iter functions, so probably yes.

Otherwise looks good:

Reviewed-by: Christoph Hellwig<hch@xxxxxx>

Thanks



[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux