diff options
author | Mathieu Malaterre <malat@debian.org> | 2018-03-22 17:03:18 -0400 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2018-04-01 08:15:34 -0400 |
commit | 19e68b2aec3c0a2bd770d3c358a296a1849f308a (patch) | |
tree | c662dd0f83f036f947c7460a20f2774d3532a051 | |
parent | 0e524e761fc2157f1037e0f5d616cd39e468d89c (diff) |
powerpc/mm/radix: Fix always false comparison against MMU_NO_CONTEXT
In commit 9690c1574268 ("powerpc/mm/radix: Fix always false comparison
against MMU_NO_CONTEXT") an issue was discovered where `mm->context.id` was
being truncated to an `unsigned int`, while the PID is actually an
`unsigned long`. Update the earlier patch by fixing one remaining
occurrence. Discovered during a compilation with W=1:
arch/powerpc/mm/tlb-radix.c:702:19: error: comparison is always false due to limited range of data type [-Werror=type-limits]
Signed-off-by: Mathieu Malaterre <malat@debian.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
-rw-r--r-- | arch/powerpc/mm/tlb-radix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c index a8b178dd2e82..2fba6170ab3f 100644 --- a/arch/powerpc/mm/tlb-radix.c +++ b/arch/powerpc/mm/tlb-radix.c | |||
@@ -706,7 +706,7 @@ void radix__flush_tlb_pte_p9_dd1(unsigned long old_pte, struct mm_struct *mm, | |||
706 | #ifdef CONFIG_KVM_BOOK3S_HV_POSSIBLE | 706 | #ifdef CONFIG_KVM_BOOK3S_HV_POSSIBLE |
707 | extern void radix_kvm_prefetch_workaround(struct mm_struct *mm) | 707 | extern void radix_kvm_prefetch_workaround(struct mm_struct *mm) |
708 | { | 708 | { |
709 | unsigned int pid = mm->context.id; | 709 | unsigned long pid = mm->context.id; |
710 | 710 | ||
711 | if (unlikely(pid == MMU_NO_CONTEXT)) | 711 | if (unlikely(pid == MMU_NO_CONTEXT)) |
712 | return; | 712 | return; |