From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.953.1585672335943034660 for ; Tue, 31 Mar 2020 09:32:16 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7185A30E; Tue, 31 Mar 2020 09:32:15 -0700 (PDT) Received: from [10.37.8.12] (unknown [10.37.8.12]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id BAEC73F71E; Tue, 31 Mar 2020 09:32:14 -0700 (PDT) Subject: Re: [edk2-devel] [PATCH] .azurepipelines: Enable CI for OvmfPkg and EmulatorPkg To: devel@edk2.groups.io, sean.brogan@microsoft.com, Ard Biesheuvel References: <10604.1585671987556224046@groups.io> From: "Ard Biesheuvel" Message-ID: <15bdbf11-56c8-54fc-9607-0b6cf7283825@arm.com> Date: Tue, 31 Mar 2020 18:32:13 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 MIME-Version: 1.0 In-Reply-To: <10604.1585671987556224046@groups.io> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 3/31/20 6:26 PM, Sean via Groups.Io wrote: > On Mon, Mar 30, 2020 at 11:41 PM, Ard Biesheuvel wrote: >=20 > Not sure I follow. Which command line are we talking about? >=20 > @Ard - In this Platform CI, ArmVirt is building and running AARCH64 but= =20 > not ARM 32bit.=C2=A0 Would it be valuable to build for ARM too? >=20 Yes, absolutely. > I prototyped it but want to make sure I am calling QEMU with the=20 > parameters you would expect to work with ArmVirtPkg >=20 > qemu-system-arm -M virt -cpu cortex-a15 -pflash=20 > /home/vsts/work/1/s/Build/ArmVirtQemu-ARM/DEBUG_GCC5/FV/QEMU_EFI.fd -m=20 > 1024 -net none -serial stdio -drive=20 > file=3Dfat:rw:/home/vsts/work/1/s/Build/ArmVirtQemu-ARM/DEBUG_GCC5/Virt= ualDrive,format=3Draw,media=3Ddisk=20 > -display none >=20 The parameters look fine. Note that you can use the qemu-system-aarch64=20 binary as well, so no need to install both. >=20 > PR build results can be seen here:=20 > https://dev.azure.com/tianocore/edk2-ci-play/_build/results?buildId=3D5= 074&view=3Dresults > PR code change:=C2=A0https://github.com/spbrogan/edk2/pull/14 Yep, looking good. The main focus is obviously on X64 and AARCH64, but=20 that only increases the risk that we might regress on IA32 or ARM=20 without anyone noticing. So having both IA32 and ARM is a good thing.