<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 28, 2023 at 9:47 AM Sunil V L <<a href="mailto:sunilvl@ventanamicro.com">sunilvl@ventanamicro.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On Fri, Jun 23, 2023 at 11:39:32AM -0700, Tuan Phan wrote:<br>
> Make sure VirtNorFlashDxe loaded before VariableRuntimeDxe as it<br>
> is the backend flash driver.<br>
> <br>
> Signed-off-by: Tuan Phan <<a href="mailto:tphan@ventanamicro.com" target="_blank">tphan@ventanamicro.com</a>><br>
> ---<br>
>  OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf | 10 ++++++++++<br>
>  1 file changed, 10 insertions(+)<br>
> <br>
> diff --git a/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf b/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf<br>
> index 21e4ba67379f..9ab8eb3ba7d8 100644<br>
> --- a/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf<br>
> +++ b/OvmfPkg/RiscVVirt/RiscVVirtQemu.fdf<br>
> @@ -53,6 +53,16 @@ READ_STATUS        = TRUE<br>
>  READ_LOCK_CAP      = TRUE<br>
>  READ_LOCK_STATUS   = TRUE<br>
>  <br>
> +APRIORI DXE {<br>
> +  INF  MdeModulePkg/Universal/DevicePathDxe/DevicePathDxe.inf<br>
> +  INF  MdeModulePkg/Universal/PCD/Dxe/Pcd.inf<br>
> +  INF  MdeModulePkg/Universal/ReportStatusCodeRouter/RuntimeDxe/ReportStatusCodeRouterRuntimeDxe.inf<br>
> +  INF  MdeModulePkg/Universal/StatusCodeHandler/RuntimeDxe/StatusCodeHandlerRuntimeDxe.inf<br>
> +  INF  EmbeddedPkg/Drivers/FdtClientDxe/FdtClientDxe.inf<br>
> +  INF  UefiCpuPkg/CpuDxeRiscV64/CpuDxeRiscV64.inf<br>
> +  INF  OvmfPkg/VirtNorFlashDxe/VirtNorFlashDxe.inf<br>
> +}<br>
> +<br>
Hi Tuan,<br>
<br>
Actually, Ard had recommended not to use APRIORI and hence we avoided<br>
it when we upstreamed RiscVVirt. So, I am wondering whether this can be<br>
avoided by using depex in CpuDxe on gEfiVariableArchProtocolGuid? <br>
<br></blockquote><div>Hi Sunil,</div><div>Not sure what the reason behind avoiding APRIORI besides it is a workaround for broken DEPEX. BTW, what we need is to put VirtNorFlashDxe loaded before VariableRuntimeDxe which doesn't depend on any modules. I don't see any other clearer way than modifying VirNorFlashDxe as shown in the first version of this series.</div><div><br></div><div>The CpuDxeRiscV64 in the aprioriy list as VirNorFlashDxe depends on it.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Thanks,<br>
Sunil<br>
</blockquote></div></div>


<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>


Groups.io Links:<p>


  
    You receive all messages sent to this group.
  
  


<p>
<a target="_blank" href="https://edk2.groups.io/g/devel/message/106487">View/Reply Online (#106487)</a> |


  

|

  <a target="_blank" href="https://groups.io/mt/99724477/1813853">Mute This Topic</a>


| <a href="https://edk2.groups.io/g/devel/post">New Topic</a>

<br>




<a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> |
<a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> |

<a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a>

 [edk2-devel-archive@redhat.com]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>