At 14:59 +0100 on 04 Aug (1438700361), Ian Campbell wrote: > On Tue, 2015-08-04 at 10:34 +0100, Wei Liu wrote: > > On Mon, Aug 03, 2015 at 06:05:43PM +0100, Andrew Cooper wrote: > > > It is not used, and can cause a spurious failure of the set_gdt() > > > hypercall in > > > low memory situations. > > > > > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > > > --- > > > CC: Jan Beulich <jbeul...@suse.com> > > > --- > > > > This array does appear to be write-only and never gets read. > > > > FWIW > > > > Reviewed-by: Wei Liu <wei.l...@citrix.com> > > The use was removed by: > > commit 5889a3e1d123bdad4a3d150310d647db55459966 > Author: Tim Deegan <t...@xen.org> > Date: Thu May 17 10:24:54 2012 +0100 > > x86/mm: Use get_page_from_gfn() instead of get_gfn()/put_gfn. > > Signed-off-by: Tim Deegan <t...@xen.org> > Signed-off-by: Andres Lagar-Cavilla <and...@lagarcavilla.org>
Yep, clearly an oversight in this patch. > after it was introduced by: > > commit 51032ca058e43fbd37ea1f7c7c003496f6451340 > Author: Andres Lagar-Cavilla <and...@lagarcavilla.org> > Date: Fri Nov 11 18:11:34 2011 +0000 > > Modify naming of queries into the p2m > [...] > Signed-off-by: Andres Lagar-Cavilla <and...@lagarcavilla.org> > Acked-by: Tim Deegan <t...@xen.org> > Committed-by: Keir Fraser <k...@xen.org> > > > It's obviously correct and fixes a problem, so it can be applied to 4.6 > > tree. +1, FWIW. Cheers, Tim. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
