lkml.org 
[lkml]   [2002]   [Nov]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Linux 2.4.20-rc4-ac1
On Thu, Nov 28, 2002 at 08:15:02AM -0800, Martin J. Bligh wrote:

> > Martin, could you please review whether the patch below which does the
> > following is correct?
> >
> > - kill the last occurence of CLUSTERED_APIC_NUMAQ
> > - only one definition of PCI_CONF1_ADDRESS is needed
> > (#ifndef CONFIG_MULTIQUAD the BUS2LOCAL() has no effect)
> > - fix an #endif comment
>
> I'm confused at to what made this break recently .... what release
> did this break on? Either it was something very recent, everyone
> else is asleep, or you're doing something wierd (like compling with
> MULTIQUAD on?)

Yes, I tried a compilation with MULTIQUAD on.

The "very recent" might be that I'm talking about -ac (look at the
Subject).

> CLUSTERED_APIC_NUMAQ is a part of the Summit patch introduction,
> where John (cc'ed) splits clustered_apic_mode from a boolean into
> a switch (in 2.4 ... I did 2.5 differently). Maybe we merged half
> of a summit patch somehow, and didn't get the define?
>...

First of all:
I noticed this problem while doing a compile-only test, I have no real
knowledge of the code.

In 2.4.20-pre the following change was made in several places:
clustered_apic_mode -> (clustered_apic_mode == CLUSTERED_APIC_NUMAQ)

In 2.4.20-pre-ac they were changed again, this time
(clustered_apic_mode == CLUSTERED_APIC_NUMAQ) -> clustered_apic_logical

According to grep the occurence of CLUSTERED_APIC_NUMAQ my patch changes
was the last one _in the whole kernel sources of 2.4.20-rc4-ac1_.

It might be that my patch is wrong but there's undoubtable something
that needs to be fixed.

> M.

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
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 13:31    [from the cache]
©2003-2011 Jasper Spaans