On 31/01/07, Robert P. J. Day <[EMAIL PROTECTED]> wrote:
On Wed, 31 Jan 2007, Michal Piotrowski wrote:> On 31/01/07, Robert P. J. Day <[EMAIL PROTECTED]> wrote: > > > > Centralize the kernel config option for debugging semaphores and > > modify the macro for frv to use that config option instead. > > > > Signed-off-by: Robert P. J. Day <[EMAIL PROTECTED]> > [..] > > +config DEBUG_SEMAPHORE > > + bool "Semaphore debugging" > > + depends on DEBUG_KERNEL > > + default n > > + help > > + If you say Y here then semaphore processing will issue lots of > > + verbose debugging messages. If you suspect a semaphore problem or > > a > > + kernel hacker asks for this option then say Y. Otherwise say N. > > + > > + At the moment, this is implemented only by alpha and frv. > > IMHO this option should stay in arch/{alpha,frv}/Kconfig.debug any particular reason?
Only this "At the moment, this is implemented only by alpha and frv." It depends on architecture.
and note that it's currently *not* in arch/frv/Kconfig.debug -- it's hard-coded with a macro name that's not the same as the config option, and i don't really see the advantage of having each architecture implement precisely the same feature in different and incompatible ways. but i'm willing to be convinced otherwise. rday
Regards, Michal -- Michal K. K. Piotrowski LTG - Linux Testers Group (http://www.stardust.webpages.pl/ltg/) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
