linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: linuxram@us.ibm.com (Ram Pai)
Cc: agruen@suse.de, akpm@osdl.org, arjan@infradead.org,
	bunk@stusta.de, greg@kroah.com, hch@infradead.org,
	jbeulich@novell.com, linux-kernel@vger.kernel.org,
	mathur@us.ibm.com
Subject: Re: [PATCH 4/4] KBUILD: export-symbol usage report generator
Date: 11 May 2006 13:43:27 +0200	[thread overview]
Message-ID: <p73r7307pnk.fsf@bragg.suse.de> (raw)
In-Reply-To: <20060510235546.8A006470034@localhost>

linuxram@us.ibm.com (Ram Pai) writes:

> From: Ram Pai <linuxram@us.ibm.com>
> 
> The following patch provides the ability to generate a report of
>      (1) All the exported symbols and their in-kernel-module usage count
>      (2) For each module, lists the modules and their exported symbols, on
>                 which it depends.

Very nice.

One thing I always wanted to see was a more focussed EXPORT_SYMBOL.

A lot of symbols are only exported for a single other module (e.g. most
of the networking exports are for IPv6) but are actually internal 
and shouldn't be messed with by other modules. It would be nice
if name spaces could be defined that say "this export is only for 
modules in this name space" and then e.g. have IPv6 be in the TCPINTERNALS
name space and nobody else.

I think adding something like this could clean up the bewildering
jungle of exports greatly.

_GPL is kind of like that already, but it is not fine grained enough.

-Andi

  reply	other threads:[~2006-05-11 11:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-10 23:55 [PATCH 4/4] KBUILD: export-symbol usage report generator Ram Pai
2006-05-11 11:43 ` Andi Kleen [this message]
2006-05-11 13:43   ` Christoph Hellwig
2006-05-11 14:13     ` Andi Kleen

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=p73r7307pnk.fsf@bragg.suse.de \
    --to=ak@suse.de \
    --cc=agruen@suse.de \
    --cc=akpm@osdl.org \
    --cc=arjan@infradead.org \
    --cc=bunk@stusta.de \
    --cc=greg@kroah.com \
    --cc=hch@infradead.org \
    --cc=jbeulich@novell.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxram@us.ibm.com \
    --cc=mathur@us.ibm.com \
    /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;
as well as URLs for NNTP newsgroup(s).