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

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

To: "xiakaixu1987@xxxxxxxxx" <xiakaixu1987@xxxxxxxxx>
Subject: Re: [PATCH] iomap: set did_zero to true when zeroing successfully
From: Chaitanya Kulkarni <chaitanyak@xxxxxxxxxx>
Date: Wed, 15 Jun 2022 22:07:59 +0000
Accept-language: en-US
Cc: "djwong@xxxxxxxxxx" <djwong@xxxxxxxxxx>, "linux-xfs@xxxxxxxxxxxxxxx" <linux-xfs@xxxxxxxxxxxxxxx>, "linux-fsdevel@xxxxxxxxxxxxxxx" <linux-fsdevel@xxxxxxxxxxxxxxx>, "hch@xxxxxxxxxxxxx" <hch@xxxxxxxxxxxxx>, Kaixu Xia <kaixuxia@xxxxxxxxxxx>
In-reply-to: <1655198062-13288-1-git-send-email-kaixuxia@tencent.com>
Thread-index: AQHYf880DMY9PWiyM0OMC2jpqIQfjq1RCSCA
Thread-topic: [PATCH] iomap: set did_zero to true when zeroing successfully
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0

On 6/14/22 02:14, 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.
> 
> Signed-off-by: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
> ---

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx>

-ck

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

Prev by Date: Re: [syzbot] KASAN: use-after-free Read in copy_page_from_iter_atomic (2)
Next by Date: Re: [PATCH v3 5/6] userfaultfd: selftests: make /dev/userfaultfd testing configurable
Previous by thread: [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] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]