Re: [PATCH RFC] [1/9] Core module symbol namespaces code and intro.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Rusty Russell
Date: Friday, November 23, 2007 - 9:53 pm

On Saturday 24 November 2007 06:53:30 Andi Kleen wrote:

So, you're saying that there's a problem with in-tree modules using symbols 
they shouldn't?  Can you give an example?


If people aren't reviewing, this won't make them review.  I don't think the 
problem is that people are conniving to avoid review.


Probably, but this doesn't reduce it.  


*This* makes sense.  But it's not clear that the burden should be placed on 
kernel coders.  You can create a list yourself.  How do I tell the difference 
between "truly publicly exported" symbols and others?

If a symbol has more than one in-tree user, it's hard to argue against an 
out-of-tree module using the symbol, unless you're arguing against *all* 
out-of-tree modules.

Sorry,
Rusty.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH RFC] [8/9] Put UDP exports into a namespace, Andi Kleen, (Wed Nov 21, 7:43 pm)
[PATCH RFC] [9/9] Add a inet namespace, Andi Kleen, (Wed Nov 21, 7:43 pm)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Christoph Hellwig, (Thu Nov 22, 4:06 am)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Arnaldo Carvalho de Melo, (Thu Nov 22, 5:01 am)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Christoph Hellwig, (Thu Nov 22, 5:03 am)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Rusty Russell, (Fri Nov 23, 9:53 pm)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Stephen Hemminger, (Mon Nov 26, 11:25 am)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Christoph Hellwig, (Tue Nov 27, 9:33 am)
Re: [PATCH RFC] [1/9] Core module symbol namespaces code a ..., Stephen Hemminger, (Tue Nov 27, 4:00 pm)
Re: [PATCH RFC] [4/9] modpost: Fix format string warnings, Sam Ravnborg, (Mon Dec 10, 11:50 am)