qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Erik Rull <erik.rull@rdsoftware.de>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] git bisect results
Date: Tue, 24 Jan 2012 19:55:28 +0100	[thread overview]
Message-ID: <4F1EFEA0.2090708@rdsoftware.de> (raw)
In-Reply-To: <4F1EF639.8000108@siemens.com>

Jan Kiszka wrote:
> On 2012-01-24 18:24, Erik Rull wrote:
>> Hi all,
>>
>> I assume that I found a possible source of the bad usbtablet update rate.
>>
>> I did some git bisectioning but I didn't get a usable result due to too
>> many merges (or maybe my little knowledge to git), so I proceeded with some
>> manual bisectioning by manually selecting commits and tested them.
>>
>> The problem was that the usbtablet update-rate in qemu-kvm-1.0 was really
>> bad compared to qemu-kvm-0.15.0.
>
> Did you bisect qemu or qemu-kvm?

qemu-kvm.

>> The first commit where the cursor worked but the update rate was bad is
>> 7cb78eec5cdf6e4131613c64cc1a29476f327242
>>
>> Before this commit the usbtablet didn't work (no grabbing was possible).
>
> That's a merge. Can you dig into the merged patches to find the one that
> resolved the grabbing issue? Might be 21635e121a. Then that can be
> backported while bisecting the tree for the other issue.

How can I do that (digging into the merge)? I'm not too familiar with git, 
I see only the whole merge as one commit. Or do you mean digging manually 
in the single diffs that are in the patch and try each of them?
How does the backport work? If I change something during bisecting, git 
complains that it cannot proceed due to changed files.

>> But in 0.15.0 it worked. So I continued searching for interesting points
>> between these versions.
>>
>> One point was the SDL commit done 2011-08-05 by Jan Kiszka.
>> There I found changes that affected the -full-screen option.
>>
>> So I took the version from the commit above and just started with the
>> -full-screen option.
>> And everything was fine (qemu-kvm-1.0 as well)! The update rate is very
>> good with this option.
>
> If you go full-screen there is constant grabbing. But I do not see the
> correlation with the update rate when in windowed mode.
>
>>
>> But I was not able to find the real change that caused the bad update rate.
>>
>> Jan - is it possible to track down with this information the cause of the
>> bad update rate? It seems to be related to SDL - the VNC option is working
>> fine without -full-screen.
>> I would like to work without the -full-screen option.
>
> I still cannot follow, specifically as I have XP with tablet running
> here. Haven't noticed any problems so far, just rechecked against
> qemu-kvm head.
>
> Jan
>

Which CPU do you have on your host? I have a Core2Duo T6800 and the guest 
running on one core. There the update rate is significant worse than on 
another system with a Core i7 (guest on a single core as well), on the 
faster system it is still visible. Maybe its related to the onboard 
graphics? I don't know, I just see the significant differences between 
fullscreen and windowed mode.

Thanks for your help.

Best regards,

Erik

  reply	other threads:[~2012-01-24 18:55 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-23  8:57 [Qemu-devel] bad USB tablet update rate on qemu-1.0 erik.rull
2012-01-24 17:24 ` [Qemu-devel] git bisect results (was: Re: bad USB tablet update rate on qemu-1.0) Erik Rull
2012-01-24 18:19   ` [Qemu-devel] git bisect results Jan Kiszka
2012-01-24 18:55     ` Erik Rull [this message]
2012-01-24 20:15       ` Jan Kiszka
  -- strict thread matches above, loose matches on Subject: below --
2012-01-25 11:48 erik.rull
2012-01-25 14:19 ` Jan Kiszka
2012-01-25 21:13   ` Erik Rull
2012-01-26 13:10 Erik Rull
2012-01-26 16:24 ` Jan Kiszka
2012-01-27 22:52 ` Jan Kiszka
2012-01-28 11:55   ` Jan Kiszka
2012-01-28 12:39     ` Erik Rull
2012-01-28 12:43       ` Jan Kiszka
2012-01-28 13:01         ` Erik Rull
2012-01-28 14:52           ` Jan Kiszka
2012-01-30 11:34 Erik Rull
2012-01-30 11:52 ` Jan Kiszka
2012-01-30 13:17   ` Erik Rull
2012-01-30 13:48     ` Jan Kiszka
2012-01-30 14:17       ` Erik Rull
2012-01-30 14:48         ` Jan Kiszka
2012-01-31  8:31           ` Erik Rull

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=4F1EFEA0.2090708@rdsoftware.de \
    --to=erik.rull@rdsoftware.de \
    --cc=jan.kiszka@siemens.com \
    --cc=qemu-devel@nongnu.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 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).