[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86/p2m: clear_identity_p2m_entry() must cope with 'relaxed' RDM mode

On 12/08/15 16:05, Jan Beulich wrote:
> Tearing down a 1:1 mapping that was never established isn't really nice
> (and in fact hits an ASSERT() in p2m_remove_page()). Convert from a
> wrapper macro to a proper function which then can take care of the
> situation.
> Also take the opportunity to remove the 'page_order' parameter of
> clear_identity_p2m_entry(), to make it match set_identity_p2m_entry().

What about identity mapping with 2MB frames?  IMO, the RMRR code is
incorrect in assuming that only 4K frames will be wanted, but that is a
separate issue.

As this at least matches the set_identity side of things, and improves
the situation, so

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

(I presume you intended to include a SoB tag?)


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.