linux-kernel - Re: [PATCH v5 RESEND 1/3] ARM: EXYNOS: Move code from hotplug.c to platsmp.c

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sun, 28 Sep 2014 13:42:21 +0900
From:	Kukjin Kim <kgene.kim@...sung.com>
To:	Krzysztof Kozlowski <k.kozlowski@...sung.com>
CC:	Arnd Bergmann <arnd@...db.de>,
	linux-arm-kernel@...ts.infradead.org,
	Russell King <linux@....linux.org.uk>,
	Kukjin Kim <kgene.kim@...sung.com>,
	linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
	Sachin Kamat <sachin.kamat@...aro.org>,
	Daniel Lezcano <daniel.lezcano@...aro.org>,
	Tomasz Figa <tomasz.figa@...il.com>,
	Kyungmin Park <kyungmin.park@...sung.com>,
	Marek Szyprowski <m.szyprowski@...sung.com>
Subject: Re: [PATCH v5 RESEND 1/3] ARM: EXYNOS: Move code from hotplug.c to
 platsmp.c

On 09/27/14 17:39, Krzysztof Kozlowski wrote:
> W dniu 26.09.2014 o 23:13, Arnd Bergmann pisze:
>> On Friday 05 September 2014, Krzysztof Kozlowski wrote:
>>> The commit only moves code around with one additional observable change:
>>> the hotplug.c was compiled with custom CFLAGS (-march=armv7-a). These
>>> CFLAGS are not necessary any more.
>>
>> This turns out to be wrong, and your change broke 'allmodconfig' builds
>> in linux-next. Please apply this patch on top.
>
Arnd, Krzysztof commented its fix has been submitted and landed in my 
-test tree not -next because it should be handled in rmk's tree I think.

I sent the patch to RMK patch tracking system just now and it should be 
fine in there.

BTW, I just applied the fix in my -next until its ladning in RMK tree 
but as you know it will be not be sent to arm-soc via samsung tree...

One more, Arnd please pull my pull-request 2nd round and 3rd round for 
samsung stuff for 3.18.

> My patch definitely needed more testing.
> I posted a fix here:
> https://lkml.org/lkml/2014/9/24/163
> However it seems that it wasn't picked up by anyone yet.
>
> Russell, could you pick up the patch (with acks from Nicolas and Kukjin)?
>
I believe Russell will take the patch in his tree.

Thanks,
Kukjin

>> 8<----------
>> From 4ba6bf8806caec386e35930314dbad071284c837 Mon Sep 17 00:00:00 2001
>> From: Arnd Bergmann <arnd@...db.de>
>> Date: Fri, 26 Sep 2014 23:09:38 +0200
>> Subject: [PATCH] ARM: EXYNOS: fix build error in platsmp.c
>>
>> /tmp/ccYeWL3V.s: Assembler messages:
>> /tmp/ccYeWL3V.s:659: Error: selected processor does not support ARM
>> mode `isb '
>> /tmp/ccYeWL3V.s:664: Error: selected processor does not support ARM
>> mode `isb '
>> /tmp/ccYeWL3V.s:665: Error: selected processor does not support ARM
>> mode `dsb '
>> make[3]: *** [arch/arm/mach-exynos/platsmp.o] Error 1
>>
>> Signed-off-by: Arnd Bergmann <arnd@...db.de>
>> Fixes: 17342534e1d932 ("ARM: EXYNOS: Move code from hotplug.c to
>> platsmp.c")
>>
>> diff --git a/arch/arm/mach-exynos/Makefile
>> b/arch/arm/mach-exynos/Makefile
>> index 4e49d4efb264..64324bf5edb4 100644
>> --- a/arch/arm/mach-exynos/Makefile
>> +++ b/arch/arm/mach-exynos/Makefile
>> @@ -21,6 +21,7 @@ obj-$(CONFIG_PM_SLEEP) += suspend.o
>> obj-$(CONFIG_PM_GENERIC_DOMAINS) += pm_domains.o
>>
>> obj-$(CONFIG_SMP) += platsmp.o headsmp.o
>> +CFLAGS_platsmp.o := -march=armv7-a
>>
>> plus_sec := $(call as-instr,.arch_extension sec,+sec)
>> AFLAGS_exynos-smc.o :=-Wa,-march=armv7-a$(plus_sec)
--
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/