From: "Theodore Y. Ts'o" <tytso@mit.edu> To: Sasha Levin <Alexander.Levin@microsoft.com> Cc: Greg KH <gregkh@linuxfoundation.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "w@1wt.eu" <w@1wt.eu>, "ksummit-discuss@lists.linuxfoundation.org" <ksummit-discuss@lists.linuxfoundation.org> Subject: Re: [Ksummit-discuss] bug-introducing patches Date: Tue, 1 May 2018 16:54:48 -0400 [thread overview] Message-ID: <20180501205448.GE10479@thunk.org> (raw) In-Reply-To: <20180501200019.GA7397@sasha-vm> On Tue, May 01, 2018 at 08:00:21PM +0000, Sasha Levin wrote: > > Yes, linux-next users want it fixed *now* and I completely agree it > should be done that way, but the fix should not be immediately pushed to > Linus as well. I should have linux-head/linux-rc said testers, sorry. The fact that we have very few live users testing linux-next is a separate problem, which I accidentally conflated. But if a user who is testing -rc2 finds a problem, it is highly desirable to send a fix for -rc3, instead of making that user wait to -rc4 or -rc5. And *that* is more important than AUTOSEL. > I've just finished reading an interesting article on LWN about the > PostgreSQL fsync issues (https://lwn.net/Articles/752952/). If you > look at Willy's commit, he wrote the final version of it about 5 days > ago, Jeff merged it in 3 days ago, and Linus merged it in the tree > today. Did it spend any time getting -next testing? nope. I agree that having the errseq patch go straight into Linus's tree is certainly unfortunate. The justification was this was a regression fix, which I don't think it qualifies, since errseq_t went in some 9+ months ago. It might be a good thing to quantify whether the patches you are talking about are new features, bug fixes, or fixing a bug that was introduced during the merge window or subsequently (e.g., a regression). > What's worse is that that commit is tagged for stable, which means > that (given Greg's schedule) it may find it's way to -stable users > even before some -next users/bots had a chance to test it out. Well, it used to be that things tagged for stable most-merge window are *supposed* to marinate for at least a week or before they would get pulled into a stable release. Part of the whole problem is that people are wanting to be a lot more aggressive (both in time and volume) in shovelling things into stable. > This is less about AUTOSEL, and more about asking maintainers to > improve the testing commits get before they are sent to Linus. > Linus would rant about commits during merge window that didn't go > through -next, but for -rc commits this rule is somehow forgiven, > which is what I'm trying to change. I do think it's about AUTOSEL, because when I'm dealing with a regression, I want to get it fixed fast. Because the alternative is the merge-window commit getting reverted. AUTOSEL seems wants perfect patches that it can cherry pick once, as opposed to a case where if the user confirms that it fixes the regression, I want to send it to Linus quickly. I do *not* want it to marinate in linux-next for 1-2 weeks. I would much rather that *stable* hold off on picking up the patch for 1-2 weeks, but get it fixed in Linux HEAD sooner. If that means that the regression fix needs a further clean up, so be it. Post -rc3 or -rc4, in my opinion bug fixes should wait until the next merge window before they get merged at all. (And certainly features bugs should be Right Out.) And sure, bug fixes should certainly get more testing. So I guess my main objection is your making a blanket statement about all fixes, instead of breaking out regression fixes versus bug fixes. Since in my opinion they are very different animals... - Ted
WARNING: multiple messages have this Message-ID (diff)
From: "Theodore Y. Ts'o" <tytso@mit.edu> To: Sasha Levin <Alexander.Levin@microsoft.com> Cc: Greg KH <gregkh@linuxfoundation.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "w@1wt.eu" <w@1wt.eu>, "ksummit-discuss@lists.linuxfoundation.org" <ksummit-discuss@lists.linuxfoundation.org> Subject: Re: [Ksummit-discuss] bug-introducing patches Date: Tue, 1 May 2018 16:54:48 -0400 [thread overview] Message-ID: <20180501205448.GE10479@thunk.org> (raw) In-Reply-To: <20180501200019.GA7397@sasha-vm> On Tue, May 01, 2018 at 08:00:21PM +0000, Sasha Levin wrote: > > Yes, linux-next users want it fixed *now* and I completely agree it > should be done that way, but the fix should not be immediately pushed to > Linus as well. I should have linux-head/linux-rc said testers, sorry. The fact that we have very few live users testing linux-next is a separate problem, which I accidentally conflated. But if a user who is testing -rc2 finds a problem, it is highly desirable to send a fix for -rc3, instead of making that user wait to -rc4 or -rc5. And *that* is more important than AUTOSEL. > I've just finished reading an interesting article on LWN about the > PostgreSQL fsync issues (https://lwn.net/Articles/752952/). If you > look at Willy's commit, he wrote the final version of it about 5 days > ago, Jeff merged it in 3 days ago, and Linus merged it in the tree > today. Did it spend any time getting -next testing? nope. I agree that having the errseq patch go straight into Linus's tree is certainly unfortunate. The justification was this was a regression fix, which I don't think it qualifies, since errseq_t went in some 9+ months ago. It might be a good thing to quantify whether the patches you are talking about are new features, bug fixes, or fixing a bug that was introduced during the merge window or subsequently (e.g., a regression). > What's worse is that that commit is tagged for stable, which means > that (given Greg's schedule) it may find it's way to -stable users > even before some -next users/bots had a chance to test it out. Well, it used to be that things tagged for stable most-merge window are *supposed* to marinate for at least a week or before they would get pulled into a stable release. Part of the whole problem is that people are wanting to be a lot more aggressive (both in time and volume) in shovelling things into stable. > This is less about AUTOSEL, and more about asking maintainers to > improve the testing commits get before they are sent to Linus. > Linus would rant about commits during merge window that didn't go > through -next, but for -rc commits this rule is somehow forgiven, > which is what I'm trying to change. I do think it's about AUTOSEL, because when I'm dealing with a regression, I want to get it fixed fast. Because the alternative is the merge-window commit getting reverted. AUTOSEL seems wants perfect patches that it can cherry pick once, as opposed to a case where if the user confirms that it fixes the regression, I want to send it to Linus quickly. I do *not* want it to marinate in linux-next for 1-2 weeks. I would much rather that *stable* hold off on picking up the patch for 1-2 weeks, but get it fixed in Linux HEAD sooner. If that means that the regression fix needs a further clean up, so be it. Post -rc3 or -rc4, in my opinion bug fixes should wait until the next merge window before they get merged at all. (And certainly features bugs should be Right Out.) And sure, bug fixes should certainly get more testing. So I guess my main objection is your making a blanket statement about all fixes, instead of breaking out regression fixes versus bug fixes. Since in my opinion they are very different animals... - Ted _______________________________________________ Ksummit-discuss mailing list Ksummit-discuss@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/ksummit-discuss
next prev parent reply other threads:[~2018-05-01 20:54 UTC|newest] Thread overview: 289+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-05-01 16:38 bug-introducing patches Sasha Levin 2018-05-01 16:38 ` [Ksummit-discuss] " Sasha Levin 2018-05-01 19:44 ` Theodore Y. Ts'o 2018-05-01 19:44 ` Theodore Y. Ts'o 2018-05-01 20:00 ` Sasha Levin 2018-05-01 20:00 ` [Ksummit-discuss] " Sasha Levin 2018-05-01 20:33 ` Willy Tarreau 2018-05-01 20:48 ` [Ksummit-discuss] " Willy Tarreau 2018-05-01 20:42 ` Sasha Levin 2018-05-01 20:42 ` [Ksummit-discuss] " Sasha Levin 2018-05-01 20:54 ` Theodore Y. Ts'o [this message] 2018-05-01 20:54 ` Theodore Y. Ts'o 2018-05-01 21:15 ` Mark Brown 2018-05-01 21:15 ` Mark Brown 2018-05-02 8:11 ` Daniel Vetter 2018-05-02 8:11 ` Daniel Vetter 2018-05-02 19:46 ` Sasha Levin 2018-05-02 19:46 ` Sasha Levin via Ksummit-discuss 2018-05-03 2:05 ` Mark Brown 2018-05-03 2:05 ` Mark Brown via Ksummit-discuss 2018-05-03 3:10 ` Theodore Y. Ts'o 2018-05-03 3:10 ` Theodore Y. Ts'o 2018-05-03 3:52 ` Guenter Roeck 2018-05-03 3:52 ` Guenter Roeck 2018-05-03 12:03 ` Greg KH 2018-05-03 12:03 ` Greg KH 2018-05-03 22:42 ` Mark Brown 2018-05-03 22:42 ` Mark Brown 2018-05-03 23:09 ` Tony Lindgren 2018-05-03 23:09 ` Tony Lindgren 2018-05-04 14:21 ` Ulf Hansson 2018-05-04 14:21 ` Ulf Hansson 2018-05-09 8:44 ` Mark Brown 2018-05-09 8:44 ` Mark Brown 2018-05-09 8:47 ` Daniel Vetter 2018-05-09 8:47 ` Daniel Vetter 2018-05-09 8:51 ` Geert Uytterhoeven 2018-05-09 8:51 ` Geert Uytterhoeven 2018-05-09 9:03 ` Mark Brown 2018-05-09 9:03 ` Mark Brown 2018-05-09 10:47 ` Stephen Rothwell 2018-05-09 10:47 ` Stephen Rothwell 2018-05-09 10:55 ` Vinod Koul 2018-05-09 10:55 ` Vinod Koul 2018-05-09 12:43 ` Stephen Rothwell 2018-05-09 12:43 ` Stephen Rothwell 2018-05-09 12:47 ` Vinod Koul 2018-05-09 12:47 ` Vinod Koul 2018-05-15 10:42 ` Krzysztof Kozlowski 2018-05-15 10:42 ` Krzysztof Kozlowski 2018-05-15 11:54 ` Stephen Rothwell 2018-05-15 11:54 ` Stephen Rothwell 2018-05-09 14:05 ` Mark Brown 2018-05-09 14:05 ` Mark Brown 2018-05-09 22:09 ` Stephen Rothwell 2018-05-09 22:09 ` Stephen Rothwell 2018-05-10 13:36 ` Mark Brown 2018-05-10 13:36 ` Mark Brown 2018-05-10 22:01 ` Stephen Rothwell 2018-05-10 22:01 ` Stephen Rothwell 2018-05-09 15:57 ` Guenter Roeck 2018-05-09 15:57 ` Guenter Roeck 2018-05-09 21:45 ` Stephen Rothwell 2018-05-09 21:45 ` Stephen Rothwell 2018-05-09 16:04 ` Dan Williams 2018-05-09 16:04 ` Dan Williams 2018-05-09 16:04 ` Dan Williams 2018-05-09 21:51 ` Stephen Rothwell 2018-05-09 21:51 ` Stephen Rothwell 2018-05-09 21:51 ` Stephen Rothwell 2018-05-09 19:35 ` Boris Brezillon 2018-05-09 19:35 ` Boris Brezillon 2018-05-09 21:58 ` Stephen Rothwell 2018-05-09 21:58 ` Stephen Rothwell 2018-05-10 3:15 ` Sasha Levin 2018-05-10 3:15 ` Sasha Levin via Ksummit-discuss 2018-05-10 15:57 ` Tony Lindgren 2018-05-10 15:57 ` Tony Lindgren 2018-05-10 22:05 ` Stephen Rothwell 2018-05-10 22:05 ` Stephen Rothwell 2018-05-11 8:47 ` David Sterba 2018-05-11 8:49 ` David Sterba 2018-05-12 4:03 ` Stephen Rothwell 2018-05-12 4:03 ` Stephen Rothwell 2018-05-12 4:38 ` Stephen Rothwell 2018-05-12 4:38 ` Stephen Rothwell 2018-05-12 18:34 ` Guenter Roeck 2018-05-13 13:53 ` Andy Shevchenko 2018-05-13 13:53 ` Andy Shevchenko 2018-05-14 8:36 ` Ulf Hansson 2018-05-14 8:36 ` Ulf Hansson 2018-05-14 21:45 ` Stephen Rothwell 2018-05-14 21:45 ` Stephen Rothwell 2018-05-17 5:10 ` Mark Brown 2018-05-17 5:10 ` Mark Brown 2018-05-10 16:03 ` Jiri Kosina 2018-05-10 16:03 ` Jiri Kosina 2018-05-10 16:47 ` Sasha Levin 2018-05-10 16:47 ` Sasha Levin via Ksummit-discuss 2018-05-14 7:53 ` Geert Uytterhoeven 2018-05-14 7:53 ` Geert Uytterhoeven 2018-05-14 8:00 ` Geert Uytterhoeven 2018-05-14 8:00 ` Geert Uytterhoeven 2018-05-14 8:12 ` Boris Brezillon 2018-05-14 8:12 ` Boris Brezillon 2018-05-14 8:29 ` Geert Uytterhoeven 2018-05-14 8:29 ` Geert Uytterhoeven 2018-05-14 8:34 ` Boris Brezillon 2018-05-14 8:34 ` Boris Brezillon 2018-05-14 8:40 ` Geert Uytterhoeven 2018-05-14 8:40 ` Geert Uytterhoeven 2018-05-14 8:48 ` Boris Brezillon 2018-05-14 8:48 ` Boris Brezillon 2018-05-14 9:25 ` Fengguang Wu 2018-05-14 9:25 ` Fengguang Wu 2018-05-11 2:10 ` Mark Brown 2018-05-11 2:10 ` Mark Brown 2018-05-08 2:34 ` Sasha Levin 2018-05-08 2:34 ` Sasha Levin 2018-05-08 3:48 ` Theodore Y. Ts'o 2018-05-08 3:48 ` Theodore Y. Ts'o 2018-05-08 14:49 ` Tony Lindgren 2018-05-09 8:13 ` Mark Brown 2018-05-09 8:13 ` Mark Brown 2018-05-10 15:36 ` Tony Lindgren 2018-05-10 15:36 ` Tony Lindgren 2018-05-08 20:29 ` Sasha Levin 2018-05-08 20:29 ` Sasha Levin via Ksummit-discuss 2018-05-08 20:40 ` Matthew Wilcox 2018-05-08 20:40 ` Matthew Wilcox 2018-05-08 20:55 ` Sasha Levin 2018-05-08 20:55 ` Sasha Levin 2018-05-08 20:59 ` David Lang 2018-05-08 21:06 ` David Lang 2018-05-08 21:43 ` Sasha Levin 2018-05-08 21:43 ` Sasha Levin via Ksummit-discuss 2018-05-08 21:51 ` Dan Williams 2018-05-08 21:51 ` Dan Williams 2018-05-08 22:41 ` James Bottomley 2018-05-08 22:41 ` James Bottomley 2018-05-08 21:26 ` Justin Forbes 2018-05-08 21:26 ` Justin Forbes 2018-05-08 21:00 ` Ken Moffat 2018-05-08 21:08 ` Ken Moffat 2018-05-08 22:15 ` Theodore Y. Ts'o 2018-05-10 16:39 ` Sasha Levin 2018-05-09 4:47 ` Willy Tarreau 2018-05-09 4:47 ` Willy Tarreau 2018-05-08 13:58 ` Justin Forbes 2018-05-08 13:58 ` Justin Forbes 2018-05-08 2:39 ` Sasha Levin 2018-05-08 2:39 ` Sasha Levin via Ksummit-discuss 2018-05-01 22:02 ` Sasha Levin 2018-05-01 22:02 ` [Ksummit-discuss] " Sasha Levin 2018-05-02 4:30 ` Willy Tarreau 2018-05-02 4:30 ` [Ksummit-discuss] " Willy Tarreau 2018-05-02 19:42 ` Sasha Levin 2018-05-02 19:42 ` Sasha Levin 2018-05-02 20:02 ` Willy Tarreau 2018-05-02 20:02 ` [Ksummit-discuss] " Willy Tarreau 2018-07-14 17:38 ` Pavel Machek 2018-07-14 17:38 ` Pavel Machek 2018-07-14 18:37 ` [Ksummit-discuss] " Guenter Roeck 2018-07-14 18:37 ` Guenter Roeck 2018-07-14 19:47 ` Pavel Machek 2018-07-14 19:47 ` Pavel Machek 2018-07-14 20:40 ` Guenter Roeck 2018-07-14 20:40 ` Guenter Roeck 2018-07-14 21:09 ` Pavel Machek 2018-07-14 21:09 ` Pavel Machek 2018-07-15 5:57 ` Willy Tarreau 2018-07-15 5:57 ` Willy Tarreau 2018-07-15 8:54 ` Greg KH 2018-07-15 8:54 ` Greg KH 2018-07-15 14:50 ` [Ksummit-discuss] " Theodore Y. Ts'o 2018-07-15 14:50 ` Theodore Y. Ts'o 2018-07-15 20:15 ` [Ksummit-discuss] " Pavel Machek 2018-07-15 20:15 ` Pavel Machek 2018-05-03 11:08 ` [Ksummit-discuss] " Jani Nikula 2018-05-03 11:08 ` Jani Nikula 2018-05-03 14:33 ` James Bottomley 2018-05-03 14:33 ` James Bottomley 2018-05-03 14:48 ` Willy Tarreau 2018-05-03 14:49 ` Willy Tarreau 2018-05-03 15:06 ` Sasha Levin 2018-05-03 15:06 ` Sasha Levin via Ksummit-discuss 2018-05-03 15:27 ` James Bottomley 2018-05-03 15:27 ` James Bottomley 2018-05-03 15:43 ` Sasha Levin 2018-05-03 15:43 ` Sasha Levin via Ksummit-discuss 2018-05-03 17:17 ` Randy Dunlap 2018-05-03 17:17 ` Randy Dunlap 2018-05-03 17:39 ` Sasha Levin 2018-05-03 17:39 ` Sasha Levin via Ksummit-discuss 2018-05-03 18:10 ` James Bottomley 2018-05-03 18:10 ` James Bottomley 2018-05-03 15:56 ` Willy Tarreau 2018-05-03 15:57 ` Willy Tarreau 2018-05-03 18:58 ` Theodore Y. Ts'o 2018-05-03 18:58 ` Theodore Y. Ts'o 2018-05-01 23:28 ` Stephen Rothwell 2018-05-01 23:10 ` Stephen Rothwell 2018-05-02 15:32 ` Geert Uytterhoeven 2018-05-02 15:32 ` Geert Uytterhoeven 2018-05-02 19:51 ` Sasha Levin 2018-05-02 19:51 ` Sasha Levin via Ksummit-discuss 2018-05-02 20:41 ` Geert Uytterhoeven 2018-05-02 20:41 ` Geert Uytterhoeven 2018-05-03 0:06 ` [Ksummit-discuss] " Theodore Y. Ts'o 2018-05-03 0:06 ` Theodore Y. Ts'o 2018-05-03 0:38 ` Guenter Roeck 2018-05-03 0:38 ` Guenter Roeck 2018-05-03 2:30 ` Willy Tarreau 2018-05-03 2:30 ` Willy Tarreau 2018-05-03 14:55 ` Sasha Levin 2018-05-03 14:55 ` Sasha Levin 2018-05-03 15:49 ` Guenter Roeck 2018-05-03 15:49 ` Guenter Roeck 2018-05-03 16:02 ` Sasha Levin 2018-05-03 16:02 ` Sasha Levin via Ksummit-discuss 2018-05-03 16:50 ` Justin Forbes 2018-05-03 16:50 ` Justin Forbes 2018-05-03 17:09 ` Guenter Roeck 2018-05-03 17:09 ` Guenter Roeck 2018-05-03 11:48 ` Al Viro 2018-05-03 14:46 ` Sasha Levin 2018-05-03 14:46 ` Sasha Levin via Ksummit-discuss 2018-05-03 14:52 ` Willy Tarreau 2018-05-03 14:52 ` Willy Tarreau 2018-05-03 15:01 ` Sasha Levin 2018-05-03 15:01 ` Sasha Levin via Ksummit-discuss 2018-05-03 16:00 ` Willy Tarreau 2018-05-03 16:01 ` Willy Tarreau 2018-05-03 16:14 ` Sasha Levin 2018-05-03 16:15 ` Sasha Levin 2018-05-03 16:35 ` Willy Tarreau 2018-05-03 16:35 ` Willy Tarreau 2018-05-03 17:29 ` Sasha Levin 2018-05-03 17:29 ` Sasha Levin via Ksummit-discuss 2018-05-03 17:57 ` Willy Tarreau 2018-05-03 17:57 ` Willy Tarreau 2018-05-03 18:12 ` Sasha Levin 2018-05-03 18:12 ` Sasha Levin 2018-05-03 18:46 ` Guenter Roeck 2018-05-03 18:46 ` Guenter Roeck 2018-05-03 19:03 ` Willy Tarreau 2018-05-03 19:03 ` Willy Tarreau 2018-05-03 16:54 ` Al Viro 2018-05-03 16:54 ` Al Viro 2018-05-03 17:34 ` Sasha Levin 2018-05-03 17:34 ` Sasha Levin via Ksummit-discuss 2018-05-03 18:20 ` Al Viro 2018-05-03 18:20 ` Al Viro 2018-05-03 18:55 ` Greg KH 2018-05-03 18:55 ` Greg KH 2018-05-03 19:14 ` Willy Tarreau 2018-05-03 19:14 ` Willy Tarreau 2018-05-03 19:17 ` Sasha Levin 2018-05-03 19:17 ` Sasha Levin via Ksummit-discuss 2018-05-03 19:04 ` Sasha Levin 2018-05-03 19:04 ` Sasha Levin 2018-05-04 9:57 ` David Howells 2018-05-04 9:57 ` David Howells 2018-05-04 12:31 ` Jani Nikula 2018-05-04 12:31 ` Jani Nikula 2018-05-04 13:09 ` Theodore Y. Ts'o 2018-05-04 13:09 ` Theodore Y. Ts'o 2018-05-04 17:40 ` Greg KH 2018-05-04 17:40 ` Greg KH 2018-05-04 21:13 ` Theodore Y. Ts'o 2018-05-04 21:13 ` Theodore Y. Ts'o 2018-05-04 21:38 ` James Bottomley 2018-05-04 21:38 ` James Bottomley 2018-05-04 21:51 ` Sasha Levin 2018-05-04 21:51 ` Sasha Levin 2018-05-04 23:35 ` Theodore Y. Ts'o 2018-05-04 23:35 ` Theodore Y. Ts'o 2018-05-05 4:23 ` Willy Tarreau 2018-05-05 4:24 ` Willy Tarreau 2018-05-05 5:02 ` Eric W. Biederman 2018-05-05 5:02 ` Eric W. Biederman 2018-05-05 16:37 ` Greg KH 2018-05-05 16:37 ` Greg KH 2018-05-05 5:27 ` Sasha Levin 2018-05-05 5:27 ` Sasha Levin via Ksummit-discuss 2018-05-03 11:43 ` Al Viro 2018-05-03 11:43 ` Al Viro 2018-05-02 15:32 ` Geert Uytterhoeven 2018-05-02 15:32 ` Geert Uytterhoeven
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=20180501205448.GE10479@thunk.org \ --to=tytso@mit.edu \ --cc=Alexander.Levin@microsoft.com \ --cc=gregkh@linuxfoundation.org \ --cc=ksummit-discuss@lists.linuxfoundation.org \ --cc=linux-kernel@vger.kernel.org \ --cc=w@1wt.eu \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.