The sole caller doesn't check the return value, and both vendors implement the hook.
Signed-off-by: Andrew Cooper <[email protected]> --- CC: Jan Beulich <[email protected]> CC: Roger Pau Monné <[email protected]> CC: Wei Liu <[email protected]> CC: Tamas K Lengyel <[email protected]> CC: Alexandru Isaila <[email protected]> CC: Petre Pircalabu <[email protected]> --- xen/include/asm-x86/hvm/hvm.h | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/xen/include/asm-x86/hvm/hvm.h b/xen/include/asm-x86/hvm/hvm.h index bd2cbb0e7baf..2767427aa938 100644 --- a/xen/include/asm-x86/hvm/hvm.h +++ b/xen/include/asm-x86/hvm/hvm.h @@ -631,15 +631,9 @@ static inline enum hvm_intblk nhvm_interrupt_blocked(struct vcpu *v) return hvm_funcs.nhvm_intr_blocked(v); } -static inline bool_t hvm_enable_msr_interception(struct domain *d, uint32_t msr) +static inline void hvm_enable_msr_interception(struct domain *d, uint32_t msr) { - if ( hvm_funcs.enable_msr_interception ) - { - hvm_funcs.enable_msr_interception(d, msr); - return 1; - } - - return 0; + hvm_funcs.enable_msr_interception(d, msr); } static inline bool_t hvm_is_singlestep_supported(void) -- 2.11.0
