public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Paulo Marques <pmarques@grupopie.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: /proc/kallsyms broken in 2.6.26-rc1-git6
Date: Mon, 12 May 2008 10:55:27 +0100	[thread overview]
Message-ID: <4828140F.9030609@grupopie.com> (raw)
In-Reply-To: <4824DB36.8000500@firstfloor.org>

Andi Kleen wrote:
>> That's not for me to judge, but I believe it has always been like that.
> 
> No, normally /proc/kallsyms looks similar to System.tap. Where are all
> these DW.* symbols coming from? 

Hum... System.map is generated by running

nm -n vmlinux | grep -v '\( [aUw] \)\|\(__crc_\)\|\( \$[adt]\)' > System.map

whereas the kallsyms symbols come from:

nm -n vmlinux

So, the only difference is the filter made by that "grep -v" to exclude 
a few classes of symbols.

Maybe I lost myself in that expression, but it doesn't seem like it 
would be able to filter out the symbols you're seeing. Are you sure the 
same symbols don't appear in System.map?

> They didn't use to be there and don't
> make any sense because they don't have any valid kernel addresses.

I don't know enough about the markers infrastructure but I guess these 
"addresses" are more like an "offset" into a markers structure that is 
automatically produced by putting these symbols into a special section 
that starts at offset 0.

>> I just wanted to understand if you noticed a change in behavior (which
>> is probably a bug) or if it has always been like that but you just
>> noticed how ugly it is.
> 
> I noticed a change in behavior.

Ok.

>> Maybe you also have some debug or markers configuration or something
>> that is generating extra symbols to a special section that is just
>> making the problem look worse now.
> 
> Nothing particular. I uploaded the config at
> http://halobates.de/basil-config

Well, my first suspects would be these:

CONFIG_KPROBES=y
CONFIG_KRETPROBES=y
CONFIG_HAVE_KPROBES=y
CONFIG_HAVE_KRETPROBES=y

>> Anyway, I can change the way kallsyms works, but that has to be done
>> with some care because there are some userspace tools that read
>> /proc/kallsyms and we don't want to break those. A proper testing period
>> through -mm should take care of that, though.
> 
> It's the other way round -- kallsyms changed and that change will likely
> break programs.

I don't have the time right now to try your configuration and pinpoint 
the problem, but if you can come up with a plan, like: "we need to 
filter out symbols from the output of "nm" whose type is 'N'", I'll be 
more than happy to provide a patch to fix it...

-- 
Paulo Marques - www.grupopie.com

"Nostalgia isn't what it used to be."

  reply	other threads:[~2008-05-12  9:55 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-09 17:41 /proc/kallsyms broken in 2.6.26-rc1-git6 Andi Kleen
2008-05-09 18:03 ` Paulo Marques
2008-05-09 19:36   ` Andi Kleen
2008-05-09 19:59     ` Paulo Marques
2008-05-09 23:16       ` Andi Kleen
2008-05-12  9:55         ` Paulo Marques [this message]
2008-05-12 10:08           ` Andi Kleen
2008-05-09 23:37       ` Alexey Dobriyan
2008-05-12 10:00         ` Paulo Marques
2008-05-12 15:50           ` Cyrill Gorcunov
2008-05-12 17:23             ` Sam Ravnborg
2008-05-12 17:28               ` Cyrill Gorcunov
2008-05-13  9:54               ` Andi Kleen
2008-05-19 18:11                 ` Sam Ravnborg
2008-05-19 18:15                   ` Cyrill Gorcunov
2008-05-19 18:21                     ` Sam Ravnborg
2008-05-19 18:22                       ` Cyrill Gorcunov

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=4828140F.9030609@grupopie.com \
    --to=pmarques@grupopie.com \
    --cc=andi@firstfloor.org \
    --cc=linux-kernel@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox