Re: [PATCH 0/8] zbd: Fix initial zone write pointer of regular block devices

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

 



On Jun 30, 2020 / 04:53, Damien Le Moal wrote:
> On 2020/06/26 18:43, Shin'ichiro Kawasaki wrote:
> > With zonemode=zbd and regular block devices, fio initializes zone status as
> > empty. However, write pointers are set not at zone start but at zone end. This
> > inconsistency between write pointer position and zone status caused different
> > behavior of regular block devices from zoned block devices. This patch series
> > fixes the inconsistency and the behavior gap.
> > 
> > The 1st patch fixes the zone status initialization code by setting write
> > pointers at zone start. Following patches fixes t/zbd test script to handle
> > regular block devices and zoned block devices in same manner.
> > 
> > Shin'ichiro Kawasaki (8):
> >   zbd: Fix initial zone write pointer of regular block devices
> >   t/zbd: Fix pass condition of test case #3
> >   t/zbd: Add write_and_run_one_fio_job() helper function
> >   t/zbd: Combine write and read fio commands for test case #6
> >   t/zbd: Combine write and read fio commands for test case #15
> >   t/zbd: Combine write and read fio commands for test case #16
> >   t/zbd: Remove write before random read/write from test case #17
> >   t/zbd: Enable regular block devices for test case #47
> > 
> >  t/zbd/test-zbd-support | 78 +++++++++++++++++++++++-------------------
> >  zbd.c                  |  2 +-
> >  2 files changed, 44 insertions(+), 36 deletions(-)
> > 
> 
> Jens,
> 
> I reviewed this series internally with Shin'ichiro before he posted.
> So:
> 
> Reviewed-by: Damien Le Moal <damien.lemoal@xxxxxxx>

Hi Jens, could you consider to merge this series to the upstream?
It will make zonemode=zbd implementation less confusing.

-- 
Best Regards,
Shin'ichiro Kawasaki



[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux