[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option |
Date: |
Mon, 3 Jul 2017 13:09:45 +0100 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Mon, Jul 03, 2017 at 12:55:04PM +0200, Paolo Bonzini wrote:
>
>
> On 03/07/2017 12:33, Daniel P. Berrange wrote:
> >> i386-softmmu and x86_64-softmmu are singled out here, because they're
> >> the only targets where --disable-tcg compiles. For the others, more
> >> work is needed (see patches 6-15 in Yang Zhong's series).
> >
> > Even with that, you still can't disable TCG if building on a non-x86
> > host, since that'd leave you with no available CPU at all. So the
> > code still needs refactoring to check architectures properly.
>
> It would leave you with a tools-only build; whether that's a good idea,
> it's another story. I think it's acceptable, but others may disagree.
If you do 'configure --target-list=x86_64-softmmu --disable-tcg' on a
non-x86 host I would be surprised to find x86_64-softmmu not built.
IMHO if you explicitly request system emulators and its not posisble
to build then, an error should be report, rather than silently ignoring
the request.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [Qemu-devel] [PATCH v2 00/15] add disable-tcg option for x86 build, Yang Zhong, 2017/07/03
- [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Yang Zhong, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Daniel P. Berrange, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Paolo Bonzini, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Daniel P. Berrange, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Paolo Bonzini, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Thomas Huth, 2017/07/03
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option,
Daniel P. Berrange <=
- Re: [Qemu-devel] [PATCH v2 01/15] configure: add the disable-tcg option, Paolo Bonzini, 2017/07/03
[Qemu-devel] [PATCH v2 02/15] vl: add tcg_enabled() for tcg related code, Yang Zhong, 2017/07/03
[Qemu-devel] [PATCH v2 03/15] tcg: tcg_handle_interrupt() function, Yang Zhong, 2017/07/03
[Qemu-devel] [PATCH v2 04/15] tcg: change tcg_enabled(), Yang Zhong, 2017/07/03
[Qemu-devel] [PATCH v2 05/15] tcg: move page_size_init() function, Yang Zhong, 2017/07/03
[Qemu-devel] [PATCH v2 07/15] tcg: move cpu_sync_bndcs_hflags() function, Yang Zhong, 2017/07/03
[Qemu-devel] [PATCH v2 06/15] kvmvapic: remove tcg related code, Yang Zhong, 2017/07/03