Re: [Xen-devel] [PATCH] xen: arm: Support <32MB frametables

2015-08-14 Thread Julien Grall
On 07/08/15 18:29, Chris (Christopher) Brand wrote: > Hi Julien, > > Thanks for the review. > >> OOI, you win 30MB of RAM but how does this affect the performance? > > Fair question. :-) All I can say is that I don't see any noticeable > difference on my > system. Are there performance tests th

Re: [Xen-devel] [PATCH] xen: arm: Support <32MB frametables

2015-08-07 Thread Chris (Christopher) Brand
>> OOI, you win 30MB of RAM but how does this affect the performance? > > Fair question. :-) All I can say is that I don't see any noticeable > difference on my system. > Are there performance tests that you suggest I run ? > Also, note that the new code is > only executed if you specify a previou

Re: [Xen-devel] [PATCH] xen: arm: Support <32MB frametables

2015-08-07 Thread Chris (Christopher) Brand
Hi Julien, Thanks for the review. > OOI, you win 30MB of RAM but how does this affect the performance? Fair question. :-) All I can say is that I don't see any noticeable difference on my system. Are there performance tests that you suggest I run ? Also, note that the new code is only executed

Re: [Xen-devel] [PATCH] xen: arm: Support <32MB frametables

2015-08-07 Thread Julien Grall
Hi Chris, On 06/08/15 18:54, Chris (Christopher) Brand wrote: > setup_frametable_mappings() rounds frametable_size up to a multiple > of 32MB. This is wasteful on systems with less than 4GB of RAM, > although it does allow the "contig" bit to be set in the PTEs. > > Where the frametable is less t

[Xen-devel] [PATCH] xen: arm: Support <32MB frametables

2015-08-06 Thread Chris (Christopher) Brand
setup_frametable_mappings() rounds frametable_size up to a multiple of 32MB. This is wasteful on systems with less than 4GB of RAM, although it does allow the "contig" bit to be set in the PTEs. Where the frametable is less than 32MB in size, instead round up to a multiple of 2MB, not setting the