From: Thomas Huth <thuth@redhat.com>
To: "Paolo Bonzini" <pbonzini@redhat.com>,
"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
qemu-devel@nongnu.org, "Richard Henderson" <rth@twiddle.net>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Emilio G . Cota" <cota@braap.org>,
"Stefan Weil" <sw@weilnetz.de>,
"Peter Maydell" <peter.maydell@linaro.org>
Subject: Re: [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter)
Date: Thu, 29 Jun 2017 18:02:25 +0200 [thread overview]
Message-ID: <e4d97293-8cdd-eda6-ec86-e2460f61837e@redhat.com> (raw)
In-Reply-To: <db5c43ba-10b1-9fce-30e6-c40368ecc46c@redhat.com>
On 29.06.2017 17:47, Paolo Bonzini wrote:
>
>
> On 29/06/2017 17:46, Thomas Huth wrote:
>>> TCI is enabled only with a specific configure argument if your machine
>>> is not supported by TCG. This would break _build_ configurations, not
>>> user configurations. It's a remote possibility that users are building
>>> their own QEMU, with TCI enabled, to work around a TCG bug. So we can
>>> be more speedy in removing the code.
>> You never know ... it's unlikely, but there might be people around who
>> run configure with "--enable-tcg-interpreter" on purpose.
>
> And they have never reported a bug? :) But I agree it's not a big deal.
> Richard should decide.
>
> Paolo
>
>> And why the
>> hurry for removing this? It's been around in the current shape since
>> years, so waiting for two more releases does not hurt, does it?
The various BSD folks also decided to rather maintain their patches
downstream instead of trying hard to get them included in the upstream
repository... now that we've got a deprecation warning in the configure
script, things fortunately start to change :-)
Likely nobody is maintaining downstream TCI patches ... but we don't
really now for sure - it could be the same situation as with BSD. So we
should give the users a chance to speak up first before we remove it.
Thomas
next prev parent reply other threads:[~2017-06-29 16:02 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-29 1:02 [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter) Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [PATCH 1/8] MAINTAINERS: update TCG entries Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [PATCH 2/8] MAINTAINERS: update KVM entries Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [PATCH 3/8] MAINTAINERS: update Xen entry Philippe Mathieu-Daudé
2017-06-29 10:39 ` Anthony PERARD
2017-06-29 11:54 ` Philippe Mathieu-Daudé
2017-06-29 11:47 ` Paolo Bonzini
2017-06-29 1:02 ` [Qemu-devel] [PATCH 4/8] MAINTAINERS: update TCI entry Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [PATCH RESEND 5/8] tcg/tci: enable bswap16_i64 Philippe Mathieu-Daudé
2017-06-29 16:29 ` Eric Blake
2017-06-29 16:41 ` Philippe Mathieu-Daudé
2017-06-29 16:52 ` Stefan Weil
2017-06-29 17:02 ` Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [PATCH 6/8] tcg/tci: disable MTTCG if TCI is enabled Philippe Mathieu-Daudé
2017-07-26 1:28 ` Philippe Mathieu-Daudé
2017-07-26 8:02 ` Alex Bennée
2017-08-03 17:40 ` Philippe Mathieu-Daudé
2017-08-03 17:56 ` Richard Henderson
2017-08-03 18:04 ` Philippe Mathieu-Daudé
2017-06-29 1:02 ` [Qemu-devel] [RFC PATCH 7/8] tcg/tci: time to remove it :( Philippe Mathieu-Daudé
2017-06-29 8:36 ` Daniel P. Berrange
2017-06-29 1:03 ` [Qemu-devel] [RFC PATCH 8/8] travis: remove tcg/tci job Philippe Mathieu-Daudé
2017-06-29 8:46 ` [Qemu-devel] [RFC PATCH 0/8] removal of tci (tcg interpreter) Thomas Huth
2017-06-29 14:57 ` Philippe Mathieu-Daudé
2017-06-29 15:06 ` Paolo Bonzini
2017-06-29 15:46 ` Thomas Huth
2017-06-29 15:47 ` Paolo Bonzini
2017-06-29 16:02 ` Thomas Huth [this message]
2017-06-29 16:18 ` Philippe Mathieu-Daudé
2017-06-29 16:45 ` Stefan Weil
2017-06-29 16:59 ` Peter Maydell
2017-06-29 16:07 ` Daniel P. Berrange
2017-06-29 16:13 ` Peter Maydell
2017-06-29 16:17 ` Thomas Huth
2017-06-29 18:09 ` Stefan Weil
2017-06-30 9:24 ` Peter Maydell
2017-07-06 23:49 ` no-reply
2017-07-07 0:04 ` Fam Zheng
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=e4d97293-8cdd-eda6-ec86-e2460f61837e@redhat.com \
--to=thuth@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=cota@braap.org \
--cc=f4bug@amsat.org \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=sw@weilnetz.de \
/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).