Re: [Qemu-devel] git bisect results

[Jan Kiszka <jan.kiszka@siemens.com>]

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?

> 
> 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.

> 
> 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

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux