Re: [RFC 01/22] Generic show_mem() implementation

From: Heiko Carstens
Date: Sat Apr 05 2008 - 03:51:43 EST


> >> I can not follow you. Of course the arch selects what they use. But
> >> they should not _all_ have to be flagged with an extra select. So what
> >> default-value are you arguing for?
> > The normal pattern is to let arch select the generic implmentation they
> > use.
> > Just because the majority does use the generic version should not
> > make us start to use the inverse logic as in your case.
> >
> > So I want all archs that uses the generic show_mem() to
> > do an explicit:
> >
> > config MYARCH
> > select HAVE_GENERIC_SHOWMEM
> >
> > Sam
>
> What is the rationale behind this? It is not a function the arch should
> select at all because it is VM code. The remaining arch-specific
> versions are meant to be removed too.
>
> It would be like forcing all architectures to select HAVE_GENERIC_PRINTK
> just because one architecture oopses on printk() and needs to replace it
> with its own version.

Positive logic and consistency with the CONFIG_HAVE_WHATEVER is the reason.

But you can solve this problem with no ifdefs and config options at all,
since you may as well just use __attribute__((weak)) for the generic
implementation.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/