[PATCH] powerpc/64s/radix: Fix radix_kvm_prefetch_workaround paca access of not possible CPU

Subsystems: linux for powerpc (32-bit and 64-bit), the rest

STALE3014d REVIEWED: 1 (0M)

1 review trailer.

2 messages, 2 authors, 2018-06-23 · open the first message on its own page

[PATCH] powerpc/64s/radix: Fix radix_kvm_prefetch_workaround paca access of not possible CPU

From: Nicholas Piggin <npiggin@gmail.com>
Date: 2018-06-12 09:38:19

If possible CPUs are limited (e.g., by kexec), then the kvm prefetch
workaround function can access the paca pointer for a !possible CPU.

Fixes: d2e60075a3d44 ("powerpc/64: Use array of paca pointers and allocate pacas individually")
Cc: stable@kernel.org
Reported-by: Pridhiviraj Paidipeddi <redacted>
Tested-by: Pridhiviraj Paidipeddi <redacted>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>

---
[kdump still seems to have a problem upstream, but this solves one crash]

 arch/powerpc/mm/tlb-radix.c | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c
index 67a6e86d3e7e..06ea845d8033 100644
--- a/arch/powerpc/mm/tlb-radix.c
+++ b/arch/powerpc/mm/tlb-radix.c
@@ -1043,6 +1043,8 @@ extern void radix_kvm_prefetch_workaround(struct mm_struct *mm)
 		for (; sib <= cpu_last_thread_sibling(cpu) && !flush; sib++) {
 			if (sib == cpu)
 				continue;
+			if (!cpu_possible(sib))
+				continue;
 			if (paca_ptrs[sib]->kvm_hstate.kvm_vcpu)
 				flush = true;
 		}
-- 
2.17.0

Re: powerpc/64s/radix: Fix radix_kvm_prefetch_workaround paca access of not possible CPU

From: Michael Ellerman <hidden>
Date: 2018-06-23 12:56:03

On Tue, 2018-06-12 at 09:38:08 UTC, Nicholas Piggin wrote:
If possible CPUs are limited (e.g., by kexec), then the kvm prefetch
workaround function can access the paca pointer for a !possible CPU.

Fixes: d2e60075a3d44 ("powerpc/64: Use array of paca pointers and allocate pacas individually")
Cc: stable@kernel.org
Reported-by: Pridhiviraj Paidipeddi <redacted>
Tested-by: Pridhiviraj Paidipeddi <redacted>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Applied to powerpc fixes, thanks.

https://git.kernel.org/powerpc/c/758380b8155f69b4e2f77f27562f8a

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help