]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
KVM: s390/vfio-ap: Use kvm_is_gpa_in_memslot() instead of open coded equivalent
authorSean Christopherson <seanjc@google.com>
Fri, 19 Sep 2025 00:32:59 +0000 (17:32 -0700)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 30 Sep 2025 17:38:06 +0000 (13:38 -0400)
commit15463eece957be34da64b4d6fe18fc98981bf487
treebc1ada2a690b5e529cf5e243115e58fefc2a4f33
parent12abeb81c87331bf53940947694cc51b7507aa38
KVM: s390/vfio-ap: Use kvm_is_gpa_in_memslot() instead of open coded equivalent

Use kvm_is_gpa_in_memslot() to check the validity of the notification
indicator byte address instead of open coding equivalent logic in the VFIO
AP driver.

Opportunistically use a dedicated wrapper that exists and is exported
expressly for the VFIO AP module.  kvm_is_gpa_in_memslot() is generally
unsuitable for use outside of KVM; other drivers typically shouldn't rely
on KVM's memslots, and using the API requires kvm->srcu (or slots_lock) to
be held for the entire duration of the usage, e.g. to avoid TOCTOU bugs.
handle_pqap() is a bit of a special case, as it's explicitly invoked from
KVM with kvm->srcu already held, and the VFIO AP driver is in many ways an
extension of KVM that happens to live in a separate module.

Providing a dedicated API for the VFIO AP driver will allow restricting
the vast majority of generic KVM's exports to KVM submodules (e.g. to x86's
kvm-{amd,intel}.ko vendor mdoules).

No functional change intended.

Acked-by: Anthony Krowiak <akrowiak@linux.ibm.com>
Reviewed-by: Christian Borntraeger <borntraeger@linux.ibm.com>
Link: https://lore.kernel.org/r/20250919003303.1355064-2-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/s390/include/asm/kvm_host.h
arch/s390/kvm/priv.c
drivers/s390/crypto/vfio_ap_ops.c