Re: [PATCH] iomap: set did_zero to true when zeroing successfully

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

Subject: Re: [PATCH] iomap: set did_zero to true when zeroing successfully
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 16 Jun 2022 00:28:26 -0700
Cc: linux-xfs@xxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, djwong@xxxxxxxxxx, hch@xxxxxxxxxxxxx, Kaixu Xia <kaixuxia@xxxxxxxxxxx>
In-reply-to: <1655198062-13288-1-git-send-email-kaixuxia@tencent.com>
References: <1655198062-13288-1-git-send-email-kaixuxia@tencent.com>

On Tue, Jun 14, 2022 at 05:14:22PM +0800, xiakaixu1987@xxxxxxxxx wrote:
> From: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
> 
> It is unnecessary to check and set did_zero value in while() loop,
> we can set did_zero to true only when zeroing successfully at last.

Looks good, but this really should be separate patches for dax and
iomap.

Follow-Ups:
- Re: [PATCH] iomap: set did_zero to true when zeroing successfully
  - From: Kaixu Xia

References:
- [PATCH] iomap: set did_zero to true when zeroing successfully
  - From: xiakaixu1987

Prev by Date: Re: [RFC PATCH v2 1/7] statx: add I/O alignment information
Next by Date: Re: [PATCH 01/50] xfs: make last AG grow/shrink perag centric
Previous by thread: Re: [PATCH] iomap: set did_zero to true when zeroing successfully
Next by thread: Re: [PATCH] iomap: set did_zero to true when zeroing successfully
Index(es):
- Date
- Thread

[Index of Archives] [XFS Filesystem Development (older mail)] [Linux Filesystem Development] [Linux Audio Users] [Yosemite Trails] [Linux Kernel] [Linux RAID] [Linux SCSI]

Powered by Linux