From: Sasha Levin <sashal@kernel.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
Chris Wilson <chris@chris-wilson.co.uk>,
Linux List Kernel Mailing <linux-kernel@vger.kernel.org>,
Bandan Das <bsd@redhat.com>
Subject: Re: Linux 5.3-rc7
Date: Sun, 8 Sep 2019 07:02:53 -0400 [thread overview]
Message-ID: <20190908110253.GC2012@sasha-vm> (raw)
In-Reply-To: <CAHk-=wg5dS9QsC5Ay0BpLBQdBRcy0qCE2hP=K4_nJ4b6Lumf_Q@mail.gmail.com>
On Sat, Sep 07, 2019 at 02:13:22PM -0700, Linus Torvalds wrote:
>On Sat, Sep 7, 2019 at 1:44 PM Thomas Gleixner <tglx@linutronix.de> wrote:
>>
>> That's what I just replied to Chris. Can you do it right away or should I queue it up?
>
>Done.
I'd like to bring back a discussion we had last year on ksummit-discuss:
https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2018-May/005122.html
. I've pointed out that some of the commits that go in the -rc cycles
are of low quality and are untested, you seemed to agree but said that
it's "by-design" because late -rc cycle commits are more complex.
Is this commit and it's fallout really how our development process
should be working?
This commit was rushed through the process: it was authored and merged
into -tip of the same day, and pulled in by you just a few days later.
There was no meaningful time for review, testing, or really any sort of
QA.
We really do have a better story now for catching the sort of issues
introduced by these patch: multiple CI systems tripped on this, but
people still need the time to look into it, make sure that the failure
is real and bisect it.
What was the rush in making it skip all of our safeguards? The "bug" has
been there forever, the fix isn't urgent, and no one seemed to care for
quite a while.
Even if this patch was fixing a bug introduced in this merge window, is
the tradeoff around rushing an untested fix worth it vs giving it more
time and shipping it as part of our stable tree?
I'm not trying to pick on this patch in particular - I feel that this is
a systematic issue and should be addressed as part of our process.
--
Thanks,
Sasha
prev parent reply other threads:[~2019-09-08 11:02 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-02 17:28 Linux 5.3-rc7 Linus Torvalds
2019-09-07 10:10 ` Chris Wilson
2019-09-07 14:29 ` Thomas Gleixner
2019-09-07 14:41 ` Chris Wilson
2019-09-07 15:00 ` Thomas Gleixner
2019-09-07 15:24 ` Chris Wilson
2019-09-07 20:12 ` Thomas Gleixner
2019-09-07 19:17 ` Linus Torvalds
2019-09-07 19:27 ` Linus Torvalds
2019-09-09 13:54 ` Bandan Das
2019-09-07 20:44 ` Thomas Gleixner
2019-09-07 21:13 ` Linus Torvalds
2019-09-08 11:02 ` Sasha Levin [this message]
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=20190908110253.GC2012@sasha-vm \
--to=sashal@kernel.org \
--cc=bsd@redhat.com \
--cc=chris@chris-wilson.co.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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