Re: [PATCH] fix screen corruption bug in vga_draw_graphic()

[Avi Kivity <avi@qumranet.com>]

Andreas Winkelbauer wrote:
> hi,
>
> the attached patch fixes the screen corruption issues which were 
> reported by others, see:
>
> http://article.gmane.org/gmane.comp.emulators.kvm.devel/13543
> http://article.gmane.org/gmane.comp.emulators.kvm.devel/13409
>
> The bug is kvm specific and can only be observed in graphics mode 
> using relatively high resolutions (when one line uses more than one 
> page of memory). As far as I've seen this bug is around since commit 
> dd9591e0fea25a1414f4a6b2faa61ed733e0acc6 (5 nov 2006).
>
> I've attached two versions of the patch. One just changes the relevant 
> line and the other one also cleans up formatting (indention) of the 
> kvm specific code.

Applied the no-formatting version, thanks.  Please sign off on patches 
next time.

The way to apply formatting changes while fixing something is with a 
separate patch that does nothing but fix the formatting, either on top 
of the fix, or to be applied before the fix.  This way the fixing patch 
is kept small and easily understandable.


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/