[Xen-devel] [PATCH] x86: fix !CONFIG_HVM build for clang 3.8

2018-10-04 Thread Wei Liu
It is discovered that hvm_funcs made it into monitor.o even when HVM is disabled. This version of clang doesn't seem to completely eliminate the code after is_hvm_domain() in arch_monitor_get_capabilities(). Signed-off-by: Wei Liu --- xen/include/asm-x86/hvm/hvm.h | 10 ++ xen/include/as

Re: [Xen-devel] [PATCH] x86: fix !CONFIG_HVM build for clang 3.8

2018-10-04 Thread Razvan Cojocaru
On 10/4/18 7:42 PM, Wei Liu wrote: > It is discovered that hvm_funcs made it into monitor.o even when HVM > is disabled. This version of clang doesn't seem to completely > eliminate the code after is_hvm_domain() in > arch_monitor_get_capabilities(). > > Signed-off-by: Wei Liu Acked-by: Razvan C

Re: [Xen-devel] [PATCH] x86: fix !CONFIG_HVM build for clang 3.8

2018-10-05 Thread Jan Beulich
>>> On 04.10.18 at 18:42, wrote: > It is discovered that hvm_funcs made it into monitor.o even when HVM > is disabled. This version of clang doesn't seem to completely > eliminate the code after is_hvm_domain() in > arch_monitor_get_capabilities(). > > Signed-off-by: Wei Liu Acked-by: Jan Beuli