From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) by mx.groups.io with SMTP id smtpd.web11.22031.1687970834114277421 for ; Wed, 28 Jun 2023 09:47:14 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@ventanamicro.com header.s=google header.b=d61bneko; spf=pass (domain: ventanamicro.com, ip: 209.85.210.177, mailfrom: sunilvl@ventanamicro.com) Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-666eef03ebdso3588671b3a.1 for ; Wed, 28 Jun 2023 09:47:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ventanamicro.com; s=google; t=1687970833; x=1690562833; 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=zf6VQ4hJ80ZBlybm9umLaKj1PS3EkVFx7izQM1N5v7Q=; b=d61bnekovJG51RLJvIBfaSo8hiPakvt8BRl9T69IDYtkwUo8+7M9o5A8FRH0GywfnJ yeFdn42zEK+BRytRLGMw9RMVGmVBcchKqH1XfowlboGYroe3RiB9qJwXPS7a01tadMH3 gNRWqazJxKnHz62gEm2G1hPnY2e8JQ2TCc5kCvkXUj5GmNuSnJHch10lvKv4lFezzn6T BTnIQW3q/OVY+kgjESCV96neLSrjrQnEqfxZAdISphzF+GO91t/kO5SwPzPvuFKCMvzg MWi0UKRm2mhnqGO03tNkmrPaVIE2Ov836T+lIzm5Qc33DvNGPQmEFKS+nLn7peBJuDM0 8u/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687970833; x=1690562833; 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=zf6VQ4hJ80ZBlybm9umLaKj1PS3EkVFx7izQM1N5v7Q=; b=K7VxKyQX9gNejQm1mzd79qbE0SrphL5yeBzxRijyDENhi90aS3VERB2yN0pDhZdSjb 1V1qCiS5h0DeOR0elcmFOI4P38g9vWNo7+6imGDkKgKL5Igvie+kGQ7A7qqI5hOTJz12 iTS0mngW/Sk3MDOeBTE5zF9j2AoWZTGy+mvpq9huFrGtoKhov3ZwTkNRZCBVVN5hhnsK RkKqOR/oAbb7Vde4s5rluabxyxVIRTGMLG5V97q31t5RB9beWTKL4RALRePfHMxDUlRy wHQXru/IRf7kElaWPrq29JY6kiztPErWNqVqHjeyjEdCyiXWg+w/KWxNA36ZodphuHbn CyiA== X-Gm-Message-State: AC+VfDyMona4YKX73YyaKqIxo91uY2LqoBeYpUamREXBUAAfF2wrBQP/ xOfNmeozqAowbCtl/nq2xSy1Cg== X-Google-Smtp-Source: ACHHUZ6Ix6lh1HS9rQE+55brdf6vi3JE19nDgCVxx2dlolwIhrg4ke62Kkd3Q8/udxP4nQ4wwDoZ9g== X-Received: by 2002:a05:6a00:995:b0:681:50fd:2b89 with SMTP id u21-20020a056a00099500b0068150fd2b89mr1992342pfg.30.1687970833457; Wed, 28 Jun 2023 09:47:13 -0700 (PDT) Return-Path: Received: from sunil-laptop ([106.51.184.72]) by smtp.gmail.com with ESMTPSA id x16-20020a62fb10000000b0066875f17266sm7237287pfm.135.2023.06.28.09.47.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 28 Jun 2023 09:47:12 -0700 (PDT) Date: Wed, 28 Jun 2023 22:17:07 +0530 From: "Sunil V L" To: Tuan Phan Cc: devel@edk2.groups.io, michael.d.kinney@intel.com, gaoliming@byosoft.com.cn, zhiguang.liu@intel.com, git@danielschaefer.me, andrei.warkentin@intel.com, ardb+tianocore@kernel.org Subject: Re: [PATCH v4 5/7] OvmfPkg/RiscVVirt: Add VirtNorFlashDxe to APRIORI list Message-ID: References: <20230623183934.23905-1-tphan@ventanamicro.com> <20230623183934.23905-6-tphan@ventanamicro.com> MIME-Version: 1.0 In-Reply-To: <20230623183934.23905-6-tphan@ventanamicro.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Jun 23, 2023 at 11:39:32AM -0700, Tuan Phan wrote: > Make sure VirtNorFlashDxe loaded before VariableRuntimeDxe as it > is the backend flash driver. > > Signed-off-by: Tuan Phan > --- > OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf b/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf > index 21e4ba67379f..9ab8eb3ba7d8 100644 > --- a/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf > +++ b/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf > @@ -53,6 +53,16 @@ READ_STATUS = TRUE > READ_LOCK_CAP = TRUE > READ_LOCK_STATUS = TRUE > > +APRIORI DXE { > + INF MdeModulePkg/Universal/DevicePathDxe/DevicePathDxe.inf > + INF MdeModulePkg/Universal/PCD/Dxe/Pcd.inf > + INF MdeModulePkg/Universal/ReportStatusCodeRouter/RuntimeDxe/ReportStatusCodeRouterRuntimeDxe.inf > + INF MdeModulePkg/Universal/StatusCodeHandler/RuntimeDxe/StatusCodeHandlerRuntimeDxe.inf > + INF EmbeddedPkg/Drivers/FdtClientDxe/FdtClientDxe.inf > + INF UefiCpuPkg/CpuDxeRiscV64/CpuDxeRiscV64.inf > + INF OvmfPkg/VirtNorFlashDxe/VirtNorFlashDxe.inf > +} > + Hi Tuan, Actually, Ard had recommended not to use APRIORI and hence we avoided it when we upstreamed RiscVVirt. So, I am wondering whether this can be avoided by using depex in CpuDxe on gEfiVariableArchProtocolGuid? Thanks, Sunil