Re: [Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-30 Thread Julien Grall
Hi Ian, On 25/11/15 12:40, Ian Campbell wrote: > On Wed, 2015-11-18 at 15:49 +, Julien Grall wrote: >> Currently, the translation table is left in place even if no entries is >> inuse. Because of how the p2m code has been implemented, replacing a >> translation table by a block (i.e

Re: [Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 14:26 +, Julien Grall wrote: > + > > > +p2m->stats.mappings[level - 1]--; > > > +update_reference_mapping(pages[level - 1], > > > old_entry, *entry); > > > + > > > +/* > > > + * We can't free the page now

Re: [Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-25 Thread Ian Campbell
On Wed, 2015-11-18 at 15:49 +, Julien Grall wrote: > Currently, the translation table is left in place even if no entries is > inuse. Because of how the p2m code has been implemented, replacing a > translation table by a block (i.e superpage) is not supported. Therefore, > any mapping of a

[Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-18 Thread Julien Grall
Currently, the translation table is left in place even if no entries is inuse. Because of how the p2m code has been implemented, replacing a translation table by a block (i.e superpage) is not supported. Therefore, any mapping of a superpage size will be split in smaller chunks making the