qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: Kevin Wolf <kwolf@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Markus Armbruster <armbru@redhat.com>,
	Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>,
	qemu-devel <qemu-devel@nongnu.org>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] Re: [0.14] Queue of 0.14 patches/pull?
Date: Fri, 28 Jan 2011 14:33:36 +0100	[thread overview]
Message-ID: <4D42C5B0.6000007@redhat.com> (raw)
In-Reply-To: <4D4293D6.2080802@msgid.tls.msk.ru>

On 01/28/2011 11:00 AM, Michael Tokarev wrote:
> 28.01.2011 11:06, Yoshiaki Tamura wrote:
> []
>> I would like Kemari to be included for 0.14.  Thanks to many
>> valuable comments from various reviewers, it got better than
>> ever, I believe.  For those who may object, it wouldn't affect
>> any functions including live migration unless one turns it on.
>> Besides, it is a good application of live migration.  By having
>> merged, I would be able to focus on increasing functionality and
>> optimization.
>
> It's somewhat unexpected to see merge request for any new
> functionality into a "stable" branch.  Having nothing bad
> against Kemari, I still think we shouldn't include any new
> functionality into 0.14, which were planned to be released
> before new year... ;)
>
> I mean, if it were not merged so far, for reasons that can
> be discussed separately etc, regardless of its good shape
> and so on... let's don't do any last-minute merges.

To be fair to Yoshiaki, the first 17 patches in his 19-patch series have 
no impact on the operation of QEMU, and even the last two are 
Kemari-only.  I know almost nothing about Kemari so I cannot comment on 
the technical side of those patches, only that it w^Hshouldn't be 
destabilizing.  I made some comments on a couple of patches, so I'm 
inclined to say no as well.

http://permalink.gmane.org/gmane.comp.emulators.qemu/91096 should be 
applied in any case, as it is a regression from 0.12.

Paolo

  reply	other threads:[~2011-01-28 13:33 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-27  8:29 [Qemu-devel] [0.14] Queue of 0.14 patches/pull? Paolo Bonzini
2011-01-27 10:26 ` [Qemu-devel] [PULL 0.14] spice + usb Gerd Hoffmann
2011-01-27 12:34 ` [Qemu-devel] [0.14] Queue of 0.14 patches/pull? Markus Armbruster
2011-01-27 12:42   ` Kevin Wolf
2011-01-27 15:13   ` Markus Armbruster
2011-01-27 16:30 ` Anthony Liguori
2011-01-27 16:44   ` [Qemu-devel] [PING 0.14] fix crash when a read handler calls qemu_set_fd_handler2() Rick Vernam
2011-01-27 17:14     ` Anthony Liguori
2011-01-27 19:50       ` Stefan Hajnoczi
2011-01-28  0:38         ` Anthony Liguori
2011-01-27 16:51   ` [Qemu-devel] [PING 0.14] seabios virtio bug Rick Vernam
2011-01-27 17:12     ` [Qemu-devel] " Paolo Bonzini
2011-01-27 17:30       ` Gleb Natapov
2011-01-28  7:51   ` [Qemu-devel] [0.14] Queue of 0.14 patches/pull? Gleb Natapov
2011-01-27 18:22 ` [Qemu-devel] " Michael S. Tsirkin
2011-01-28  8:06   ` Yoshiaki Tamura
2011-01-28 10:00     ` Michael Tokarev
2011-01-28 13:33       ` Paolo Bonzini [this message]
2011-01-28 14:21         ` Yoshiaki Tamura
2011-02-02  9:31           ` Paolo Bonzini
2011-02-03  4:44             ` Yoshiaki Tamura
2011-01-28 16:48         ` Anthony Liguori
2011-01-29 13:43           ` Yoshiaki Tamura
2011-01-28  6:43 ` [Qemu-devel] " Amit Shah
2011-02-02 19:28 ` [Qemu-devel] [PING 0.14] Missing patches (mostly fixes) Stefan Weil
2011-02-02 21:25   ` [Qemu-devel] " Michael S. Tsirkin
2011-02-03  7:00   ` Riku Voipio
2011-02-03 10:05   ` [Qemu-devel] " Markus Armbruster
2011-02-04 12:30   ` Anthony Liguori
2011-02-04 12:37     ` Paolo Bonzini
2011-02-04 14:40     ` Alexander Graf
2011-02-04 15:27     ` Markus Armbruster
2011-02-04 15:38       ` Anthony Liguori
2011-02-04 16:42       ` Luiz Capitulino
2011-02-04 17:18       ` Stefan Weil
2011-02-04 17:21         ` Anthony Liguori
2011-02-04 17:36           ` Stefan Weil
2011-02-04 17:59             ` Anthony Liguori
2011-02-07 18:54             ` Luiz Capitulino
2011-02-07 19:18               ` Stefan Weil
2011-02-16 20:26       ` Stefan Weil
2011-02-16 20:39         ` Markus Armbruster
2011-02-04 15:45     ` malc
2011-02-05 19:23     ` Michael S. Tsirkin
2011-02-05 19:34     ` riku voipio
2011-02-23 21:50     ` Stefan Weil

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=4D42C5B0.6000007@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=armbru@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mjt@tls.msk.ru \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=tamura.yoshiaki@lab.ntt.co.jp \
    /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).