[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 at 18:06, <andrew.cooper3@xxxxxxxxxx> wrote:
> 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.

Why would it be incorrect? It would be an optimization to use larger
pages for larger regions, but I don't see it being wrong.

> 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?)

Oops, yes, of course.


Xen-devel mailing list



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