linux-kernel - [arm-integrator:virt-to-pfn-v6.0-rc1 11/24] arch/arm64/kernel/vdso.c:317:45: warning: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <202209150113.lrrSroX1-lkp@intel.com>
Date:   Thu, 15 Sep 2022 02:08:55 +0800
From:   kernel test robot <lkp@...el.com>
To:     Linus Walleij <linus.walleij@...aro.org>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org
Subject: [arm-integrator:virt-to-pfn-v6.0-rc1 11/24]
 arch/arm64/kernel/vdso.c:317:45: warning: passing argument 1 of
 'virt_to_pfn' makes pointer from integer without a cast

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git virt-to-pfn-v6.0-rc1
head:   bc69ecf9b2a3d1f9699edab614580d6a6e090407
commit: 5bccc479c1ea26fbab6d882aba31356bbb435bc7 [11/24] arm64: memory: Make virt_to_pfn() a static inline
config: arm64-buildonly-randconfig-r006-20220914
compiler: aarch64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git/commit/?id=5bccc479c1ea26fbab6d882aba31356bbb435bc7
        git remote add arm-integrator https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
        git fetch --no-tags arm-integrator virt-to-pfn-v6.0-rc1
        git checkout 5bccc479c1ea26fbab6d882aba31356bbb435bc7
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/kernel/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   In file included from arch/arm64/include/asm/memory.h:322,
                    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:78,
                    from include/linux/smp.h:110,
                    from arch/arm64/include/asm/arch_timer.h:18,
                    from arch/arm64/include/asm/timex.h:8,
                    from include/linux/timex.h:67,
                    from include/linux/clocksource.h:13,
                    from arch/arm64/kernel/vdso.c:11:
   arch/arm64/kernel/vdso.c: In function 'aarch32_alloc_kuser_vdso_page':
>> arch/arm64/kernel/vdso.c:317:45: warning: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
     317 |         aarch32_vectors_page = virt_to_page(vdso_page);
         |                                             ^~~~~~~~~
         |                                             |
         |                                             long unsigned int
   include/asm-generic/memory_model.h:25:45: note: in definition of macro '__pfn_to_page'
      25 | #define __pfn_to_page(pfn)      (vmemmap + (pfn))
         |                                             ^~~
   arch/arm64/kernel/vdso.c:317:32: note: in expansion of macro 'virt_to_page'
     317 |         aarch32_vectors_page = virt_to_page(vdso_page);
         |                                ^~~~~~~~~~~~
   arch/arm64/include/asm/memory.h:324:53: note: expected 'const void *' but argument is of type 'long unsigned int'
     324 | static inline unsigned long virt_to_pfn(const void *kaddr)
         |                                         ~~~~~~~~~~~~^~~~~


vim +/virt_to_pfn +317 arch/arm64/kernel/vdso.c

9031fefde6f2ac Will Deacon       2012-03-05  301  
1255a7341bee6c Vincenzo Frascino 2019-04-15  302  static int aarch32_alloc_kuser_vdso_page(void)
9031fefde6f2ac Will Deacon       2012-03-05  303  {
9031fefde6f2ac Will Deacon       2012-03-05  304  	extern char __kuser_helper_start[], __kuser_helper_end[];
9031fefde6f2ac Will Deacon       2012-03-05  305  	int kuser_sz = __kuser_helper_end - __kuser_helper_start;
1255a7341bee6c Vincenzo Frascino 2019-04-15  306  	unsigned long vdso_page;
9031fefde6f2ac Will Deacon       2012-03-05  307  
af1b3cf2c2a3f4 Vincenzo Frascino 2019-04-15  308  	if (!IS_ENABLED(CONFIG_KUSER_HELPERS))
af1b3cf2c2a3f4 Vincenzo Frascino 2019-04-15  309  		return 0;
af1b3cf2c2a3f4 Vincenzo Frascino 2019-04-15  310  
7cd6ca1d790226 Will Deacon       2021-03-18  311  	vdso_page = get_zeroed_page(GFP_KERNEL);
1255a7341bee6c Vincenzo Frascino 2019-04-15  312  	if (!vdso_page)
0d747f6585954d Vincenzo Frascino 2019-04-15  313  		return -ENOMEM;
9031fefde6f2ac Will Deacon       2012-03-05  314  
1255a7341bee6c Vincenzo Frascino 2019-04-15  315  	memcpy((void *)(vdso_page + 0x1000 - kuser_sz), __kuser_helper_start,
1255a7341bee6c Vincenzo Frascino 2019-04-15  316  	       kuser_sz);
74fc72e77dc5c8 Mark Rutland      2020-04-28 @317  	aarch32_vectors_page = virt_to_page(vdso_page);
1255a7341bee6c Vincenzo Frascino 2019-04-15  318  	return 0;
0d747f6585954d Vincenzo Frascino 2019-04-15  319  }
9031fefde6f2ac Will Deacon       2012-03-05  320  

:::::: The code at line 317 was first introduced by commit
:::::: 74fc72e77dc5c8033d1b47d2c8a7229b4b83a746 arm64: vdso: remove aarch32_vdso_pages[]

:::::: TO: Mark Rutland <mark.rutland@....com>
:::::: CC: Will Deacon <will@...nel.org>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

View attachment "config" of type "text/plain" (161259 bytes)