On 30.11.2020 15:13, Hongyan Xia wrote: > On Mon, 2020-11-30 at 13:50 +0100, Jan Beulich wrote: >> Not very efficient, but not needed anywhere anyway - the sole user of >> the construct is domain_page_map_to_mfn(), which maps only individual >> pages. (An even better option would be to avoid the recurring walk of >> the higher levels by using only virt_to_xen_l3e() and then doing the >> remaining steps "by hand". This would at once allow to avoid the here >> unwanted / unneeded checking for whether page tables need >> allocating.) > > The "even better option" looks more promising to me, and is what I want > to go forward with. At any rate, this fix grows larger than intended > and I want to send this as an individual patch. Any objections?
Definitely not - separate changes are almost always easier to look at and faster to get in. Jan
