All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: Greg Kurz <groug@kaod.org>
Cc: qemu-devel@nongnu.org, Laurent Vivier <lvivier@redhat.com>,
	qemu-ppc@nongnu.org, Cedric Le Goater <clg@kaod.org>
Subject: Re: [Qemu-devel] [PATCH v5 3/3] spapr: fix migration of ICPState objects from/to older QEMU
Date: Wed, 14 Jun 2017 22:39:14 +0800	[thread overview]
Message-ID: <20170614143914.GB2614@umbus> (raw)
In-Reply-To: <149744695984.5507.11604157257247801492.stgit@bahia.lan>

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

On Wed, Jun 14, 2017 at 03:29:19PM +0200, Greg Kurz wrote:
> Commit 5bc8d26de20c ("spapr: allocate the ICPState object from under
> sPAPRCPUCore") moved ICPState objects from the machine to CPU cores.
> This is an improvement since we no longer allocate ICPState objects
> that will never be used. But it has the side-effect of breaking
> migration of older machine types from older QEMU versions.
> 
> This patch allows spapr to register dummy "icp/server" entries to vmstate.
> These entries use a dedicated VMStateDescription that can swallow and
> discard state of an incoming migration stream, and that don't send anything
> on outgoing migration.
> 
> As for real ICPState objects, the instance_id is the cpu_index of the
> corresponding vCPU, which happens to be equal to the generated instance_id
> of older machine types.
> 
> The machine can unregister/register these entries when CPUs are dynamically
> plugged/unplugged.
> 
> This is only available for pseries-2.9 and older machines, thanks to a
> compat property.
> 
> Signed-off-by: Greg Kurz <groug@kaod.org>

Applied to ppc-for-2.10.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

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

  reply	other threads:[~2017-06-14 14:44 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-14 13:28 [Qemu-devel] [PATCH v5 0/3] spapr/xics: fix migration of older machine types Greg Kurz
2017-06-14 13:29 ` [Qemu-devel] [PATCH v5 1/3] spapr: manage hotplugged devices while the VM is not started Greg Kurz
2017-06-14 13:29 ` [Qemu-devel] [PATCH v5 2/3] xics: directly register ICPState objects to vmstate Greg Kurz
2017-06-14 14:04   ` Laurent Vivier
2017-06-14 14:09     ` Greg Kurz
2017-06-14 14:30   ` David Gibson
2017-06-14 13:29 ` [Qemu-devel] [PATCH v5 3/3] spapr: fix migration of ICPState objects from/to older QEMU Greg Kurz
2017-06-14 14:39   ` David Gibson [this message]
2017-06-14 13:48 ` [Qemu-devel] [PATCH v5 0/3] spapr/xics: fix migration of older machine types no-reply
2017-06-14 14:41 ` David Gibson
2017-06-14 15:19   ` Greg Kurz
2017-06-15  6:39 ` Cédric Le Goater

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=20170614143914.GB2614@umbus \
    --to=david@gibson.dropbear.id.au \
    --cc=clg@kaod.org \
    --cc=groug@kaod.org \
    --cc=lvivier@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@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.