On 29.05.2020 13:11, Hongyan Xia wrote: > From: Wei Liu <wei.l...@citrix.com> > > Map and unmap pages instead of relying on the direct map. > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > Signed-off-by: Hongyan Xia <hongy...@amazon.com>
Reviewed-by: Jan Beulich <jbeul...@suse.com> albeit preferably with ... > --- a/xen/arch/x86/x86_64/mm.c > +++ b/xen/arch/x86/x86_64/mm.c > @@ -481,6 +481,7 @@ void __init paging_init(void) > l3_pgentry_t *l3_ro_mpt; > l2_pgentry_t *pl2e = NULL, *l2_ro_mpt = NULL; > struct page_info *l1_pg; > + mfn_t l3_ro_mpt_mfn, l2_ro_mpt_mfn; ... just a single variable named "mfn" here. Afaics the life ranges allow for this (imo) readability improvement. Jan
