>>> On 04.08.15 at 15:59, <[email protected]> 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 <[email protected]> >> > --- >> > CC: Jan Beulich <[email protected]> >> > --- >> >> This array does appear to be write-only and never gets read. >> >> FWIW >> >> Reviewed-by: Wei Liu <[email protected]> > > The use was removed by: > > commit 5889a3e1d123bdad4a3d150310d647db55459966 > Author: Tim Deegan <[email protected]> > 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 <[email protected]> > Signed-off-by: Andres Lagar-Cavilla <[email protected]> > > after it was introduced by: > > commit 51032ca058e43fbd37ea1f7c7c003496f6451340 > Author: Andres Lagar-Cavilla <[email protected]> > Date: Fri Nov 11 18:11:34 2011 +0000 > > Modify naming of queries into the p2m > [...] > Signed-off-by: Andres Lagar-Cavilla <[email protected]> > Acked-by: Tim Deegan <[email protected]> > Committed-by: Keir Fraser <[email protected]>
Indeed pointing out the commit that introduced the unused array right in the commit message would have helped both in review and in determination whether (and to which branches) to backport. Jan _______________________________________________ Xen-devel mailing list [email protected] http://lists.xen.org/xen-devel
