lkml.org 
[lkml]   [2004]   [Aug]   [24]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 24 Aug 2004 08:17:29 +0200
FromAndi Kleen <>
SubjectRe: Fix MTRR strings definition.
On Tue, 24 Aug 2004 00:23:20 +0100
Dave Jones <davej@redhat.com> wrote:

> Instead of deleting the extern from include/asm/mtrr.h, I believe
> the correct fix would be to move the strings back to the include file
> where they belong.> The reason behind this, is that there are userspace apps (admittedly
> few, but we even ship two in Documentation/mtrr.txt) that rely upon
> these definitions being in that header.  This has been broken for
> all 2.6 releases so far. Patch below fixes things back the way it
> was in 2.4

That's rather ugly. It would be cleaner to just have a 
macro that expands to the strings, and everybody who wants to use
it declares an own array using that macro.


> Andi, I don't have gcc 3.5 to hand, I trust this fixes whatever
> problem you saw there too ?

3.5 doesn't like it when something is declared both extern and static.
Your new patch has this problem again.

-Andi

-
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 14:05    [from the cache]
©2003-2008