Re: [PATCH] percpu: fix compile warnings

[Randy Dunlap <randy.dunlap@oracle.com>]

Tejun Heo wrote:
> Fix the following two compile warnings which show up on i386.
> 
> mm/percpu.c:1873: warning: comparison of distinct pointer types lacks a cast
> mm/percpu.c:1879: warning: format '%lx' expects type 'long unsigned int', but argument 2 has type 'size_t'
> 
> Signed-off-by: Tejun Heo <tj@kernel.org>
> Reported-by: Randy Dunlap <randy.dunlap@oracle.com>

Acked-by: Randy Dunlap <randy.dunlap@oracle.com>

Thanks.

> ---
> This patch has been committed to percpu#for-next.  Thanks.
> 
>  mm/percpu.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/mm/percpu.c b/mm/percpu.c
> index e4e08b8..ec158bb 100644
> --- a/mm/percpu.c
> +++ b/mm/percpu.c
> @@ -1868,13 +1868,14 @@ int __init pcpu_embed_first_chunk(size_t reserved_size, ssize_t dyn_size,
>  	max_distance = 0;
>  	for (group = 0; group < ai->nr_groups; group++) {
>  		ai->groups[group].base_offset = areas[group] - base;
> -		max_distance = max(max_distance, ai->groups[group].base_offset);
> +		max_distance = max_t(size_t, max_distance,
> +				     ai->groups[group].base_offset);
>  	}
>  	max_distance += ai->unit_size;
>  
>  	/* warn if maximum distance is further than 75% of vmalloc space */
>  	if (max_distance > (VMALLOC_END - VMALLOC_START) * 3 / 4) {
> -		pr_warning("PERCPU: max_distance=0x%lx too large for vmalloc "
> +		pr_warning("PERCPU: max_distance=0x%zx too large for vmalloc "
>  			   "space 0x%lx\n",
>  			   max_distance, VMALLOC_END - VMALLOC_START);
>  #ifdef CONFIG_NEED_PER_CPU_PAGE_FIRST_CHUNK


-- 
~Randy