[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: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 22/42] configure: factor out list of supported Xen/KVM/HAX targets |
Date: |
Mon, 10 Jul 2017 17:24:16 +0100 |
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.
thanks
-- PMM
- [Qemu-devel] [PULL 12/42] hw/misc/edu: Compile the edu device as common object, (continued)
- [Qemu-devel] [PULL 12/42] hw/misc/edu: Compile the edu device as common object, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 17/42] main_loop: Make main_loop_wait() return void, Paolo Bonzini, 2017/07/05
- [Qemu-devel] [PULL 10/42] include/exec/poison: Mark CONFIG_SOFTMMU as poisoned, Paolo Bonzini, 2017/07/05
- [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