ksummit.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Guenter Roeck <linux@roeck-us.net>
Cc: ksummit-discuss@lists.linux-foundation.org
Subject: Re: [Ksummit-discuss] Kernel Summit Agenda -- 2nd draft
Date: Wed, 21 Oct 2015 18:25:28 +0100	[thread overview]
Message-ID: <20151021172528.GQ32054@sirena.org.uk> (raw)
In-Reply-To: <5627BF30.4080703@roeck-us.net>

[-- Attachment #1: Type: text/plain, Size: 1665 bytes --]

On Wed, Oct 21, 2015 at 09:37:04AM -0700, Guenter Roeck wrote:

> 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.

Yeah, I mostly just report problems (partly because it's just less time
consuming).  On the reviewer side fixes like that only get to be an
issue when submitters ignore feedback and keep on sending the same stuff
even after discussion as to why other approaches are better.

> >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.

It's definitely not the norm but I have encountered it.  Obviously
indivudal cases will differ, sometimes there will be value in exposure
in -next (eg, if it's likely to get validation from the boot farms),
it's a question of what the issue is, what the coverage is and what the
risks of the fix are.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

  parent reply	other threads:[~2015-10-21 17:25 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
2015-10-21 17:24           ` Luck, Tony
2015-10-21 18:53             ` Jonathan Corbet
2015-10-21 17:25           ` Mark Brown [this message]
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=20151021172528.GQ32054@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=ksummit-discuss@lists.linux-foundation.org \
    --cc=linux@roeck-us.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