[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <27f3817b-0c92-5f92-e079-078a8b92eab4@eikelenboom.it>
Date: Tue, 13 Feb 2018 11:00:49 +0100
From: Sander Eikelenboom <linux@...elenboom.it>
To: Masahiro Yamada <yamada.masahiro@...ionext.com>,
Woody Suwalski <terraluna977@...il.com>
Cc: Ulf Magnusson <ulfalizer@...il.com>, srivasta@...ian.org,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: Linux 4.16-rc1: regression bisected, Debian kernel package tool
make-kpkg stalls indefinitely during kernel build due to commit "kconfig:
remove check_stdin()"
On 13/02/18 05:09, Masahiro Yamada wrote:
> 2018-02-13 12:00 GMT+09:00 Woody Suwalski <terraluna977@...il.com>:
>> Sander Eikelenboom wrote:
>>>
>>> L.S.,
>>>
>>> The Debian kernel-package tool make-kpkg for easy building of upstream
>>> kernels on Debian fails with linux 4.16-rc1.
>>>
>>> The tool (perl script) while invoked with:
>>> make-kpkg --initrd --append_to_version -20180212 kernel_image
>>>
>>> On a git tree with a .config from the previous kernel release, so new
>>> KConfig questions have to be asked on new or changed options.
>>>
>>> The script stalls indefinitely while it seems to be excuting:
>>> exec make kpkg_version=13.018+nmu1 -f
>>> /usr/share/kernel-package/ruleset/minimal.mk debian
>>> APPEND_TO_VERSION=-t440s-20180212 INITRD=YES
>>>
>>> After using ctrl-c to break out it, i get:
>>> ^CFailed to create a ./debian directory: No such file or directory at
>>> /usr/bin/make-kpkg line 970.
>>>
>>> Bisection turned up as culprit:
>>> commit d2a04648a5dbc3d1d043b35257364f0197d4d868
>>> kconfig: remove check_stdin()
>>> Except silentoldconfig, valid_stdin is 1, so check_stdin() is
>>> no-op.
>>> oldconfig and silentoldconfig work almost in the same way except
>>> that
>>> the latter generates additional files under include/. Both ask users
>>> for input for new symbols.
>>> I do not know why only silentoldconfig requires stdio be tty.
>>> $ rm -f .config; touch .config
>>> $ yes "" | make oldconfig > stdout
>>> $ rm -f .config; touch .config
>>> $ yes "" | make silentoldconfig > stdout
>>> make[1]: *** [silentoldconfig] Error 1
>>> make: *** [silentoldconfig] Error 2
>>> $ tail -n 4 stdout
>>> Console input/output is redirected. Run 'make oldconfig' to update
>>> configuration.
>>> scripts/kconfig/Makefile:40: recipe for target
>>> 'silentoldconfig' failed
>>> Makefile:507: recipe for target 'silentoldconfig' failed
>>> Redirection is useful, for example, for testing where we want to
>>> give
>>> particular key inputs from a test file, then check the result.
>>> Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
>>> Reviewed-by: Ulf Magnusson <ulfalizer@...il.com>
>>>
>>> Reverting this specific commit makes make-kpkg work again as usual.
>>>
>>> Version of the kernel-package used:
>>> ii kernel-package
>>> 13.018+nmu1
>>>
>>>
>>> I also cc'ed the Debian developer who maintains the kernel-package
>>> package: Manoj Srivastava
>>>
>>> --
>>> Sander
>>>
>> I have noticed today the same - the kernel-build blockage was in (as I
>> recall)
>> srcipts/kconfig/conf -s --silentoldconfig Kbuild
>>
>> I have bypassed it by regenerating the .config "by hand"...
>
>
> silentoldconfig asks you values for new symbols.
> So, you must answer questions to proceed.
I know, but it stalls before asking the questions.
>
> How does 'make-kpkg' handle silentoldconfig?
>
> Re-direct stdio, then make it forcibly fail?
I don't know, it is a bunch of perl and shell scripts that gets invoked, not the most easy to comprehend if you are not familiar with them. I'm just a user of the tool.
So i would have to defer that question to the Debian package maintainer, hopefully he will chime in.
--
Sander
>
>
>
Powered by blists - more mailing lists