Re: [PATCH v3 2/8] zbd: fix write zone accounting

[Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>]

On Mar 02, 2026 / 12:41, Damien Le Moal wrote:
[...]
> > diff --git a/zbd.c b/zbd.c
> > index b71f842c..c511b709 100644
> > --- a/zbd.c
> > +++ b/zbd.c
> > @@ -1693,8 +1693,17 @@ retry:
> >  
> >  		zone_lock(td, f, z);
> >  		if (zbd_zone_remainder(z) >= min_bs) {
> > -			need_zone_finish = false;
> > -			goto out;
> > +			/*
> > +			 * The zone might be already removed from
> > +			 * zbdi->write_zones[] by other jobs at this moment.
> > +			 * Even if the zone has remainder, call
> > +			 * zbd_write_zone_get() to ensure that it is in the
> > +			 * array.
> > +			 */
> > +			if (zbd_write_zone_get(td, f, z)) {
> > +				need_zone_finish = false;
> > +				goto out;
> > +			}
> 
> Please change this to:
> 
> 		if (zbd_zone_remainder(z) >= min_bs &&
> 		    zbd_write_zone_get(td, f, z)) {
> 			need_zone_finish = false;
> 			goto out;
> 		}
> 
> And move the comment block above the if. You could also improve the comment to
> explain why we look at "zbd_zone_remainder(z) >= min_bs"
> 
> With that (and the much better commit message), feel free to add:
> 
> Reviewed-by: Damien Le Moal <dlemoal@kernel.org>

Thanks, will reflect the comments in v4.