From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) by mx.groups.io with SMTP id smtpd.web10.8649.1675927142397222147 for ; Wed, 08 Feb 2023 23:19:02 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@ventanamicro.com header.s=google header.b=JKJQCaDl; spf=pass (domain: ventanamicro.com, ip: 209.85.216.49, mailfrom: sunilvl@ventanamicro.com) Received: by mail-pj1-f49.google.com with SMTP id v6-20020a17090ad58600b00229eec90a7fso5738919pju.0 for ; Wed, 08 Feb 2023 23:19:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ventanamicro.com; s=google; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=Xi7q5tY33FX0hMHBRRiUDUneOK/RXHISemAsjB1mfFs=; b=JKJQCaDlwh30C5pFFee1E5xKnSA0uN/WBEDKDWQgfDmfBRVU72KEFaTJp+Kq6binfi W10GekOjEYxZGUPCI++PMsfjaye/7Lm3DU7xJw/n2Q8qSeRIiCsHHD6rdNUZ8BObgxrA oD2Mk0PO9FxSvq0rtZbdRsMeqxouK/QD0mJQ+/VECliNuIzSbji9amiEpXhE+P8+2TF0 EjcKrF58vWs7SRvrLj27VPUu7ovf4dSwhCdVAJ5rcJSywzUBHLDJ75l2HPULRAdN/thE mIqgGrL4WdNhJZQfo8/gp42NES2wa/WV4Pb78Vf6RdSa+///k6EL5YylTcKkn0cjnfpm h4yw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Xi7q5tY33FX0hMHBRRiUDUneOK/RXHISemAsjB1mfFs=; b=mPluLZL6uPlwEDzsF3dYO5LT+LdtIXiolfwjm96pWmEyh3AVbrgKLQ1Et3XiNM2yPO QntzEIHAWSkFNHow92a96UmeQbPMn8t2FiKmu3BqiPc1ahosHczZUB8kTagsncwT0ooa fnq0wlvFzDgU3srs5Db1/SBno0815SO9tUG8LCDNvKE45K6pN+JPBeBNcF3rEc/VuhYc UHHsmy/sAczLEI5TjCZux2LiWnVBMqsm2eBTmarD4OWtF27oAwL9zthXPZiBaGNrkDHD DscmF8wP6UyjwVi8K/7ToZ44Kx/L6CEBw4N1RvlZ3EJzkJIOAUgADwCX4LIplNLkdlrB 3abQ== X-Gm-Message-State: AO0yUKX7LXjB3PXQfh7k4nhKzQ+fnGOQND5SmZixvUhP51DmHPYXNZRx 5vFSw95egu+LVyCYE4QzlwAcFI23Iu87rF2S X-Google-Smtp-Source: AK7set/ktS9dCupU/tCw8681lht9kyGxALfkW+NjN458xg02SHs+eiWxcJLLdAtflgxMxD3EofJWdg== X-Received: by 2002:a17:90a:7f98:b0:230:f874:3600 with SMTP id m24-20020a17090a7f9800b00230f8743600mr9259794pjl.28.1675927136875; Wed, 08 Feb 2023 23:18:56 -0800 (PST) Return-Path: Received: from sunil-laptop ([49.206.14.226]) by smtp.gmail.com with ESMTPSA id mn19-20020a17090b189300b00231224439c1sm1633609pjb.27.2023.02.08.23.18.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 Feb 2023 23:18:56 -0800 (PST) Date: Thu, 9 Feb 2023 12:48:50 +0530 From: "Sunil V L" To: devel@edk2.groups.io, michael.d.kinney@intel.com Cc: "Gao, Liming" , "Liu, Zhiguang" , Abner Chang Subject: Re: [edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 03/20] MdePkg: Add BaseRiscVSbiLib Library for RISC-V Message-ID: References: <20230128191807.2080547-1-sunilvl@ventanamicro.com> <20230128191807.2080547-4-sunilvl@ventanamicro.com> MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Mike, On Thu, Feb 09, 2023 at 01:45:48AM +0000, Michael D Kinney wrote: > One comment below. > > Mike > > > -----Original Message----- > > From: Sunil V L > > Sent: Saturday, January 28, 2023 11:18 AM > > To: devel@edk2.groups.io > > Cc: Kinney, Michael D ; Gao, Liming ; Liu, Zhiguang ; > > Abner Chang > > Subject: [edk2-staging/RiscV64QemuVirt PATCH V7 03/20] MdePkg: Add BaseRiscVSbiLib Library for RISC-V > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 > > > > This library is required to make SBI ecalls from the S-mode EDK2. > > This is mostly copied from > > edk2-platforms/Silicon/RISC-V/ProcessorPkg/Library/RiscVEdk2SbiLib > > > > Cc: Michael D Kinney > > Cc: Liming Gao > > Cc: Zhiguang Liu > > Signed-off-by: Sunil V L > > Acked-by: Abner Chang > > --- > > MdePkg/MdePkg.dec | 4 + > > MdePkg/MdePkg.dsc | 3 + > > MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.inf | 25 +++ > > MdePkg/Include/Library/BaseRiscVSbiLib.h | 127 +++++++++++ > > MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.c | 227 ++++++++++++++++++++ > > 5 files changed, 386 insertions(+) > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > > index 3d08f20d15b0..ca2e4dcf815c 100644 > > --- a/MdePkg/MdePkg.dec > > +++ b/MdePkg/MdePkg.dec > > @@ -316,6 +316,10 @@ [LibraryClasses.IA32, LibraryClasses.X64] > > ## @libraryclass Provides function to support TDX processing. > > TdxLib|Include/Library/TdxLib.h > > > > +[LibraryClasses.RISCV64] > > + ## @libraryclass Provides function to make ecalls to SBI > > + BaseRiscVSbiLib|Include/Library/BaseRiscVSbiLib.h > > + > > [Guids] > > # > > # GUID defined in UEFI2.1/UEFI2.0/EFI1.1 > > diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc > > index 32a852dc466e..0ac7618b4623 100644 > > --- a/MdePkg/MdePkg.dsc > > +++ b/MdePkg/MdePkg.dsc > > @@ -190,4 +190,7 @@ [Components.ARM, Components.AARCH64] > > MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicArmVirt.inf > > MdePkg/Library/BaseStackCheckLib/BaseStackCheckLib.inf > > > > +[Components.RISCV64] > > + MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.inf > > + > > [BuildOptions] > > diff --git a/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.inf b/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.inf > > new file mode 100644 > > index 000000000000..d03132bf01c1 > > --- /dev/null > > +++ b/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.inf > > @@ -0,0 +1,25 @@ > > +## @file > > +# RISC-V Library to call SBI ecalls > > +# > > +# Copyright (c) 2020, Hewlett Packard Enterprise Development LP. All rights reserved.
> > +# > > +# SPDX-License-Identifier: BSD-2-Clause-Patent > > +# > > +## > > + > > +[Defines] > > + INF_VERSION = 0x0001001b > > + BASE_NAME = BaseRiscVSbiLib > > + FILE_GUID = D742CF3D-E600-4009-8FB5-318073008508 > > + MODULE_TYPE = BASE > > + VERSION_STRING = 1.0 > > + LIBRARY_CLASS = RiscVSbiLib > > + > > +[Sources] > > + BaseRiscVSbiLib.c > > + > > +[Packages] > > + MdePkg/MdePkg.dec > > + > > +[LibraryClasses] > > + BaseLib > > diff --git a/MdePkg/Include/Library/BaseRiscVSbiLib.h b/MdePkg/Include/Library/BaseRiscVSbiLib.h > > new file mode 100644 > > index 000000000000..3a3cbfb879f1 > > --- /dev/null > > +++ b/MdePkg/Include/Library/BaseRiscVSbiLib.h > > @@ -0,0 +1,127 @@ > > +/** @file > > + Library to call the RISC-V SBI ecalls > > + > > + Copyright (c) 2021-2022, Hewlett Packard Development LP. All rights reserved.
> > + > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > + > > + @par Glossary: > > + - Hart - Hardware Thread, similar to a CPU core > > + > > + Currently, EDK2 needs to call SBI only to set the time and to do system reset. > > + > > +**/ > > + > > +#ifndef RISCV_SBI_LIB_H_ > > +#define RISCV_SBI_LIB_H_ > > + > > +#include > > + > > +/* SBI Extension IDs */ > > +#define SBI_EXT_TIME 0x54494D45 > > +#define SBI_EXT_SRST 0x53525354 > > + > > +/* SBI function IDs for TIME extension*/ > > +#define SBI_EXT_TIME_SET_TIMER 0x0 > > + > > +/* SBI function IDs for SRST extension */ > > +#define SBI_EXT_SRST_RESET 0x0 > > + > > +#define SBI_SRST_RESET_TYPE_SHUTDOWN 0x0 > > +#define SBI_SRST_RESET_TYPE_COLD_REBOOT 0x1 > > +#define SBI_SRST_RESET_TYPE_WARM_REBOOT 0x2 > > + > > +#define SBI_SRST_RESET_REASON_NONE 0x0 > > +#define SBI_SRST_RESET_REASON_SYSFAIL 0x1 > > + > > +/* SBI return error codes */ > > +#define SBI_SUCCESS 0 > > +#define SBI_ERR_FAILED -1 > > +#define SBI_ERR_NOT_SUPPORTED -2 > > +#define SBI_ERR_INVALID_PARAM -3 > > +#define SBI_ERR_DENIED -4 > > +#define SBI_ERR_INVALID_ADDRESS -5 > > +#define SBI_ERR_ALREADY_AVAILABLE -6 > > +#define SBI_ERR_ALREADY_STARTED -7 > > +#define SBI_ERR_ALREADY_STOPPED -8 > > + > > +#define SBI_LAST_ERR SBI_ERR_ALREADY_STOPPED > > + > > +typedef struct { > > + UINT64 BootHartId; > > + VOID *PeiServiceTable; // PEI Service table > > + VOID *PrePiHobList; // Pre PI Hob List > > + UINT64 FlattenedDeviceTree; // Pointer to Flattened Device tree > > +} EFI_RISCV_FIRMWARE_CONTEXT; > > + > > +// > > +// EDK2 OpenSBI firmware extension return status. > > +// > > +typedef struct { > > + UINTN Error; ///< SBI status code > > + UINTN Value; ///< Value returned > > +} SBI_RET; > > + > > +VOID > > +EFIAPI > > +SbiSetTimer ( > > + IN UINT64 Time > > + ); > > + > > +EFI_STATUS > > +EFIAPI > > +SbiSystemReset ( > > + IN UINTN ResetType, > > + IN UINTN ResetReason > > + ); > > + > > +/** > > + Get firmware context of the calling hart. > > + > > + @param[out] FirmwareContext The firmware context pointer. > > +**/ > > +VOID > > +EFIAPI > > +GetFirmwareContext ( > > + OUT EFI_RISCV_FIRMWARE_CONTEXT **FirmwareContext > > + ); > > + > > +/** > > + Set firmware context of the calling hart. > > + > > + @param[in] FirmwareContext The firmware context pointer. > > +**/ > > +VOID > > +EFIAPI > > +SetFirmwareContext ( > > + IN EFI_RISCV_FIRMWARE_CONTEXT *FirmwareContext > > + ); > > + > > +/** > > + Get pointer to OpenSBI Firmware Context > > + > > + Get the pointer of firmware context. > > + > > + @param FirmwareContextPtr Pointer to retrieve pointer to the > > + Firmware Context. > > +**/ > > +VOID > > +EFIAPI > > +GetFirmwareContextPointer ( > > + IN OUT EFI_RISCV_FIRMWARE_CONTEXT **FirmwareContextPtr > > + ); > > + > > +/** > > + Set pointer to OpenSBI Firmware Context > > + > > + Set the pointer of firmware context. > > + > > + @param FirmwareContextPtr Pointer to Firmware Context. > > +**/ > > +VOID > > +EFIAPI > > +SetFirmwareContextPointer ( > > + IN EFI_RISCV_FIRMWARE_CONTEXT *FirmwareContextPtr > > + ); > > + > > +#endif > > diff --git a/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.c b/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.c > > new file mode 100644 > > index 000000000000..15222a528753 > > --- /dev/null > > +++ b/MdePkg/Library/BaseRiscVSbiLib/BaseRiscVSbiLib.c > > @@ -0,0 +1,227 @@ > > +/** @file > > + Instance of the SBI ecall library. > > + > > + It allows calling an SBI function via an ecall from S-Mode. > > + > > + Copyright (c) 2021-2022, Hewlett Packard Development LP. All rights reserved.
> > + > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > + > > +**/ > > + > > +#include > > +#include > > +#include > > +#include > > + > > +// > > +// Maximum arguments for SBI ecall > > +#define SBI_CALL_MAX_ARGS 6 > > + > > +/** > > + Call SBI call using ecall instruction. > > + > > + Asserts when NumArgs exceeds SBI_CALL_MAX_ARGS. > > + > > + @param[in] ExtId SBI extension ID. > > + @param[in] FuncId SBI function ID. > > + @param[in] NumArgs Number of arguments to pass to the ecall. > > + @param[in] ... Argument list for the ecall. > > + > > + @retval Returns SBI_RET structure with value and error code. > > + > > +**/ > > +STATIC > > +SBI_RET > > +EFIAPI > > +SbiCall ( > > + IN UINTN ExtId, > > + IN UINTN FuncId, > > + IN UINTN NumArgs, > > + ... > > + ) > > +{ > > + UINTN I; > > + SBI_RET Ret; > > + UINTN Args[SBI_CALL_MAX_ARGS]; > > + VA_LIST ArgList; > > + > > + VA_START (ArgList, NumArgs); > > + > > + if (NumArgs > SBI_CALL_MAX_ARGS) { > > + Ret.Error = SBI_ERR_INVALID_PARAM; > > + Ret.Value = -1; > > + return Ret; > > + } > > + > > + for (I = 0; I < SBI_CALL_MAX_ARGS; I++) { > > + if (I < NumArgs) { > > + Args[I] = VA_ARG (ArgList, UINTN); > > + } else { > > + // Default to 0 for all arguments that are not given > > + Args[I] = 0; > > + } > > + } > > + > > + VA_END (ArgList); > > + > > + register UINTN a0 asm ("a0") = Args[0]; > > + register UINTN a1 asm ("a1") = Args[1]; > > + register UINTN a2 asm ("a2") = Args[2]; > > + register UINTN a3 asm ("a3") = Args[3]; > > + register UINTN a4 asm ("a4") = Args[4]; > > + register UINTN a5 asm ("a5") = Args[5]; > > + register UINTN a6 asm ("a6") = (UINTN)(FuncId); > > + register UINTN a7 asm ("a7") = (UINTN)(ExtId); > > + > > + asm volatile ("ecall" \ > > + : "+r" (a0), "+r" (a1) \ > > + : "r" (a2), "r" (a3), "r" (a4), "r" (a5), "r" (a6), "r" (a7) \ > > + : "memory"); \ > > + Ret.Error = a0; > > + Ret.Value = a1; > > Why is inline assembly used here? I see use of .S files in BaseLib > Can this be converted to .S file? > It was ported as is from edk2-platforms. I can convert into .S file and I prefer to add it in this library itself than BaseLib. Thanks!, Sunil