Re: [Xen-devel] [PATCH 6/8] x86/HVM: Hook up miscellaneous AVIC functions

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 04.04.18 at 01:01, <Janakarajan.Natarajan@xxxxxxx> wrote:
> --- a/xen/arch/x86/hvm/svm/svm.c
> +++ b/xen/arch/x86/hvm/svm/svm.c
> @@ -1711,7 +1711,10 @@ const struct hvm_function_table * __init 
> start_svm(void)
>          svm_avic = 0;
>  
>      if ( svm_avic )
> +    {
>          svm_function_table.deliver_posted_intr  = 
> svm_avic_deliver_posted_intr;
> +        svm_function_table.virtual_intr_delivery_enabled = svm_avic;

Considering the controlling expression of the if() we're in, why not simply 
"true"?

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel