Re: [Qemu-devel] [PATCH 1/3] qxl: add fb_size_mb and fb_size

[Alon Levy]

On Mon, Jun 11, 2012 at 12:18:14PM +0200, Gerd Hoffmann wrote:
> On 06/11/12 11:33, Alon Levy wrote:
> > On Mon, Jun 11, 2012 at 10:29:59AM +0200, Gerd Hoffmann wrote:
> >>   Hi,
> >>
> >>>> Reminds me that I have some pending work to make vga ram size
> >>>> configurable which I should finish and repost ...
> >>>>
> >>>> Current state pushed to http://www.kraxel.org/cgit/qemu/log/?h=vga.1
> >>>>
> >>>> It probably makes sense to base this on top.
> >>>
> >>> So I can rebase on top and send the patches, with the understanding you
> >>> will do any later changes and repost the whole thing.
> >>
> >> Yes, makes sense to put them into one series I think.
> > 
> > git://people.freedesktop.org/~alon/qemu fb_async_fix.v2
> 
> http://www.kraxel.org/cgit/qemu/log/?h=vga.2
> 
> Did some small fixups [no need for the -1 dance, the reason the other
> properties do that is that we have one accepting byte values and one
> accepting megabyte values].
> 
> looks ok to you?

Looks like you squashed and removed the -1 (you're right of course, it
was just me doing a straight copy), other then that I didn't notice
anything. Looks good.

> 
> cheers,
>   Gerd