RE: Re: [Xen-devel] [PATCH] ept: Fix bug in changeset 22526:7a5ee3800417

[George Dunlap <george.dunlap@xxxxxxxxxx>]

I think so, yes.

 -George

On Wed, 2011-03-16 at 11:08 +0000, MaoXiaoyun wrote:
>  
> So system doesn't use VTD is not affectted, right?
>  
> > Date: Wed, 16 Mar 2011 09:40:15 +0000
> > From: Tim Deegan <Tim.Deegan@xxxxxxxxxx>
> > Subject: Re: [Xen-devel] [PATCH] ept: Fix bug in changeset
> > 22526:7a5ee3800417
> > To: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> > Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
> > 
> > At 15:41 +0000 on 15 Mar (1300203711), George Dunlap wrote:
> > > 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.
> > > 
> > > old_entry is left untouched in the second case, since having a
> present
> > > old_entry will cause the an unnecessary check to be taken at the
> end
> > > of the function.
> > > 
> > > Spotted-by: Zhang, Peng Fei <zpfalpc23@xxxxxxxxx>
> > > Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> > 
> > Applied, thanks.
> > 
> > Tim.
> > 
> > -- 
> > Tim Deegan <Tim.Deegan@xxxxxxxxxx>
> > Principal Software Engineer, Xen Platform Team
> > Citrix Systems UK Ltd. (Company #02937203, SL9 0BG)
> > 
> > 
> > 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel