All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoffer Dall <christoffer.dall@linaro.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Anup Patel <anup.patel@linaro.org>,
	patches@apm.com, Alexander Graf <agraf@suse.de>,
	qemu-devel <qemu-devel@nongnu.org>,
	"kvmarm@lists.cs.columbia.edu" <kvmarm@lists.cs.columbia.edu>,
	Pranavkumar Sawargaonkar <pranavkumar@linaro.org>
Subject: Re: [Qemu-devel] [RFC PATCH 5/7] target-arm: Enable KVM_ARM_VCPU_PSCI_0_2 feature when possible
Date: Mon, 17 Mar 2014 15:48:40 -0700	[thread overview]
Message-ID: <20140317224840.GB3692@cbox> (raw)
In-Reply-To: <CAFEAcA-kfbjVkX1eM8oF8t+utffD6g41VP9OH4mS-XPm3vSPqg@mail.gmail.com>

On Mon, Mar 17, 2014 at 07:35:43PM +0000, Peter Maydell wrote:
> [another resend for qemu-devel...]
> 
> On 27 February 2014 07:16, Pranavkumar Sawargaonkar
> <pranavkumar@linaro.org> wrote:
> > On 27 February 2014 12:21, Pranavkumar Sawargaonkar
> > <pranavkumar@linaro.org> wrote:
> >> Latest linux kernel supports in-kernel emulation of PSCI v0.2 but
> >> to enable it we need to select KVM_ARM_VCPU_PSCI_0_2 feature using
> >> KVM_ARM_VCPU_INIT ioctl.
> >>
> >> Also, we can use KVM_ARM_VCPU_PSCI_0_2 feature for VCPU only when
> >> linux kernel has KVM_CAP_ARM_PSCI_0_2 capability.
> >>
> >> This patch updates kvm_arch_init_vcpu() to enable KVM_ARM_VCPU_PSCI_0_2
> >> feature for VCPU when KVM ARM/ARM64 has KVM_CAP_ARM_PSCI_0_2 capability.
> 
> I think this has implications for migration. What happens
> if you migrate a VM from a host system where the host
> kernel supports this capability to one where it doesn't?
> (Presumably the answer is "it won't work" and we need
> to ensure we don't allow that.)
> Conversely, is it OK to migrate a guest from a host
> which didn't support PSCI 0.2 to one which does? Does
> that work even if the destination has PSCI 0.2 enabled
> for the vCPUs, or do we need to support starting a
> VM with 0.2 disabled even on 0.2-capable hosts, for
> use as a migration target?

The init feature should be a switch, turn 0.2 on or off, if off, we're
emulating 0.1.  If 0.2 is on, 0.1 call numbers should generate errors.

You're right, we need to fix this for migration.

> 
> Also we still don't seem to have done anything about
> migration of the PSCI CPU powerup/down state.
> 

We need to create a card for this in Linaro.  Thought I would deal with
it when working through migration for ARMv8.

Thanks!
-Christoffer

  reply	other threads:[~2014-03-17 22:48 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1393483868-6151-1-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:11 ` [Qemu-devel] [RFC PATCH 0/7] PSCI v0.2 support for KVM ARM/ARM64 Pranavkumar Sawargaonkar
     [not found] ` <1393483868-6151-2-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:12   ` [Qemu-devel] [RFC PATCH 1/7] linux-headers: Update KVM headers from v3.14-rc3 Pranavkumar Sawargaonkar
     [not found] ` <1393483868-6151-3-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:13   ` [Qemu-devel] [RFC PATCH 2/7] target-arm: Try X-Gene Potenza target type for KVM Aarch64 Pranavkumar Sawargaonkar
     [not found] ` <1393483868-6151-4-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:14   ` [Qemu-devel] [RFC PATCH 3/7] target-arm: Handle exit reason KVM_EXIT_SYSTEM_EVENT Pranavkumar Sawargaonkar
     [not found]   ` <20140314034348.GE24808@cbox>
2014-03-17 19:34     ` Peter Maydell
     [not found] ` <1393483868-6151-5-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:15   ` [Qemu-devel] [RFC PATCH 4/7] target-arm: Provide PSCI v0.2 constants to generic QEMU code Pranavkumar Sawargaonkar
     [not found] ` <1393483868-6151-6-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:16   ` [Qemu-devel] [RFC PATCH 5/7] target-arm: Enable KVM_ARM_VCPU_PSCI_0_2 feature when possible Pranavkumar Sawargaonkar
2014-03-17 19:35     ` Peter Maydell
2014-03-17 22:48       ` Christoffer Dall [this message]
     [not found] ` <1393483868-6151-7-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:16   ` [Qemu-devel] [RFC PATCH 6/7] hw/arm/virt: Use PSCI v0.2 function IDs when kernel supports its Pranavkumar Sawargaonkar
     [not found]   ` <20140314034926.GF24808@cbox>
2014-03-14  6:53     ` Pranavkumar Sawargaonkar
2014-03-14 10:23       ` Mark Rutland
2014-03-14 17:10         ` Rob Herring
2014-03-17 10:09           ` Mark Rutland
     [not found] ` <1393483868-6151-8-git-send-email-pranavkumar@linaro.org>
2014-02-27  7:17   ` [Qemu-devel] [RFC PATCH 7/7] target-arm: Hacky implementation of kvm_arch_reset_vcpu() for KVM ARM64 Pranavkumar Sawargaonkar
     [not found]   ` <CAFEAcA9VcpVS_FJKYruCaWB=jr9unkor1aAzAAJX_GLC0LBouw@mail.gmail.com>
2014-02-27 10:35     ` Pranavkumar Sawargaonkar
2014-02-27 11:26       ` Peter Maydell
2014-03-14  3:51       ` Christoffer Dall
2014-03-14  6:58         ` Pranavkumar Sawargaonkar

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=20140317224840.GB3692@cbox \
    --to=christoffer.dall@linaro.org \
    --cc=agraf@suse.de \
    --cc=anup.patel@linaro.org \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=patches@apm.com \
    --cc=peter.maydell@linaro.org \
    --cc=pranavkumar@linaro.org \
    --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.