Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Gabriel Krisman Bertazi <krisman@collabora.com>
To:     Arnaud Ferraris <arnaud.ferraris@collabora.com>
Cc:     linux-ext4@vger.kernel.org, Daniel Rosenberg <drosen@google.com>
Subject: Re: [PATCH v2 01/12] tune2fs: Allow enabling casefold feature after
 fs creation
Organization: Collabora
References: <20201127170116.197901-1-arnaud.ferraris@collabora.com>
        <20201127170116.197901-2-arnaud.ferraris@collabora.com>
Date:   Mon, 07 Dec 2020 14:17:19 -0300
In-Reply-To: <20201127170116.197901-2-arnaud.ferraris@collabora.com> (Arnaud
        Ferraris's message of "Fri, 27 Nov 2020 18:01:05 +0100")
Message-ID: <87o8j57cyo.fsf@collabora.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Arnaud Ferraris <arnaud.ferraris@collabora.com> writes:

> From: Gabriel Krisman Bertazi <krisman@collabora.com>
>
> The main reason we didn't allow this before was because !CASEFOLDED
> directories were expected to be normalized().  Since this is no longer
> the case, and as long as the encrypt feature is not enabled, it should
> be safe to enable this feature.
>
> Disabling the feature is trickier, since we need to make sure there are
> no existing +F directories in the filesystem.  Leave that for a future
> patch.
>
> Also, enabling strict mode requires some filesystem-wide verification,
> so ignore that for now.
>

For some reason, I wasn't CC'd in the cover letter, only on the patches
themselves.  Anyway, in order to get more/the right attention, I think
you should CC Ted and Eric Biggers on the entire thread.

Can you send a v3 with a proper CC list?

I haven't taken a second look at the patches yet, but it is no my to list.

-- 
Gabriel Krisman Bertazi