linux-kernel - Re: [PATCH] kconfig: add KCONFIG_ALLCONFIG support for tinyconfig

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <87zgm41xwp.fsf@irisa.fr>
Date:   Sat, 05 Mar 2022 10:05:18 +0100
From:   RANDRIANAINA Georges Aaron <georges-aaron.randrianaina@...sa.fr>
To:     Masahiro Yamada <masahiroy@...nel.org>
Cc:     Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Mathieu Acher <mathieu.acher@...sa.fr>
Subject: Re: [PATCH] kconfig: add KCONFIG_ALLCONFIG support for tinyconfig


Masahiro Yamada <masahiroy@...nel.org> writes:

> On Fri, Mar 4, 2022 at 2:27 AM Randrianaina Georges Aaron
> <georges-aaron.randrianaina@...sa.fr> wrote:
>>
>> Since f8f0d06438e5, tinyconfig overrides KCONFIG_ALLCONFIG to
>> include kernel/configs/tiny-base.config. However, this ignores
>> user's preset if defined.
>>
>> This modification checks if the user has set KCONFIG_ALLCONFIG
>> and if so, concatenates it with kernel/configs/tiny-base.config
>> to be used as preset config symbols.
>>
>> Signed-off-by: Randrianaina Georges Aaron <georges-aaron.randrianaina@...sa.fr>
>> ---
>>  scripts/kconfig/Makefile | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
>> index b8ef0fb4bbef..337693fb4762 100644
>> --- a/scripts/kconfig/Makefile
>> +++ b/scripts/kconfig/Makefile
>> @@ -102,7 +102,13 @@ configfiles=$(wildcard $(srctree)/kernel/configs/$@ $(srctree)/arch/$(SRCARCH)/c
>>
>>  PHONY += tinyconfig
>>  tinyconfig:
>> +ifeq ($(KCONFIG_ALLCONFIG),)
>>         $(Q)KCONFIG_ALLCONFIG=kernel/configs/tiny-base.config $(MAKE) -f $(srctree)/Makefile allnoconfig
>> +else
>> +       $(Q)cat $(KCONFIG_ALLCONFIG) <(echo) kernel/configs/tiny-base.config > .tmp.config
>
>
> I assume "<(echo)" exists here to insert a blank line.
> The process substitution is a bash'ism.
>
> If you execute it on Debian or its variants, where /bin/sh is a symlink
> to dash, it fails.
>
> masahiro@...ver:~/ref/linux$ make  KCONFIG_ALLCONFIG=/tmp/dummy.config
> tinyconfig
> /bin/sh: 1: Syntax error: "(" unexpected
> make[1]: *** [scripts/kconfig/Makefile:108: tinyconfig] Error 2
> make: *** [Makefile:619: tinyconfig] Error 2
>
>
>
> We can delete '<(echo)', but another issue is that this does not work
> with O=<dir> option.

Yes, we can delete `<(echo)`. We can solve the remaining issue by adding
`$(srctree)` in the presets' path to make it work with O=<dir>.

>
>
> masahiro@...ver:~/ref/linux$ make O=foo
> KCONFIG_ALLCONFIG=/tmp/dummy.config tinyconfig
> make[1]: Entering directory '/home/masahiro/ref/linux/foo'
>   GEN     Makefile
>   HOSTCC  scripts/basic/fixdep
> cat: kernel/configs/tiny-base.config: No such file or directory
> make[2]: *** [../scripts/kconfig/Makefile:108: tinyconfig] Error 1
> make[1]: *** [/home/masahiro/ref/linux/Makefile:619: tinyconfig] Error 2
> make[1]: Leaving directory '/home/masahiro/ref/linux/foo'
> make: *** [Makefile:219: __sub-make] Error 2
>
>
>
>
>
>> +       $(Q)KCONFIG_ALLCONFIG=.tmp.config $(MAKE) -f $(srctree)/Makefile allnoconfig
>> +       $(Q)rm -f .tmp.config
>> +endif
>>         $(Q)$(MAKE) -f $(srctree)/Makefile tiny.config
>>
>>  # CHECK: -o cache_dir=<path> working?
>> --
>> 2.34.1