From: Guenter Roeck <linux@roeck-us.net>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>,
Michael Ellerman <mpe@ellerman.id.au>,
Kees Cook <keescook@chromium.org>,
Andrew Morton <akpm@linux-foundation.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: Problems with commit 'kallsyms: add support for relative offsets in kallsyms address table' (in mmotm)
Date: Sat, 23 Jan 2016 23:06:09 -0800 [thread overview]
Message-ID: <56A477E1.50509@roeck-us.net> (raw)
In-Reply-To: <EA1328A5-F57B-4370-B27D-F09DD1C5429C@linaro.org>
On 01/23/2016 10:10 PM, Ard Biesheuvel wrote:
>
>
>> On 24 jan. 2016, at 03:35, Guenter Roeck <linux@roeck-us.net> wrote:
>>
>>> On 01/23/2016 06:06 PM, Guenter Roeck wrote:
>>> Hi,
>>>
>>> I see runtime problems with the current mmotm branch. All qemu mips targets
>>> (32 and 64 bit, big and little endian) are stuck in boot after this commit.
>>>
>>> Bisect points to commit d13682e4d9d2 ("kallsyms: add support for relative offsets
>>> in kallsyms address table". Disabling CONFIG_KALLSYMS_BASE_RELATIVE fixes the problem,
>>> ie I can boot the image with qemu.
>>>
>>> Bisect log is attached.
>>>
>>> Playing with the problem, I found the following:
>>>
>>> 1) The problem is only seen with a toolchain using binutils 2.22, but not
>>> with a toolchain using binutils 2.25. The compiler configuration may be
>>> different for both toolchains.
>>> 2) Message "kallsyms failure: absolute symbol value 0xffffffff807afd14 out of range
>>> in relative mode" (twice) when using the toolchain with binutils 2.22.
>>> This does not cause the build to fail, though.
>>> 3) kallsyms_sym_address() parameter variable type is "int". In the calling code,
>>> the variable type used is "unsigned long". That has no impact on the problem,
>>> though.
>>
>> An additional data point: When using the older toolchain, many symbols in System.map
>> are marked "A".
>> ffffffff80100000 A _text
>> With the more recent toolchain, the same symbols are marked "T".
>> ffffffff80100000 T _text
>>
>
> Thanks for the analysis. It is surprising that the build does not fail when this occurs, and the subsequent hangs themselves are probably caused by missing kallsyms data.
>
Yes, I wondered why the build doesn't fail. Seems odd.
> scripts/kallsyms.c ignores all A symbols except _text, which is actually a relative symbol by nature so we can simply assume it is relative (i.e., override it as T)
>
> Re x86_64 !SMP, any build time errors there as well? Likewise for sparc32?
>
Yes, same kind of errors for both. For x86_64/nosmp I also get the error message
when using the Ubuntu native toolchain, so it doesn't seem to be (directly)
related to binutils 2.22 vs. 2.25 for that architecture.
Runtime behavior is a bit different for the different architectures.
x86_64 dies silently without any console output, mips just hangs,
and sparc32 gets a panic with NULL pointer access.
Of course, with missing kallsyms data all bets are off.
>
> Thanks again, and sorry for the trouble,
No worries. Hope you'll get this sorted out.
Guenter
next prev parent reply other threads:[~2016-01-24 7:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-24 2:06 Problems with commit 'kallsyms: add support for relative offsets in kallsyms address table' (in mmotm) Guenter Roeck
2016-01-24 2:35 ` Guenter Roeck
2016-01-24 6:10 ` Ard Biesheuvel
2016-01-24 7:06 ` Guenter Roeck [this message]
2016-01-24 8:21 ` Ard Biesheuvel
2016-01-24 17:05 ` Guenter Roeck
2016-01-24 17:20 ` Ard Biesheuvel
2016-01-24 18:01 ` Guenter Roeck
2016-01-24 19:01 ` Ard Biesheuvel
2016-01-24 19:16 ` Ard Biesheuvel
2016-01-24 4:11 ` Guenter Roeck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56A477E1.50509@roeck-us.net \
--to=linux@roeck-us.net \
--cc=akpm@linux-foundation.org \
--cc=ard.biesheuvel@linaro.org \
--cc=heiko.carstens@de.ibm.com \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mpe@ellerman.id.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.