Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 28 Nov 2022 19:03:17 +0000
From:   Eric Biggers <ebiggers@kernel.org>
To:     linux-ext4@vger.kernel.org
Cc:     linux-fscrypt@vger.kernel.org,
        Harshad Shirwadkar <harshadshirwadkar@gmail.com>
Subject: Re: [PATCH 0/7] ext4 fast-commit fixes
Message-ID: <Y4UF9VntxPnoA+SW@gmail.com>
References: <20221106224841.279231-1-ebiggers@kernel.org>
 <Y3Q6U3jd3dI33xdJ@sol.localdomain>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y3Q6U3jd3dI33xdJ@sol.localdomain>
Precedence: bulk

On Tue, Nov 15, 2022 at 05:18:11PM -0800, Eric Biggers wrote:
> On Sun, Nov 06, 2022 at 02:48:34PM -0800, Eric Biggers wrote:
> > 
> > This series fixes several bugs in the fast-commit feature.
> > 
> > Patch 6 may be the most controversial patch of this series, since it
> > would make old kernels unable to replay fast-commit journals created by
> > new kernels.  I'd appreciate any thoughts on whether that's okay.  I can
> > drop that patch if needed.
> > 
> > I've tested that this series doesn't introduce any regressions with
> > 'gce-xfstests -c ext4/fast_commit -g auto'.  Note that ext4/039,
> > ext4/053, and generic/475 fail both before and after.
> > 
> > Eric Biggers (7):
> >   ext4: disable fast-commit of encrypted dir operations
> >   ext4: don't set up encryption key during jbd2 transaction
> >   ext4: fix leaking uninitialized memory in fast-commit journal
> >   ext4: add missing validation of fast-commit record lengths
> >   ext4: fix unaligned memory access in ext4_fc_reserve_space()
> >   ext4: fix off-by-one errors in fast-commit block filling
> >   ext4: simplify fast-commit CRC calculation
> > 
> >  fs/ext4/ext4.h              |   4 +-
> >  fs/ext4/fast_commit.c       | 203 ++++++++++++++++++------------------
> >  fs/ext4/fast_commit.h       |   3 +-
> >  fs/ext4/namei.c             |  44 ++++----
> >  include/trace/events/ext4.h |   7 +-
> >  5 files changed, 132 insertions(+), 129 deletions(-)
> > 
> > 
> > base-commit: 089d1c31224e6b266ece3ee555a3ea2c9acbe5c2
> 
> Any thoughts on this patch series?
> 
> - Eric

Ping?

- Eric