linux-kernel - Re: [PATCH 2.4.34-pre1] fix x86_64 etc build failure due to memchr change

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-next>] [day] [month] [year] [list]

Message-Id: <200608201024.k7KAO4i5023339@harpo.it.uu.se>
Date:	Sun, 20 Aug 2006 12:24:04 +0200 (MEST)
From:	Mikael Pettersson <mikpe@...uu.se>
To:	w@....eu
Cc:	ak@...e.de, davem@...emloft.net, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2.4.34-pre1] fix x86_64 etc build failure due to memchr change

On Sun, 20 Aug 2006 03:31:09 +0200, Willy Tarreau wrote:
>On Sun, Aug 20, 2006 at 12:19:06AM +0200, Mikael Pettersson wrote:
>> 2.4.34-pre1 doesn't build on x86_64:
>> 
>> kernel/kernel.o(__ksymtab+0x1c10): multiple definition of `__ksymtab_memchr'
>> arch/x86_64/kernel/kernel.o(__ksymtab+0x3f0): first defined here
>> kernel/kernel.o(.kstrtab+0x3960): multiple definition of `__kstrtab_memchr'
>> arch/x86_64/kernel/kernel.o(.kstrtab+0x5fd): first defined here
>> ld: Warning: size of symbol `__kstrtab_memchr' changed from 7 in arch/x86_64/kernel/kernel.o to 17 in kernel/kernel.o
>> make: *** [vmlinux] Error 1
>> 
>> This is because the 'export memchr() which is used by smbfs and lp driver'
>> change in 2.4.34-pre1 added an EXPORT_SYMBOL of memchr to kernel/ksyms.c
>> without also removing the existing one in arch/x86_64/kernel/x8664_ksyms.c.
>> Alpha, ARM, ppc32, and SH also have EXPORTs of memchr so they probably
>> also broke.
>> 
>> This patch removes the EXPORTs of memchr under arch/, which fixes x86_64
>> and should fix the other architectures as well.
>
>OK Mikael,
>
>I have fixed sparc and sparc64 instead, and it's OK now without having to
>export memchr() in kernel/ksyms.c. So the fix is shorter and more logical.
>It brings non-sparc architectures to their state before my wrong fix. However,
>sparc needs to export memchr() for smbfs and lp.

Works for me. Tested on i386, x86_64, and ppc32. I'll try to test on sparc64
later today.

/Mikael
-
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/