[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] ppc/kvm: check some capabilities with kvm_v
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] ppc/kvm: check some capabilities with kvm_vm_check_extension() |
Date: |
Fri, 15 Sep 2017 10:52:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 15.09.2017 10:43, Greg Kurz wrote:
> On Fri, 15 Sep 2017 07:15:41 +0200
> Thomas Huth <address@hidden> wrote:
>
>> On 14.09.2017 21:25, Greg Kurz wrote:
>>> The following capabilities are VM specific:
>>> - KVM_CAP_PPC_SMT_POSSIBLE
>>> - KVM_CAP_PPC_HTAB_FD
>>
>> BTW, looks like kvmppc_has_cap_htab_fd() is dead code ... should we
>> either remove it or check it somewhere?
>>
>
> Heh you're right :)
>
> The only user for cap_htab_fd is kvmppc_get_htab_fd(), in order to provide a
> sensible error message that KVM doesn't allow saving HPTEs (ie, migration
> isn't
> supported with some older KVM HV). It doesn't need kvmppc_has_cap_htab_fd()
> for
> that since all the code sits in target/ppc/kvm.c.
>
> I can't think of any other use, so I guess we can drop it.
OK. If you've got some spare minutes, could you maybe send a patch?
[...]
>>> @@ -2353,7 +2353,7 @@ int kvmppc_reset_htab(int shift_hint)
>>> /* Full emulation, tell caller to allocate htab itself */
>>> return 0;
>>> }
>>> - if (kvm_check_extension(kvm_state, KVM_CAP_PPC_ALLOC_HTAB)) {
>>> + if (kvm_vm_check_extension(kvm_state, KVM_CAP_PPC_ALLOC_HTAB)) {
>>> int ret;
>>> ret = kvm_vm_ioctl(kvm_state, KVM_PPC_ALLOCATE_HTAB, &shift);
>>> if (ret == -ENOTTY) {
>>
>> Looking at the comment in the code after the "if (ret == -ENOTTY)" line,
>> it sounds like there is a bug in the kernel and the
>> KVM_CAP_PPC_ALLOC_HTAB should depend on the hv_enabled variable, too?
>
> It already does :)
>
> commit a8acaece5d88db234d0b82b8692dea15d602f622
Oh, right, seems like I was looking at an older kernel branch :-/ So
never mind, we're fine here.
Thomas
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 1/3] kvm: check KVM_CAP_SYNC_MMU with kvm_vm_check_extension(), (continued)
Re: [Qemu-devel] [PATCH 3/3] ppc/kvm: check some capabilities with kvm_vm_check_extension(), David Gibson, 2017/09/15