[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.