From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH] ept: Fix bug in changeset 22526:7a5ee3800417 Date: Thu, 17 Mar 2011 17:45:24 +0000 Message-ID: <1300383924.7041.21.camel@elijah> References: <56b32e6fd3158e34288e.1300203711@elijah> <19842.13401.654173.18115@mariner.uk.xensource.com> <20110317163057.GB5200@whitby.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20110317163057.GB5200@whitby.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Tim Deegan Cc: George Dunlap , "xen-devel@lists.xensource.com" , Ian Jackson List-Id: xen-devel@lists.xenproject.org On Thu, 2011-03-17 at 16:30 +0000, Tim Deegan wrote: > At 16:18 +0000 on 17 Mar (1300378713), Ian Jackson wrote: > > George Dunlap writes ("[Xen-devel] [PATCH] ept: Fix bug in changeset 22526:7a5ee3800417"): > > > This fixes a bug in changeset 22526:7a5ee3800417, where the wrong > > > value is read when deciding whether to flush the VTd tables. The > > > effect is minor: in situations where the p2m entry is changed but the > > > mfn is the same, the VTd tables will be unnecessarily flushed. > > > > Does this need applying to 4.1 before the release ? > > I don't think so, but it should be backported afterwards. I tend to agree. -George