From patchwork Wed May 29 19:55:01 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Oleksii Kurochko X-Patchwork-Id: 13679452 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org 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 smtp.lore.kernel.org (Postfix) with ESMTPS id A37D6C25B7C for ; Wed, 29 May 2024 19:55:29 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.732319.1138250 (Exim 4.92) (envelope-from ) id 1sCPOG-0003Ns-CC; Wed, 29 May 2024 19:55:16 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 732319.1138250; Wed, 29 May 2024 19:55:16 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1sCPOG-0003Nl-9D; Wed, 29 May 2024 19:55:16 +0000 Received: by outflank-mailman (input) for mailman id 732319; Wed, 29 May 2024 19:55:15 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1sCPOF-0003Nf-4S for xen-devel@lists.xenproject.org; Wed, 29 May 2024 19:55:15 +0000 Received: from mail-lf1-x12b.google.com (mail-lf1-x12b.google.com [2a00:1450:4864:20::12b]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 5d3c7546-1df5-11ef-90a1-e314d9c70b13; Wed, 29 May 2024 21:55:14 +0200 (CEST) Received: by mail-lf1-x12b.google.com with SMTP id 2adb3069b0e04-529614b8c29so216246e87.2 for ; Wed, 29 May 2024 12:55:14 -0700 (PDT) Received: from fedora.. ([94.75.70.14]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5297066b249sm1344203e87.178.2024.05.29.12.55.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 May 2024 12:55:12 -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: 5d3c7546-1df5-11ef-90a1-e314d9c70b13 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1717012513; x=1717617313; darn=lists.xenproject.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=pAzpqmChn/xM5jNHeWdfIx1rNGhRNDTsnTQUp2N3w0c=; b=I2t0OkCHFl8rSoZPn8ZRHNqYHKWWp2EIQv2LyQ13V393SpKpOyzoHOMbhcW4p8amAR KWq2x7UNpPNQ7Os/cHlUT1MMhhWH/sGDDDpfD4yNMc5+i5FvJm7YLNIQLsGwBOHoN+8M ChA8Bo+uOVZJ0fu07pAoyUu/fZr74UjZLQiFMSqJ0ksfoME1WsXZG9438RJiCKdDrGTw HjOK4VBeENpsz7iBkeMowiyTOFaoATT2r9qCc+ElMlcIXgUWuVubgQcXNJfU3NuOE9kW SS1XCD0lxbZ4bO5ia8O0x71OqkSEq7VA420/kp1Z0SvMPsJVm8OsiipAtFrmoo4mhzBJ 2dxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717012513; x=1717617313; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=pAzpqmChn/xM5jNHeWdfIx1rNGhRNDTsnTQUp2N3w0c=; b=Zj3tXmioLBG6m/HhQQEy9lYPfZMh/U6eJxFvHpi6TtVuMDKNpCigbRx/4jh8cXRD6W DFs7PO8EmrU6iGFD6uAkfDpTV+jGJbmRV3zUEtmuiRd+pj3qfVbXWgdG0LkaniBkvBt6 T0mJJgiW6DDh/vU8tRjd/WuPbhGJOt7vWAE6QkJJ2ceCHt1t87YrPrhzd0VDGlEVjV90 ezS4r1kFhCIMdmF+ppbIZBFpEXjtrCfOsPMDaS2V9VfQ8R2iNXv6XH9FymC4gV8bC0LN Q9ZHyzyRu1epCK4wMTs9li9h8q5YU77SOzOFkwAIgR9nAeq3FAMZe82BhwGSFLAWsS6D XRsA== X-Gm-Message-State: AOJu0YzSE4GDOCaLNLzUsbsGIDhcvcg2HuKzpESyh2HG0MdbICLSPxS5 Mj3AaqKZnSaAOXggKwtSvafnQw7SOLcjnCMdqnmt19asuI8Y9kxvsgnVjBht X-Google-Smtp-Source: AGHT+IEhwUhr5SI6KtlUMzXiIkwF4VeWu3DSGpjuardRodBn6EAmuWCRKQgr6QpcNaG3tPdKcFtcjA== X-Received: by 2002:ac2:42c7:0:b0:51d:d630:365c with SMTP id 2adb3069b0e04-52b7d419a32mr132874e87.4.1717012512944; Wed, 29 May 2024 12:55:12 -0700 (PDT) From: Oleksii Kurochko To: xen-devel@lists.xenproject.org Cc: Oleksii Kurochko , Doug Goldstein , Stefano Stabellini , Alistair Francis , Bob Eshleman , Connor Davis , "Daniel P. Smith" , Julien Grall , Bertrand Marquis , Michal Orzel , Volodymyr Babchuk , Andrew Cooper , George Dunlap , Jan Beulich , Shawn Anastasio , =?utf-8?q?Roger_Pau_Mon?= =?utf-8?q?n=C3=A9?= Subject: [PATCH for-4.19? v12 0/8] *Enable build of full Xen for RISC-V Date: Wed, 29 May 2024 21:55:01 +0200 Message-ID: X-Mailer: git-send-email 2.45.0 MIME-Version: 1.0 This patch series performs all of the additions necessary to drop the build overrides for RISCV and enable the full Xen build. Except in cases where compatibile implementations already exist (e.g. atomic.h and bitops.h), the newly added definitions are simple. The patch series is based on the following patch series: - [PATCH v2 for-4.19 00/13] xen/bitops: Untangle ffs()/fls() infrastructure [1] - [PATCH] move __read_mostly to xen/cache.h [2] Right now, the patch series doesn't have a direct dependency on [2] and it provides __read_mostly in the patch: [PATCH v3 26/34] xen/riscv: add definition of __read_mostly However, it will be dropped as soon as [2] is merged or at least when the final version of the patch [2] is provided. As an option, it can be considered to merge arch-specific patch and then just rebase [2] and drop arch-specific changes. The link to the branch with rebased patches on top of [1] could be found here: https://gitlab.com/xen-project/people/olkur/xen/-/commits/riscv-full-xen-build-v12 [1] https://lore.kernel.org/xen-devel/20240524200338.1232391-1-andrew.cooper3@citrix.com/ [2] https://lore.kernel.org/xen-devel/f25eb5c9-7c14-6e23-8535-2c66772b333e@suse.com/ *** I think that I need to write a separate email requesting approval for this patch series to be included in Xen 4.19. Most of the patches are RISC-V specific, so there is a low risk of breaking anything else. There is only one patch that touches common or arch-specific files, and it doesn't introduce any functional changes. Since I can't approve it myself, I am asking for someone else to do so. *** --- Changes in V12: - Rebase the patch series on top of [1] mentioned above. - Update the cover letter message. - "[PATCH v11 3/9] xen/bitops: implement fls{l}() in common logic" was droped as it is a part of patch series [1] mentioned above. - Other changes are specific to specific patches. Please look at changes for specific patch. --- Changes in V11: - Patch was merged to staging: - [PATCH v10 05/14] xen/riscv: introduce cmpxchg.h - [PATCH v10 06/14] xen/riscv: introduce atomic.h - [PATCH v10 07/14] xen/riscv: introduce monitor.h - [PATCH v10 09/14] xen/riscv: add required things to current.h - [PATCH v10 11/14] xen/riscv: introduce vm_event_*() functions - Other changes are specific to specific patches. Please look at changes for specific patch. --- Changes in V10: - Patch was merged to staging: - [PATCH v9 04/15] xen/bitops: put __ffs() into linux compatible header - Other changes are specific to specific patches. Please look at changes for specific patch. --- Changes in V9: - Patch was merged to staging: - [PATCH v8 07/17] xen/riscv: introduce io.h - [PATCH v7 14/19] xen/riscv: add minimal stuff to page.h to build full Xen - Other changes are specific to specific patches. Please look at changes for specific patch. --- Changes in V8: - Patch was merged to staging: - [PATCH v7 01/19] automation: introduce fixed randconfig for RISC-V - [PATCH v7 03/19] xen/riscv: introduce extenstion support check by compiler - Other changes are specific to specific patches. Please look at changes for specific patch. - Update the commit message: - drop the dependency from STATIC_ASSERT_UNREACHABLE() implementation. - Add suggestion to merge arch-specific changes related to __read_mostly. --- Changes in V7: - Patch was merged to staging: [PATCH v6 15/20] xen/riscv: add minimal stuff to processor.h to build full Xen. - Other changes are specific to specific patches. Please look at changes for specific patch. --- Changes in V6: - Update the cover letter message: drop already merged dependecies and add a new one. - Patches were merged to staging: - [PATCH v5 02/23] xen/riscv: use some asm-generic headers ( even v4 was merged to staging branch, I just wasn't apply changes on top of the latest staging branch ) - [PATCH v5 03/23] xen/riscv: introduce nospec.h - [PATCH v5 10/23] xen/riscv: introduces acrquire, release and full barriers - Introduce new patches: - xen/riscv: introduce extenstion support check by compiler - xen/bitops: put __ffs() and ffz() into linux compatible header - xen/bitops: implement fls{l}() in common logic - The following patches were dropped: - drop some patches related to bitops operations as they were introduced in another patch series [...] - introduce new version for generic __ffs(), ffz() and fls{l}(). - Merge patch from patch series "[PATCH v9 0/7] Introduce generic headers" to this patch series as only one patch left in the generic headers patch series and it is more about RISC-V. - Other changes are specific to specific patches. please look at specific patch. --- Changes in V5: - Update the cover letter as one of the dependencies were merged to staging. - Was introduced asm-generic for atomic ops and separate patches for asm-generic bit ops - Moved fence.h to separate patch to deal with some patches dependecies on fence.h - Patches were dropped as they were merged to staging: * [PATCH v4 03/30] xen: add support in public/hvm/save.h for PPC and RISC-V * [PATCH v4 04/30] xen/riscv: introduce cpufeature.h * [PATCH v4 05/30] xen/riscv: introduce guest_atomics.h * [PATCH v4 06/30] xen: avoid generation of empty asm/iommu.h * [PATCH v4 08/30] xen/riscv: introduce setup.h * [PATCH v4 10/30] xen/riscv: introduce flushtlb.h * [PATCH v4 11/30] xen/riscv: introduce smp.h * [PATCH v4 15/30] xen/riscv: introduce irq.h * [PATCH v4 16/30] xen/riscv: introduce p2m.h * [PATCH v4 17/30] xen/riscv: introduce regs.h * [PATCH v4 18/30] xen/riscv: introduce time.h * [PATCH v4 19/30] xen/riscv: introduce event.h * [PATCH v4 22/30] xen/riscv: define an address of frame table - Other changes are specific to specific patches. please look at specific patch --- Changes in V4: - Update the cover letter message: new patch series dependencies. - Some patches were merged to staging, so they were dropped in this patch series: [PATCH v3 09/34] xen/riscv: introduce system.h [PATCH v3 18/34] xen/riscv: introduce domain.h [PATCH v3 19/34] xen/riscv: introduce guest_access.h - Was sent out of this patch series: [PATCH v3 16/34] xen/lib: introduce generic find next bit operations - [PATCH v3 17/34] xen/riscv: add compilation of generic find-next-bit.c was droped as CONFIG_GENERIC_FIND_NEXT_BIT was dropped. - All other changes are specific to a specific patch. --- Changes in V3: - Update the cover letter message - The following patches were dropped as they were merged to staging: [PATCH v2 03/39] xen/riscv:introduce asm/byteorder.h [PATCH v2 04/39] xen/riscv: add public arch-riscv.h [PATCH v2 05/39] xen/riscv: introduce spinlock.h [PATCH v2 20/39] xen/riscv: define bug frame tables in xen.lds.S [PATCH v2 34/39] xen: add RISCV support for pmu.h [PATCH v2 35/39] xen: add necessary headers to common to build full Xen for RISC-V - Instead of the following patches were introduced new: [PATCH v2 10/39] xen/riscv: introduce asm/iommu.h [PATCH v2 11/39] xen/riscv: introduce asm/nospec.h - remove "asm/" for commit messages which start with "xen/riscv:" - code style updates. - add emulation of {cmp}xchg_* for 1 and 2 bytes types. - code style fixes. - add SPDX and footer for the newly added headers. - introduce generic find-next-bit.c. - some other mionor changes. ( details please find in a patch ) --- Changes in V2: - Drop the following patches as they are the part of [2]: [PATCH v1 06/57] xen/riscv: introduce paging.h [PATCH v1 08/57] xen/riscv: introduce asm/device.h [PATCH v1 10/57] xen/riscv: introduce asm/grant_table.h [PATCH v1 12/57] xen/riscv: introduce asm/hypercall.h [PATCH v1 13/57] xen/riscv: introduce asm/iocap.h [PATCH v1 15/57] xen/riscv: introduce asm/mem_access.h [PATCH v1 18/57] xen/riscv: introduce asm/random.h [PATCH v1 21/57] xen/riscv: introduce asm/xenoprof.h [PATCH v1 24/57] xen/riscv: introduce asm/percpu.h [PATCH v1 29/57] xen/riscv: introduce asm/hardirq.h [PATCH v1 33/57] xen/riscv: introduce asm/altp2m.h [PATCH v1 38/57] xen/riscv: introduce asm/monitor.h [PATCH v1 39/57] xen/riscv: introduce asm/numa.h [PATCH v1 42/57] xen/riscv: introduce asm/softirq.h - xen/lib.h in most of the cases were changed to xen/bug.h as mostly functionilty of bug.h is used. - align arch-riscv.h with Arm's version of it. - change the Author of commit with introduction of asm/atomic.h. - update some definition from spinlock.h. - code style changes. --- Oleksii Kurochko (8): xen/riscv: disable unnecessary configs xen: introduce generic non-atomic test_*bit() xen/riscv: introduce bitops.h xen/riscv: add definition of __read_mostly xen/riscv: add minimal stuff to mm.h to build full Xen xen/riscv: add minimal amount of stubs to build full Xen xen/riscv: enable full Xen build xen/README: add compiler and binutils versions for RISC-V64 README | 4 + automation/gitlab-ci/build.yaml | 4 + xen/arch/arm/include/asm/bitops.h | 67 ---- xen/arch/ppc/include/asm/bitops.h | 52 --- xen/arch/ppc/include/asm/page.h | 2 +- xen/arch/ppc/mm-radix.c | 2 +- xen/arch/riscv/Makefile | 17 +- xen/arch/riscv/arch.mk | 4 - xen/arch/riscv/configs/tiny64_defconfig | 12 +- xen/arch/riscv/early_printk.c | 168 ---------- xen/arch/riscv/include/asm/bitops.h | 137 ++++++++ xen/arch/riscv/include/asm/cache.h | 2 + xen/arch/riscv/include/asm/domain.h | 2 + xen/arch/riscv/include/asm/mm.h | 240 ++++++++++++++ xen/arch/riscv/include/asm/p2m.h | 2 + xen/arch/riscv/mm.c | 52 ++- xen/arch/riscv/setup.c | 10 +- xen/arch/riscv/stubs.c | 415 ++++++++++++++++++++++++ xen/arch/riscv/traps.c | 25 ++ xen/arch/x86/include/asm/bitops.h | 31 +- xen/include/xen/bitops.h | 146 +++++++++ 21 files changed, 1069 insertions(+), 325 deletions(-) create mode 100644 xen/arch/riscv/include/asm/bitops.h create mode 100644 xen/arch/riscv/stubs.c