lkml.org 
[lkml]   [2002]   [Mar]   [14]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateThu, 14 Mar 2002 06:26:40 -0500
FromJeff Garzik <>
SubjectRe: [PATCH] 2.5.1-pre5: per-cpu areas
Rusty Russell wrote:

>In message <3C902FA5.5010208@mandrakesoft.com> you write:
>
>>Your other changes look good, but RELOC_HIDE really does belong in 
>>compiler.h... and percpu.h is a particularly poor choice of destination. 
>>
>
>How?  compiler.h is for things which vary based on compiler versions.
>

The name "linux/compiler.h" does not imply that to me, nor do the 
comments in the file, which are related specifically to __ builtin_expect.

RELOC_HIDE is a potentially general facility (with the caveat below), 
that does not seem to directly relate to the name "linux/percpu.h" at 
all, except by happenstance due to its origins.  Subjectively it seemed 
to me that compiler.h was the most appropriate.  Maybe kernel.h is a 
better choice, in others' eyes.  But I think percpu.h is probably the 
wrong home.

>
>It was an arbitrary and relatively crappy place to put it: I only put
>it there so PPC could use it...
>
Will other arches -ever- use the macro?  If not, include/asm-ppc is a 
better place...

    Jeff "mountain out of a molehill" Garzik



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:21    [W:0.229 / U:0.070 seconds]
©2003-2008 Jasper Spaans