Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote:
>
>
> On 13/04/2015 16:17, Nadav Amit wrote:
>> Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote:
>>
>>> On 13/04/2015 01:32, Nadav Amit wrote:
>>>> + if (kvm_check_extension(s, KVM_CAP_ENABLE_CAP_VM)) {
>>>
>>> The right capability to check here is KVM_CAP_DISABLE_QUIRKS, not
>>> KVM_CAP_ENABLE_CAP_VM.
>>>
>>> Paolo
>>>
>>>> + ret = kvm_vm_enable_cap(s, KVM_CAP_DISABLE_QUIRKS, 0,
>>>> + KVM_QUIRK_LINT0_DISABLED |
>>>> + KVM_QUIRK_CD_NW_CLEARED);
>>>> + if (ret < 0) {
>>>> + return ret;
>>>> + }
>>>> + }
>>
>> Of course…
>> Do you want a v2 now, later (after 4.2), or would you change it yourself?
>
> Later, close to 4.2 but not necessarily after it.
Anyhow, in that case the KVM patch is also wrong (not reporting
KVM_CAP_DISABLE_QUIRKS is supported). I don’t want to spam, so I’ll run some
tests and resubmit.
Nadav--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
