From: Ingo Molnar <mingo@elte.hu>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Sam Ravnborg <sam@ravnborg.org>,
LKML <linux-kernel@vger.kernel.org>,
Nicolas Pitre <nico@fluxnic.net>,
"Luck, Tony" <tony.luck@intel.com>,
Stephen Rothwell <sfr@canb.auug.org.au>,
"Luis R. Rodriguez" <mcgrof@gmail.com>,
Jeff Garzik <jeff@garzik.org>,
Robert Richter <robert.richter@amd.com>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Jean Delvare <khali@linux-fr.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
"David S. Miller" <davem@davemloft.net>
Subject: Re: [RFC] to rebase or not to rebase on linux-next
Date: Wed, 28 Oct 2009 07:40:09 +0100 [thread overview]
Message-ID: <20091028064009.GA6353@elte.hu> (raw)
In-Reply-To: <1256657956.26028.408.camel@gandalf.stny.rr.com>
* Steven Rostedt <rostedt@goodmis.org> wrote:
> On Tue, 2009-10-27 at 08:59 +0100, Ingo Molnar wrote:
>
> > For any reasonably complex change you simply need to wait a bit
> > anyway to gather feedback. And for trivial/obvious/small patches it
> > makes little sense to upset the Git history just to add the tags.
> >
> > (And if there's frequent problems with small changes that were
> > supposed to be easy you need to revisit the quality process.)
>
> Sure, but I'm not talking about changing the patch, I'm talking about
> a late "reviewed by" or "tested-by". These usually do come after a
> patch set has been moved into the final git push. It would be nice to
> flag a commit that it was tested by someone.
If you want to make your patches dependent on acks then you first need
to send out those patches with an '[RFC]' added, and put into a branch
hierarchy (rfc/* would be fine) that i sure wont pull from you.
You can also send obvious/small/own patches out for immediate pull. In
that case extra review can still be done after you sent patches out.
Mistakes noticed during review will be credited in terms of future
Reported-by tags.
People do this frequently.
Ingo
next prev parent reply other threads:[~2009-10-28 6:40 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4ADD0793.3000707@kernel.org>
[not found] ` <20091020020449.GF24370@core.coreip.homeip.net>
[not found] ` <20091020034829.GA12833@elte.hu>
[not found] ` <20091020140750.GH11972@erda.amd.com>
[not found] ` <alpine.LFD.2.00.0910210654480.5125@eeepc.linux-foundation.org>
[not found] ` <20091022122042.e535d43c.sfr@canb.auug.org.au>
[not found] ` <20091023112732.GB5886@elte.hu>
[not found] ` <4AE19A74.1090709@garzik.org>
[not found] ` <20091023123555.GA25366@elte.hu>
[not found] ` <57C9024A16AD2D4C97DC78E552063EA3E33D0174@orsmsx505.amr.corp.intel.com>
[not found] ` <20091023134115.GD27097@elte.hu>
[not found] ` <alpine.LFD.2.00.0910231424440.21460@xanadu.home>
[not found] ` <20091023191631.GA1879@elte.hu>
2009-10-23 19:35 ` [RFC] to rebase or not to rebase on liunx-next Steven Rostedt
2009-10-23 20:37 ` [RFC] to rebase or not to rebase on linux-next Ingo Molnar
2009-10-23 20:54 ` Ingo Molnar
2009-10-23 21:59 ` Sam Ravnborg
2009-10-26 23:26 ` Steven Rostedt
2009-10-26 23:30 ` David Miller
2009-10-26 23:51 ` Steven Rostedt
2009-10-27 0:15 ` David Miller
2009-10-27 0:30 ` Steven Rostedt
2009-10-27 1:34 ` David Miller
2009-10-27 3:02 ` Steven Rostedt
2009-10-27 4:56 ` Theodore Tso
2009-10-27 5:18 ` David Miller
2009-10-27 8:13 ` Ingo Molnar
2009-10-27 9:48 ` Catalin Marinas
2009-10-27 7:59 ` Ingo Molnar
2009-10-27 15:39 ` Steven Rostedt
2009-10-27 17:18 ` Nicolas Pitre
2009-10-28 15:15 ` Steven Rostedt
2009-10-28 6:40 ` Ingo Molnar [this message]
2009-10-24 8:03 ` Theodore Tso
2009-10-24 12:20 ` Stefan Richter
2009-10-24 19:43 ` Sam Ravnborg
2009-10-27 19:06 ` Linus Torvalds
2009-10-27 19:27 ` Steven Rostedt
2009-10-27 19:39 ` Valdis.Kletnieks
2009-10-27 19:43 ` Steven Rostedt
2009-10-27 19:42 ` Linus Torvalds
2009-10-26 4:53 ` Luck, Tony
2009-10-26 5:21 ` Theodore Tso
2009-10-26 18:01 ` Stefan Richter
2009-10-27 18:07 ` Theodore Tso
2009-10-27 18:54 ` Stefan Richter
2009-10-28 7:26 ` Ingo Molnar
2009-10-24 12:51 ` Stefan Richter
[not found] ` <4AE21839.3000000@s5r6.in-berlin.de>
2009-10-23 21:03 ` [kernel.org users] Please remember to run 'git gc' on your repositories Steven Rostedt
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=20091028064009.GA6353@elte.hu \
--to=mingo@elte.hu \
--cc=davem@davemloft.net \
--cc=dmitry.torokhov@gmail.com \
--cc=jeff@garzik.org \
--cc=khali@linux-fr.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mcgrof@gmail.com \
--cc=nico@fluxnic.net \
--cc=robert.richter@amd.com \
--cc=rostedt@goodmis.org \
--cc=sam@ravnborg.org \
--cc=sfr@canb.auug.org.au \
--cc=tony.luck@intel.com \
--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