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=-12.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS 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 884CEC47097 for ; Thu, 3 Jun 2021 23:33:48 +0000 (UTC) Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 40E80613FA for ; Thu, 3 Jun 2021 23:33:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 40E80613FA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Received: from list by lists.xenproject.org with outflank-mailman.136594.253187 (Exim 4.92) (envelope-from ) id 1lowqH-0005gs-7s; Thu, 03 Jun 2021 23:33:37 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 136594.253187; Thu, 03 Jun 2021 23:33:37 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lowqH-0005gl-4m; Thu, 03 Jun 2021 23:33:37 +0000 Received: by outflank-mailman (input) for mailman id 136594; Thu, 03 Jun 2021 23:33:36 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lowqF-0005gf-TC for xen-devel@lists.xenproject.org; Thu, 03 Jun 2021 23:33:35 +0000 Received: from mail-il1-x136.google.com (unknown [2607:f8b0:4864:20::136]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id 58fe0324-ed44-4528-88e9-18cdc8f5dc4e; Thu, 03 Jun 2021 23:33:33 +0000 (UTC) Received: by mail-il1-x136.google.com with SMTP id i13so1471650ilk.3 for ; Thu, 03 Jun 2021 16:33:33 -0700 (PDT) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 58fe0324-ed44-4528-88e9-18cdc8f5dc4e DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=LzTmbLr1R+Utddp1aVDlm+2T7NkmZxeRM9ikLAhGO7M=; b=sYdd5i96yOUGzz6QFOfuFpPZYQ04fumcSMhdJaG55I4uIWIosqIden5FadbuD3bGtX wGVFJDPbEXzQ4FKW4gCzaPanhdeaZP0vJV6ySeC2RWsIqTpVTTQNgwtzIuGCo+6+OWwe G/6fZhmjB57VrqPqGgQxl4Zr4SPTEMpvu1Kao8AQOI8zZ7OG0jyeYVmh+R7ZvI9nrrzg WPr7T0sGdR/lUJEZTyXzh4E1yN2Ihf9EI7iJv7t1233L+Q2ZiYget0FXa3GMY4IwUoF/ iO/pqB+hTbC9+7cEktqsxR4S46/Ftl65M6r6+6dihFvQHs9QvrWMbPBmsPadUQAkvsqX u1gw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=LzTmbLr1R+Utddp1aVDlm+2T7NkmZxeRM9ikLAhGO7M=; b=ie4nwLHnCMyZwPF4HQ7deAOflLUcRIWCLuuXbZFl5RKRHArMooPb+fl5pUTyexQvb9 6Fmo57wSbHbsuwJtQdjFlTf29uVL0QTLpEtC2+a2Z2zLT06tRZPgouWKLup+MBeHJLSe +/qhGHJntViA50vjnJpz8V7GX/qqGRujxa57j0oPPq2Ru0JNG+w2pD77jUk8+pRsBeZ+ O2PtY/zUP5B/QaAvg2cKtNbs1Mn4MilqFHczT7yAZ10qnV3BzEdwAkKGf/K2yGDAdEsN i2IF8iW7v/9i10NfkHZbsxodxWJl8wnqKiBIBSuaRv070YEdBMD4CiloocPXyIFb3gGA 381g== X-Gm-Message-State: AOAM533ks1lMq2yf/O+zTmRxbWg7hKl/wOUHqBRzbd8DAYx+nTxPk6Gk M71aBy/ded1sk6UXrNoLHaEFZoRSeLea+GqiI94= X-Google-Smtp-Source: ABdhPJxULwAzKpZZFgf0OJV/4LZF4D8z+3BNELae9HyhiM4Tu50EN0T8TxZaCF034cLPQCWdE3Xjh0psbEP2odEQb50= X-Received: by 2002:a05:6e02:d08:: with SMTP id g8mr1288789ilj.40.1622763213291; Thu, 03 Jun 2021 16:33:33 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Alistair Francis Date: Fri, 4 Jun 2021 09:33:07 +1000 Message-ID: Subject: Re: [PATCH v7 2/2] xen: Add files needed for minimal riscv build To: Connor Davis Cc: "open list:X86" , Bobby Eshleman , Andrew Cooper , George Dunlap , Ian Jackson , Jan Beulich , Julien Grall , Stefano Stabellini , Wei Liu Content-Type: text/plain; charset="UTF-8" On Thu, Jun 3, 2021 at 9:38 AM Connor Davis wrote: > > Add arch-specific makefiles and configs needed to build for > riscv. Also add a minimal head.S that is a simple infinite loop. > head.o can be built with > > $ make XEN_TARGET_ARCH=riscv64 SUBSYSTEMS=xen -C xen tiny64_defconfig > $ make XEN_TARGET_ARCH=riscv64 SUBSYSTEMS=xen -C xen TARGET=riscv64/head.o > > No other TARGET is supported at the moment. > > Signed-off-by: Connor Davis > --- > Bob: I moved back to XEN_TARGET_ARCH=riscv64 because supplying > just XEN_TARGET_ARCH=riscv causes TARGET_ARCH == TARGET_SUBARCH, and > that broke the build after the recent commit b6ecd5c8bc > "build: centralize / unify asm-offsets generation". It also deviates > from how x86 and arm work now, so I think this change is for the best > for now. That commit is also why the PHONY include target is added > in the riscv/Makefile. > --- > MAINTAINERS | 8 +++++ > config/riscv64.mk | 5 +++ > xen/Makefile | 8 +++-- > xen/arch/riscv/Kconfig | 47 +++++++++++++++++++++++++ > xen/arch/riscv/Kconfig.debug | 0 > xen/arch/riscv/Makefile | 2 ++ > xen/arch/riscv/Rules.mk | 0 > xen/arch/riscv/arch.mk | 14 ++++++++ > xen/arch/riscv/configs/tiny64_defconfig | 13 +++++++ > xen/arch/riscv/riscv64/asm-offsets.c | 0 > xen/arch/riscv/riscv64/head.S | 6 ++++ > xen/include/asm-riscv/config.h | 47 +++++++++++++++++++++++++ > 12 files changed, 148 insertions(+), 2 deletions(-) > create mode 100644 config/riscv64.mk > create mode 100644 xen/arch/riscv/Kconfig > create mode 100644 xen/arch/riscv/Kconfig.debug > create mode 100644 xen/arch/riscv/Makefile > create mode 100644 xen/arch/riscv/Rules.mk > create mode 100644 xen/arch/riscv/arch.mk > create mode 100644 xen/arch/riscv/configs/tiny64_defconfig > create mode 100644 xen/arch/riscv/riscv64/asm-offsets.c > create mode 100644 xen/arch/riscv/riscv64/head.S > create mode 100644 xen/include/asm-riscv/config.h > > diff --git a/MAINTAINERS b/MAINTAINERS > index d46b08a0d2..956e71220d 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -456,6 +456,14 @@ F: tools/libs/light/libxl_nonetbuffer.c > F: tools/hotplug/Linux/remus-netbuf-setup > F: tools/hotplug/Linux/block-drbd-probe > > +RISCV > +M: Bob Eshleman > +R: Connor Davis > +S: Supported > +F: config/riscv64.mk > +F: xen/arch/riscv/ > +F: xen/include/asm-riscv/ I volunteer to be a maintainer as well, feel free to say no :) I did the QEMU RISC-V H extension port and have a pretty good understanding of the RISC-V Hypervisor extension. > + > RTDS SCHEDULER > M: Dario Faggioli > M: Meng Xu > diff --git a/config/riscv64.mk b/config/riscv64.mk > new file mode 100644 > index 0000000000..a5a21e5fa2 > --- /dev/null > +++ b/config/riscv64.mk > @@ -0,0 +1,5 @@ > +CONFIG_RISCV := y > +CONFIG_RISCV_64 := y > +CONFIG_RISCV_$(XEN_OS) := y > + > +CONFIG_XEN_INSTALL_SUFFIX := > diff --git a/xen/Makefile b/xen/Makefile > index 7ce7692354..89879fad4c 100644 > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -26,7 +26,9 @@ MAKEFLAGS += -rR > EFI_MOUNTPOINT ?= $(BOOT_DIR)/efi > > ARCH=$(XEN_TARGET_ARCH) > -SRCARCH=$(shell echo $(ARCH) | sed -e 's/x86.*/x86/' -e s'/arm\(32\|64\)/arm/g') > +SRCARCH=$(shell echo $(ARCH) | \ > + sed -e 's/x86.*/x86/' -e s'/arm\(32\|64\)/arm/g' \ > + -e s'/riscv.*/riscv/g') > > # Don't break if the build process wasn't called from the top level > # we need XEN_TARGET_ARCH to generate the proper config > @@ -35,7 +37,8 @@ include $(XEN_ROOT)/Config.mk > # Set ARCH/SUBARCH appropriately. > export TARGET_SUBARCH := $(XEN_TARGET_ARCH) > export TARGET_ARCH := $(shell echo $(XEN_TARGET_ARCH) | \ > - sed -e 's/x86.*/x86/' -e s'/arm\(32\|64\)/arm/g') > + sed -e 's/x86.*/x86/' -e s'/arm\(32\|64\)/arm/g' \ > + -e s'/riscv.*/riscv/g') > > # Allow someone to change their config file > export KCONFIG_CONFIG ?= .config > @@ -335,6 +338,7 @@ _clean: delete-unfresh-files > $(MAKE) $(clean) xsm > $(MAKE) $(clean) crypto > $(MAKE) $(clean) arch/arm > + $(MAKE) $(clean) arch/riscv > $(MAKE) $(clean) arch/x86 > $(MAKE) $(clean) test > $(MAKE) -f $(BASEDIR)/tools/kconfig/Makefile.kconfig ARCH=$(ARCH) SRCARCH=$(SRCARCH) clean > diff --git a/xen/arch/riscv/Kconfig b/xen/arch/riscv/Kconfig > new file mode 100644 > index 0000000000..bd8381c5e0 > --- /dev/null > +++ b/xen/arch/riscv/Kconfig > @@ -0,0 +1,47 @@ > +config RISCV > + def_bool y > + > +config RISCV_64 > + def_bool y > + select 64BIT > + > +config ARCH_DEFCONFIG > + string > + default "arch/riscv/configs/tiny64_defconfig" > + > +menu "Architecture Features" > + > +source "arch/Kconfig" > + > +endmenu > + > +menu "ISA Selection" > + > +choice > + prompt "Base ISA" > + default RISCV_ISA_RV64IMA if RISCV_64 > + help > + This selects the base ISA extensions that Xen will target. > + > +config RISCV_ISA_RV64IMA > + bool "RV64IMA" > + help > + Use the RV64I base ISA, plus the "M" and "A" extensions > + for integer multiply/divide and atomic instructions, respectively. > + > +endchoice > + > +config RISCV_ISA_C > + bool "Compressed extension" > + help > + Add "C" to the ISA subsets that the toolchain is allowed to > + emit when building Xen, which results in compressed instructions > + in the Xen binary. > + > + If unsure, say N. I would change this to y if you are unsure. I don't expect any hardware to have an MMU (yet along the H extension) and no compressed instruction extension. Linux won't run without the C extension. Otherwise looks good: Reviewed-by: Alistair Francis Now the hard part of getting it to boot. Alistair > + > +endmenu > + > +source "common/Kconfig" > + > +source "drivers/Kconfig" > diff --git a/xen/arch/riscv/Kconfig.debug b/xen/arch/riscv/Kconfig.debug > new file mode 100644 > index 0000000000..e69de29bb2 > diff --git a/xen/arch/riscv/Makefile b/xen/arch/riscv/Makefile > new file mode 100644 > index 0000000000..942e4ffbc1 > --- /dev/null > +++ b/xen/arch/riscv/Makefile > @@ -0,0 +1,2 @@ > +.PHONY: include > +include: > diff --git a/xen/arch/riscv/Rules.mk b/xen/arch/riscv/Rules.mk > new file mode 100644 > index 0000000000..e69de29bb2 > diff --git a/xen/arch/riscv/arch.mk b/xen/arch/riscv/arch.mk > new file mode 100644 > index 0000000000..53dadb8975 > --- /dev/null > +++ b/xen/arch/riscv/arch.mk > @@ -0,0 +1,14 @@ > +######################################## > +# RISCV-specific definitions > + > +CFLAGS-$(CONFIG_RISCV_64) += -mabi=lp64 > + > +riscv-march-$(CONFIG_RISCV_ISA_RV64IMA) := rv64ima > +riscv-march-$(CONFIG_RISCV_ISA_C) := $(riscv-march-y)c > + > +# Note that -mcmodel=medany is used so that Xen can be mapped > +# into the upper half _or_ the lower half of the address space. > +# -mcmodel=medlow would force Xen into the lower half. > + > +CFLAGS += -march=$(riscv-march-y) -mstrict-align -mcmodel=medany > +CFLAGS += -I$(BASEDIR)/include > diff --git a/xen/arch/riscv/configs/tiny64_defconfig b/xen/arch/riscv/configs/tiny64_defconfig > new file mode 100644 > index 0000000000..3c9a2ff941 > --- /dev/null > +++ b/xen/arch/riscv/configs/tiny64_defconfig > @@ -0,0 +1,13 @@ > +# CONFIG_SCHED_CREDIT is not set > +# CONFIG_SCHED_RTDS is not set > +# CONFIG_SCHED_NULL is not set > +# CONFIG_SCHED_ARINC653 is not set > +# CONFIG_TRACEBUFFER is not set > +# CONFIG_HYPFS is not set > +# CONFIG_GRANT_TABLE is not set > +# CONFIG_SPECULATIVE_HARDEN_ARRAY is not set > + > +CONFIG_RISCV_64=y > +CONFIG_DEBUG=y > +CONFIG_DEBUG_INFO=y > +CONFIG_EXPERT=y > diff --git a/xen/arch/riscv/riscv64/asm-offsets.c b/xen/arch/riscv/riscv64/asm-offsets.c > new file mode 100644 > index 0000000000..e69de29bb2 > diff --git a/xen/arch/riscv/riscv64/head.S b/xen/arch/riscv/riscv64/head.S > new file mode 100644 > index 0000000000..0dbc27ba75 > --- /dev/null > +++ b/xen/arch/riscv/riscv64/head.S > @@ -0,0 +1,6 @@ > +#include > + > + .text > + > +ENTRY(start) > + j start > diff --git a/xen/include/asm-riscv/config.h b/xen/include/asm-riscv/config.h > new file mode 100644 > index 0000000000..e2ae21de61 > --- /dev/null > +++ b/xen/include/asm-riscv/config.h > @@ -0,0 +1,47 @@ > +#ifndef __RISCV_CONFIG_H__ > +#define __RISCV_CONFIG_H__ > + > +#if defined(CONFIG_RISCV_64) > +# define LONG_BYTEORDER 3 > +# define ELFSIZE 64 > +# define MAX_VIRT_CPUS 128u > +#else > +# error "Unsupported RISCV variant" > +#endif > + > +#define BYTES_PER_LONG (1 << LONG_BYTEORDER) > +#define BITS_PER_LONG (BYTES_PER_LONG << 3) > +#define POINTER_ALIGN BYTES_PER_LONG > + > +#define BITS_PER_LLONG 64 > + > +/* xen_ulong_t is always 64 bits */ > +#define BITS_PER_XEN_ULONG 64 > + > +#define CONFIG_RISCV_L1_CACHE_SHIFT 6 > +#define CONFIG_PAGEALLOC_MAX_ORDER 18 > +#define CONFIG_DOMU_MAX_ORDER 9 > +#define CONFIG_HWDOM_MAX_ORDER 10 > + > +#define OPT_CONSOLE_STR "dtuart" > +#define INVALID_VCPU_ID MAX_VIRT_CPUS > + > +/* Linkage for RISCV */ > +#ifdef __ASSEMBLY__ > +#define ALIGN .align 2 > + > +#define ENTRY(name) \ > + .globl name; \ > + ALIGN; \ > + name: > +#endif > + > +#endif /* __RISCV_CONFIG_H__ */ > +/* > + * Local variables: > + * mode: C > + * c-file-style: "BSD" > + * c-basic-offset: 4 > + * indent-tabs-mode: nil > + * End: > + */ > -- > 2.31.1 >