On Mon, 2015-04-20 at 17:06 +0200, Tamas K Lengyel wrote: > -paddr_t p2m_lookup(struct domain *d, paddr_t paddr, p2m_type_t *t) > +static paddr_t __p2m_lookup(struct domain *d, paddr_t paddr, p2m_type_t *t)
I'd have been inclined to make this take a p2m_domain* rather than a domain*, on the basis that the caller must have one in hand to have locked it. I don't think __p2m_lookup uses d other than to find the p2m. Not worth changing unless there is some other reason to resend though, so with or without that: Acked-by: Ian Campbell <ian.campb...@citrix.com> Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel