From mboxrd@z Thu Jan 1 00:00:00 1970 From: Razvan Cojocaru Subject: Re: [PATCH V7 for-4.5 4/4] xen: Handle resumed instruction based on previous mem_event reply Date: Thu, 11 Sep 2014 16:29:26 +0300 Message-ID: <5411A3B6.5010601@bitdefender.com> References: <1410441334-10603-1-git-send-email-rcojocaru@bitdefender.com> <1410441334-10603-5-git-send-email-rcojocaru@bitdefender.com> <5411BEFD0200007800033E7C@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1XS4R9-0000C5-0Z for xen-devel@lists.xenproject.org; Thu, 11 Sep 2014 13:29:23 +0000 Received: from smtp02.buh.bitdefender.net (smtp.bitdefender.biz [10.17.80.76]) by mx-sr.buh.bitdefender.com (Postfix) with ESMTP id 2425C80086 for ; Thu, 11 Sep 2014 16:29:20 +0300 (EEST) In-Reply-To: <5411BEFD0200007800033E7C@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: kevin.tian@intel.com, keir@xen.org, ian.campbell@citrix.com, stefano.stabellini@eu.citrix.com, andrew.cooper3@citrix.com, eddie.dong@intel.com, tim@xen.org, jun.nakajima@intel.com, xen-devel@lists.xenproject.org, ian.jackson@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On 09/11/2014 04:25 PM, Jan Beulich wrote: >>>> On 11.09.14 at 15:15, wrote: >> In a scenario where a page fault that triggered a mem_event occured, >> p2m_mem_access_check() will now be able to either 1) emulate the >> current instruction, or 2) emulate it, but don't allow it to perform >> any writes. >> >> Signed-off-by: Razvan Cojocaru >> Acked-by: Kevin Tian >> >> --- >> Changes since V6: >> - Removed filtering for GLA events: all events are now being sent. > > With a functional change like this not requested by the person > having given their ack under that condition, any earlier ack > necessarily becomes invalid imo. I understand. Please consider it un-Acked in that case, and if a new series comes about (without Kevin't explicit Ack) I'll make sure to remove it. Thanks, Razvan Cojocaru