Re: [PATCH] x86: Disable CONFIG_X86_DECODER_SELFTEST in allmod/allyesconfigs

[Paul Gortmaker]

On Wed, Feb 5, 2014 at 12:51 AM, Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
> * Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
>
>> > BTW you probably also want to do the same for the x86 instruction
>> > decoder test. It seems to be the biggest compile time waster in
>> > allyesconfig and I usually disable it too.
>>
>> Agreed, that should be disabled too by default. Although in that
>> case I think the COMPILE_TEST combination makes more sense, since
>> it's a coverage issue.
>
> I'd say randconfig will cover it often enough, and the failure is also
> borderline build coverage related: you cannot really make the decoder
> test fail via source level changes, only with changes in the build
> environment, so I agree with Andi that we can disable this one too -
> via the patch below.

For the record, even with a (currently) modern CPU such as a
Xeon E5-2630 @ 2.30GHz, the script takes 4m30s on 3.14-rc1
i386 allyesconfig.  And since it runs single threaded at the end
of the build, it really adds that amount to the wall time spent
on the build. So enthusiastically Acked, FWIW.

Paul.
--

>
> Thanks,
>
>         Ingo
>
> Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx>
>
> diff --git a/arch/x86/Kconfig.debug b/arch/x86/Kconfig.debug
> index 0f3621e..f0fca39 100644
> --- a/arch/x86/Kconfig.debug
> +++ b/arch/x86/Kconfig.debug
> @@ -184,6 +184,11 @@ config HAVE_MMIOTRACE_SUPPORT
>  config X86_DECODER_SELFTEST
>         bool "x86 instruction decoder selftest"
>         depends on DEBUG_KERNEL && KPROBES
> +       #
> +       # It can take some time to validate the image,
> +       # make sure {allyes|allmod}config doesn't enable it:
> +       #
> +       depends on !COMPILE_TEST
>         ---help---
>          Perform x86 instruction decoder selftests at build time.
>          This option is useful for checking the sanity of x86 instruction
> --
> 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/
--
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/