RE: [Xen-devel] [PATCH 3/3] Add shadow VRAM

Dugger, Donald D
Thu, 16 Mar 2006 22:00:22 -0000

Keir-

I''ve sent out two new patches (1a and 3a) that address this concern.
I''ve removed the check for SSE2 at compile time for the VGA patch so
that patch no longer touches the `configure'' script.

Unfortunately, the `configure'' script still needs to be touched for VNC
mouse support.  The current version of `libvncserver.a'' checks for 1
input event and then sleeps in its main loop.  The sleep is short enough
that you don''t notice anything on the keyboard but this causes
noticable
latency for the mouse.  I''ve talked Johannes Schindelin into adding a
new field to the VNC control structuce that modifies this behavior to
check for all input events and then go to sleep.  Since it will be a
while before this version of `libvncserver.a'' is generally available
the
`qemu-dm'' build procedure has to check for this new field and only set
it if it''s available.

All the other feature of these paches remain the same and, I should
point out, with the Summagraphics Tablet changes it''s now possible to
run Windows on a VMX guest under VNC.

--
Don Dugger
"Censeo Toto nos in Kansa esse decisse." - D. Gale
Donald.D.Dugger@intel.com
Ph: (303)440-1368

>-----Original Message-----
>From: Keir Fraser [mailto:Keir.Fraser@cl.cam.ac.uk]
>Sent: Wednesday, March 15, 2006 4:14 PM
>To: Dugger, Donald D
>Cc: xen-devel@lists.xensource.com
>Subject: Re: [Xen-devel] [PATCH 3/3] Add shadow VRAM
>
>
>On 15 Mar 2006, at 23:13, Dugger, Donald D wrote:
>
>> Nice try but I thought of that :-)  The code checks at run time also,
>> for just the reason you state.  Also, I did measure and the runtime
>> check adds no measurable overhead.
>
>Can you get rid of the compile-time checks from your patch then, since
>they''re redundant.
>
>I think that, although we''d like to track qemu''s source
files, we''ll
>avoid their build system as much as possible. So I''d
>definitely like to
>avoid adding stuff into their configuration scripts.