All of lore.kernel.org
 help / color / mirror / Atom feed
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 0/3] Target CPU=Host implementation for KVM ARM/ARM64
Date: Fri, 6 Sep 2013 11:52:40 +0100	[thread overview]
Message-ID: <20130906105240.GD30450@mudshark.cambridge.arm.com> (raw)
In-Reply-To: <CAAhSdy3cUA7hh-Q6dL+=AowfVnzPEVxTR3CyXhUbo1W1fq=KSw@mail.gmail.com>

On Fri, Sep 06, 2013 at 11:51:07AM +0100, Anup Patel wrote:
> On Fri, Sep 6, 2013 at 4:19 PM, Will Deacon <will.deacon@arm.com> wrote:
> > On Fri, Sep 06, 2013 at 11:09:09AM +0100, Anup Patel wrote:
> >> On Fri, Sep 6, 2013 at 2:36 PM, Will Deacon <will.deacon@arm.com> wrote:
> >> > On Fri, Sep 06, 2013 at 09:54:23AM +0100, Alexander Graf wrote:
> >> >> On 06.09.2013, at 09:44, Anup Patel wrote:
> >> >> > it hard to come up with a use case where having a separate ioctl for
> >> >> > "get best CPU for this host" would be better for user space.
> >> >>
> >> >> It can store it and migrate it as part of its migration protocol (probably hard for QEMU's current work flow, but that's QEMU's issue).
> >> >
> >> > It's also better from the point of view of devicetree generation. For
> >> > example, kvmtool needs to generate the correct architected timer interrupt
> >> > numbers for the target CPU, so munging this into KVM_ARM_VCPU_INIT means we
> >> > have to go off and figure out the host CPU separately anyway.
> >>
> >> Please look at the patch carefully we are returning VCPU
> >> target type back to user space so, you can generate correct
> >> architected timer interrupt numbers for the target CPU.
> >
> > I did look at the patch, but I also looked at the overriding consensus in
> > the feedback saying that you can't change the direction of the ioctl, so
> > that would require what I said above.
> >
> > Will
> 
> Instead of arguing more, I'll save everyone's time and revise this
> patch as needed by everyone.

Cheers :)

Will

  reply	other threads:[~2013-09-06 10:52 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-05 14:45 [RFC PATCH 0/3] Target CPU=Host implementation for KVM ARM/ARM64 Anup Patel
2013-09-05 14:46 ` [RFC PATCH 1/3] ARM: KVM: Implement target CPU=Host Anup Patel
2013-09-06  8:08   ` Claudio Fontana
2013-09-05 14:46 ` [RFC PATCH 2/3] ARM64: " Anup Patel
2013-09-06  8:09   ` Claudio Fontana
2013-09-05 14:46 ` [RFC PATCH 3/3] KVM: ARM: Update documentation for KVM_ARM_VCPU_INIT ioctl Anup Patel
2013-09-06  8:05   ` Claudio Fontana
2013-09-06 10:13     ` Anup Patel
2013-09-05 14:51 ` [RFC PATCH 0/3] Target CPU=Host implementation for KVM ARM/ARM64 Peter Maydell
2013-09-06  7:44   ` Anup Patel
2013-09-06  8:54     ` Alexander Graf
2013-09-06  9:06       ` Will Deacon
2013-09-06 10:09         ` Anup Patel
2013-09-06 10:49           ` Will Deacon
2013-09-06 10:51             ` Anup Patel
2013-09-06 10:52               ` Will Deacon [this message]
2013-09-06 10:05       ` Anup Patel
2013-09-06 10:24         ` Alexander Graf
2013-09-06 10:34           ` Marc Zyngier
2013-09-06 10:38             ` Anup Patel
2013-09-06 10:34           ` Anup Patel

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=20130906105240.GD30450@mudshark.cambridge.arm.com \
    --to=will.deacon@arm.com \
    --cc=linux-arm-kernel@lists.infradead.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.