From mboxrd@z Thu Jan  1 00:00:00 1970
From: Don Slutz <dslutz@verizon.com>
Subject: Re: [PATCH v4 04/16] xen: Add is_vmware_port_enabled
Date: Tue, 16 Sep 2014 12:20:47 -0400
Message-ID: <5418635F.3050001@terremark.com>
References: <1410460610-14759-1-git-send-email-dslutz@verizon.com>	<1410460610-14759-5-git-send-email-dslutz@verizon.com>
	<5412128C.9020001@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5412128C.9020001@citrix.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Andrew Cooper <andrew.cooper3@citrix.com>, Don Slutz <dslutz@verizon.com>, xen-devel@lists.xen.org
Cc: Kevin Tian <kevin.tian@intel.com>, Keir Fraser <keir@xen.org>, Ian Campbell <ian.campbell@citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, George Dunlap <George.Dunlap@eu.citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, Eddie Dong <eddie.dong@intel.com>, Tim Deegan <tim@xen.org>, Jan Beulich <jbeulich@suse.com>, Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>, Jun Nakajima <jun.nakajima@intel.com>, Boris Ostrovsky <boris.ostrovsky@oracle.com>, Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
List-Id: xen-devel@lists.xenproject.org

On 09/11/14 17:22, Andrew Cooper wrote:
> On 11/09/2014 19:36, Don Slutz wrote:
>> --- a/xen/arch/x86/hvm/vmx/vmx.c
>> +++ b/xen/arch/x86/hvm/vmx/vmx.c
>> @@ -44,6 +44,7 @@
>>   #include <asm/hvm/support.h>
>>   #include <asm/hvm/vmx/vmx.h>
>>   #include <asm/hvm/vmx/vmcs.h>
>> +#include <asm/hvm/vmport.h>
>>   #include <public/sched.h>
>>   #include <public/hvm/ioreq.h>
>>   #include <asm/hvm/vpic.h>
>> @@ -1279,6 +1280,8 @@ static void vmx_update_guest_cr(struct vcpu *v, unsigned int cr)
>>                   v->arch.hvm_vmx.exception_bitmap = HVM_TRAP_MASK
>>                             | (paging_mode_hap(v->domain) ?
>>                                0 : (1U << TRAP_page_fault))
>> +                          | v->domain->arch.hvm_domain.is_vmware_port_enabled ?
>> +                             (1U << TRAP_gp_fault) : 0
> I am fairly certain that you need some brackets here.

I will add hem.
    -Don Slutz

> ~Andrew
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel