qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Amit Shah <amit.shah@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Bernhard Kauer <kauer@os.inf.tu-dresden.de>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] [RESEND2] Qemu unmaintained?
Date: Thu, 10 Sep 2009 20:45:56 +0530	[thread overview]
Message-ID: <20090910151556.GC28534@amit-x200.redhat.com> (raw)
In-Reply-To: <4AA90945.3000709@codemonkey.ws>

On (Thu) Sep 10 2009 [09:12:21], Anthony Liguori wrote:
>
> Yeah, I tried that without a lot of success.  Basically, my work flow is  
> the following:
>
> 1) Mail client filters things that look like patches into a folder
> 2) I do a once through to remove obvious dups or non-patches
> 3) I have some scripts that pull patches into an mbox format.  They are  
> smart about identifying threads and trying to find a sane sort order.
> 4) I have more scripts that let me split the mbox into smaller mboxes.   
> This is because git am is not very forgiving and having to do a git am  
> --abort can lead to a lot of wasted effort if the mbox has 100+ patches
> 5) I git am the smaller mboxes into staging
> 6) I push to staging
> 7) I do a build check, and fix errors, and push to staging
> 8) I start the testing cycle, removing patches based on testing results  
> and pushing to staging
> 9) I manually review each patch that's survived testing
> 10) I do final build/test, then push to master

Obviously there's a lot of manual intervention and can get you
overloaded. But if some developer knows that his patch is in some queue
or getting some consideration, he'll rest easier than to check if his
patch has made it either to master or to some staging queue somewhere.
If it can't be automated, it sadly has to drop back to manual acking.

> And commit mails are sent whenever something goes to master.  So someone  
> should get a mail when a patch goes to master or when I reject it.  To  
> me, the area to optimize is reducing the time things are spent in  
> staging.  I don't think people really need visibility into staging.

Absolutely. I think the trend has been anywhere from 2 to 4 weeks for
patches landing on the list to the master tree. Also, this results in
delays for important patches to get to the master tree because they're
delayed behind a huge pile of other patches. Avi has mentioned this
before but I don't know if there was any plan to address it.

		Amit

  reply	other threads:[~2009-09-10 15:16 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-02  7:49 [Qemu-devel] [PATCH] [RESEND] RTC polling mode broken Bernhard Kauer
2009-09-09 12:18 ` [Qemu-devel] [PATCH] [RESEND2] Qemu unmaintained? Bernhard Kauer
2009-09-09 12:23   ` Amit Shah
2009-09-09 12:58     ` Bernhard Kauer
2009-09-09 12:25   ` Kevin Wolf
2009-09-09 13:00   ` Anthony Liguori
2009-09-09 13:26     ` Gleb Natapov
2009-09-09 13:45       ` Anthony Liguori
2009-09-09 14:31       ` Avi Kivity
2009-09-09 13:34     ` Bernhard Kauer
2009-09-10  7:03     ` Amit Shah
2009-09-10  7:56       ` Reimar Döffinger
2009-09-10 10:08         ` Amit Shah
2009-09-10 11:47           ` Luiz Capitulino
2009-09-10 12:01             ` [Qemu-devel] QEMU patch management Amit Shah
2009-09-10 12:29               ` [Qemu-devel] " Luiz Capitulino
2009-09-10 12:51                 ` Reimar Döffinger
2009-09-10 13:11                   ` Luiz Capitulino
2009-09-10 17:24                     ` Reimar Döffinger
2009-09-10 13:58             ` [Qemu-devel] [PATCH] [RESEND2] Qemu unmaintained? Anthony Liguori
2009-09-10 13:56       ` Anthony Liguori
2009-09-10 14:04         ` Amit Shah
2009-09-10 14:12           ` Anthony Liguori
2009-09-10 15:15             ` Amit Shah [this message]
2009-09-10 14:38         ` Avi Kivity
2009-09-10 15:54           ` Anthony Liguori
2009-09-10 16:09             ` Avi Kivity
2009-09-10 16:22               ` Anthony Liguori
2009-09-10 16:35                 ` Avi Kivity
2009-09-10 16:38                   ` Anthony Liguori
2009-09-10 16:46                     ` Avi Kivity
2009-09-10 17:19                       ` Reimar Döffinger
2009-09-11 12:39                         ` Amit Shah
2009-09-12  5:55                         ` Blue Swirl
2009-09-13 15:44                           ` Avi Kivity
2009-09-13 16:30                             ` Blue Swirl
2009-09-11  7:06                     ` Gerd Hoffmann
2009-09-10 18:29                   ` Mark McLoughlin
2009-09-10 18:40                     ` Avi Kivity
2009-09-10 19:31                       ` Anthony Liguori
2009-09-13 15:49                         ` Avi Kivity
2009-09-10 20:36                       ` Mark McLoughlin
2009-09-13 16:19                         ` Avi Kivity
2009-09-14  7:49                           ` Mark McLoughlin
2009-09-14  7:59                             ` Avi Kivity
2009-09-10 18:59                     ` Reimar Döffinger
2009-09-10 16:05           ` Anthony Liguori
2009-09-10 16:14             ` Avi Kivity
2009-09-11  9:16           ` [Qemu-devel] commit e09a5267 (was: [PATCH] [RESEND2] Qemu unmaintained?) Jan Kiszka
2009-09-11 12:56             ` [Qemu-devel] Re: commit e09a5267 Anthony Liguori
2009-09-11 13:04               ` Jan Kiszka

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=20090910151556.GC28534@amit-x200.redhat.com \
    --to=amit.shah@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=kauer@os.inf.tu-dresden.de \
    --cc=qemu-devel@nongnu.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;
as well as URLs for NNTP newsgroup(s).