From: kbuild test robot <lkp@intel.com> To: Atish Patra <atish.patra@wdc.com> Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org Subject: Re: [v2 PATCH 5/5] RISC-V: Add EFI stub support. Date: Tue, 14 Apr 2020 08:20:43 +0800 [thread overview] Message-ID: <202004140827.fIxyYP7R%lkp@intel.com> (raw) In-Reply-To: <20200413212907.29244-6-atish.patra@wdc.com> [-- Attachment #1: Type: text/plain, Size: 3176 bytes --] Hi Atish, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.7-rc1 next-20200413] [cannot apply to efi/next arm/for-next arm64/for-next/core linux/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Atish-Patra/Add-UEFI-support-for-RISC-V/20200414-053224 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 8f3d9f354286745c751374f5f1fcafee6b3f3136 config: riscv-nommu_virt_defconfig (attached as .config) compiler: riscv64-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=9.3.0 make.cross ARCH=riscv If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): drivers/firmware/efi/earlycon.c: In function 'efi_earlycon_map': >> drivers/firmware/efi/earlycon.c:59:34: error: implicit declaration of function 'pgprot_writecombine' [-Werror=implicit-function-declaration] 59 | fb_prot = fb_wb ? PAGE_KERNEL : pgprot_writecombine(PAGE_KERNEL); | ^~~~~~~~~~~~~~~~~~~ >> drivers/firmware/efi/earlycon.c:59:32: error: type mismatch in conditional expression 59 | fb_prot = fb_wb ? PAGE_KERNEL : pgprot_writecombine(PAGE_KERNEL); | ^ cc1: some warnings being treated as errors -- drivers/firmware/efi/libstub/riscv-stub.c: In function 'efi_enter_kernel': >> drivers/firmware/efi/libstub/riscv-stub.c:64:33: warning: initialization makes '__attribute__((noreturn))' qualified function pointer from unqualified [-Wdiscarded-qualifiers] 64 | jump_kernel_func jump_kernel = (void (*)(unsigned int, unsigned long))kernel_entry; | ^ vim +64 drivers/firmware/efi/libstub/riscv-stub.c 53 54 /* 55 * Jump to real kernel here with following constraints. 56 * 1. MMU should be disabled. 57 * 2. a0 should contain hartid 58 * 3. a1 should DT address 59 */ 60 void __noreturn efi_enter_kernel(unsigned long entrypoint, unsigned long fdt, 61 unsigned long fdt_size) 62 { 63 unsigned long kernel_entry = entrypoint + (unsigned long)stext_offset; > 64 jump_kernel_func jump_kernel = (void (*)(unsigned int, unsigned long))kernel_entry; 65 u32 hartid = get_boot_hartid_from_fdt(fdt); 66 67 if (hartid == U32_MAX) 68 /* We can not use panic or BUG at this point */ 69 __asm__ __volatile__ ("ebreak"); 70 /* Disable MMU */ 71 csr_write(CSR_SATP, 0); 72 jump_kernel(hartid, fdt); 73 } 74 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 7060 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [v2 PATCH 5/5] RISC-V: Add EFI stub support. Date: Tue, 14 Apr 2020 08:20:43 +0800 [thread overview] Message-ID: <202004140827.fIxyYP7R%lkp@intel.com> (raw) In-Reply-To: <20200413212907.29244-6-atish.patra@wdc.com> [-- Attachment #1: Type: text/plain, Size: 3246 bytes --] Hi Atish, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.7-rc1 next-20200413] [cannot apply to efi/next arm/for-next arm64/for-next/core linux/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Atish-Patra/Add-UEFI-support-for-RISC-V/20200414-053224 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 8f3d9f354286745c751374f5f1fcafee6b3f3136 config: riscv-nommu_virt_defconfig (attached as .config) compiler: riscv64-linux-gcc (GCC) 9.3.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=9.3.0 make.cross ARCH=riscv If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): drivers/firmware/efi/earlycon.c: In function 'efi_earlycon_map': >> drivers/firmware/efi/earlycon.c:59:34: error: implicit declaration of function 'pgprot_writecombine' [-Werror=implicit-function-declaration] 59 | fb_prot = fb_wb ? PAGE_KERNEL : pgprot_writecombine(PAGE_KERNEL); | ^~~~~~~~~~~~~~~~~~~ >> drivers/firmware/efi/earlycon.c:59:32: error: type mismatch in conditional expression 59 | fb_prot = fb_wb ? PAGE_KERNEL : pgprot_writecombine(PAGE_KERNEL); | ^ cc1: some warnings being treated as errors -- drivers/firmware/efi/libstub/riscv-stub.c: In function 'efi_enter_kernel': >> drivers/firmware/efi/libstub/riscv-stub.c:64:33: warning: initialization makes '__attribute__((noreturn))' qualified function pointer from unqualified [-Wdiscarded-qualifiers] 64 | jump_kernel_func jump_kernel = (void (*)(unsigned int, unsigned long))kernel_entry; | ^ vim +64 drivers/firmware/efi/libstub/riscv-stub.c 53 54 /* 55 * Jump to real kernel here with following constraints. 56 * 1. MMU should be disabled. 57 * 2. a0 should contain hartid 58 * 3. a1 should DT address 59 */ 60 void __noreturn efi_enter_kernel(unsigned long entrypoint, unsigned long fdt, 61 unsigned long fdt_size) 62 { 63 unsigned long kernel_entry = entrypoint + (unsigned long)stext_offset; > 64 jump_kernel_func jump_kernel = (void (*)(unsigned int, unsigned long))kernel_entry; 65 u32 hartid = get_boot_hartid_from_fdt(fdt); 66 67 if (hartid == U32_MAX) 68 /* We can not use panic or BUG at this point */ 69 __asm__ __volatile__ ("ebreak"); 70 /* Disable MMU */ 71 csr_write(CSR_SATP, 0); 72 jump_kernel(hartid, fdt); 73 } 74 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 7060 bytes --]
next prev parent reply other threads:[~2020-04-14 0:21 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-04-13 21:29 [v2 PATCH 0/5] Add UEFI support for RISC-V Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` [v2 PATCH 1/5] efi: Move arm-stub to a common file Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-25 9:47 ` Heinrich Schuchardt 2020-04-25 9:47 ` Heinrich Schuchardt 2020-04-25 9:47 ` Heinrich Schuchardt 2020-04-25 9:54 ` Ard Biesheuvel 2020-04-25 9:54 ` Ard Biesheuvel 2020-04-25 9:54 ` Ard Biesheuvel 2020-04-13 21:29 ` [v2 PATCH 2/5] include: pe.h: Add RISC-V related PE definition Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-13 21:29 ` [v2 PATCH 3/5] RISC-V: Define fixmap bindings for generic early ioremap support Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-21 19:19 ` Palmer Dabbelt 2020-04-13 21:29 ` [v2 PATCH 4/5] RISC-V: Add PE/COFF header for EFI stub Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` [v2 PATCH 5/5] RISC-V: Add EFI stub support Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-13 21:29 ` Atish Patra 2020-04-14 0:20 ` kbuild test robot [this message] 2020-04-14 0:20 ` kbuild test robot 2020-04-14 1:28 ` kbuild test robot 2020-04-14 1:28 ` kbuild test robot 2020-04-14 7:41 ` Ard Biesheuvel 2020-04-14 7:41 ` Ard Biesheuvel 2020-04-14 7:41 ` Ard Biesheuvel 2020-04-15 0:10 ` Atish Patra 2020-04-15 0:10 ` Atish Patra 2020-04-15 0:10 ` Atish Patra 2020-04-15 7:38 ` Ard Biesheuvel 2020-04-15 7:38 ` Ard Biesheuvel 2020-04-15 7:38 ` Ard Biesheuvel
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202004140827.fIxyYP7R%lkp@intel.com \ --to=lkp@intel.com \ --cc=atish.patra@wdc.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.