All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: "Thomas Huth" <thuth@redhat.com>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	"Laurent Vivier" <lvivier@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"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 04:13:19 +0300	[thread overview]
Message-ID: <20161005041245-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <8035787f-e9ed-f493-6575-ce5282e9afe1@redhat.com>

On Tue, Oct 04, 2016 at 07:16:49PM +0200, 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...
> 
> Paolo

AFAIK pxe firmware typically can't handle interrupts
so it polls for packets.

-- 
MST

  reply	other threads:[~2016-10-05  1:13 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 [this message]
2016-10-05  7:21               ` Paolo Bonzini
2016-10-05  7:00             ` Thomas Huth
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=20161005041245-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=berrange@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=thuth@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.