lkml.org 
[lkml]   [2007]   [Mar]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch] KVM: T60 resume fix
On Sat, 03 Mar 2007 10:21:38 +0200 Avi Kivity <avi@qumranet.com> wrote:

> Ingo Molnar wrote:
> > Subject: [patch] KVM: T60 resume fix
> > From: Ingo Molnar <mingo@elte.hu>
> >
> > my T60 laptop does not resume correctly due to KVM attempting to send an
> > IPI to a CPU that might be down (or not up yet). (Doing so also triggers
> > the send_IPI_mask_bitmask() warning in arch/i386/kernel/smp.c, line
> > 732.)
> >
> > with this fix applied my laptop does not hang during resume.
> >
> > [ KVM will have to disable/enable virtualization on the CPU itself that
> > goes down / comes up, not via an IPI sent from the requesting CPU. ]
> >
> > Signed-off-by: Ingo Molnar <mingo@elte.hu>
> > ---
> > drivers/kvm/kvm_main.c | 6 ------
> > 1 file changed, 6 deletions(-)
> >
> > Index: linux/drivers/kvm/kvm_main.c
> > ===================================================================
> > --- linux.orig/drivers/kvm/kvm_main.c
> > +++ linux/drivers/kvm/kvm_main.c
> > @@ -2083,12 +2083,6 @@ static int kvm_cpu_hotplug(struct notifi
> > case CPU_DEAD:
> > case CPU_UP_CANCELED:
> > decache_vcpus_on_cpu(cpu);
> > - smp_call_function_single(cpu, kvm_arch_ops->hardware_disable,
> > - NULL, 0, 1);
> > - break;
> > - case CPU_UP_PREPARE:
> > - smp_call_function_single(cpu, kvm_arch_ops->hardware_enable,
> > - NULL, 0, 1);
> > break;
> > }
> > return NOTIFY_OK;
> >
>
> That is already CPU_ONLINE in my tree

I noticed ;)

So this mean that Ingo's bug should already be fixed there. And in rc2-mm1.

> (and in the pull request sent to
> Linus a couple of days ago).

Resend, please.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-03-03 13:09    [W:0.454 / U:0.364 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site