Re: Regression: sched/preempt: Add static_key() to preempt_notifiers breaks my KVM

From: Josh Boyer
Date: Tue Jun 30 2015 - 09:47:55 EST


On Thu, Jun 25, 2015 at 8:15 AM, Pontus Fuchs <pontus.fuchs@xxxxxxxxx> wrote:
> On 2015-06-25 14:09, Peter Zijlstra wrote:
>>
>> On Thu, Jun 25, 2015 at 02:00:02PM +0200, Pontus Fuchs wrote:
>>>
>>> Hi,
>>>
>>
>> That seems pointless..
>>
>> ---
>> virt/kvm/kvm_main.c | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c
>> index 90977418aeb6..d7aafa0458a0 100644
>> --- a/virt/kvm/kvm_main.c
>> +++ b/virt/kvm/kvm_main.c
>> @@ -128,8 +128,9 @@ int vcpu_load(struct kvm_vcpu *vcpu)
>>
>> if (mutex_lock_killable(&vcpu->mutex))
>> return -EINTR;
>> - cpu = get_cpu();
>> preempt_notifier_register(&vcpu->preempt_notifier);
>> +
>> + cpu = get_cpu();
>> kvm_arch_vcpu_load(vcpu, cpu);
>> put_cpu();
>> return 0;
>> @@ -139,8 +140,8 @@ void vcpu_put(struct kvm_vcpu *vcpu)
>> {
>> preempt_disable();
>> kvm_arch_vcpu_put(vcpu);
>> - preempt_notifier_unregister(&vcpu->preempt_notifier);
>> preempt_enable();
>> + preempt_notifier_unregister(&vcpu->preempt_notifier);
>> mutex_unlock(&vcpu->mutex);
>> }
>
>
> Tested ok. Thanks.

We've had a report of this in Fedora now. Is the above patch queued anywhere?

https://bugzilla.redhat.com/show_bug.cgi?id=1237143

josh
--
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/