From: Jan Kiszka <jan.kiszka@web.de>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: gleb@redhat.com, kvm@vger.kernel.org, mtosatti@redhat.com,
qemu-devel@nongnu.org, avi@redhat.com,
Anthony Liguori <anthony@codemonkey.ws>
Subject: Re: [PATCH for-1.2 0/2] migrate PV EOI MSR
Date: Mon, 27 Aug 2012 08:20:19 +0200 [thread overview]
Message-ID: <503B11A3.8060901@web.de> (raw)
In-Reply-To: <cover.1345996760.git.mst@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 639 bytes --]
On 2012-08-26 17:59, Michael S. Tsirkin wrote:
> It turns out PV EOI gets disabled after migration -
> until next guest reset.
> This is because we are missing code to actually migrate it.
> This patch fixes it up: it does not do anything useful
> without kvm irqchip but applies cleanly to qemu.git
> as well as qemu-kvm.git, so I think it's cleaner
> to apply it in qemu.git to keep diff to minimum.
There is nothing except pci-assign left in qemu-kvm (which will be
posted for upstream in a minute), so you are intuitively doing the right
thing.
Patch 2 looks good to me, see patch 1 for the clean procedure.
Jan
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Jan Kiszka <jan.kiszka@web.de>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: gleb@redhat.com, kvm@vger.kernel.org, mtosatti@redhat.com,
qemu-devel@nongnu.org, avi@redhat.com,
Anthony Liguori <anthony@codemonkey.ws>
Subject: Re: [Qemu-devel] [PATCH for-1.2 0/2] migrate PV EOI MSR
Date: Mon, 27 Aug 2012 08:20:19 +0200 [thread overview]
Message-ID: <503B11A3.8060901@web.de> (raw)
In-Reply-To: <cover.1345996760.git.mst@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 639 bytes --]
On 2012-08-26 17:59, Michael S. Tsirkin wrote:
> It turns out PV EOI gets disabled after migration -
> until next guest reset.
> This is because we are missing code to actually migrate it.
> This patch fixes it up: it does not do anything useful
> without kvm irqchip but applies cleanly to qemu.git
> as well as qemu-kvm.git, so I think it's cleaner
> to apply it in qemu.git to keep diff to minimum.
There is nothing except pci-assign left in qemu-kvm (which will be
posted for upstream in a minute), so you are intuitively doing the right
thing.
Patch 2 looks good to me, see patch 1 for the clean procedure.
Jan
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]
next prev parent reply other threads:[~2012-08-27 6:20 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-26 15:59 [PATCH for-1.2 0/2] migrate PV EOI MSR Michael S. Tsirkin
2012-08-26 15:59 ` [Qemu-devel] " Michael S. Tsirkin
2012-08-26 15:59 ` [PATCH for-1.2 1/2] linux-headers: update asm/kvm_para.h to 3.6 Michael S. Tsirkin
2012-08-26 15:59 ` [Qemu-devel] " Michael S. Tsirkin
2012-08-27 6:22 ` Jan Kiszka
2012-08-27 6:22 ` [Qemu-devel] " Jan Kiszka
2012-08-26 15:59 ` [PATCH for-1.2 2/2] kvm: get/set PV EOI MSR Michael S. Tsirkin
2012-08-26 15:59 ` [Qemu-devel] " Michael S. Tsirkin
2012-08-27 6:20 ` Jan Kiszka [this message]
2012-08-27 6:20 ` [Qemu-devel] [PATCH for-1.2 0/2] migrate " 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=503B11A3.8060901@web.de \
--to=jan.kiszka@web.de \
--cc=anthony@codemonkey.ws \
--cc=avi@redhat.com \
--cc=gleb@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mst@redhat.com \
--cc=mtosatti@redhat.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.