Re: [PATCH] x86: vdso: fix pvclock races with task migration

From: Radim KrÄmÃÅ
Date: Tue Apr 07 2015 - 08:47:47 EST


2015-04-07 13:11+0200, Paolo Bonzini:
> On 02/04/2015 20:44, Radim KrÄmÃÅ wrote:
> > If we were migrated right after __getcpu, but before reading the
> > migration_count, we wouldn't notice that we read TSC of a different
> > VCPU, nor that KVM's bug made pvti invalid, as only migration_count
> > on source VCPU is increased.
> >
> > Change vdso instead of updating migration_count on destination.
> >
> > Fixes: 0a4e6be9ca17 ("x86: kvm: Revert "remove sched notifier for cross-cpu migrations"")
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Radim KrÄmÃÅ <rkrcmar@xxxxxxxxxx>
>
> Applying this, but removing the "Fixes" tag because a guest patch cannot
> fix a host patch (it can work around it or complement it).

I think it was correct. Both are guest only, the revert just missed
some races. (0a4e6be9ca17 has misleading commit message ...)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/