* Avi Kivity <[EMAIL PROTECTED]> wrote:
> Ingo Molnar wrote:
>> * Avi Kivity <[EMAIL PROTECTED]> wrote:
>>
>>
>>> Enabling Intel VT has the curious side effect whereby the INIT signal
>>> is blocked. Rather than comment on the wisdom of this side effect,
>>> this patch adds an emergency restart reboot notifier, and modifies
>>> the kvm reboot notifier to disable VT on emergency reboot.
>>>
>>
>> looks good to me - i was bitten by that problem on a testbox.
>>
>
> I'm a little worried about making emergency restart more complex.
>
> Another thing that worries me is that emergency_restart() doesn't
> reset the box -- it sends INIT. We could do better by using the ACPI
> FADT reset register (hopefully that's connected to RESET).
reboot was always a bit fragile - i think we should only do that if we
find a box where the FADT reset works better than the first-wave
approaches we try.
> Which seems to be what we want? Maybe we should just try acpi_reboot()
> before the other stuff.
perhaps in a separate commit, for v2.6.28 at the earliest.
Ingo
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
