Re: [PATCH v3 1/8] block: fix zone write plug removal

[Christoph Hellwig <hch@infradead.org>]

On Fri, Feb 27, 2026 at 10:49:44AM +0900, Damien Le Moal wrote:
>  again:
>  	zwplug = disk_get_zone_wplug(disk, sector);
>  	if (zwplug) {
> -		/*
> -		 * Check that a BIO completion or a zone reset or finish
> -		 * operation has not already removed the zone write plug from
> -		 * the hash table and dropped its reference count. In such case,
> -		 * we need to get a new plug so start over from the beginning.
> -		 */
>  		spin_lock_irqsave(&zwplug->lock, *flags);
> -		if (zwplug->flags & BLK_ZONE_WPLUG_UNHASHED) {
> -			spin_unlock_irqrestore(&zwplug->lock, *flags);
> -			disk_put_zone_wplug(zwplug);
> -			goto again;
> -		}
>  		return zwplug;

So this special case now simplify goes away, and we let writes to
reset/finished zoned go down the normal path, and then they'll hit the
incorrect write pointer check and error out.  Nice!

The entire patch looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>