Hi Joerg,

I love your patch! Yet something to improve:

[auto build test ERROR on kvm/linux-next]
[also build test ERROR on v5.6-rc2]
[cannot apply to tip/x86/core tip/x86/mm next-20200214]
[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/Joerg-Roedel/Linux-as-SEV-ES-Guest-Support/20200214-032020
base:   https://git.kernel.org/pub/scm/virt/kvm/kvm.git linux-next
config: x86_64-randconfig-s1-20200217 (attached as .config)
compiler: gcc-7 (Debian 7.5.0-5) 7.5.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   arch/x86//kernel/sev-es.c: In function 'encrypted_state_init_ghcbs':
>> arch/x86//kernel/sev-es.c:315:2: error: 'smp_ops' undeclared (first use in this function)
     smp_ops.play_dead = sev_es_play_dead;
     ^~~~~~~
   arch/x86//kernel/sev-es.c:315:2: note: each undeclared identifier is reported only once for each function it appears in
>> arch/x86//kernel/sev-es.c:296:26: error: 'native_play_dead' undeclared (first use in this function); did you mean 'native_io_delay'?
    #define sev_es_play_dead native_play_dead
                             ^
   arch/x86//kernel/sev-es.c:315:22: note: in expansion of macro 'sev_es_play_dead'
     smp_ops.play_dead = sev_es_play_dead;
                         ^~~~~~~~~~~~~~~~

vim +/smp_ops +315 arch/x86//kernel/sev-es.c

   279	
   280	void sev_es_play_dead(void)
   281	{
   282		play_dead_common();
   283	
   284		/* IRQs now disabled */
   285	
   286		sev_es_ap_hlt_loop();
   287	
   288		/*
   289		 * If we get here, the VCPU was woken up again. Jump to CPU
   290		 * startup code to get it back online.
   291		 */
   292	
   293		start_cpu();
   294	}
   295	#else  /* CONFIG_HOTPLUG_CPU */
 > 296	#define sev_es_play_dead	native_play_dead
   297	#endif /* CONFIG_HOTPLUG_CPU */
   298	
   299	void encrypted_state_init_ghcbs(void)
   300	{
   301		int cpu;
   302	
   303		if (!sev_es_active())
   304			return;
   305	
   306		/* Initialize per-cpu GHCB pages */
   307		for_each_possible_cpu(cpu) {
   308			struct ghcb *ghcb = &per_cpu(ghcb_page, cpu);
   309	
   310			set_memory_decrypted((unsigned long)ghcb,
   311					     sizeof(ghcb_page) >> PAGE_SHIFT);
   312			memset(ghcb, 0, sizeof(*ghcb));
   313		}
   314	
 > 315		smp_ops.play_dead = sev_es_play_dead;
   316	}
   317	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org