Re: [PATCH v2 4/4] ext4: Do not fallback to buffered-io for DIO atomic write
From: IBM
Date: Tue Oct 29 2024 - 05:28:05 EST
John Garry <john.g.garry@xxxxxxxxxx> writes:
> On 27/10/2024 18:17, Ritesh Harjani (IBM) wrote:
>> iomap can return -ENOTBLK if pagecache invalidation fails.
>> Let's make sure if -ENOTBLK is ever returned for atomic
>> writes than we fail the write request (-EIO) instead of
>> fallback to buffered-io.
>>
>> Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx>
>> Signed-off-by: Ritesh Harjani (IBM) <ritesh.list@xxxxxxxxx>
>
> I am not sure if you plan on dropping this patch...
>
As discussed in the other thread, EXT4 has got a fallback to buffered-io
logic at certain places including during short DIO writes unlike XFS
which never fallsback to buffered-io in case of short DIO writes.
Let me send another patch for this one.
Thanks for the review.
-ritesh