Hi Sean, I love your patch! Yet something to improve:
[auto build test ERROR on tip/x86/core] [also build test ERROR on tip/auto-latest v5.7 next-20200605] [cannot apply to bp/for-next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Sean-Christopherson/x86-cpu-Reinitialize-IA32_FEAT_CTL-MSR-on-BSP-during-wakeup/20200606-040921 base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git 9cb1fd0efd195590b828b9b865421ad345a4a145 config: i386-tinyconfig (attached as .config) compiler: gcc-9 (Debian 9.3.0-13) 9.3.0 reproduce (this is a W=1 build): # save the attached .config to linux build tree make W=1 ARCH=i386 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <l...@intel.com> All errors (new ones prefixed by >>, old ones prefixed by <<): arch/x86/kernel/cpu/centaur.c: In function 'init_centaur': >> arch/x86/kernel/cpu/centaur.c:219:2: error: implicit declaration of function >> 'init_ia32_feat_ctl' [-Werror=implicit-function-declaration] 219 | init_ia32_feat_ctl(c); | ^~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors -- arch/x86/kernel/cpu/zhaoxin.c: In function 'init_zhaoxin': >> arch/x86/kernel/cpu/zhaoxin.c:110:2: error: implicit declaration of function >> 'init_ia32_feat_ctl' [-Werror=implicit-function-declaration] 110 | init_ia32_feat_ctl(c); | ^~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +/init_ia32_feat_ctl +219 arch/x86/kernel/cpu/centaur.c 60882cc159e141 arch/x86/kernel/cpu/centaur.c David Wang 2018-04-20 218 501444905fcb41 arch/x86/kernel/cpu/centaur.c Sean Christopherson 2019-12-20 @219 init_ia32_feat_ctl(c); ^1da177e4c3f41 arch/i386/kernel/cpu/centaur.c Linus Torvalds 2005-04-16 220 } ^1da177e4c3f41 arch/i386/kernel/cpu/centaur.c Linus Torvalds 2005-04-16 221 :::::: The code at line 219 was first introduced by commit :::::: 501444905fcb4166589fda99497c273ac5efc65e x86/centaur: Use common IA32_FEAT_CTL MSR initialization :::::: TO: Sean Christopherson <sean.j.christopher...@intel.com> :::::: CC: Borislav Petkov <b...@suse.de> --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
