RE: [GIT PULL][PATCH v11 0/4] Update to zstd-1.4.10

From: Paul Jones <paul@pauljones.id.au>
To: "dsterba@suse.cz" <dsterba@suse.cz>, Nick Terrell <terrelln@fb.com>
Cc: "B093B859-53CC-4818-8CC3-A317F4872AD6@fb.com" 
	<B093B859-53CC-4818-8CC3-A317F4872AD6@fb.com>,
	"linux-btrfs@vger.kernel.org" <linux-btrfs@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [GIT PULL][PATCH v11 0/4] Update to zstd-1.4.10
Date: Wed, 29 Sep 2021 10:23:03 +0000	[thread overview]
Message-ID: <SYXPR01MB1918E9CBD97CF904DDB4DC539EA99@SYXPR01MB1918.ausprd01.prod.outlook.com> (raw)
In-Reply-To: <20210929100659.GK9286@twin.jikos.cz>

> -----Original Message-----
> From: David Sterba <dsterba@suse.cz>
> Sent: Wednesday, 29 September 2021 8:07 PM
> To: Nick Terrell <terrelln@fb.com>
> Cc: B093B859-53CC-4818-8CC3-A317F4872AD6@fb.com; linux-
> btrfs@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [GIT PULL][PATCH v11 0/4] Update to zstd-1.4.10
> 
> On Wed, Sep 29, 2021 at 01:30:26AM +0000, Nick Terrell wrote:
> > > On Sep 28, 2021, at 5:22 PM, Tom Seewald <tseewald@gmail.com>
> wrote:
> > > Has this been abandoned or will there be future attempts at syncing
> > > the in-kernel zstd with the upstream project?
> >
> > Sorry for the lack of action, but this has not been abandoned. I’ve
> > just been preparing a rebased patch-set last week, so expect to see some
> action soon.
> > Since we’re not in a merge window, I’m unsure if it is best to send
> > out the updated patches now, or wait until the merge window is open,
> > but I’m about to pose that question to the LKML.
> 
> If you send it once merge window is open it's unlikely to be merged. The
> code must be ready before it opens and part of linux-next for a week at least
> if not more.
> 
> > This work has been on my back burner, because I’ve been busy with work
> > on Zstd and other projects, and have had a hard time justifying to
> > myself spending too much time on this, since progress has been so slow.
> 
> What needs to be done from my POV:
> 
> - refresh the patches on top of current mainline, eg. v5.15-rc3
> 
> - make sure it compiles and works with current in-kernel users of zstd,
>   ie. with btrfs in particular, I can do some tests too

I have been running this patchset with btrfs on the latest stable kernels (5.12-14), and have not encountered any issues at all.
For AMD64 and AARCH64 you can add my
Tested-by: Paul Jones <paul@pauljones.id.au>

> - push the patches to a public branch eg. on k.org or github
> 
> - ask for adding the branch to linux-next
> 
> - try to get some feedback from people that were objecting in the past,
>   and of course gather acks or supportive feedback
> 
> - once merge window opens, send a pull request to Linus, write the
>   rationale why we want this change and summarize the evolution of the
>   patchset and why the full version update is perhaps the way forward