On Mon, Aug 04, 2008 at 03:08:15PM -0500, Robin Holt wrote:
> On Mon, Aug 04, 2008 at 12:27:25PM -0700, Luck, Tony wrote:
> > I've locally applied these config updates, but I ran into
> > a regression while running my usual set of test builds.
> >
> > I build a few uniprocessor configs (by taking files from
> > arch/ia64/configs/ and turning off CONFIG_SMP). The
> > resulting uniprocessor generic_defconfig fails to
> > build with:
> >
> > drivers/misc/sgi-gru/grutables.h:308: error: field `ms_notifier' has incomplete type
> >
> > I tried this:
> > diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> > index 82af385..6e2dcd8 100644
> > --- a/drivers/misc/Kconfig
> > +++ b/drivers/misc/Kconfig
> > @@ -430,7 +430,7 @@ config SGI_XP
> > depends on IA64_GENERIC || IA64_SGI_SN2 || IA64_SGI_UV || (X86_64 && SMP)
> > select IA64_UNCACHED_ALLOCATOR if IA64_GENERIC || IA64_SGI_SN2
> > select GENERIC_ALLOCATOR if IA64_GENERIC || IA64_SGI_SN2
> > - select SGI_GRU if IA64_GENERIC || IA64_SGI_UV || (X86_64 && SMP)
> > + select SGI_GRU if (IA64_GENERIC || IA64_SGI_UV || X86_64) && SMP
>
> Wouldn't this allow a !SMP build to not select GRU? That is not
> correct. I think this is more a depends on thing, but Dean/Jack are
> more appropriate for that question.
Dean and I just talked. We think it should really be:
depends on (IA64_GENERIC || IA64_SGI_SN2 || IA64_SGI_UV || X86_64) && SMP
select IA64_UNCACHED_ALLOCATOR if IA64_GENERIC || IA64_SGI_SN2
select GENERIC_ALLOCATOR if IA64_GENERIC || IA64_SGI_SN2
select SGI_GRU if (IA64_GENERIC || IA64_SGI_UV || X86_64) && SMP
Could you give it a try? If not, Dean or I can try it tomorrow.
Thanks,
Robin
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|