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

Re: [Xen-devel] [PATCH 05/14] x86/p2m: Don't overwrite p2m_altp2m_lazy_copy()'s callers p2m pointer



>>> On 21.11.18 at 14:21, <andrew.cooper3@xxxxxxxxxx> wrote:
> The final parameter to p2m_altp2m_lazy_copy() appears to be unnecessary, and
> results in very hard-to-follow code.  Have the sole caller set its local p2m
> pointer appropriately, and drop the parameter.
> 
> With that done, a level of indirection of ap2m can be dropped inside
> p2m_altp2m_lazy_copy().  While changing the API, switch it from bool_t to
> bool, and drop printing of the altp2m's virtual address, because it is of no
> use even when debugging.
> 
> No (intended) overall change in behaviour.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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