linux-kernel - Re: [PATCH 2/3] Kconfig: improve handling for all{rand,yes,no,}.config fragments

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK7LNARvbsMc3JpcxZjvmc3UjvACx3t66cLM_aSDcADiPA0V7g@mail.gmail.com>
Date:   Tue, 20 Feb 2018 18:26:55 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Arnd Bergmann <arnd@...db.de>
Cc:     Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        "Yann E. MORIN" <yann.morin.1998@...e.fr>,
        Michal Marek <mmarek@...e.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 2/3] Kconfig: improve handling for all{rand,yes,no,}.config
 fragments

2018-02-17 6:41 GMT+09:00 Arnd Bergmann <arnd@...db.de>:
> The kernel currently supports two methods of dealing with config
> fragments in the tree:
>
> a) Running "make foo.config" looks for arch/$(ARCH)/configs/foo.config
>    and kernel/configs/foo.config, and applies the defaults from those
>    files on top of the current configuration.
>
> b) Running "KCONFIG_ALLCONFIG=1 make randconfig" (or the equivalent
>    allmodconfig/allnoconfig/allyesconfig/alldefconfig) will look
>    for a "allrandconfig.config" file in the current directory or the
>    top of the $(srctree). These are used as defaults before we generate
>    the remaining options.
>
> This is rather inconsistent, and prevents us from easily shipping
> good defaults for "randconfig". I'm extending the logic here so that
> the second case also looks for the hardcoded file names in the standard
> directories (first arch/$(ARCH)/configs/, then kernel/configs) in the
> source tree.
>
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> ---
>  scripts/kconfig/conf.c | 27 +++++++++++++++++++--------
>  1 file changed, 19 insertions(+), 8 deletions(-)
>
> diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
> index 866369f10ff8..848bf4d15e9a 100644
> --- a/scripts/kconfig/conf.c
> +++ b/scripts/kconfig/conf.c
> @@ -493,8 +493,9 @@ int main(int ac, char **av)
>  {
>         const char *progname = av[0];
>         int opt;
> -       const char *name, *defconfig_file = NULL /* gcc uninit */;
> +       const char *arch, *name, *defconfig_file = NULL /* gcc uninit */;
>         struct stat tmpstat;
> +       char fullname[PATH_MAX+1];
>
>         setlocale(LC_ALL, "");
>         bindtextdomain(PACKAGE, LOCALEDIR);
> @@ -621,14 +622,24 @@ int main(int ac, char **av)
>                 case randconfig:        name = "allrandom.config"; break;
>                 default: break;
>                 }
> -               if (conf_read_simple(name, S_DEF_USER) &&
> -                   conf_read_simple("all.config", S_DEF_USER)) {
> -                       fprintf(stderr,
> -                               _("*** KCONFIG_ALLCONFIG set, but no \"%s\" or \"all.config\" file found\n"),
> -                               name);
> -                       exit(1);
> +               /* try ./name, arch/$(ARCH)/configs/name and kernel/config/name */
> +               if (!conf_read_simple(name, S_DEF_USER))
> +                       break;
> +               arch = getenv("ARCH");
> +               if (arch) {
> +                       snprintf(fullname, sizeof(fullname), "arch/%s/configs/%s",
> +                                arch, name);
> +                       if (!conf_read_simple(fullname, S_DEF_USER))
> +                               break;
>                 }


I am not a big fan of hard-coding the kernel directory structure.

We already do this [1], but I am thinking of kicking this out.
[1] https://github.com/torvalds/linux/blob/v4.16-rc1/scripts/kconfig/confdata.c#L33



BTW, I am trying to compiler capability check to Kconfig.
If this work is done, I hope some "depends on !COMPILE_TEST" will go
away (but not all?)
https://patchwork.kernel.org/patch/10225375/



> -               break;
> +               snprintf(fullname, sizeof(fullname), "kernel/configs/%s", name);
> +               if (!conf_read_simple(fullname, S_DEF_USER))
> +                       break;
> +
> +               fprintf(stderr,
> +                       _("*** KCONFIG_ALLCONFIG set, but no \"%s\" or \"all.config\" file found\n"),
> +                               name);
> +               exit(1);
>         default:
>                 break;
>         }
> --
> 2.9.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Best Regards
Masahiro Yamada