[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xe
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xen/KVM/HAX targets |
Date: |
Fri, 14 Jul 2017 12:58:03 -0400 (EDT) |
----- Original Message -----
> From: "Philippe Mathieu-Daudé" <address@hidden>
> To: "Paolo Bonzini" <address@hidden>, "Peter Maydell" <address@hidden>
> Cc: "Anthony PERARD" <address@hidden>, "Stefano Stabellini" <address@hidden>,
> "Alex Bennée"
> <address@hidden>, "QEMU Developers" <address@hidden>
> Sent: Friday, July 14, 2017 5:26:36 PM
> Subject: Re: [Qemu-devel] [PULL 22/42] configure: factor out list of
> supported Xen/KVM/HAX targets
>
> what about this RFC series?
> http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg02513.html
Too late for 2.10, but the way to go is to define a new "xenpv" target.
In fact, --disable-tcg was the prerequisite for that if I remember correctly
my old discussions with Citrix folks (could it have been Wei Liu?)
Paolo
> On 07/10/2017 01:28 PM, Paolo Bonzini wrote:
> > On 10/07/2017 18:24, Peter Maydell wrote:
> >> On 10 July 2017 at 17:14, Paolo Bonzini <address@hidden> wrote:
> >>> On 10/07/2017 17:49, Peter Maydell wrote:
> >>>> On 5 July 2017 at 08:14, Paolo Bonzini <address@hidden> wrote:
> >>>>> This will be useful when the functions are called, early in the
> >>>>> configure
> >>>>> process, to filter out targets that do not support hardware
> >>>>> acceleration.
> >>>>>
> >>>>> Signed-off-by: Paolo Bonzini <address@hidden>
> >>>>
> >>>>> +supported_xen_target() {
> >>>>> + test "$xen" = "yes" || return 1
> >>>>> + glob "$1" "*-softmmu" || return 1
> >>>>> + case "${1%-softmmu}:$cpu" in
> >>>>> + arm:arm | aarch64:aarch64 | \
> >>>>> + i386:i386 | i386:x86_64 | x86_64:i386 | x86_64:x86_64)
> >>>>> + return 0
> >>>>> + ;;
> >>>>
> >>>> This says that arm-on-arm and aarch64-on-aarch64 are supported
> >>>> Xen targets...
> >>>
> >>> Hmm, this comes from my old patches. IIRC the reason for the change,
> >>> when it wasn't a change (many conflicts ago) was that Xen folks were
> >>> using --disable-tcg because their device model for Xen PV on ARM was
> >>> actually an x86_64 QEMU.
> >>>
> >>> Stefano and Anthony, is this still true? If so, would it make sense to
> >>> add the Xen PV machine type to qemu-system-arm---that is, is it
> >>> something you can whip up easily, or should I just remove that line?
> >>
> >> I think you should just fix configure for the moment, because
> >> this patch wasn't supposed to change anything about what we
> >> build (AIUI). We can think about changing the Xen PV on ARM
> >> build setup as a separate thing if we want to, I suspect it
> >> is more invasive than a couple of lines changing in configure.
> >
> > Yes, definitely more invasive.
> >
> > I'll prepare a fix.
> >
> > Paolo
> >
>
- [Qemu-devel] [PULL 18/42] checkpatch: should not use signal except for SIG_DFL or SIG_IGN, (continued)
- [Qemu-devel] [PULL 18/42] checkpatch: should not use signal except for SIG_DFL or SIG_IGN, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 14/42] xsave_helper: pull xsave and xrstor out of kvm.c into helper function, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 16/42] tests/test-char.c: Don't use main_loop_wait()'s return value, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 19/42] virtio-scsi: finalize IOMMU support, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xen/KVM/HAX targets, Paolo Bonzini, 2017/07/05
- Re: [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xen/KVM/HAX targets, Stefano Stabellini, 2017/07/11
- Re: [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xen/KVM/HAX targets, Anthony PERARD, 2017/07/10
[Qemu-devel] [PULL 24/42] configure: add --disable-tcg configure option, Paolo Bonzini, 2017/07/05
[Qemu-devel] [PULL 26/42] vl: add tcg_enabled() for tcg related code, Paolo Bonzini, 2017/07/05
[Qemu-devel] [PULL 21/42] qemu-doc: do not refer to years-old version numbers, Paolo Bonzini, 2017/07/05
[Qemu-devel] [PULL 23/42] configure: early test for supported targets, Paolo Bonzini, 2017/07/05
[Qemu-devel] [PULL 20/42] qemu-thread: Assert locks are initialized before using, Paolo Bonzini, 2017/07/05