[PATCH] powerpc/mm/radix: Make the pid unsigned long

[Balbir Singh]

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.kumar at 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 <bsingharora at gmail.com>