From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6EC7DC4321D for ; Mon, 20 Aug 2018 18:37:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 267AA2173A for ; Mon, 20 Aug 2018 18:37:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 267AA2173A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726626AbeHTVyC (ORCPT ); Mon, 20 Aug 2018 17:54:02 -0400 Received: from mx2.suse.de ([195.135.220.15]:58354 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726077AbeHTVyB (ORCPT ); Mon, 20 Aug 2018 17:54:01 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay1.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id A791FAD96; Mon, 20 Aug 2018 18:37:13 +0000 (UTC) Date: Mon, 20 Aug 2018 20:37:12 +0200 Message-ID: From: Takashi Iwai To: Masahiro Yamada Cc: Michal =?UTF-8?B?U3VjaMOhbmVr?= , Linux Kernel Mailing List , Andreas Schwab , Michal Kubecek , Michal Marek , Jonathan Corbet , Yoshinori Sato , Rich Felker , "David S. Miller" , Jeff Dike , Richard Weinberger , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , X86 ML , Kees Cook , Philippe Ombredanne , Greg Kroah-Hartman , Ulf Magnusson , Jeff Mahoney , "Peter Zijlstra," , Mathieu Desnoyers , Frederic Weisbecker , Randy Dunlap , Dominik Brodowski , Nicholas Piggin , Linux Kbuild mailing list , "open list:DOCUMENTATION" , Linux-sh list , sparclinux , linux-um@lists.infradead.org Subject: Re: due to kconfig changes kernel config file is no longer sufficient for configuring the kernel In-Reply-To: References: <20180627143705.5a1fed1c@kitsune.suse.cz> <20180628111623.3807fe9b@naga.suse.cz> <20180806200725.4efa5d35@kitsune.suse.cz> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/26 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 20 Aug 2018 20:15:12 +0200, Masahiro Yamada wrote: > > 2018-08-07 3:07 GMT+09:00 Michal Suchánek : > > On Mon, 30 Jul 2018 17:02:42 +0900 > > Masahiro Yamada wrote: > > > >> 2018-06-28 18:16 GMT+09:00 Michal Suchánek : > >> > On Wed, 27 Jun 2018 23:07:21 +0900 > >> > Masahiro Yamada wrote: > >> > > >> >> Hi. > >> >> > >> >> > >> >> 2018-06-27 21:37 GMT+09:00 Michal Suchánek : > >> >> > Hello, > >> >> > > >> >> > in the x86 Kconfig we have this: > >> >> > > >> >> > # Select 32 or 64 bit > >> >> > config 64BIT > >> >> > bool "64-bit kernel" if "$(ARCH)" = "x86" > >> >> > default "$(ARCH)" != "i386" > >> >> > ---help--- > >> >> > Say yes to build a 64-bit kernel - formerly known as > >> >> > x86_64 Say no to build a 32-bit kernel - formerly known as i386 > >> >> > > >> >> > Since commit 104daea149c4 ("kconfig: reference environment > >> >> > variables directly and remove 'option env='") the value of ARCH > >> >> > is not saved in the kernel config. > >> >> > >> >> I think this commit is unrelated. It was just a syntax change. > >> > > >> > This does not look like syntax only change to me: > >> > > >> > diff --git a/init/Kconfig b/init/Kconfig > >> > index 15aae32e0719..1217fc62ca61 100644 > >> > --- a/init/Kconfig > >> > +++ b/init/Kconfig > >> > @@ -1,20 +1,12 @@ > >> > -config ARCH > >> > - string > >> > - option env="ARCH" > >> > - > >> > -config KERNELVERSION > >> > - string > >> > - option env="KERNELVERSION" > >> > - > >> > >> This is just syntax change. > >> > >> 'option env=' was used to reference an environment variable. > >> > >> Now, $(ARCH), $(KERNELVERSION) are simpler forms. > >> > >> > >> >> > >> >> Unless I am missing something, > >> >> we have never saved ARCH in the .config in the past. > >> > > >> > There was a config symbol defined for it before the commit removed > >> > it. > >> > >> No. > >> > >> CONFIG symbols with'option env=' > >> are not written out to the .config file. > >> > >> We have never had CONFIG_ARCH or CONFIG_KERNELVERSION. Maybe it sounds like a stupid question, but... if passing ARCH= is almost mandatory for distinguishing the bi-arch or cross-compile cases, why don't we save it in .config? The whole "regression" we've seen can be worked around by passing ARCH at each time. So, by having it in .config, everything would work more easily, no? thanks, Takashi