| 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
| ||
|
Message-ID: <202209302007.LxFlIRDW-lkp@intel.com>
Date: Fri, 30 Sep 2022 20:21:10 +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:kernel-in-vmalloc-v6.0-rc1 11/30]
arch/arm64/kernel/vdso.c:317:45: warning: passing argument 1 of
'virt_to_pfn' makes pointer from integer without a cast
Hi Linus,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git kernel-in-vmalloc-v6.0-rc1
head: c52106732066b75afcd0be7304d57d1c5d7e4a69
commit: abccaa86889322e58b320ccc8bfc587c304e20fd [11/30] arm64: memory: Make virt_to_pfn() a static inline
config: arm64-allyesconfig
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=abccaa86889322e58b320ccc8bfc587c304e20fd
git remote add arm-integrator https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
git fetch --no-tags arm-integrator kernel-in-vmalloc-v6.0-rc1
git checkout abccaa86889322e58b320ccc8bfc587c304e20fd
# 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/ drivers/net/xen-netback/
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)
| ~~~~~~~~~~~~^~~~~
--
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/time32.h:13,
from include/linux/time.h:60,
from arch/arm64/include/asm/stat.h:12,
from include/linux/stat.h:6,
from include/linux/module.h:13,
from drivers/net/xen-netback/common.h:32,
from drivers/net/xen-netback/netback.c:35:
drivers/net/xen-netback/netback.c: In function 'xenvif_fill_frags':
>> drivers/net/xen-netback/netback.c:600:37: warning: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion]
600 | page = virt_to_page(idx_to_kaddr(queue, pending_idx));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| 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))
| ^~~
drivers/net/xen-netback/netback.c:600:24: note: in expansion of macro 'virt_to_page'
600 | page = virt_to_page(idx_to_kaddr(queue, pending_idx));
| ^~~~~~~~~~~~
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" (358139 bytes)
Powered by blists - more mailing lists