Re: [PATCH v6 1/6] acpi: Create subtable parsing infrastructure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Keith,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.20-rc7 next-20181218]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Dan-Williams/mm-Randomize-free-memory/20181218-130230
config: arm64-defconfig (attached as .config)
compiler: aarch64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.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=7.2.0 make.cross ARCH=arm64 

All errors (new ones prefixed by >>):

   arch/arm64/kernel/smp.c: In function 'acpi_parse_and_init_cpus':
>> arch/arm64/kernel/smp.c:577:11: error: passing argument 2 of 'acpi_table_parse_madt' from incompatible pointer type [-Werror=incompatible-pointer-types]
              acpi_parse_gic_cpu_interface, 0);
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/arm64/kernel/smp.c:20:0:
   include/linux/acpi.h:249:5: note: expected 'acpi_tbl_entry_handler {aka int (*)(union acpi_subtable_headers *, const long unsigned int)}' but argument is of type 'int (*)(struct acpi_subtable_header *, const long unsigned int)'
    int acpi_table_parse_madt(enum acpi_madt_type id,
        ^~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   arch/arm64/kernel/acpi_numa.c: In function 'acpi_map_cpus_to_nodes':
>> arch/arm64/kernel/acpi_numa.c:88:10: error: passing argument 4 of 'acpi_table_parse_entries' from incompatible pointer type [-Werror=incompatible-pointer-types]
             acpi_parse_gicc_pxm, 0);
             ^~~~~~~~~~~~~~~~~~~
   In file included from arch/arm64/kernel/acpi_numa.c:19:0:
   include/linux/acpi.h:242:12: note: expected 'acpi_tbl_entry_handler {aka int (*)(union acpi_subtable_headers *, const long unsigned int)}' but argument is of type 'int (*)(struct acpi_subtable_header *, const long unsigned int)'
    int __init acpi_table_parse_entries(char *id, unsigned long table_size,
               ^~~~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/acpi_table_parse_madt +577 arch/arm64/kernel/smp.c

e1896249 Lorenzo Pieralisi 2018-06-25  566  
e1896249 Lorenzo Pieralisi 2018-06-25  567  static void __init acpi_parse_and_init_cpus(void)
e1896249 Lorenzo Pieralisi 2018-06-25  568  {
e1896249 Lorenzo Pieralisi 2018-06-25  569  	int i;
e1896249 Lorenzo Pieralisi 2018-06-25  570  
e1896249 Lorenzo Pieralisi 2018-06-25  571  	/*
e1896249 Lorenzo Pieralisi 2018-06-25  572  	 * do a walk of MADT to determine how many CPUs
e1896249 Lorenzo Pieralisi 2018-06-25  573  	 * we have including disabled CPUs, and get information
e1896249 Lorenzo Pieralisi 2018-06-25  574  	 * we need for SMP init.
e1896249 Lorenzo Pieralisi 2018-06-25  575  	 */
e1896249 Lorenzo Pieralisi 2018-06-25  576  	acpi_table_parse_madt(ACPI_MADT_TYPE_GENERIC_INTERRUPT,
e1896249 Lorenzo Pieralisi 2018-06-25 @577  				      acpi_parse_gic_cpu_interface, 0);
e1896249 Lorenzo Pieralisi 2018-06-25  578  
e1896249 Lorenzo Pieralisi 2018-06-25  579  	/*
e1896249 Lorenzo Pieralisi 2018-06-25  580  	 * In ACPI, SMP and CPU NUMA information is provided in separate
e1896249 Lorenzo Pieralisi 2018-06-25  581  	 * static tables, namely the MADT and the SRAT.
e1896249 Lorenzo Pieralisi 2018-06-25  582  	 *
e1896249 Lorenzo Pieralisi 2018-06-25  583  	 * Thus, it is simpler to first create the cpu logical map through
e1896249 Lorenzo Pieralisi 2018-06-25  584  	 * an MADT walk and then map the logical cpus to their node ids
e1896249 Lorenzo Pieralisi 2018-06-25  585  	 * as separate steps.
e1896249 Lorenzo Pieralisi 2018-06-25  586  	 */
e1896249 Lorenzo Pieralisi 2018-06-25  587  	acpi_map_cpus_to_nodes();
e1896249 Lorenzo Pieralisi 2018-06-25  588  
e1896249 Lorenzo Pieralisi 2018-06-25  589  	for (i = 0; i < nr_cpu_ids; i++)
e1896249 Lorenzo Pieralisi 2018-06-25  590  		early_map_cpu_to_node(i, acpi_numa_get_nid(i));
e1896249 Lorenzo Pieralisi 2018-06-25  591  }
0f078336 Lorenzo Pieralisi 2015-05-13  592  #else
e1896249 Lorenzo Pieralisi 2018-06-25  593  #define acpi_parse_and_init_cpus(...)	do { } while (0)
0f078336 Lorenzo Pieralisi 2015-05-13  594  #endif
0f078336 Lorenzo Pieralisi 2015-05-13  595  

:::::: The code at line 577 was first introduced by commit
:::::: e189624916961c735c18e3c75acc478661403830 arm64: numa: rework ACPI NUMA initialization

:::::: TO: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
:::::: CC: Will Deacon <will.deacon@xxxxxxx>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux