From: Guenter Roeck <linux@roeck-us.net>
To: Mark Brown <broonie@kernel.org>
Cc: ksummit-discuss@lists.linux-foundation.org
Subject: Re: [Ksummit-discuss] Kernel Summit Agenda -- 2nd draft
Date: Wed, 21 Oct 2015 09:37:04 -0700 [thread overview]
Message-ID: <5627BF30.4080703@roeck-us.net> (raw)
In-Reply-To: <20151021160907.GM32054@sirena.org.uk>
On 10/21/2015 09:09 AM, Mark Brown wrote:
> On Wed, Oct 21, 2015 at 08:20:54AM -0700, Guenter Roeck wrote:
>
>> Number of build and runtime breakages in -next is a bit high, and fixes are
>> sometimes slow to roll in. At least in part this is because those responsible
>> for breakages are not informed, but I have also seen problems which were
>> known for weeks to propagate into mainline before they got fixed, even if
>> the culprit was informed. This could use some improvement, though I am not
>> really sure how we could get there. Make more noise ?
>
> Noise is probably a large part of it, having a human following up about
> issues seems to help a lot.
>
Sure, and I do that if I can find the time. In my experience, submitting
patches to fix observed problems turns out to be the best approach.
Even (or especially) if plain wrong or less than perfect, patches are
almost guaranteed to trigger a response.
Doing this is just very time consuming, and time is always short.
Also, while beneficial for the system as a whole, I am not sure if it is
beneficial for the submitter. Touching multiple subsystems almost
guarantees for the submitter to get something wrong, either because
of unfamiliarity with the code or because of maintainer preferences.
Does submitting patches all over the place benefit or hurt my reputation
with other maintainers, given that the percentage of rejected patches
is quite high ? I don't know, and I don't really care that much since my
ultimate goal is to get problems fixed, not to get my patches accepted.
However, for others it may play a role when deciding if or if not to
spend the time, track down a problem, and submit a patch for it.
> I've seen some active resistance to pushing fixes to mainline without
> lengthy soaks in -next in a very rules based fashion which isn't super
> awesome when it takes out other testing due to the breakage. As the
> test coverage improves this is going to be getting to be more and more
> of an issue as failures to build or boot will cause gaps in other
> testing.
>
For fixes ? I don't recall seeing that reaction, at least not to patches
I have been involved in. Unless in very special cases, it doesn't seem
to make much sense to me to require bug fixes to soak in -next.
Guenter
next prev parent reply other threads:[~2015-10-21 16:37 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-20 22:03 Theodore Ts'o
2015-10-20 23:17 ` Jason Cooper
2015-10-21 2:36 ` Olof Johansson
2015-10-21 14:56 ` Theodore Ts'o
2015-10-21 15:20 ` Guenter Roeck
2015-10-21 16:09 ` Mark Brown
2015-10-21 16:37 ` Guenter Roeck [this message]
2015-10-21 17:24 ` Luck, Tony
2015-10-21 18:53 ` Jonathan Corbet
2015-10-21 17:25 ` Mark Brown
2015-10-22 15:25 ` Theodore Ts'o
2015-10-22 20:01 ` Alexandre Belloni
2015-10-24 15:19 ` Rafael J. Wysocki
2015-10-26 5:56 ` Luis R. Rodriguez
2015-10-26 6:12 ` Eric W. Biederman
2015-10-26 6:28 ` Josh Triplett
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=5627BF30.4080703@roeck-us.net \
--to=linux@roeck-us.net \
--cc=broonie@kernel.org \
--cc=ksummit-discuss@lists.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