Re: [PATCH] x86: kill arch/x86/kernel/mpparse.c debugging printk.

From: Ingo Molnar
Date: Mon Aug 11 2008 - 12:39:29 EST



* Rene Herman <rene.herman@xxxxxxxxxxxx> wrote:

>> The way to make the printout depend on apic=debug/verbose is to do
>> something like this:
>>
>> apic_printk(APIC_VERBOSE, "Scan SMP from %p for %ld bytes.\n", bp, length);
>>
>> Would you mind to send a patch for that?
>
> I wouldn't. Like this? This turns the printk's that used to be
> Dprintk's into apic_printk's.

applied to tip/x86/urgent - thanks Rene.

> I am myself only interested in the one at the top of smp_scan_config()
> (it made me think I had misconfigured something upon all of a sudden
> seeing SMP printk's on my UP machine on 2.6.27-rc) but I guess this is
> the more complete version.
>
> One problem; on 32-bit, "apic=" is a __setup() param and isn't
> actually early enough for us here so this needs it turned into an
> early_param() (followup).

good spotting, applied that one to tip/x86/urgent as well - thanks.

Ingo
--
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/