Eugen Hristev <[email protected]> writes:
> On 4/5/24 15:18, Matthew Wilcox wrote:
>> On Fri, Apr 05, 2024 at 03:13:23PM +0300, Eugen Hristev wrote:
>>> Hello,
>>>
>>> I am trying to respin the series here :
>>> https://www.spinics.net/lists/linux-ext4/msg85081.html
>>
>> The subject here is "Cache insensitive cleanup for ext4/f2fs".
>> Cache insensitive means something entirely different
>> https://en.wikipedia.org/wiki/Cache-oblivious_algorithm
>>
>> I suspect you mean "Case insensitive".
>
> You are correct, I apologize for the typo.
Heh. I completely missed it in the previous submissions. I guess we both
just mentally auto-corrected.
Since we are here, I think I contributed to the typo in the cover letter
with the summary lines of patch 1 and 2. Differently from the rest of
the series, these two are actually working on a "cache of
casefolded strings". But their summary lines are misleading.
Can you rename them to:
[PATCH v16 1/9] ext4: Simplify the handling of cached casefolded names
[PATCH v16 2/9] f2fs: Simplify the handling of cached casefolded names
From a quick look, the series is looking good and the strict mode issue
pointed in the last iteration seems fixed, though I didn't run it yet.
I'll take a closer look later today and fully review.
--
Gabriel Krisman Bertazi
Hello Krisman,
On 4/5/24 19:37, Gabriel Krisman Bertazi wrote:
> Eugen Hristev <[email protected]> writes:
>
>> On 4/5/24 15:18, Matthew Wilcox wrote:
>>> On Fri, Apr 05, 2024 at 03:13:23PM +0300, Eugen Hristev wrote:
>>>> Hello,
>>>>
>>>> I am trying to respin the series here :
>>>> https://www.spinics.net/lists/linux-ext4/msg85081.html
>>>
>>> The subject here is "Cache insensitive cleanup for ext4/f2fs".
>>> Cache insensitive means something entirely different
>>> https://en.wikipedia.org/wiki/Cache-oblivious_algorithm
>>>
>>> I suspect you mean "Case insensitive".
>>
>> You are correct, I apologize for the typo.
>
> Heh. I completely missed it in the previous submissions. I guess we both
> just mentally auto-corrected.
>
> Since we are here, I think I contributed to the typo in the cover letter
> with the summary lines of patch 1 and 2. Differently from the rest of
> the series, these two are actually working on a "cache of
> casefolded strings". But their summary lines are misleading.
>
> Can you rename them to:
>
> [PATCH v16 1/9] ext4: Simplify the handling of cached casefolded names
> [PATCH v16 2/9] f2fs: Simplify the handling of cached casefolded names
>
> From a quick look, the series is looking good and the strict mode issue
> pointed in the last iteration seems fixed, though I didn't run it yet.
> I'll take a closer look later today and fully review.
>
Have you managed to take a look ? What would be the future of the series ? I didn't
want to send another version for just a subject change, but I can if that's the
only change required .
Thanks,
Eugen