Re: [PATCH v4 bpf-next 2/3] mm/bpf: Add bpf_get_kmem_cache() kfunc

[kernel test robot <lkp@intel.com>]

Hi Namhyung,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 9502a7de5a61bec3bda841a830560c5d6d40ecac]

url:    https://github.com/intel-lab-lkp/linux/commits/Namhyung-Kim/bpf-Add-kmem_cache-iterator/20241003-021144
base:   9502a7de5a61bec3bda841a830560c5d6d40ecac
patch link:    https://lore.kernel.org/r/20241002180956.1781008-3-namhyung%40kernel.org
patch subject: [PATCH v4 bpf-next 2/3] mm/bpf: Add bpf_get_kmem_cache() kfunc
config: arm64-defconfig (https://download.01.org/0day-ci/archive/20241005/202410050149.QCPhh01T-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 13.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241005/202410050149.QCPhh01T-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410050149.QCPhh01T-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from arch/arm64/include/asm/thread_info.h:17,
                    from include/linux/thread_info.h:60,
                    from arch/arm64/include/asm/preempt.h:6,
                    from include/linux/preempt.h:79,
                    from include/linux/spinlock.h:56,
                    from include/linux/mmzone.h:8,
                    from include/linux/gfp.h:7,
                    from include/linux/slab.h:16,
                    from mm/slab_common.c:7:
   mm/slab_common.c: In function 'bpf_get_kmem_cache':
>> arch/arm64/include/asm/memory.h:427:66: warning: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
     427 |         __is_lm_address(__addr) && pfn_is_map_memory(virt_to_pfn(__addr));      \
         |                                                                  ^~~~~~
         |                                                                  |
         |                                                                  u64 {aka long long unsigned int}
   mm/slab_common.c:1334:14: note: in expansion of macro 'virt_addr_valid'
    1334 |         if (!virt_addr_valid(addr))
         |              ^~~~~~~~~~~~~~~
   arch/arm64/include/asm/memory.h:382:53: note: expected 'const void *' but argument is of type 'u64' {aka 'long long unsigned int'}
     382 | static inline unsigned long virt_to_pfn(const void *kaddr)
         |                                         ~~~~~~~~~~~~^~~~~


vim +/virt_to_pfn +427 arch/arm64/include/asm/memory.h

4f04d8f0054511 Catalin Marinas 2012-03-05  424  
68dd8ef321626f Will Deacon     2019-08-13  425  #define virt_addr_valid(addr)	({					\
91cb2c8b072e00 Catalin Marinas 2021-02-01  426  	__typeof__(addr) __addr = __tag_reset(addr);			\
873ba463914cf4 Mike Rapoport   2021-06-30 @427  	__is_lm_address(__addr) && pfn_is_map_memory(virt_to_pfn(__addr));	\
68dd8ef321626f Will Deacon     2019-08-13  428  })
4f04d8f0054511 Catalin Marinas 2012-03-05  429  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki