On Sat, Aug 08, 2020 at 11:35:54PM +0800, Jiaxun Yang wrote:
在 2020/8/8 下午11:31, Greg KH 写道:
On Sat, Aug 08, 2020 at 08:50:52PM +0800, Huacai Chen wrote:
Only Loongson64 platform has and needs loongson_regs.h, including it unconditionally will cause build errors.
Fixes: 7f2a83f1c2a941ebfee5 ("KVM: MIPS: Add CPUCFG emulation for Loongson-3") Cc: stable@vger.kernel.org Reported-by: kernel test robot lkp@intel.com Signed-off-by: Huacai Chen chenhc@lemote.com
arch/mips/kvm/vz.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/arch/mips/kvm/vz.c b/arch/mips/kvm/vz.c index 3932f76..a474578 100644 --- a/arch/mips/kvm/vz.c +++ b/arch/mips/kvm/vz.c @@ -29,7 +29,9 @@ #include <linux/kvm_host.h> #include "interrupt.h" +#ifdef CONFIG_CPU_LOONGSON64 #include "loongson_regs.h" +#endif
The fix for this should be in the .h file, no #ifdef should be needed in a .c file.
The header file can only be reached when CONFIG_CPU_LOONGSON64 is selected... Otherwise the include path of this file won't be a part of CFLAGS.
That sounds like you should fix up the path of this file in the #include line as well :)