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