Re: [Xen-devel] [PATCH] x86/HVM: restore parentheses lost by 3d4d4f9336

[Don Slutz <dslutz@xxxxxxxxxxx>]

On 09/03/14 08:51, Jan Beulich wrote:
> ("x86: consolidate boolean inputs in hvm and p2m into a shared bitmap")
>
> Coverity-ID: 1234366
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Looks good.

Reviewed-by:  Don Slutz <dslutz@xxxxxxxxxxx>



> --- a/xen/arch/x86/hvm/hvm.c
> +++ b/xen/arch/x86/hvm/hvm.c
> @@ -2796,7 +2796,7 @@ int hvm_hap_nested_page_fault(paddr_t gp
>   
>       p2m = p2m_get_hostp2m(v->domain);
>       mfn = get_gfn_type_access(p2m, gfn, &p2mt, &p2ma,
> -                              P2M_ALLOC | npfec.write_access ? P2M_UNSHARE : 0,
> +                              P2M_ALLOC | (npfec.write_access ? P2M_UNSHARE : 
> 0),
>                                 NULL);
>   
>       /* Check access permissions first, then handle faults */
>
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel