| 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: <202208231055.QZtD1nxQ-lkp@intel.com>
Date: Tue, 23 Aug 2022 10:48:13 +0800
From: kernel test robot <lkp@...el.com>
To: Ard Biesheuvel <ardb@...nel.org>
Cc: llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
linux-kernel@...r.kernel.org
Subject: [ardb:for-kernelci 7/7]
drivers/firmware/efi/libstub/arm64-stub.c:178:25: error: incompatible
pointer to integer conversion passing 'void *' to parameter of type
'unsigned long'
tree: git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git for-kernelci
head: 6aca304c2ee44597ad1f6989d4905ed9f52ad16b
commit: 6aca304c2ee44597ad1f6989d4905ed9f52ad16b [7/7] arm64: efi/libstub: enter with the MMU on
config: arm64-randconfig-r025-20220821 (https://download.01.org/0day-ci/archive/20220823/202208231055.QZtD1nxQ-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project abce7acebd4c06c977bc4bd79170697f1122bc5e)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git/commit/?id=6aca304c2ee44597ad1f6989d4905ed9f52ad16b
git remote add ardb git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git
git fetch --no-tags ardb for-kernelci
git checkout 6aca304c2ee44597ad1f6989d4905ed9f52ad16b
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/firmware/efi/libstub/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>
All errors (new ones prefixed by >>):
>> drivers/firmware/efi/libstub/arm64-stub.c:178:25: error: incompatible pointer to integer conversion passing 'void *' to parameter of type 'unsigned long' [-Wint-conversion]
caches_clean_inval_pou((void *)*image_addr,
^~~~~~~~~~~~~~~~~~~
arch/arm64/include/asm/cacheflush.h:72:50: note: passing argument to parameter 'start' here
extern void caches_clean_inval_pou(unsigned long start, unsigned long end);
^
drivers/firmware/efi/libstub/arm64-stub.c:179:11: error: incompatible pointer to integer conversion passing 'void *' to parameter of type 'unsigned long' [-Wint-conversion]
(void *)*image_addr + kernel_codesize);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/arm64/include/asm/cacheflush.h:72:71: note: passing argument to parameter 'end' here
extern void caches_clean_inval_pou(unsigned long start, unsigned long end);
^
2 errors generated.
vim +178 drivers/firmware/efi/libstub/arm64-stub.c
81
82 efi_status_t handle_kernel_image(unsigned long *image_addr,
83 unsigned long *image_size,
84 unsigned long *reserve_addr,
85 unsigned long *reserve_size,
86 efi_loaded_image_t *image,
87 efi_handle_t image_handle)
88 {
89 efi_status_t status;
90 unsigned long kernel_size, kernel_codesize, kernel_memsize = 0;
91 u32 phys_seed = 0;
92
93 /*
94 * Although relocatable kernels can fix up the misalignment with
95 * respect to MIN_KIMG_ALIGN, the resulting virtual text addresses are
96 * subtly out of sync with those recorded in the vmlinux when kaslr is
97 * disabled but the image required relocation anyway. Therefore retain
98 * 2M alignment if KASLR was explicitly disabled, even if it was not
99 * going to be activated to begin with.
100 */
101 u64 min_kimg_align = efi_nokaslr ? MIN_KIMG_ALIGN : SEGMENT_ALIGN;
102
103 if (IS_ENABLED(CONFIG_RANDOMIZE_BASE)) {
104 efi_guid_t li_fixed_proto = LINUX_EFI_LOADED_IMAGE_FIXED_GUID;
105 void *p;
106
107 if (efi_nokaslr) {
108 efi_info("KASLR disabled on kernel command line\n");
109 } else if (efi_bs_call(handle_protocol, image_handle,
110 &li_fixed_proto, &p) == EFI_SUCCESS) {
111 efi_info("Image placement fixed by loader\n");
112 } else {
113 status = efi_get_random_bytes(sizeof(phys_seed),
114 (u8 *)&phys_seed);
115 if (status == EFI_NOT_FOUND) {
116 efi_info("EFI_RNG_PROTOCOL unavailable\n");
117 efi_nokaslr = true;
118 } else if (status != EFI_SUCCESS) {
119 efi_err("efi_get_random_bytes() failed (0x%lx)\n",
120 status);
121 efi_nokaslr = true;
122 }
123 }
124 }
125
126 if (image->image_base != _text)
127 efi_err("FIRMWARE BUG: efi_loaded_image_t::image_base has bogus value\n");
128
129 if (!IS_ALIGNED((u64)_text, SEGMENT_ALIGN))
130 efi_err("FIRMWARE BUG: kernel image not aligned on %dk boundary\n",
131 SEGMENT_ALIGN >> 10);
132
133 kernel_size = _edata - _text;
134 kernel_codesize = __inittext_end - _text;
135 kernel_memsize = kernel_size + (_end - _edata);
136 *reserve_size = kernel_memsize;
137
138 if (IS_ENABLED(CONFIG_RANDOMIZE_BASE) && phys_seed != 0) {
139 /*
140 * If KASLR is enabled, and we have some randomness available,
141 * locate the kernel at a randomized offset in physical memory.
142 */
143 status = efi_random_alloc(*reserve_size, min_kimg_align,
144 reserve_addr, phys_seed,
145 EFI_LOADER_CODE);
146 if (status != EFI_SUCCESS)
147 efi_warn("efi_random_alloc() failed: 0x%lx\n", status);
148 } else {
149 status = EFI_OUT_OF_RESOURCES;
150 }
151
152 if (status != EFI_SUCCESS) {
153 if (!check_image_region((u64)_text, kernel_memsize)) {
154 efi_err("FIRMWARE BUG: Image BSS overlaps adjacent EFI memory region\n");
155 } else if (IS_ALIGNED((u64)_text, min_kimg_align)) {
156 /*
157 * Just execute from wherever we were loaded by the
158 * UEFI PE/COFF loader if the alignment is suitable.
159 */
160 *image_addr = (u64)_text;
161 *reserve_size = 0;
162 return EFI_SUCCESS;
163 }
164
165 status = efi_allocate_pages_aligned(*reserve_size, reserve_addr,
166 ULONG_MAX, min_kimg_align,
167 EFI_LOADER_CODE);
168
169 if (status != EFI_SUCCESS) {
170 efi_err("Failed to relocate kernel\n");
171 *reserve_size = 0;
172 return status;
173 }
174 }
175
176 *image_addr = *reserve_addr;
177 memcpy((void *)*image_addr, _text, kernel_size);
> 178 caches_clean_inval_pou((void *)*image_addr,
179 (void *)*image_addr + kernel_codesize);
180
181 return EFI_SUCCESS;
182 }
183
--
0-DAY CI Kernel Test Service
https://01.org/lkp
Powered by blists - more mailing lists