[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 7/7] qxl: add allocator
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [RFC 7/7] qxl: add allocator |
Date: |
Mon, 20 Feb 2012 14:18:19 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120210 Thunderbird/10.0.1 |
Hi,
> I'll send a series that works with vnc+spice and sdl+spice (didn't test
> sdl+vnc+spice), and screendumps at the same time.
>
>>
>> Setting the QEMU_ALLOCATED_FLAG flag sounds hackish too.
QEMU_ALLOCATED_FLAG is just a flag which is used by the
defaultallocator_free_displaysurface function to figure whenever the
displaysurface memory was allocated by qemu_alloc_display or not.
I think vga.c shouldn't look at it in the first place, and faking it
because vga.c looks at it is even worse.
I'm also not sure it is the right approach to to have qxl register a
display allocator in the first place. The only other place doing this
is sdl, which is a user interface. None of the gfx card emulations in
the tree do that ...
> Actually I think the right thing is to move/copy the 24bit->32bit convertion
> from vga.c to pflib.c, what do you think?
Agree, although that easily gets a patch series of its own when you
collect optimized format conversion functions to move them over to pflib ...
BTW: qxl insisting on a shared displaysurface isn't very clean too, it
better should be able to fallback to just copying/converting for the
non-shared case.
cheers,
Gerd
- Re: [Qemu-devel] [RFC 3/7] qxl: introduce QXLCookie, (continued)
[Qemu-devel] [RFC 4/7] qxl: make qxl_render_update async, Alon Levy, 2012/02/19
[Qemu-devel] [RFC 6/7] qxl: use spice_qxl_update_area_dirty_async, Alon Levy, 2012/02/19
[Qemu-devel] [RFC 7/7] qxl: add allocator, Alon Levy, 2012/02/19
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Gerd Hoffmann, 2012/02/20
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Alon Levy, 2012/02/20
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator,
Gerd Hoffmann <=
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Alon Levy, 2012/02/20
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Gerd Hoffmann, 2012/02/21
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Alon Levy, 2012/02/21
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Gerd Hoffmann, 2012/02/21
- Re: [Qemu-devel] [RFC 7/7] qxl: add allocator, Alon Levy, 2012/02/21
[Qemu-devel] [RFC 5/7] qxl-render: call ppm_save on callback, Alon Levy, 2012/02/19