From: Thomas Huth <thuth@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>,
"Daniel P. Berrange" <berrange@redhat.com>
Cc: "Laurent Vivier" <lvivier@redhat.com>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Michael S . Tsirkin" <mst@redhat.com>,
"Stefano Stabellini" <stefano.stabellini@eu.citrix.com>,
qemu-devel@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>
Subject: Re: [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator
Date: Wed, 5 Oct 2016 09:00:37 +0200 [thread overview]
Message-ID: <37e541f9-9daa-f8a2-f172-d3ea37fbe36a@redhat.com> (raw)
In-Reply-To: <8035787f-e9ed-f493-6575-ce5282e9afe1@redhat.com>
On 04.10.2016 19:16, Paolo Bonzini wrote:
>
>
> On 04/10/2016 16:18, Thomas Huth wrote:
>>>>>> Using only tcg has also some disadvantages: For some tests, it's
>>>>>> interesting to know whether they also work properly with KVM (e.g.
>>>>>> migration tests), and only using tcg by default slows down the "make
>>>>>> check" quite a bit - which might become an issue now that we're adding
>>>>>> more and more tests.
>>>>
>>>> Which tests are you seeing a slow-down for ?
>> Well, everything that is using accel=tcg in tests/ could be accelerated.
>> For example, the new ipv6/ppc64 unit test is quite slow with TCG:
>>
>> sudo QTEST_QEMU_BINARY=ppc64-softmmu/qemu-system-ppc64 time tests/pxe-test
>>
>> 48.46user 0.07system 0:48.48elapsed
>
> Could that point to a firmware bug? 10 network-bound seconds for a boot
> makes some sense, but 10 CPU-bound seconds don't...
SLOF is incredibly slow with TCG - most parts are written in Forth that
gets interpreted during runtime, and that seems to perform quite badly
with TCG for some reasons. We're in progress of speeding up the boot
process of SLOF a little bit (see
https://github.com/aik/SLOF/commit/b3fde41bc75269df2 for example), but
it will likely always be slower than a firmware that has been written in
C only.
Thomas
next prev parent reply other threads:[~2016-10-05 7:00 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-04 11:30 [Qemu-devel] [PATCH 0/2] introduce "accel=best" Laurent Vivier
2016-10-04 11:30 ` [Qemu-devel] [PATCH 1/2] accel: allows to select the "best" accelerator Laurent Vivier
2016-10-04 12:00 ` Daniel P. Berrange
2016-10-04 12:29 ` Laurent Vivier
2016-10-04 12:32 ` Peter Maydell
2016-10-04 12:41 ` Paolo Bonzini
2016-10-04 12:49 ` Thomas Huth
2016-10-04 13:31 ` Daniel P. Berrange
2016-10-04 14:18 ` Thomas Huth
2016-10-04 17:16 ` Paolo Bonzini
2016-10-05 1:13 ` Michael S. Tsirkin
2016-10-05 7:21 ` Paolo Bonzini
2016-10-05 7:00 ` Thomas Huth [this message]
2016-10-05 7:48 ` Paolo Bonzini
2016-10-04 11:30 ` [Qemu-devel] [PATCH 2/2] tests: use accel=best instead of accel=kvm:tcg Laurent Vivier
2016-10-09 20:50 ` Michael S. Tsirkin
2016-10-04 11:35 ` [Qemu-devel] [PATCH 0/2] introduce "accel=best" Laurent Vivier
2016-10-04 12:00 ` Andrew Jones
2016-10-04 12:07 ` Daniel P. Berrange
2016-10-04 12:26 ` Andrew Jones
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=37e541f9-9daa-f8a2-f172-d3ea37fbe36a@redhat.com \
--to=thuth@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=berrange@redhat.com \
--cc=lvivier@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
/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).