Re: [Qemu-devel] [PATCH for-1.6 V2 0/2] pvpanic: Separate pvpanic from machine type

[Paolo Bonzini]

Il 21/08/2013 12:16, Michael S. Tsirkin ha scritto:
> > 
> > By the way, this means two things:
> > 
> > - I am now sold on the idea that explicitly enabling of pvpanic is the
> > right thing to do;
> > 
> > - on the other hand this is the proof that the change was not fully
> > understood, and rushing it in 1.6 was the wrong thing to do.
> 
> You mean 1.5.
> pvpanic was a builtin in 1.5 and that was clearly the wrong thing to do.
> We fixed that in 1.6, thankfully.

No, even taking it out in 1.6 is a mistake.  It happened too close to
the release.

Now we have 1.5 which has it, 1.6 which doesn't have it and requires
reset after panic, 1.7 which doesn't have it and in all likelihood will
not require reset after panic.

Paolo