From mboxrd@z Thu Jan  1 00:00:00 1970
From: Olaf Hering <olaf@aepfle.de>
Subject: Re: [PATCH] x86/hvm: implement save/restore for posted
 interrupts
Date: Thu, 9 Oct 2014 15:31:16 +0200
Message-ID: <20141009133116.GA8259@aepfle.de>
References: <20140919133227.GA2611@aepfle.de>
	<542AF41C020000780003B2EC@mail.emea.novell.com>
	<20141001080706.GA11230@aepfle.de>
	<542BD772020000780003B5ED@mail.emea.novell.com>
	<20141001201302.GA30085@aepfle.de>
	<A9667DDFB95DB7438FA9D7D576C3D87E0ABA67F8@SHSMSX104.ccr.corp.intel.com>
	<20141008082846.GA4978@aepfle.de>
	<A9667DDFB95DB7438FA9D7D576C3D87E0ABA68A1@SHSMSX104.ccr.corp.intel.com>
	<20141008085411.GA6512@aepfle.de>
	<A9667DDFB95DB7438FA9D7D576C3D87E0ABA76D8@SHSMSX104.ccr.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Content-Disposition: inline
In-Reply-To: <A9667DDFB95DB7438FA9D7D576C3D87E0ABA76D8@SHSMSX104.ccr.corp.intel.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: "Zhang, Yang Z" <yang.z.zhang@intel.com>
Cc: "Tian, Kevin" <kevin.tian@intel.com>, "Dong, Eddie" <eddie.dong@intel.com>, "Nakajima, Jun" <jun.nakajima@intel.com>, Jan Beulich <JBeulich@suse.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On Thu, Oct 09, Zhang, Yang Z wrote:

> +    for ( vector = 0; vector < NR_VECTORS; vector++ )
> +        if (vlapic_test_vector(vector, &s->regs->data[APIC_TMR]))
> +            set_bit(vector,  v->arch.hvm_vmx.eoi_exit_bitmap);

Did you send out the wrong version a few weeks ago? I tested the old
patch at that time. Now I notice the new version has this change:

-        if (vlapic_test_vector(vector, &s->regs->data[APIC_ISR]))$
+        if (vlapic_test_vector(vector, &s->regs->data[APIC_TMR]))$

I will try the new version.

Olaf