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
next prev parent 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 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).