linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Theodore Ts'o" <tytso@mit.edu>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Amir Goldstein <amir73il@gmail.com>,
	Sasha Levin <sashal@kernel.org>,
	lsf-pc <lsf-pc@lists.linux-foundation.org>,
	linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	Jan Kara <jack@suse.cz>,
	"Darrick J. Wong" <darrick.wong@oracle.com>,
	Josef Bacik <josef@toxicpanda.com>,
	"Luis R. Rodriguez" <mcgrof@kernel.org>,
	Matthew Wilcox <willy@infradead.org>
Subject: Re: [LSF/MM TOPIC] FS, MM, and stable trees
Date: Tue, 8 Mar 2022 11:40:18 -0500	[thread overview]
Message-ID: <YieG8rZkgnfwygyu@mit.edu> (raw)
In-Reply-To: <YicrMCidylefTC3n@kroah.com>

On Tue, Mar 08, 2022 at 11:08:48AM +0100, Greg KH wrote:
> > When one looks at xfstest bug reports on the list for xfs on kernels > v4.19
> > one has to wonder if using xfs on kernels v5.x.y is a wise choice.
> 
> That's up to the xfs maintainers to discuss.
> 
> > Which makes me wonder: how do the distro kernel maintainers keep up
> > with xfs fixes?
> 
> Who knows, ask the distro maintainers that use xfs.  What do they do?

This is something which is being worked, so I'm not sure we'll need to
discuss the specifics of the xfs stable backports at LSF/MM.  I'm
hopeful that by May, we'll have come to some kind of resolution of
that topic.

One of my team members has been working with Darrick to set up a set
of xfs configs[1] recommended by Darrick, and she's stood up an
automated test spinner using gce-xfstests which can watch a git branch
and automatically kick off a set of tests whenever it is updated.
Sasha has also provided her with a copy of his scripts so we can do
automated cherry picks of commits with Fixes tags.  So the idea is
that we can, hopefully in a mostly completely automated fashion, do
the backports and do a full set of regression tests on those stable
backports of XFS bug fixes.

[1] https://github.com/tytso/xfstests-bld/tree/master/kvm-xfstests/test-appliance/files/root/fs/xfs/cfg

Next steps are to get a first tranche of cherry-picks for 5.10 and
probably 5.15, and use the test spinner to demonstrate that they don't
have any test regressions (if there are, we'll drop those commits).
Once we have a first set of proposed stable backports for XFS, we'll
present them to the XFS development community for their input.  There
are a couple of things that could happen at this point, depending on
what the XFS community is willing to accept.

The first is that we'll send these tested stable patches directly to
Greg and Sasha for inclusion in the LTS releases, with the linux-xfs
list cc'ed so they know what's going into the stable trees.

The second is that we send them only to the linux-xfs list, and they
have to do whatever approval they want before they go into the
upstream stable trees.

And the third option, if they aren't willing to take our work or they
choose to require manual approvals and those approvals are taking too
long, is that we'll feed the commits into Google's Container-Optimized
OS (COS) kernel, so that our customers can get those fixes and so we
can support XFS fully.  This isn't our preferred path; we'd prefer to
take the backports into the COS tree via the stable trees if at all
possible.  (Note: if requested, we could also publish these
backported-and-tested commits on a git tree for other distros to
take.)

There are still some details we'll need to work out; for example, will
the XFS maintainers let us do minor/obvious patch conflict
resolutions, or perhaps those commits which don't cherry-pick cleanly
will need to go through some round of approval by the linux-xfs list,
if the "we've run a full set of tests and there are no test
regressions" isn't good enough for them.

There is also the problem that sometimes commits aren't marked with
Fixes tag, but maybe there are some other signals we could use (for
example, maybe an indication in a comment in an xfstests test that
it's testing regressions for a specified kernel commit id).  Or
perhaps some other would be willing to contribute candidate commit
id's for backport consideration, with the approval of linux-xfs?
TBD...

Note: Darrick has been very helpful in geting this set up; the issue
is not the XFS maintainer, but rather the will of the whole of the XFS
development community, which sometimes can be a bit... fractious.

						- Ted

  parent reply	other threads:[~2022-03-08 16:41 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-12 17:00 [LSF/MM TOPIC] FS, MM, and stable trees Sasha Levin
2019-02-12 21:32 ` Steve French
2019-02-13  7:20   ` Amir Goldstein
2019-02-13  7:37     ` Greg KH
2019-02-13  9:01       ` Amir Goldstein
2019-02-13  9:18         ` Greg KH
2019-02-13 19:25           ` Sasha Levin
2019-02-13 19:52             ` Greg KH
2019-02-13 20:14               ` James Bottomley
2019-02-15  1:50                 ` Sasha Levin
2019-02-15  2:48                   ` James Bottomley
2019-02-16 18:28                     ` Theodore Y. Ts'o
2019-02-21 15:34                       ` Luis Chamberlain
2019-02-21 18:52                         ` Theodore Y. Ts'o
2019-03-20  3:46               ` Jon Masters
2019-03-20  5:06                 ` Greg KH
2019-03-20  6:14                   ` Jon Masters
2019-03-20  6:28                     ` Greg KH
2019-03-20  6:32                       ` Jon Masters
2022-03-08  9:32 ` Amir Goldstein
2022-03-08 10:08   ` Greg KH
2022-03-08 11:04     ` Amir Goldstein
2022-03-08 15:42       ` Luis Chamberlain
2022-03-08 19:06       ` Sasha Levin
2022-03-09 18:57         ` Luis Chamberlain
2022-03-11  5:23           ` Theodore Ts'o
2022-03-11 12:00             ` Jan Kara
2022-03-11 20:52             ` Luis Chamberlain
2022-03-11 22:04               ` Theodore Ts'o
2022-03-11 22:36                 ` Luis Chamberlain
2022-04-27 18:58                 ` Amir Goldstein
2022-05-01 16:25                   ` Luis Chamberlain
2022-03-10 23:59         ` Steve French
2022-03-11  0:36           ` Chuck Lever III
2022-03-11 20:54             ` Luis Chamberlain
2022-03-08 16:40     ` Theodore Ts'o [this message]
2022-03-08 17:16       ` Amir Goldstein
2022-03-09  0:43       ` Dave Chinner
2022-03-09 18:41       ` Luis Chamberlain
2022-03-09 18:49         ` Josef Bacik
2022-03-09 19:00           ` Luis Chamberlain
2022-03-09 21:19             ` Josef Bacik
2022-03-10  1:28               ` Luis Chamberlain
2022-03-10 18:51                 ` Josef Bacik
2022-03-10 22:41                   ` Luis Chamberlain
2022-03-11 12:09                     ` Jan Kara
2022-03-11 18:32                       ` Luis Chamberlain
2022-03-12  2:07                   ` Luis Chamberlain
2022-03-14 22:45                     ` btrfs profiles to test was: (Re: [LSF/MM TOPIC] FS, MM, and stable trees) Luis Chamberlain
2022-03-15 14:23                       ` Josef Bacik
2022-03-15 17:42                         ` Luis Chamberlain
2022-03-29 20:24       ` [LSF/MM TOPIC] FS, MM, and stable trees Amir Goldstein
2022-04-10 15:11         ` Amir Goldstein
2022-03-08 10:54   ` Jan Kara
2022-03-09  0:02   ` Dave Chinner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YieG8rZkgnfwygyu@mit.edu \
    --to=tytso@mit.edu \
    --cc=amir73il@gmail.com \
    --cc=darrick.wong@oracle.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=jack@suse.cz \
    --cc=josef@toxicpanda.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=lsf-pc@lists.linux-foundation.org \
    --cc=mcgrof@kernel.org \
    --cc=sashal@kernel.org \
    --cc=willy@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).