Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <c4d2f929-9a1f-d816-5a24-e2bf8a64fa8f@i-love.sakura.ne.jp>
In-Reply-To: <c4d2f929-9a1f-d816-5a24-e2bf8a64fa8f@i-love.sakura.ne.jp>
From:   Masahiro Yamada <masahiroy@kernel.org>
Date:   Wed, 14 Jul 2021 13:33:02 +0900
Message-ID: <CAK7LNAT4Jznx5_Tr8jWbdcmJ6zwqAoF9Jcvn89a5+wsGNd9G1Q@mail.gmail.com>
Subject: Re: [5.14-rc1] kbuild: stdout from cmd should not be discarded
To:     Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>
Cc:     Linus Torvalds <torvalds@linux-foundation.org>,
        LKML <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Mon, Jul 12, 2021 at 10:40 AM Tetsuo Handa
<penguin-kernel@i-love.sakura.ne.jp> wrote:
>
> Commit 174a1dcc96429efc ("kbuild: sink stdout from cmd for silent build")
> confuses users by sending prompt messages from
>
>   scripts/kconfig/conf -s --syncconfig
>
> to /dev/null before start waiting for response to prompt messages
> when building with "make -s". Reverting that commit fixes this problem.


Thanks for catching this.

Instead of reverting it,
I'd like to fix the syncconfig part.

https://patchwork.kernel.org/project/linux-kbuild/patch/20210714042349.219199-1-masahiroy@kernel.org/


> ----------
> $ make
>   SYNC    include/config/auto.conf.cmd
> *
> * Restart config...
> *
> *
> * General setup
> *
> Compile also drivers which will not load (COMPILE_TEST) [N/y/?] n
> Local version - append to kernel release (LOCALVERSION) []
> Automatically append version information to the version string (LOCALVERSION_AUTO) [N/y/?] n
> Build ID Salt (BUILD_SALT) []
> Kernel compression mode
> > 1. Gzip (KERNEL_GZIP)
>   2. Bzip2 (KERNEL_BZIP2)
>   3. LZMA (KERNEL_LZMA)
>   4. XZ (KERNEL_XZ)
>   5. LZO (KERNEL_LZO)
>   6. LZ4 (KERNEL_LZ4)
>   7. ZSTD (KERNEL_ZSTD)
> choice[1-7?]: 1
> Default init path (DEFAULT_INIT) []
> Default hostname (DEFAULT_HOSTNAME) [(none)] (none)
> Support for paging of anonymous memory (swap) (SWAP) [Y/n/?] y
> System V IPC (SYSVIPC) [Y/n/?] y
> POSIX Message Queues (POSIX_MQUEUE) [Y/n/?] y
> General notification queue (WATCH_QUEUE) [Y/n/?] y
> Enable process_vm_readv/writev syscalls (CROSS_MEMORY_ATTACH) [Y/n/?] y
> uselib syscall (USELIB) [Y/n/?] y
> Auditing support (AUDIT) [Y/n/?] y
> Preemption Model
>   1. No Forced Preemption (Server) (PREEMPT_NONE)
>   2. Voluntary Kernel Preemption (Desktop) (PREEMPT_VOLUNTARY)
> > 3. Preemptible Kernel (Low-Latency Desktop) (PREEMPT)
> choice[1-3?]: 3
> Core Scheduling for SMT (SCHED_CORE) [N/y/?] (NEW) ^Cmake[2]: *** [scripts/kconfig/Makefile:77: syncconfig] Interrupt
> make[1]: *** [Makefile:626: syncconfig] Interrupt
> make: *** [Makefile:735: include/config/auto.conf.cmd] Interrupt


-- 
Best Regards
Masahiro Yamada