[PATCH 0/3] kallsyms: remove special handling for CONFIG_ARM

Ard Biesheuvel ard.biesheuvel at linaro.org
Wed Feb 3 12:41:21 PST 2016


On 3 February 2016 at 21:30, Ard Biesheuvel <ard.biesheuvel at linaro.org> wrote:
> On 3 February 2016 at 21:23, Nicolas Pitre <nicolas.pitre at linaro.org> wrote:
>> On Wed, 3 Feb 2016, Ard Biesheuvel wrote:
>>
>>> On 3 February 2016 at 21:01, Russell King - ARM Linux
>>> <linux at arm.linux.org.uk> wrote:
>>> > On Wed, Feb 03, 2016 at 02:41:29PM +0100, Ard Biesheuvel wrote:
>>> >> > bf353edc T __stubs_start
>>> >>
>>> >> This is a global symbol
>>> >>
>>> >> > bf353edc T _etext
>>> >> > bf353ee0 t __stubs_start
>>> >>
>>> >> This is a local symbol
>>> >
>>> > Stop this madness.  This gets you an immediate NAK on the patch set.
>>> >
>>>
>>> What madness? This does not change at all with my patches, these
>>> duplicates already exist in the current code.
>>
>> That doesn't make it any better, and was cause for issue as Chris
>> demonstrated.  Can we get rid of that duplication please?
>>
>
> 4/4 in my v2 removes the redundant symbols.

... actually, that only removes the redundant __stub_start, and the
redundant __vectors_start is already removed in patch #1. Patch #4
(v2) may be shot down for other reasons, in that case I can simply
move the removal of __stubs_start to patch #1 instead.



More information about the linux-arm-kernel mailing list