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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
