On 2018/10/16 11:34, Jens Axboe wrote: > On 10/14/18 6:45 PM, Damien Le Moal wrote: >> Jens, >> >> On 2018/10/14 7:43, Jens Axboe wrote: >>> On 10/12/18 4:08 AM, Damien Le Moal wrote: >>>> This series improves zoned block device support (reduce overhead) and >>>> introduces many simplifications to the code (overall, there are more deletions >>>> than insertions). >>>> >>>> In more details: >>>> * Patches 1 to 3 are SCSI side (sd driver) cleanups and improvements reducing >>>> the overhead of report zones command execution during disk scan and >>>> revalidation. >>>> * Patches 4 to 9 improve the useability and user API of zoned block devices. >>>> * Patch 10 is the main part of this series. This patch replaces the >>>> REQ_OP_ZONE_REPORT BIO/request operation for executing report zones commands >>>> with a block device file operation, removing the need for the command reply >>>> payload in-place rewriting in the BIO buffer. This leads to major >>>> simplification of the code in many places. >>>> * Patch 11 further simplifies the code of low level drivers by providing a >>>> generic implementation of zoned block device request queue zone bitmaps >>>> initialization and revalidation. >>>> >>>> Please consider the addition of these patches in 4.20. >>>> Comments are as always welcome. >>> >>> How do we want to funnel this series? 1-3 look separate, so perhaps >>> they should go through the scsi tree. Then I can take the rest. Or do >>> some of the later ones depend on 1-3 being in, in terms of applying >>> cleanly? I can also take all of them, looks like only #3 needs a >>> SCSI ack. >> >> Patch 10 and 11 will not apply cleanly for the scsi part without 1-3 >> in first. 1-3, 10 and 11 would need an ack/review from Martin (SCSI). >> And 10-11 also probably need an ack/review from Mike for the DM >> changes. > > Probably want to start pinging people, if you're targeting 4.20 with > this... Hi Jens, Martin and Mike acked/reviewed the SCSI and DM parts respectively (thanks !). Do you think you can take this series for 4.20 ? Thanks. -- Damien Le Moal Western Digital Research
