lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 23 Apr 2012 20:52:14 +0100 From: Russell King - ARM Linux <linux@....linux.org.uk> To: Pavel Machek <pavel@...x.de> Cc: kernel list <linux-kernel@...r.kernel.org>, linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>, wd@...x.de Subject: Re: Sharing header files -- arch/arm/mach-generic ? On Mon, Apr 23, 2012 at 09:40:42PM +0200, Pavel Machek wrote: > I have new architecture I'd like to merge into the mainline. It has > some code very similar to existing plaforms (for example > real differences to mach-realview/platsmp.c are two lines.) I suspect there's more differences than just that, because 17 lines in that file are very specific to realview. I doubt that you even need a scu_base_addr() function as you probably know where the SCU is located. So, I bet the reason your example is just two lines different from mach-realview/platsmp.c is because someone copied and modified the file rather than thinking about what's really required. > Should mach-generic/platsmp.c be created, so code can be better > shared? Or are the code pieces small enough that this duplication can > be ignored? It should be ignored. > headsmp.S is another candidate for sharing. AFAICT we have 4 times > identical code in the tree already: > > ./mach-exynos/headsmp.S: ldmia r4, {r5, r6} > ./plat-versatile/headsmp.S: ldmia r4, {r5, r6} > ./mach-ux500/headsmp.S: ldmia r4, {r5, r6} > ./mach-msm/headsmp.S: ldmia r4, {r5, r6} Again, because of standard 'lets copy and modify' behaviour. I'd really like to drop the holding pen from versatile stuff - it's not really required, as we can wake each CPU individually. The only reason it hasn't happened is because the holding pen stuff is also used for holding CPUs in the unplugged state, as realview and versatile express have no way to power manage or reset individual CPUs. Most normal platforms (iow, those fabricated on real SoCs rather than test-chips) have ways to power off the unplugged CPU, and don't need this holding pen. Or they only have one additional CPU, and so don't even need any holding pen. Unfortunately, I don't have the time to hit people over the head to get them to fix their implementations here, so lots of copy-n-modify for the SMP bringup from Realview has happened over the years. It's little surprise though that OMAP (where I've been quite involved with) does not suffer from this... Basically, everything except plat-versatile probably shouldn't be using this holding pen stuff. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists