tree:
git://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git for-kernelci
head: 5d3ec29ac0d7f5b733d85148f225bf06d6ba42ec
commit: 0c97e87b7a27b4d331c3ecba955e5e08a5b708ec [3/4] efi: use const* paramaters for
get/setvar by-ref arguments annotated as IN
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
#
https://git.kernel.org/pub/scm/linux/kernel/git/ardb/linux.git/commit/?id...
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 0c97e87b7a27b4d331c3ecba955e5e08a5b708ec
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
arch/x86/platform/efi/efi_64.c: In function 'efi_thunk_get_variable':
arch/x86/platform/efi/efi_64.c:592:60: warning: passing argument 1 of
'efi_name_size' discards 'const' qualifier from pointer target type
[-Wdiscarded-qualifiers]
592 | phys_name = virt_to_phys_or_null_size(name, efi_name_size(name));
| ^~~~
arch/x86/platform/efi/efi_64.c:570:50: note: expected 'efi_char16_t *' {aka
'short unsigned int *'} but argument is of type 'const efi_char16_t *'
{aka 'const short unsigned int *'}
570 | static unsigned long efi_name_size(efi_char16_t *name)
| ~~~~~~~~~~~~~~^~~~
arch/x86/platform/efi/efi_64.c:592:40: warning: passing argument 1 of
'virt_to_phys_or_null_size' discards 'const' qualifier from pointer target
type [-Wdiscarded-qualifiers]
592 | phys_name = virt_to_phys_or_null_size(name, efi_name_size(name));
| ^~~~
arch/x86/platform/efi/efi_64.c:155:33: note: expected 'void *' but argument is
of type 'const efi_char16_t *' {aka 'const short unsigned int *'}
155 | virt_to_phys_or_null_size(void *va, unsigned long size)
| ~~~~~~^~
arch/x86/platform/efi/efi_64.c: In function 'efi_thunk_set_variable':
arch/x86/platform/efi/efi_64.c:621:60: warning: passing argument 1 of
'efi_name_size' discards 'const' qualifier from pointer target type
[-Wdiscarded-qualifiers]
621 | phys_name = virt_to_phys_or_null_size(name, efi_name_size(name));
| ^~~~
arch/x86/platform/efi/efi_64.c:570:50: note: expected 'efi_char16_t *' {aka
'short unsigned int *'} but argument is of type 'const efi_char16_t *'
{aka 'const short unsigned int *'}
570 | static unsigned long efi_name_size(efi_char16_t *name)
| ~~~~~~~~~~~~~~^~~~
arch/x86/platform/efi/efi_64.c:621:40: warning: passing argument 1 of
'virt_to_phys_or_null_size' discards 'const' qualifier from pointer target
type [-Wdiscarded-qualifiers]
621 | phys_name = virt_to_phys_or_null_size(name, efi_name_size(name));
| ^~~~
arch/x86/platform/efi/efi_64.c:155:33: note: expected 'void *' but argument is
of type 'const efi_char16_t *' {aka 'const short unsigned int *'}
155 | virt_to_phys_or_null_size(void *va, unsigned long size)
| ~~~~~~^~
arch/x86/platform/efi/efi_64.c: In function 'efi_thunk_runtime_setup':
> arch/x86/platform/efi/efi_64.c:810:31: error: assignment to
'efi_status_t (*)(const efi_char16_t *, const efi_guid_t *, u32, long unsigned int,
void *)' {aka 'long unsigned int (*)(const short unsigned int *, const struct
<anonymous> *, unsigned int, long unsigned int, void *)'} from incompatible
pointer type 'efi_status_t (*)(efi_char16_t *, efi_guid_t *, u32, long unsigned int,
void *)' {aka 'long unsigned int (*)(short unsigned int *, struct
<anonymous> *, unsigned int, long unsigned int, void *)'}
[-Werror=incompatible-pointer-types]
810 | efi.set_variable_nonblocking =
efi_thunk_set_variable_nonblocking;
| ^
cc1: some warnings being treated as errors
vim +810 arch/x86/platform/efi/efi_64.c
4f9dbcfc40299d Matt Fleming 2014-01-10 797
ea5e1919b44f09 Ard Biesheuvel 2020-01-03 798 void __init efi_thunk_runtime_setup(void)
4f9dbcfc40299d Matt Fleming 2014-01-10 799 {
ea5e1919b44f09 Ard Biesheuvel 2020-01-03 800 if (!IS_ENABLED(CONFIG_EFI_MIXED))
ea5e1919b44f09 Ard Biesheuvel 2020-01-03 801 return;
ea5e1919b44f09 Ard Biesheuvel 2020-01-03 802
4f9dbcfc40299d Matt Fleming 2014-01-10 803 efi.get_time = efi_thunk_get_time;
4f9dbcfc40299d Matt Fleming 2014-01-10 804 efi.set_time = efi_thunk_set_time;
4f9dbcfc40299d Matt Fleming 2014-01-10 805 efi.get_wakeup_time =
efi_thunk_get_wakeup_time;
4f9dbcfc40299d Matt Fleming 2014-01-10 806 efi.set_wakeup_time =
efi_thunk_set_wakeup_time;
4f9dbcfc40299d Matt Fleming 2014-01-10 807 efi.get_variable =
efi_thunk_get_variable;
4f9dbcfc40299d Matt Fleming 2014-01-10 808 efi.get_next_variable =
efi_thunk_get_next_variable;
4f9dbcfc40299d Matt Fleming 2014-01-10 809 efi.set_variable =
efi_thunk_set_variable;
83a0a2ea0b9919 Ard Biesheuvel 2018-07-20 @810 efi.set_variable_nonblocking =
efi_thunk_set_variable_nonblocking;
4f9dbcfc40299d Matt Fleming 2014-01-10 811 efi.get_next_high_mono_count =
efi_thunk_get_next_high_mono_count;
4f9dbcfc40299d Matt Fleming 2014-01-10 812 efi.reset_system =
efi_thunk_reset_system;
4f9dbcfc40299d Matt Fleming 2014-01-10 813 efi.query_variable_info =
efi_thunk_query_variable_info;
83a0a2ea0b9919 Ard Biesheuvel 2018-07-20 814 efi.query_variable_info_nonblocking =
efi_thunk_query_variable_info_nonblocking;
4f9dbcfc40299d Matt Fleming 2014-01-10 815 efi.update_capsule =
efi_thunk_update_capsule;
4f9dbcfc40299d Matt Fleming 2014-01-10 816 efi.query_capsule_caps =
efi_thunk_query_capsule_caps;
4f9dbcfc40299d Matt Fleming 2014-01-10 817 }
69829470457344 Ard Biesheuvel 2020-01-03 818
:::::: The code at line 810 was first introduced by commit
:::::: 83a0a2ea0b991927e42984be220329e776ce7137 efi/x86: Prevent reentrant firmware calls
in mixed mode
:::::: TO: Ard Biesheuvel <ard.biesheuvel(a)linaro.org>
:::::: CC: Ingo Molnar <mingo(a)kernel.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org