From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jan Beulich" <JBeulich@suse.com>
Subject: Re: [PATCH 2/2] x86/VPMU: Handle APIC_LVTPC accesses
Date: Thu, 29 Jan 2015 11:54:47 +0000
Message-ID: <54CA2D97020000780005ACA9@mail.emea.novell.com>
References: <1422474991-2001-1-git-send-email-boris.ostrovsky@oracle.com>
	<1422474991-2001-3-git-send-email-boris.ostrovsky@oracle.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta14.messagelabs.com ([193.109.254.103])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <JBeulich@suse.com>) id 1YGngP-0006Lo-CK
	for xen-devel@lists.xenproject.org; Thu, 29 Jan 2015 11:54:49 +0000
In-Reply-To: <1422474991-2001-3-git-send-email-boris.ostrovsky@oracle.com>
Content-Disposition: inline
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: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: andrew.cooper3@citrix.com, kevin.tian@intel.com, dietmar.hahn@ts.fujitsu.com, xen-devel@lists.xenproject.org
List-Id: xen-devel@lists.xenproject.org

>>> On 28.01.15 at 20:56, <boris.ostrovsky@oracle.com> wrote:
> Don't have the hypervisor update APIC_LVTPC when _it_ thinks the vector
> should be updated. Instead, handle guest's APIC_LVTPC accesses and write 
> what
> the guest explicitly wanted (but only when VPMU is enabled).
> 
> This is updated version of commit 8097616fbdda that was reverted by
> cc3404093c85. Unlike the previous version, we don't update APIC_LVTPC
> when VPMU is disabled to avoid interfering with NMI watchdog (which
> runs only when VPMU is off).
> 
> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Acked-by: Kevin Tian <kevin.tian@intel.com>
> Reviewed-by: Dietmar Hahn <dietmar.hahn@ts.fujitsu.com>
> Tested-by: Dietmar Hahn <dietmar.hahn@ts.fujitsu.com>

Even leaving aside the functionality change, on an updated patch
the previous version of which needed to be reverted, retaining
_any_ such tags is wrong from my pov. I asked you before to be
more conservative with retaining tags, and I'm now going to reserve
the right to no longer point out such issues, but silently drop such
patches coming from you.

Jan