Re: [Qemu-devel] [PATCH 07/11] kvm/x86: added hyper-v crash data and ctl msr's get/set'ers

[Denis V. Lunev]

On 23/06/15 12:51, Paolo Bonzini wrote:
> On 23/06/2015 11:47, Denis V. Lunev wrote:
> > > The current implementation is potentially more flexible but makes the
> > > MSR handling a little more awkward since the host_initiated bool needs
> > > to be passed around (patch 09). I guess either approach seems ok to
> > > me.
> > >
> > > Also, if this patchset is used then it looks like
> > > HV_X64_MSR_CRASH_CTL_CONTENTS can be removed.
> > Paolo,
> >
> > do you have any opinion on this? I prefer
> > configurable way but there is no problem
> > to enable it by default dropping 'hv_panic'
> > property.
> I also prefer the configurable way.  Would hv_panic prevent
> reboot-on-BSOD from rebooting the VM?
>
> Paolo
the behavior becomes controlled by libvirt, which has
appropriate event configuration, specified here
  https://libvirt.org/formatdomain.html
namely
  <on_crash>restart</on_crash>

Thus it looks like without libvirt help the guest will stuck
in RUN_STATE_GUEST_PANICKED state.

Den