On Monday, December 14, 2015 at 08:48:16 AM, Pavel Machek wrote:
Hi!
This patch has several effects:
- it selects proper ARMv7 translation table level 1 bit definitions;
- it provides proper ARMv7 definitions for WT/WB/WA;
- it selects proper ARMv7 settings for TTBR0.
All these are correct as per the docs I have (although I may have missed something during the readings (and cross-readings with Marek) of these last hours/days.
Now, one specific effect goes against performance, and it is the setting of bit S in all TT entries. This bit makes the corresponding region shareable, but for all I know, in U-Boot we don't have more than one core accessing the same memory or registers sets so -- at least for the major part of its execution -- there is no reason for any region to be shareable.
Well, I'm currently working on AMP patch, which will mean two processors at the same time in u-boot.
Also... we provide memory modify operations for the user. User may be trying to communicate with second core.
You have a special usecase, so you should tweak your U-Boot for that using some sort of config option.
/That/ effect I certainly don't want.
How big is the slowdown from S bit?
30-40% .
Best regards, Marek Vasut