2024-03-04 10:31:39

by Ritesh Harjani

[permalink] [raw]
Subject: Re: [RFC 3/8] iomap: Add atomic write support for direct-io

John Garry <[email protected]> writes:

>>>
>>> https://urldefense.com/v3/__https://lore.kernel.org/linux-fsdevel/[email protected]/__;!!ACWV5N9M2RV99hQ!PqMMFBeUqdWwlm0AxVyI_Vr1HPajTQ6AG2_GwK_IrhBSa-Wnz4cc-1w0LEFyTXY9Q9gT0WwhxvXloSqnOHb6Btg$
>>>
>>> and now this one.
>>>
>>> Can the two of you please co-ordinate your efforts and based your
>>> filesysetm work off the same iomap infrastructure changes?
>>
>> Sure Dave, make sense. But we are cc'ing each other in this effort
>> together so that we are aware of what is being worked upon.
>
> Just cc'ing is not enough. I was going to send my v2 for XFS/iomap
> support today. I didn't announce that as I did not think that I had to.

ok. Let me take care of this next time to avoid any overlapping change
hitting the mailing list to avoid double reviews/competing changes from
2 people. Hopefully I can find you on xfs IRC channel in case if I would
like to post anything in the related/overlapping area . My handle is riteshh.

> Admittedly it will be effectively an RFC, as the forcealign feature (now
> included) is not mature. But it's going to be a bit awkward to have 2x
> overlapping series' sent to the list.
>
> FWIW, I think that it's better to send series based on top of other
> series, rather than cherry-picking necessary parts of other series (when
> posting)
>

Ok. Sure John. Make sense. Now that I understood what I am looking for
in from iomap side of the changes, I can provide my review comments to
your series, whenever you post them.

Thanks for your feedback.

-ritesh