Le 03/03/2026 à 20:03, Sean Christopherson a écrit :
Fix a build error in kvmppc_e500_tlb_init() that was introduced by the
conversion to use kzalloc_objs(), as KVM confusingly uses the size of the
structure that is one and only field in tlbe_priv:

   arch/powerpc/kvm/e500_mmu.c:923:33: error: assignment to 'struct tlbe_priv *'
     from incompatible pointer type 'struct tlbe_ref *' 
[-Wincompatible-pointer-types]
   923 |         vcpu_e500->gtlb_priv[0] = kzalloc_objs(struct tlbe_ref,
       |                                 ^

KVM has been flawed since commit 0164c0f0c404 ("KVM: PPC: e500: clear up
confusion between host and guest entries"), but the issue went unnoticed
until kmalloc_obj() came along and enforced types, as "struct tlbe_priv"
was just a wrapper of "struct tlbe_ref" (why on earth the two ever existed
separately...).

Fixes: 69050f8d6d07 ("treewide: Replace kmalloc with kmalloc_obj for non-scalar 
types")
Cc: Kees Cook <[email protected]>
Signed-off-by: Sean Christopherson <[email protected]>

Reviewed-by: Christophe Leroy (CS GROUP) <[email protected]>

---
  arch/powerpc/kvm/e500_mmu.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/kvm/e500_mmu.c b/arch/powerpc/kvm/e500_mmu.c
index 48580c85f23b..75ed1496ead5 100644
--- a/arch/powerpc/kvm/e500_mmu.c
+++ b/arch/powerpc/kvm/e500_mmu.c
@@ -920,12 +920,12 @@ int kvmppc_e500_tlb_init(struct kvmppc_vcpu_e500 
*vcpu_e500)
        vcpu_e500->gtlb_offset[0] = 0;
        vcpu_e500->gtlb_offset[1] = KVM_E500_TLB0_SIZE;
- vcpu_e500->gtlb_priv[0] = kzalloc_objs(struct tlbe_ref,
+       vcpu_e500->gtlb_priv[0] = kzalloc_objs(struct tlbe_priv,
                                               
vcpu_e500->gtlb_params[0].entries);
        if (!vcpu_e500->gtlb_priv[0])
                goto free_vcpu;
- vcpu_e500->gtlb_priv[1] = kzalloc_objs(struct tlbe_ref,
+       vcpu_e500->gtlb_priv[1] = kzalloc_objs(struct tlbe_priv,
                                               
vcpu_e500->gtlb_params[1].entries);
        if (!vcpu_e500->gtlb_priv[1])
                goto free_vcpu;


Reply via email to