On 02/06/16 19:44, Aneesh Kumar K.V wrote:
> Semantic Issue: comparison of constant 18446744073709551615 with
> expression of type 'unsigned int' is always false.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com>
> ---
> arch/powerpc/mm/tlb-radix.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c
> index 7f2fa82bd155..0484fcdca00a 100644
> --- a/arch/powerpc/mm/tlb-radix.c
> +++ b/arch/powerpc/mm/tlb-radix.c
> @@ -115,7 +115,7 @@ static inline void _tlbie_va(unsigned long va, unsigned
> long pid,
> */
> void radix__local_flush_tlb_mm(struct mm_struct *mm)
> {
> - unsigned int pid;
> + unsigned long pid;
>
> preempt_disable();
> pid = mm->context.id;
> @@ -128,7 +128,7 @@ EXPORT_SYMBOL(radix__local_flush_tlb_mm);
> void radix___local_flush_tlb_page(struct mm_struct *mm, unsigned long vmaddr,
> unsigned long ap, int nid)
> {
> - unsigned int pid;
> + unsigned long pid;
>
> preempt_disable();
> pid = mm ? mm->context.id : 0;
> @@ -158,7 +158,7 @@ static int mm_is_core_local(struct mm_struct *mm)
>
> void radix__flush_tlb_mm(struct mm_struct *mm)
> {
> - unsigned int pid;
> + unsigned long pid;
>
> preempt_disable();
> pid = mm->context.id;
> @@ -207,7 +207,7 @@ EXPORT_SYMBOL(radix__flush_tlb_mm_pwc);
> void radix___flush_tlb_page(struct mm_struct *mm, unsigned long vmaddr,
> unsigned long ap, int nid)
> {
> - unsigned int pid;
> + unsigned long pid;
>
> preempt_disable();
> pid = mm ? mm->context.id : 0;
>
Reviewed-by: Balbir Singh <bsinghar...@gmail.com>
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
