Re: [PATCH] kvm: disable stealtime via reboot notifier to avoid mem corruption

2012-08-14 Thread Marcelo Tosatti
On Tue, Aug 14, 2012 at 11:15:06AM +0200, Florian Westphal wrote: > Marcelo Tosatti wrote: > > On Fri, Aug 10, 2012 at 12:36:22PM +0200, Florian Westphal wrote: > > > --- a/arch/x86/kernel/kvmclock.c > > > +++ b/arch/x86/kernel/kvmclock.c > > > @@ -191,7 +191,6 @@ static void kvm_crash_shutdown(st

Re: [PATCH] kvm: disable stealtime via reboot notifier to avoid mem corruption

2012-08-14 Thread Florian Westphal
Marcelo Tosatti wrote: > On Fri, Aug 10, 2012 at 12:36:22PM +0200, Florian Westphal wrote: > > --- a/arch/x86/kernel/kvmclock.c > > +++ b/arch/x86/kernel/kvmclock.c > > @@ -191,7 +191,6 @@ static void kvm_crash_shutdown(struct pt_regs *regs) > > static void kvm_shutdown(void) > > { > > nativ

Re: [PATCH] kvm: disable stealtime via reboot notifier to avoid mem corruption

2012-08-13 Thread Marcelo Tosatti
On Fri, Aug 10, 2012 at 12:36:22PM +0200, Florian Westphal wrote: > else, we get memory corruption on reboot; found when tracking down > initramfs unpack error on initial reboot (with qemu-kvm -smp 2, > no problem with single-core). > > problem with doing it via kvm_shutdown() is that this file >

[PATCH] kvm: disable stealtime via reboot notifier to avoid mem corruption

2012-08-10 Thread Florian Westphal
else, we get memory corruption on reboot; found when tracking down initramfs unpack error on initial reboot (with qemu-kvm -smp 2, no problem with single-core). problem with doing it via kvm_shutdown() is that this file depends on CONFIG_KVM_CLOCK, also its not enough to call it for one cpu only.