KVM: PPC: Book3S HV: Fix handling of secondary HPTEG in HPT resizing code
authorPaul Mackerras <paulus@ozlabs.org>
Wed, 7 Feb 2018 08:49:54 +0000 (19:49 +1100)
committerPaul Mackerras <paulus@ozlabs.org>
Fri, 9 Feb 2018 04:35:24 +0000 (15:35 +1100)
commit05f2bb0313a2855e491dadfc8319b7da261d7074
tree5e75fbfcc885ae5235eaaef4ef2ef310160ab56d
parent57ea5f161a7de5b1913c212d04f57a175b159fdf
KVM: PPC: Book3S HV: Fix handling of secondary HPTEG in HPT resizing code

This fixes the computation of the HPTE index to use when the HPT
resizing code encounters a bolted HPTE which is stored in its
secondary HPTE group.  The code inverts the HPTE group number, which
is correct, but doesn't then mask it with new_hash_mask.  As a result,
new_pteg will be effectively negative, resulting in new_hptep
pointing before the new HPT, which will corrupt memory.

In addition, this removes two BUG_ON statements.  The condition that
the BUG_ONs were testing -- that we have computed the hash value
incorrectly -- has never been observed in testing, and if it did
occur, would only affect the guest, not the host.  Given that
BUG_ON should only be used in conditions where the kernel (i.e.
the host kernel, in this case) can't possibly continue execution,
it is not appropriate here.

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Paul Mackerras <paulus@ozlabs.org>
arch/powerpc/kvm/book3s_64_mmu_hv.c