linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: Eric Sandeen <sandeen@sandeen.net>,
	Christoph Hellwig <hch@infradead.org>,
	linux-xfs <linux-xfs@vger.kernel.org>
Subject: Re: [ANNOUNCE] xfsprogs v4.15.0-rc1 released
Date: Wed, 21 Feb 2018 08:02:37 +1100	[thread overview]
Message-ID: <20180220210237.GB7000@dastard> (raw)
In-Reply-To: <20180220163347.GB27629@magnolia>

On Tue, Feb 20, 2018 at 08:33:47AM -0800, Darrick J. Wong wrote:
> On Tue, Feb 20, 2018 at 09:09:22AM -0600, Eric Sandeen wrote:
> > (resend, somehow I keep missing reply-all)
> > 
> > That would be much faster than at least our historical progression
> > for new features:
> > 
> > * Experimental for $LONG_TIME
> > * Drop experimental for $SIGNIFICANT_TIME
> > * Make default after that
> 
> ISTR Dave once telling me that he usually waited ~4 kernel releases to
> drop the experimental tag and then another year to turn it on by
> default in mkfs.  I'll grant you that seven kernels went by before
> EXPERIMENTAL went away, so I don't think we need to wait a whole other
> year.  How about turning it on by default in July or so?

The wait time for "supported -> default" is so that distro's have
time to move to a feature supported kernel before they pick up a new
xfsprogs release that turns on that feature by default. this mostly
avoids the problem of distros accidentally turning on a feature that
is still experimental in their kernel....

It also gives us "early adopter" feedback before large numbers of
unsuspecting users have it enabled for them....

> Speaking of which, hasn't enough time passed to enable spinodes by
> default in mkfs?  Given the likelihood of worse fragmentation once we
> add cow into the mix, we probably want that turned on before reflink.
> 
> Hm.... EXPERIMENTAL was removed for spinodes in July 2016, so that
> probably could be turned on "now".

Yup, enough time has passed there.

> > My sense is that it's a bit premature, having /just/ dropped the
> > EXPERIMENTAL tag, which will (in theory) get more people using
> > it in earnest and maybe shaking out more bugs.
> > 
> > But I'd like to hear what others think as well.
> 
> There's already too much new stuff in mkfs 4.15; let's wait for .17
> because that'll give the more cautious testers more time to find
> whatever other bugs still lurk. :)

I think even that is too fast. If distro's want it as the default
before we set it, then they can patch mkfs themselves.

If this is really a huge problem for distros, then wasn't this what
someone wanted mkfs config files for?

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

  reply	other threads:[~2018-02-20 21:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-13 20:29 [ANNOUNCE] xfsprogs v4.15.0-rc1 released Eric Sandeen
2018-02-20  0:36 ` Christoph Hellwig
2018-02-20 15:09   ` Eric Sandeen
2018-02-20 16:33     ` Darrick J. Wong
2018-02-20 21:02       ` Dave Chinner [this message]
2018-02-22 20:49         ` Eric Sandeen
2018-02-22 23:44           ` Dave Chinner
2018-02-23 18:33             ` Darrick J. Wong
2018-02-23 22:14               ` Dave Chinner
2018-02-23  1:56           ` Luis R. Rodriguez
2018-02-23  1:59             ` Darrick J. Wong

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=20180220210237.GB7000@dastard \
    --to=david@fromorbit.com \
    --cc=darrick.wong@oracle.com \
    --cc=hch@infradead.org \
    --cc=linux-xfs@vger.kernel.org \
    --cc=sandeen@sandeen.net \
    /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).