From: Ram Pai <linuxram@us.ibm.com>
To: Adrian Bunk <bunk@stusta.de>
Cc: linux-kernel@vger.kernel.org, akpm@osdl.org, arjan@infradead.org,
greg@kroah.com, hch@infradead.org
Subject: Re: [RFC PATCH 1/2] Makefile: export-symbol usage report generator.
Date: Wed, 19 Apr 2006 17:53:34 -0700 [thread overview]
Message-ID: <1145494414.7323.178.camel@localhost> (raw)
In-Reply-To: <20060419234112.GI25047@stusta.de>
On Thu, 2006-04-20 at 01:41 +0200, Adrian Bunk wrote:
> On Wed, Apr 19, 2006 at 04:25:58PM -0700, Ram Pai wrote:
> > On Tue, 2006-04-18 at 16:09 +0200, Adrian Bunk wrote:
> > > On Thu, Apr 13, 2006 at 05:38:26AM -0700, Ram Pai wrote:
> > >
> > > > I ran a report to extract export-symbol usage by kernel modules. The results
> > > > are at http://www.sudhaa.com/~ram/misc/export_report.txt
> > > >
> > > > The report lists:
> > > > 1. All the exported symbols and their usage counts by in-kernel modules.
> > > > 2. for each in-kernel module, lists the modules and the exported symbols
> > > > from those modules, that it depends on.
> > > >
> > > > Highlights:
> > > > On x86 architecture
> > > > (1) 880 exported symbols not used by any in-kernel modules.
> > > > (2) 1792 exported symbols used only once.
> > > >
> > > > I hope this report/tool shall help all inkernel modules to revisit their usage
> > > > of kernel interfaces.
> > > >
> > > > This patch integrates the report-generator into the kernel build process. After
> > > > applying this patch, invoke 'make export_report' and it creates the report in
> > > > Documentation/export_report.txt
> > > >...
> > >
> > > I like your patch, but I have observed two issues:
> > > - please don't force an allmodconfig, simply use the currently compiled
> > > kernel
> >
> > Looking for ideas. Its hard to extract out the imported symbols unless
> > they are compiled as modules. Running the report on a currently compiled
> > kernel will miss most of the subsystems that are not compile as modules.
> > Hence the report wont be complete. Any suggestions?
> >...
>
> - it's a tool for kernel hackers who know what they are doing
> - it's a tool to help you finding unused exports, but each one still
> requires manual verification
I am not sure I understand you. Are you saying:
A) the tool should report the export symbol usage with whatever
set of modules the user has configured in his .config ?
or
B) the tool should report the export symbol usage for all the
modules irrespective of what has been configured in the
users .config ?
RP
>
> Besides this, e.g. CONFIG_SMP=n brings you more modules and therefore a
> better coverage than allmodconfig.
>
> cu
> Adrian
>
next prev parent reply other threads:[~2006-04-20 0:53 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-13 12:38 [RFC PATCH 1/2] Makefile: export-symbol usage report generator Ram Pai
2006-04-18 14:09 ` Adrian Bunk
2006-04-19 23:25 ` Ram Pai
2006-04-19 23:41 ` Adrian Bunk
2006-04-20 0:53 ` Ram Pai [this message]
2006-04-20 1:06 ` Adrian Bunk
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=1145494414.7323.178.camel@localhost \
--to=linuxram@us.ibm.com \
--cc=akpm@osdl.org \
--cc=arjan@infradead.org \
--cc=bunk@stusta.de \
--cc=greg@kroah.com \
--cc=hch@infradead.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 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.