On Mon, Feb 20, 2023 at 09:44:22AM -0800, Oliver Steffen wrote: > Quoting Sunil V L (2023-02-17 05:27:15) > > On Thu, Feb 16, 2023 at 03:45:49PM -0700, dann frazier wrote: > > > On Fri, Feb 10, 2023 at 5:30 AM Sunil V L <suni...@ventanamicro.com> > > > wrote: > > > > > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 > > > > > > > > Add support for RISC-V qemu virt machine. Most of the changes are > > > > migrated from > > > > edk2-platforms repo and added qemu specific libraries under OvmfPkg. > > > > > > > > The series has passed CI tests > > > > (https://github.com/tianocore/edk2/pull/4023) > > > > > > > > These changes are available at: > > > > https://github.com/vlsunil/edk2/tree/RiscV64QemuVirt_V8 > > > > > > > > The series can be tested as per instructions @ > > > > https://github.com/vlsunil/riscv-uefi-edk2-docs/wiki/RISC-V-Qemu-Virt-support > > > > > > Hi Sunil, > > > > > > Thanks for your work getting this merged! In the above wiki, it > > > notes that GCC 12+ is not supported. Is that still accurate? If so, > > > can you clarify what is blocking that? > > > > > Hi Dan, > > > > Please see https://bugzilla.tianocore.org/show_bug.cgi?id=4061. > > > > My attempt to fix this issue > > (https://edk2.groups.io/g/devel/message/93831) was not accepted due to > > the concerns that it can cause weird issues in CI. > > > > So, we are left with either support gcc <12 or gcc >=12. We can mandate > > gcc 12 itself for RISC-V, but that change need to be done hand in hand > > with CI tests moving to use gcc 12. Otherwise, it will break CI. > > > > Let me check with CI owners on their roadmap to move to gcc 12. > > > > Thanks, > > Sunil > > > > Adding Chris. > > Pull request for a new contaimer image for Linux CI jobs with gcc12 is > open: > https://github.com/tianocore/containers/pull/60 > > Meanwhile you could try this image (temporary test build) in the CI: > > docker pull ghcr.io/osteffenrh/edk2-containers/fedora-37-test:latest > > - Oliver > Hey Oliver!,
Thank you very much!. However, I found a way to break this dependency. So, you can move the CI image anytime you want once this patch gets merged. https://edk2.groups.io/g/devel/message/100379 Thanks! Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#100380): https://edk2.groups.io/g/devel/message/100380 Mute This Topic: https://groups.io/mt/96874977/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
