[SeaBIOS] [PATCH] don't expose pvpanic device in the UI
Michael S. Tsirkin
mst at redhat.com
Mon Aug 5 10:10:55 CEST 2013
On Mon, Aug 05, 2013 at 03:47:23PM +0800, Hu Tao wrote:
> pvpanic device is an internal default device in qemu. It may cause
> problem when upgrading qemu from a version without pvpanic.
>
> for example: in Windows(let's say XP) the Device manager will open a
> "new device" wizard and the device will appear as an unrecognized
> device. On a cluster with hundreds of such VMs, If that cluster has
> a health monitoring service it may show all the VMs in a "not healthy"
> state.
>
> This patch is a workaround to not show pvpanic in UI to avoid the
> problem in Windows.
>
> Cc: Marcel Apfelbaum <marcel.a at redhat.com>
> Cc: "Michael S. Tsirkin" <mst at redhat.com>
> Cc: Paolo Bonzini <pbonzini at redhat.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Eric Blake <eblake at redhat.com>
> Cc: "Daniel P. Berrange" <berrange at redhat.com>
> Cc: Andreas Färber <afaerber at suse.de>
> Signed-off-by: Hu Tao <hutao at cn.fujitsu.com>
Quoting from this discussion:
>That may "fix" the issue of a windows guest showing the yellow ! mark,
>but what if, down the road, someone writes an actual windows driver that
>is aware of that port and how to make a windows BSOD write a panic
>notification to the port? How does a user go about installing such a
>driver if the device is not exposed in the user interface list of
>devices?
I think the correct way to address this is:
- don't create the device by default, only when -device pvpanic is
present
- teach management to supply said -device pvpanic for guests which
support the pvpanic device
> ---
>
> See discussion at: http://lists.nongnu.org/archive/html/qemu-devel/2013-08/msg00036.html
> and follow-ups.
>
> src/ssdt-misc.dsl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/ssdt-misc.dsl b/src/ssdt-misc.dsl
> index acc850e..19a2058 100644
> --- a/src/ssdt-misc.dsl
> +++ b/src/ssdt-misc.dsl
> @@ -75,7 +75,7 @@ DefinitionBlock ("ssdt-misc.aml", "SSDT", 0x01, "BXPC", "BXSSDTSUSP", 0x1)
> If (LEqual(Local0, Zero)) {
> Return (0x00)
> } Else {
> - Return (0x0F)
> + Return (0x0B)
> }
> }
>
> --
> 1.8.1.4
>
More information about the SeaBIOS
mailing list