[kbuild-all] [ardb:for-kernelci 3/4] arch/x86/platform/efi/efi_64.c:810:31: error: assignment to 'efi_status_t efi_char16_t const efi_guid_t u32, long unsigned int, void {aka 'long unsigned int short unsigned int const struct <anonymous> unsigned int, l...

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&gt; 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': 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&gt; :::::: CC: Ingo Molnar <mingo(a)kernel.org&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org