devel@edk2.groups.io

[edk2-test PATCHv2 4/5] SctPkg: Unify CC_FLAGS for all GCC platforms

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3042 Cc: Barton Gao <gaojie@...> Cc: Eric Jin <eric.jin@...> Cc: Leif Lindholm <leif@...> Cc: Heinrich Schuchardt

By Daniel Schaefer · #71530 ·

[edk2-test PATCHv2 1/5] SctPkg: Add RISCV64 support by using Aarch64 sources

The sources were copied from Aarch64. Follow-up commit will adapt them for Riscv64. REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3042 Cc: Barton Gao <gaojie@...> Cc: Eric Jin

By Daniel Schaefer · #71529 ·

[edk2-test PATCHv2 0/5] Port SCT to RISCV64

Hi everyone, here's my second patch series for porting the SCT to RISCV64. Nothing substantial has changed. The first one was really just an RFC like Leif mentioned. Now I have cleaned it up and it

By Daniel Schaefer · #71528 ·

[edk2-test PATCHv2 2/5] SctPkg: Adapt Aarch64 sources for Riscv64

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3042 Cc: Barton Gao <gaojie@...> Cc: Eric Jin <eric.jin@...> Cc: Leif Lindholm <leif@...> Cc: Heinrich Schuchardt

By Daniel Schaefer · #71527 ·

Re: Does EDK2 ArmVirtPkg has support for a virtio-mmio-blk device

That BLK0: alias in the UEFI shell is the NOR flash not a virtio block device. -- Ard.

By Ard Biesheuvel · #71526 ·

Re: [edk2-test PATCH v1 2/2] SctPkg: Enable RISCV64 support using AARCH64 sources

No idea why it was added for Arm and Aarch64 copied it. It's not there for the other architectures. Let's remove it.

By Daniel Schaefer · #71525 ·

Re: [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

Hi Mike Could you help review this patch and push it? Liming may take leave now. Thanks, Bob

By Bob Feng · #71524 ·

Re: [Patch] BaseTools: Fix the ending character for Linux script BuildEnv

SMTP strips the intersting aspect, but as long as the end result is a BuildEnv that has no <CR> characters: Reviewed-by: Leif Lindholm <leif@...>

By Leif Lindholm <leif@...> · #71523 ·

Re: [edk2-test PATCH v1 2/2] SctPkg: Enable RISCV64 support using AARCH64 sources

I don't think so. The GNU family of toolchains are the only ones using .s/.S sources. Unless there are GNU-compatible assemblers that don't support the directive. Visual Studio uses .asm on ARM* and

By Leif Lindholm <leif@...> · #71522 ·

[Patch] BaseTools: Fix the ending character for Linux script BuildEnv

Signed-off-by: Bob Feng <bob.c.feng@...> Cc: Yuwei Chen <yuwei.chen@...> Cc: Matthew Carlson <matthewfcarlson@...> Cc: Michael D Kinney <michael.d.kinney@...>=0D Cc: Leif

By Bob Feng · #71521 ·

[PATCH v3 4/4] UefiCpuPkg/MpInitLib: Remove unused Lock from MP_CPU_EXCHANGE_INFO

The Lock is no longer needed since "LOCK XADD" was used in MpFuncs.nasm for ApIndex atomic increment. Signed-off-by: Ray Ni <ray.ni@...> Cc: Eric Dong <eric.dong@...> Cc: Laszlo Ersek

By Ni, Ray · #71520 ·

[PATCH v3 3/4] UefiCpuPkg/MpInitLib: Use NASM struc to avoid hardcode offset

In Windows environment, "dumpbin /disasm" is used to verify the disassembly before and after using NASM struc doesn't change. Signed-off-by: Ray Ni <ray.ni@...> Cc: Eric Dong

By Ni, Ray · #71519 ·

[PATCH v3 2/4] MdePkg/Nasm.inc: add macros for C types used in structure definition

Signed-off-by: Ray Ni <ray.ni@...> Cc: Michael D Kinney <michael.d.kinney@...> Cc: Liming Gao <gaoliming@...> Cc: Zhiguang Liu <zhiguang.liu@...> ---

By Ni, Ray · #71518 ·

[PATCH v3 1/4] UefiCpuPkg/MpInitLib: Use XADD to avoid lock acquire/release

When AP firstly wakes up, MpFuncs.nasm contains below logic to assign an unique ApIndex to each AP according to who comes first: ---ASM--- TestLock: xchg [edi], eax cmp eax,

By Ni, Ray · #71517 ·

[PATCH v3 0/4] Use XADD to avoid lock acquire/release

Patch #1 follows Mike's suggestion to use XADD to avoid lock acquire/release. Patch #2 follows Laszlo's suggestion to add global NASM macros for NASM struc usage. Patch #3 simply remves all hardcode

By Ni, Ray · #71516 ·

Re: [edk2-test PATCH v1 2/2] SctPkg: Enable RISCV64 support using AARCH64 sources

Thanks for pointing out the different meanings of .align. They are documented in binutils-2.36/gas/doc/as.info chapter 7.3. .balign is GNU assembler specific. Do we have to consider other assemblers

By Heinrich Schuchardt · #71515 ·

Re: Does EDK2 ArmVirtPkg has support for a virtio-mmio-blk device

Yes. First, the MMIO transport (as I remember from checking it last time, which was quite some time ago) is very different between 0.9.5 and 1.0. Second, device initialization steps differ: -

By Laszlo Ersek · #71514 ·

Re: [PATCH v5 1/1] BaseTools: Use pip module if available, CI uses it by default

Hi Bob, Something has gone wrong with the line endings on BaseTools/BuildEnv, breaking the build on posix systems: /work/git/edk2$ . edksetup.sh -bash: /work/git/orion/edk2/BaseTools/BuildEnv: line

By Leif Lindholm <leif@...> · #71513 ·

Re: [PATCH 1/1] ArmPkg: add DebugLib to OemMiscLibNull

Acked-by: Ard Biesheuvel <ardb@...>

By Ard Biesheuvel · #71512 ·

Re: [edk2-test PATCH v1 2/2] SctPkg: Enable RISCV64 support using AARCH64 sources

The glory of the GAS .align directive is that its meaning differs between architectures and executable formats. On ARM/AArch64, as well as on RISC-V (I think?) the alignment is described as a power of

By Leif Lindholm <leif@...> · #71511 ·

Messages