[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump
From: |
andrzej zaborowski |
Subject: |
Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump |
Date: |
Sun, 20 Jul 2008 21:28:55 +0200 |
2008/7/20 Anthony Liguori <address@hidden>:
> andrzej zaborowski wrote:
>>
>> 2008/7/20 Anthony Liguori <address@hidden>:
>> With -no-graphic there's no ds->data, or it can have lower bpp than
>> what's otherwise available.
>>
>
> Screen dumping with -no-graphic probably should just show a black screen
> anyway.
I thought this was the main use case. You shouldn't need sdl installed
to perform the tests Avi talked about.
>
> I don't think a lower depth is really that big of a deal. It's what the
> user would be seeing anyway. The simplicity seems really nice to me.
>
>>> I think your patch looks better, but I can also code up a new screen
>>> dumping
>>> mechanism that doesn't take over DisplayState unless I'm missing
>>> something
>>> obvious.
>>>
>>
>> It's worth a try, dumping ds->data is perhaps the way to go but
>> ds->depth is being set to 32bit and various video cards are optimised
>> on the assumption that ds->depth doesn't change, so maybe there should
>> be a notification callback.
>>
>
> Yeah, that's a bug BTW and AFAIK it's only a problem with VMware VGA.
AFAIK also at least three other adapters.
> There's nothing that keeps ds->depth from changing after a screen resize
> with SDL. If a VNC client does SetPixelFormat that will also change
> ds->depth. Right now, some VNC clients + VMware VGA will cause a SEGV
> because of the assumptions that ds->depth doesn't change.
>
> Regards,
>
> Anthony Liguori
>
>> Regards
>>
>>
>>
>
>
>
>
- [Qemu-devel] [PATCH 1/6] kvm: qemu: fix screendump with multiple consoles, Anthony Liguori, 2008/07/17
- [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/17
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, andrzej zaborowski, 2008/07/19
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/19
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, andrzej zaborowski, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump,
andrzej zaborowski <=
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, andrzej zaborowski, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Anthony Liguori, 2008/07/20
- Re: [Qemu-devel] [PATCH 6/6] kvm: qemu: fix vga screendump, Jamie Lokier, 2008/07/20
[Qemu-devel] [PATCH 5/6] kvm: qemu: improve scsi dma speed by increasing the dma buffer size, Anthony Liguori, 2008/07/17
[Qemu-devel] [PATCH 4/6] kvm: qemu: revert ""QEMU keyboard issue with Gujin-2.2", by Etienne Lorrain.", Anthony Liguori, 2008/07/17