Re: [PATH] Fix compilation when CONFIG_TRACE_IRQFLAGS_SUPPORT isnot set

From: Hiroshi Shimamoto
Date: Thu Sep 04 2008 - 13:20:53 EST


Ingo Molnar wrote:
> * Rui Sousa <rui.p.m.sousa@xxxxxxxxx> wrote:
>
>> Hi,
>>
>> This patch fixes compilation if CONFIG_TRACE_IRQFLAGS_SUPPORT is ever
>> disabled (which is currently not allowed by Kconfig). Alternatively we
>> could just remove the option altogether and the associated code paths.
>> Since the compilation error has been in the tree for at least two
>> years and no one noticed it, I guess we don't really have the need for
>> CONFIG_TRACE_IRQFLAGS_SUPPORT=n. Boot tested on x86 UP.
>
> applied to tip/core/locking - thanks Rui.
>
> since you fixed it i prefer the fix over the removal. We could still
> remove the !TRACE_IRQFLAGS_SUPPORT now (in a separate commit) and
> simpify this header a bit that way. Thus if someone needs the
> !TRACE_IRQFLAGS_SUPPORT mode of build for future work, it can be
> restored via a simple revert.

Hi, it seems that this patch breaks uml build.

kernel/printk.c: In function 'vprintk':
kernel/printk.c:674: error: implicit declaration of function 'raw_local_irq_save'
kernel/printk.c:772: error: implicit declaration of function 'raw_local_irq_restore'

thanks,
Hiroshi Shimamoto
--
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/