Re: [PATCH 03/12] ras: add estatus vendor handling and processing

[kernel test robot <lkp@intel.com>]

Hi Ahmed,

kernel test robot noticed the following build errors:

[auto build test ERROR on efi/next]
[also build test ERROR on rafael-pm/linux-next rafael-pm/bleeding-edge robh/for-next arm64/for-next/core linus/master v6.19-rc1 next-20251219]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Ahmed-Tiba/ras-add-estatus-core-interfaces/20251217-200718
base:   https://git.kernel.org/pub/scm/linux/kernel/git/efi/efi.git next
patch link:    https://lore.kernel.org/r/20251217112845.1814119-4-ahmed.tiba%40arm.com
patch subject: [PATCH 03/12] ras: add estatus vendor handling and processing
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251222/202512220010.sdcS5LYV-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251222/202512220010.sdcS5LYV-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202512220010.sdcS5LYV-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/firmware/efi/estatus.c: In function 'estatus_source_fixmap':
>> drivers/firmware/efi/estatus.c:126:24: error: 'FIX_HOLE' undeclared (first use in this function)
     126 |                 return FIX_HOLE;
         |                        ^~~~~~~~
   drivers/firmware/efi/estatus.c:126:24: note: each undeclared identifier is reported only once for each function it appears in
   drivers/firmware/efi/estatus.c: In function 'estatus_handle_arm_hw_error':
   drivers/firmware/efi/estatus.c:656:9: error: too few arguments to function 'log_arm_hw_error'
     656 |         log_arm_hw_error(err);
         |         ^~~~~~~~~~~~~~~~
   In file included from drivers/firmware/efi/estatus.c:23:
   include/linux/ras.h:27:6: note: declared here
      27 | void log_arm_hw_error(struct cper_sec_proc_arm *err, const u8 sev);
         |      ^~~~~~~~~~~~~~~~
   In file included from drivers/firmware/efi/estatus.c:30:
   drivers/firmware/efi/estatus.c: In function 'estatus_do_proc':
   include/linux/estatus.h:231:71: warning: passing argument 2 of 'estatus_section_iter_next' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     231 |              ((_section) = estatus_section_iter_next(&__estatus_iter, (_estatus))); \
         |                                                                       ^~~~~~~~~~
   drivers/firmware/efi/estatus.c:817:9: note: in expansion of macro 'estatus_for_each_section'
     817 |         estatus_for_each_section(estatus, gdata) {
         |         ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/estatus.h:208:51: note: expected 'struct acpi_hest_generic_status *' but argument is of type 'const struct acpi_hest_generic_status *'
     208 |                           estatus_generic_status *estatus)
         |                                                   ^
   drivers/firmware/efi/estatus.c: At top level:
   drivers/firmware/efi/estatus.c:947:13: warning: 'estatus_print_queued_estatus' defined but not used [-Wunused-function]
     947 | static void estatus_print_queued_estatus(void)
         |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/firmware/efi/estatus.c:118:12: warning: 'estatus_panic_timeout' defined but not used [-Wunused-variable]
     118 | static int estatus_panic_timeout __read_mostly = 30;
         |            ^~~~~~~~~~~~~~~~~~~~~


vim +/FIX_HOLE +126 drivers/firmware/efi/estatus.c

   122	
   123	static enum fixed_addresses estatus_source_fixmap(struct estatus_source *source)
   124	{
   125		if (WARN_ON_ONCE(!source->fixmap_idx))
 > 126			return FIX_HOLE;
   127	
   128		return source->fixmap_idx;
   129	}
   130	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki