From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751513AbdFFRjJ (ORCPT ); Tue, 6 Jun 2017 13:39:09 -0400 Received: from mout.kundenserver.de ([212.227.17.10]:61452 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751291AbdFFRjI (ORCPT ); Tue, 6 Jun 2017 13:39:08 -0400 Date: Tue, 6 Jun 2017 19:39:04 +0200 From: Karsten Merker To: Palmer Dabbelt Cc: Arnd Bergmann , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/7] RISC-V: Top-Level Makefile for riscv{32,64} Message-ID: <20170606173904.GA3220@excalibur.cnev.de> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-No-Archive: yes User-Agent: Mutt/1.5.23 (2014-03-12) X-Provags-ID: V03:K0:6eCzLF2W+z2HV4lOXsSG4YiVG6rAXRhnVA9kIQZc1+5tOuSCGkL OAN/qfFS8Ao45Mi7K03ICDJAlLEoiOjMV5xOr4TXD4stvCvurrISP+se/YCbslG2Xn8D3m+ LnyCjtUtB9d9GvnFVFj3dLXXw3U1p3u9IseBm0Im3Rv9Zl+/XGuVO6VsFpoT3irNUnCske8 9L/PeOedlIYtSJbWAdWkg== X-UI-Out-Filterresults: notjunk:1;V01:K0:BfM7tvrmJOw=:frxz/+u4Bu9a25icL7tMQ+ Kci5tDsLIkH6FzNYU2USjCOFiHeG6RyXKvSSFMgZqvQatSbkAuozOUzej6nJX7VmGYvblv3Hi TdZ4zvyH6IIxXiM3IGxBUY/3w6aQMUBYnaJMs8OGTmA5lHwIuPWRdFuGMwzD0wkc0/fePbbln qivwAKSkaKCwmM3i8u2TojHpCHeAwkr9J7zIWlFSyys6pUI+t03mFhfrsf6Xu8DhzSrwCuHeF hYiD7x5n9/7f0vyC0rdUM7ezgX5UPmpnWPy5chKANg3tCH4LDTkd2Wp4TC+gZ2J2k8Y/dNH8D RMIsW4m5PSTk+zK44smPuLClJSJ8wHsyrj6pbecMoYu0rPlPB6rixJShTzhliPv8yZWkeTpup evyj/MdMWCRBZSE9oou3RxKaJcSozWwv185W1mfYFMGSS835Z6hv8n6lObHrWeWSNaFEfb+D8 jIDKOfgeW7t2e9/Ey8V3vQt/rqlpOcrZavROPDKqU9xIx+Nml0+G08KCTlOvZA8zTdgCR9CGb 6iool+gEFPuIDTwfP4/gg53+NLF5xdlwV1tpZ7q9iWLJGM4y/98HDk0cb/6mY4yASU/UQeap6 8tNPYr9XRCzqpwaGXYPMgXEqwjatMAYR6DDJbhUxBS8vkB85M4el4HXJY3f+T2eGDADAaecfX QljAdlIUTy1w0DuP8nU8TXuyPdywaczUE5zBRHGBRuirVhAePTIwrWuM84mCuv0JiZI4= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 05, 2017 at 09:56:33PM -0700, Palmer Dabbelt wrote: > On Mon, 29 May 2017 03:50:47 PDT (-0700), Arnd Bergmann wrote: > > On Sat, May 27, 2017 at 2:57 AM, Palmer Dabbelt wrote: > >> On Tue, 23 May 2017 04:30:50 PDT (-0700), Arnd Bergmann wrote: > >>> On Tue, May 23, 2017 at 2:41 AM, Palmer Dabbelt wrote: > >>>> RISC-V has both 32-bit and 64-bit base ISAs, but they are very similar. > >>>> Like some other platforms, we'd like to share one arch directory between > >>>> the two of them. > >>> > >>> I think we mainly do the others for backwards-compatibility with ancient > >>> build scripts, and we don't need that here. Instead, you could add one more > >>> line to the 'SUBARCH:=' statement that interprets the uname output. > >> > >> I don't think that does the same thing. The desired effect of this diff is: > >> > >> * "uname -m" when running on a RISC-V machine returns either riscv32 or > >> riscv64, as that's what tools like autoconf expect when trying to find > >> tuples. > >> > >> * I can cross compile for riscv32 and riscv64. That's currently controlled by > >> a Kconfig setting, but ARCH=riscv32 vs ARCH=riscv64 controlls what defconfig > >> sets. > >> > >> * I can natively compile for riscv32 and riscv64. That uses the same Kconfig > >> setting, and the same ARCH=riscv32 vs ARCH=riscv64 switch for defconfig. > > > > Right, but my point is that a new architecture should not rely on 'ARCH=' > > to pick the defconfig, we only do that on a couple of architectures for > > backwards compatibility with old scripts. > > > >> Neither of the two Kconfig issues is a big deal, but we de need "uname -m" to > >> return "riscv64" or "riscv32" not "riscv". I think the only way to do that is > >> to set SRCARCH, but I'd be happy to change it if there's a better way. I think > >> if I just do this > >> > >> diff --git a/Makefile b/Makefile > >> index 0606f28..4adc609 100644 > >> --- a/Makefile > >> +++ b/Makefile > >> @@ -232,7 +232,8 @@ SUBARCH := $(shell uname -m | sed -e s/i.86/x86/ -e s/x86_64/x86/ \ > >> -e s/arm.*/arm/ -e s/sa110/arm/ \ > >> -e s/s390x/s390/ -e s/parisc64/parisc/ \ > >> -e s/ppc.*/powerpc/ -e s/mips.*/mips/ \ > >> - -e s/sh[234].*/sh/ -e s/aarch64.*/arm64/ ) > >> + -e s/sh[234].*/sh/ -e s/aarch64.*/arm64/ \ > >> + -e s/riscv.*/riscv/ ) > >> > >> # Cross compiling and selecting different set of gcc/bin-utils > >> # --------------------------------------------------------------------------- > >> @@ -269,14 +270,6 @@ ifeq ($(ARCH),x86_64) > >> SRCARCH := x86 > >> endif > >> > >> -# Additional ARCH settings for RISC-V > >> -ifeq ($(ARCH),riscv32) > >> - SRCARCH := riscv > >> -endif > >> -ifeq ($(ARCH),riscv64) > >> - SRCARCH := riscv > >> -endif > >> - > >> # Additional ARCH settings for sparc > >> ifeq ($(ARCH),sparc32) > >> SRCARCH := sparc > >> > >> then I'll end up with "uname -m" as "riscv" -- I haven't tried it, but that's > >> why we ended up with this diff in the first place. > > > > Do you mean the "uname -m" output comes from "${SRCARCH}" at > > the time of the kernel build? That would be easy enough to change > > by simply hardcoding it depending on CONFIG_64BIT. > > OK, I didn't know about COMPAT_UTS_MACHINE. That's a much better solution, > I'll use that. Hello Palmer, I suppose the commit: https://github.com/riscv/riscv-linux/commit/8c826930d2a19ecd4f1036f10a380dc4fddd0da5 aims to address this, but it appears to be incomplete. It lacks the first fragment of the patch above, i.e. the conversion from the "uname -m" output (i.e. "riscv{64,32}") to the canonical arch string (i.e. "riscv"). As a result, a native build (which normally doesn't explicitly pass ARCH=riscv to make and therefore relies on the output of "uname -m") would fail. Regards, Karsten -- Gem. Par. 28 Abs. 4 Bundesdatenschutzgesetz widerspreche ich der Nutzung sowie der Weitergabe meiner personenbezogenen Daten für Zwecke der Werbung sowie der Markt- oder Meinungsforschung.