Re: [PATCH 4/8] block/mq-deadline: Only use zone locking if necessary

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

 



On 1/9/23 16:48, Jens Axboe wrote:
On 1/9/23 5:44?PM, Bart Van Assche wrote:
On 1/9/23 16:41, Jens Axboe wrote:
Or, probably better, a stacked scheduler where the bottom one can be zone
away. Then we can get rid of littering the entire stack and IO schedulers
with silly blk_queue_pipeline_zoned_writes() or blk_is_zoned_write() etc.

Hi Jens,

Isn't one of Damien's viewpoints that an I/O scheduler should not do
the reordering of write requests since reordering of write requests
may involve waiting for write requests, write request that will never
be received if all tags have been allocated?

It should be work conservering, it should not wait for anything. If
there are holes or gaps, then there's nothing the scheduler can do.

My point is that the strict ordering was pretty hacky when it went in,
and rather than get better, it's proliferating. That's not a good
direction.

Hi Jens,

As you know one of the deeply embedded design choices in the blk-mq code is that reordering can happen at any time between submission of a request to the blk-mq code and request completion. I agree with that design choice.

For the use cases I'm looking at the sequential write required zone type works best. This zone type works best since it guarantees that data on the storage medium is sequential. This results in optimal sequential read performance.

Combining these two approaches is not ideal and I agree that the combination of these two approaches adds some complexity. Personally I prefer to add a limited amount of complexity rather than implementing a new block layer from scratch.

Thanks,

Bart.





[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux