Re: [Qemu-devel][BUG][PATCH] Fix crash in kvm.c

[Anthony Liguori <anthony@codemonkey.ws>]

Stefan Weil wrote:
> I got a crash (array access out of bounds results in access fault)
> with the current Qemu trunk when kvm is enabled:
>
> qemu -fda fd.img -cdrom cdrom.img -hda hda.img -hdb raw.img -m 256 -boot
> c -enable-kvm
>
> Host is Debian x86_64, the crash occurs before any code is emulated.
>   

Is the patch incomplete, perhaps?  It seems to just add asserts which 
shouldn't fix anything.

I don't think any leaf should return a max leaf greater than 100 
elements so I'd be pretty surprised to see this happen.  I'd really like 
to see the back trace to see which leaf is the problematic one and what 
the greatest leaf being reported is.

> With the patch, the emulation (Win 98) boots, but has problems with the
> display of
> icons and the mouse cursor. Qemu displays lots of
> "BUG: kvm_physical_sync_dirty_bitmap: invalid parameters" messages.
>   

It looks like Avi and Glauber found some bugs in this code.  There are 
patches for kvm-userspace right now that need porting to QEMU.  I'll 
take a look at that this weekend unless Glauber was already planning too.

Regards,

Anthony Liguori

> Stefan
>
>