On Fri, Feb 25, 2022 at 12:49 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > > Hi Rafael, > > On 2/25/22 00:24, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git bleeding-edge > > head: 8276cbee5a52543c614a1e1bc2624188d5970848 > > commit: 62fabd56faafe033eb0be3ba24000b8db13d4c17 [59/73] x86/PCI: Disable exclusion of E820 reserved addresses in some cases > > config: x86_64-randconfig-a012 (https://download.01.org/0day-ci/archive/20220225/202202250758.oDiHyXAy-lkp@xxxxxxxxx/config) > > compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e) > > reproduce (this is a W=1 build): > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/?id=62fabd56faafe033eb0be3ba24000b8db13d4c17 > > git remote add rafael-pm https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git > > git fetch --no-tags rafael-pm bleeding-edge > > git checkout 62fabd56faafe033eb0be3ba24000b8db13d4c17 > > # save the config file to linux build tree > > mkdir build_dir > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot <lkp@xxxxxxxxx> > > > > All errors (new ones prefixed by >>): > > > > In file included from arch/x86/kernel/resource.c:4: > > arch/x86/include/asm/pci_x86.h:97:8: error: unknown type name 'raw_spinlock_t' > > extern raw_spinlock_t pci_config_lock; > > ^ > >>> arch/x86/include/asm/pci_x86.h:133:19: error: expected ';' after top level declarator > > extern void __init dmi_check_pciprobe(void); > > ^ > > ; > > arch/x86/include/asm/pci_x86.h:134:19: error: expected ';' after top level declarator > > extern void __init dmi_check_skip_isa_align(void); > > ^ > > ; > > So the problem here seems to be that arch/x86/include/asm/pci_x86.h does not include > all the headers which it relies on. Instead it release on the files which include it > to include those headers before hand. > > This should fix the errors reported here: > > --- a/arch/x86/include/asm/pci_x86.h > +++ b/arch/x86/include/asm/pci_x86.h > @@ -5,7 +5,9 @@ > * (c) 1999 Martin Mares <mj@xxxxxx> > */ > > +#include <linux/init.h> > #include <linux/ioport.h> > +#include <linux/spinlock.h> > > #undef DEBUG > > I believe this is best squashed into the original commit, Done, thanks! > but let me know if > you want me to send this out as a proper follow up patch; or if you want a > new version of the troublesome commit to replace the original one.
