Re: [Qemu-devel] [PATCH 3/3] pvpanic: rename to isa-pvpanic

[Michael S. Tsirkin]

On Wed, Aug 21, 2013 at 07:01:39PM +0200, Paolo Bonzini wrote:
> Il 21/08/2013 19:01, Michael S. Tsirkin ha scritto:
> >> > The pvpanic situation is already messed up enough.  Let us give our
> >> > libvirt friends an easy indication that we have untied our side.
> >> > 
> >> > Not-yet-signed-off-by: Paolo Bonzini <address@hidden>
> >> > ... because we first have to determine how to expose the device's 
> >> > existence
> >> >     in the ACPI tables or in fw_cfg.
> >> > 
> >> > Signed-off-by: Paolo Bonzini <address@hidden>
> > So it's isa-pvpanic meaning "I-am-sure-this-is-the-last-bug-pvpanic"
> 
> More like "we tested pvpanic for more than 2 weeks and did not find
> anything that's utterly broken in the design".
> 
> And more practically "you are sure there are no traces of builtin
> pvpanic; also, panicked state is reversible".
> 
> Paolo

isa-pvpanic does not look like a sane way to say that.

NACK

> > If we feel there's need to give libvirt a way to do
> > introspection into QEMU bugs, let's architect one.
> > Randomly renaming devices in the vain hope it's
> > the last major bug is not it.