[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter)
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter) |
Date: |
Thu, 29 Jun 2017 10:46:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
Hi Philippe,
On 29.06.2017 03:02, Philippe Mathieu-Daudé wrote:
> There have been some comments on the ML about the usefulness of tci.
>
> https://lists.nongnu.org/archive/html/qemu-devel/2017-06/msg04551.html
>
> Peter Maydell> I'd prefer we just got rid of it.
>
> https://lists.nongnu.org/archive/html/qemu-devel/2017-06/msg04296.html
>
> Richard Henderson> Is it time to remove it? I'm pretty sure the only hosts
> for which it will work have proper backends...
>
> Richard quotes are way clearer than me trying to paraphrase what he told me:
> - it doesn't use libffi, and as such the way it makes calls to helpers
> doesn't work for many hosts.
> - we already cover almost everthing that debian does. if debian or gentoo
> doesn't support it, one can confidently say there's little interest.
> - if someone *does* want to run qemu on something else, it isn't difficult to
> port tcg.
>
> I figured out MAINTAINERS was unsync, so added patches 1-4, they are not
> really
> tci-related.
Since they are not related to TCI at all, please submit these as
separate series.
> Patches 5,6 are trivial fixes to let the codebase quite sane if there is
> future
> need to revert/reimport tci.
I think this should go into 2.10...
> Patches 7,8 are the removal, marked RFC... let's debate!
... but NACK for a direct removal. Common sense is to make obsolete
features as deprecated first and then wait for 2 public releases before
the final removal, so that users still have a chance to speak up in case
they still need the feature and are willing to maintain it.
Please see the following URL for details (and please also add an entry
for TCI in the Miscellaneous section there):
http://wiki.qemu.org/Features/LegacyRemoval
Thomas
- Re: [Qemu-devel] [PATCH 3/8] MAINTAINERS: update Xen entry, (continued)
- [Qemu-devel] [PATCH 4/8] MAINTAINERS: update TCI entry, Philippe Mathieu-Daudé, 2017/06/28
- [Qemu-devel] [PATCH RESEND 5/8] tcg/tci: enable bswap16_i64, Philippe Mathieu-Daudé, 2017/06/28
- [Qemu-devel] [PATCH 6/8] tcg/tci: disable MTTCG if TCI is enabled, Philippe Mathieu-Daudé, 2017/06/28
- [Qemu-devel] [RFC PATCH 8/8] travis: remove tcg/tci job, Philippe Mathieu-Daudé, 2017/06/28
- [Qemu-devel] [RFC PATCH 7/8] tcg/tci: time to remove it :(, Philippe Mathieu-Daudé, 2017/06/28
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter),
Thomas Huth <=
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Philippe Mathieu-Daudé, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Paolo Bonzini, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Thomas Huth, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Paolo Bonzini, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Thomas Huth, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Philippe Mathieu-Daudé, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Stefan Weil, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Peter Maydell, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Daniel P. Berrange, 2017/06/29
- Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter), Peter Maydell, 2017/06/29