[SeaBIOS] [PATCH] don't expose pvpanic device in the UI

Vadim Rozenfeld vrozenfe at redhat.com
Tue Aug 6 10:03:17 CEST 2013



----- Original Message -----
From: "Gleb Natapov" <gleb at redhat.com>
To: "Michael S. Tsirkin" <mst at redhat.com>
Cc: "Daniel P. Berrange" <berrange at redhat.com>, "Marcel Apfelbaum" <marcel.a at redhat.com>, seabios at seabios.org, qemu-devel at nongnu.org, "Gerd Hoffmann" <kraxel at redhat.com>, "Paolo Bonzini" <pbonzini at redhat.com>, "Eric Blake" <eblake at redhat.com>, "Andreas Färber" <afaerber at suse.de>
Sent: Tuesday, August 6, 2013 5:34:06 PM
Subject: Re: [SeaBIOS] [PATCH] don't expose pvpanic device in the UI

On Mon, Aug 05, 2013 at 09:32:18PM +0300, Michael S. Tsirkin wrote:
> > > As you see we do let people change many parameters
> > > that do affect activation.
> > By editing XML user can shoot himself in the foot, we should not prevent
> > that.
> 
> So that's what I'm saying basically.
> At the moment there's no way to remove this device from XML.
> That's just wrong.
Can say the same about PV acpi hotpulg device.

> In QEMU, we have a standard way to specify devices with -device.
> That should be the interface for anything new really
> unless there's a very compelling reason for something else.
We are disagree on compelling reason in this case obviously.

> *Not* building it into the PC machine type.
> 
> > It should not be required though.
> 
> libvirt can pass -device pvpanic by default if nothing
> is specified in XML. That discussion really has to happen
> on libvirt list.
> 
As Paolo said you are just pushing the "problem" up the stack
where it is harder to "solve". I put "problem" and "solve" in
quotes because I disagree that the problem that need to be solved
is identified correctly. The correct problem to be solved IMO is
writing Windows driver for the device.

[VR]
This one shouldn't be too complicated. Can be done on weekend.
--
			Gleb.

_______________________________________________
SeaBIOS mailing list
SeaBIOS at seabios.org
http://www.seabios.org/mailman/listinfo/seabios



More information about the SeaBIOS mailing list