All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Sander Eikelenboom <linux@eikelenboom.it>
Cc: Anthony Perard <anthony.perard@citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [Qemu-devel] xen-unstable qemu-upstream: pci-passthrough timing issues due to commit 0f7b2864d0d0c3ef2801f9214d8c510c80a220d1 when vnc enabled
Date: Mon, 22 Apr 2013 13:04:55 +0200	[thread overview]
Message-ID: <51751957.2000009@redhat.com> (raw)
In-Reply-To: <1574647612.20130422121053@eikelenboom.it>

On 04/22/13 12:10, Sander Eikelenboom wrote:
> 
> Monday, April 22, 2013, 12:01:02 PM, you wrote:
> 
>> Are you sure that the bug is related to Xen PCI passthrough (it
>> doesn't happen if you don't assign any devices to the Xen guest)? I
>> am asking because the Xen PCI passthrough code doesn't use any
>> timers, while this looks like something related to timer and
>> refresh intervals...

> Ok latency issue would perhaps be a better description. I think
> something in this change make it worse, hence resulting in the driver
> of the passthroughed pci device reporting that it's buffer was empty
> (4 times) and then the buffer is filled up with 4 frames where it
> expected only 1.

irq latencies probably.  Although ... is qemu involved in the irq
delivery path in the first place?

How frequent are these latency spikes?

> Anyhow i noticed that running with "nographic" dramatically reduces
> the overhead of the qemu process in dom0 compared to running with
> graphics. Which seems strange since it's a console only linux guest
> and no vnc clients are connected ...

The gui update rate is adaptive and ranges from .03 seconds minimum to 3
seconds maximum.  With an active guest you'll see the refresh interval
close to the lowest limit.  With an idle guest (doing no screen updates)
the update rate goes down step by step until it reaches 3 seconds after
a while.

Enable the "console_refresh" tracepoint and you should see the logic at
work.

With no vnc client connected it should always stay at the maximum (3
seconds).  An vnc screen update every three seconds should not cause a
dramatic change, especially as there is next to nothing to do when the
guest doesn't update the screen.

cheers,
  Gerd

  parent reply	other threads:[~2013-04-22 11:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-20 21:48 [Qemu-devel] xen-unstable qemu-upstream: pci-passthrough timing issues due to commit 0f7b2864d0d0c3ef2801f9214d8c510c80a220d1 when vnc enabled Sander Eikelenboom
2013-04-22 10:01 ` Stefano Stabellini
2013-04-22 10:10   ` Sander Eikelenboom
2013-04-22 10:10   ` [Qemu-devel] " Sander Eikelenboom
2013-04-22 11:04     ` Gerd Hoffmann
2013-04-22 11:04     ` Gerd Hoffmann [this message]
2013-04-22 10:01 ` Stefano Stabellini

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=51751957.2000009@redhat.com \
    --to=kraxel@redhat.com \
    --cc=anthony.perard@citrix.com \
    --cc=linux@eikelenboom.it \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xen.org \
    /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.