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

Re: [Xen-devel] [PATCH v5 7/8] xen: arm: use superpages in p2m when pages are suitably aligned

On Wed, 2014-07-09 at 17:28 +0100, Julien Grall wrote:
> > +        /*
> > +         * could flush up to the next boundary, but would need to be
> > +         * careful about preemption, so just do one page now and loop.
> At the first glance I though this code didn't handle correctly superpage
> flush. It took me several minutes to understand that you rely
> P2M_ONE_PROGRESS{,_NOP} to do the job for you.
> Can you explain this trick in the commit message?

That's what the existing comment which you've quoted is attempting to
explain. Is it not clear enough? Can you suggest an improvement?

Would "up to the next superpage boundary ... do one 4K page now and
loop" suffice?

> With this changes:
> Acked-by: Julien Grall <julien.grall@xxxxxxxxxx>
> Regards,

Xen-devel mailing list



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