From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 6F70B7803CC for ; Wed, 24 Apr 2024 01:57:59 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=zF+kQCRk8Wd5+VspUGD+LKiA7JxPN+q+DhyxkUHOrwY=; c=relaxed/simple; d=groups.io; h=Message-ID:Date:MIME-Version:User-Agent:Subject:From:To:Cc:Reply-To:References:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:List-Unsubscribe-Post:List-Unsubscribe:Content-Type; s=20240206; t=1713923877; v=1; b=lFsrJSJcCHeVJaXNsWiGzAFaAQXyehCBIt10gMjcquAMYdYCdBG3SBbPe9ZsC7oR9a5lVzSP JHiPHxccrDPKLJT1/kZT+3p5ti5wis5xSiQYPgBEiWDYkZt2gVxSEsUxvn+Ljz2EEM7Imybt9yK O+N+man1f1Q/GDDfcqV3hEB1WzepBZuRP7IlZd6xSPV8dmZpPmt3pL90p/ty6yQ5ayHqOenqR9V H2HzOoVjri71dWZCx0/fD9RZYjV0cWuCA7WRSfSyRHuTHFoMVRmQJYxDXb8aN9jtCvIB86bMKzU J6S3otg5Hr3fXzs0bZ19Tdq9APViyNrfylQsHAvMFwvng== X-Received: by 127.0.0.2 with SMTP id HoqCYY7687511x5DXSsntGi0; Tue, 23 Apr 2024 18:57:57 -0700 X-Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by mx.groups.io with SMTP id smtpd.web10.7059.1713923876181696458 for ; Tue, 23 Apr 2024 18:57:56 -0700 X-Received: from loongson.cn (unknown [10.40.24.149]) by gateway (Coremail) with SMTP id _____8BxFvAgZyhm4boBAA--.9608S3; Wed, 24 Apr 2024 09:57:52 +0800 (CST) X-Received: from [10.40.24.149] (unknown [10.40.24.149]) by localhost.localdomain (Coremail) with SMTP id AQAAf8AxLN4eZyhmguYCAA--.12018S3; Wed, 24 Apr 2024 09:57:50 +0800 (CST) Message-ID: <8ce013e7-766c-4ca1-ba90-8d8fe0f4b947@loongson.cn> Date: Wed, 24 Apr 2024 09:57:50 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [edk2-devel] [PATCH v1 0/4] Adjust the QemuFwCfgLibMmio and add PEI stage From: "Chao Li" To: devel@edk2.groups.io, kraxel@redhat.com, Ard Biesheuvel Cc: Jiewen Yao , Leif Lindholm , Sami Mujawar , Sunil V L , Andrei Warkentin Reply-To: devel@edk2.groups.io,lichao@loongson.cn References: <20240417081256.3125187-1-lichao@loongson.cn> <17C70907D38C479B.20815@groups.io> <17C88FBD6FF53AEF.31749@groups.io> In-Reply-To: <17C88FBD6FF53AEF.31749@groups.io> X-CM-TRANSID: AQAAf8AxLN4eZyhmguYCAA--.12018S3 X-CM-SenderInfo: xolfxt3r6o00pqjv00gofq/1tbiAQARCGYnIZISEwAAsr X-Coremail-Antispam: 1Uk129KBjDUn29KB7ZKAUJUUUUU529EdanIXcx71UUUUU7KY7 ZEXasCq-sGcSsGvfJ3UbIjqfuFe4nvWSU5nxnvy29KBjDU0xBIdaVrnUUvcSsGvfC2Kfnx nUUI43ZEXa7xR_UUUUUUUUU== Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Tue, 23 Apr 2024 18:57:56 -0700 Resent-From: lichao@loongson.cn List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: pfrpL2VeAEWDoB2oLCTzUcerx7686176AA= Content-Type: multipart/alternative; boundary="------------yZ8cklcUFyBBpHr0dOxeAca2" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=lFsrJSJc; dmarc=none; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io --------------yZ8cklcUFyBBpHr0dOxeAca2 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable Hi Gerd and Ard, Can I submit the V2 this week? I want all OvmfPkg changes to be meged=20 before the 202405 feature freeze. Thanks, Chao On 2024/4/22 17:21, Chao Li wrote: > > Hi Ard, > > Could you take a look at this patch set and give you some suggestions? > > On 2024/4/17 18:01, Chao Li wrote: >> >> Hi Gerd, >> >> On 2024/4/17 17:35, Gerd Hoffmann wrote: >>> On Wed, Apr 17, 2024 at 04:12:56PM +0800, Chao Li wrote: >>>> Patch1: Added three PCDs for QemuFwCfgLibMmio >>>> Patch2: Sparate QemuFwCfgLibMmio.c into two files and default as DXE >>>> stage library. >>>> Patch3: Added QemuFwCfgMmiLib PEI version >>>> Patch4: Rename QemuFwCfgLibMmio.inf to QemuFwCfgMmioDxeLib.inf and >>>> enable it in AARCH64 and RISCV64. >>> Ok, I see, you are using the PCDs because global variables don't work >>> in PEI. >> Yes. :) >>> Alternative approach would be to create a HOB for that (see >>> EFI_HOB_PLATFORM_INFO used by X64). Not sure this is a good idea >>> though given that we have three different architectures using that code= . >>> Ard, any advise? >> I think HOB might be heavy, and the dynamically PCD is also uses the=20 >> HOB, so this I think this way is more simpler. I'm also looking=20 >> forward Ard's suggestions. :) >>>> OvmfPkg: Add three PCDs for QemuFwCfgLib >>>> OvmfPkg: Separate QemuFwCfgLibMmio.c into two files >>> This patch should be splitted into two, one doing the code split withou= t >>> functional change, and one which switches from global variables to PCDs >>> (or HOB). >> OK, I will split it in V2. >>> Otherwise this looks good to me (and I'd suggest to keep and merge this >>> as separate patch series). >> Sure! >>> take care, >>> Gerd >>> >>> >>> >>> >>> >=20 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#118179): https://edk2.groups.io/g/devel/message/118179 Mute This Topic: https://groups.io/mt/105573103/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --------------yZ8cklcUFyBBpHr0dOxeAca2 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

Hi Gerd and Ard,

Can I submit the V2 this week? I want all OvmfPkg changes to be meged before the 202405 feature freeze.


=
Thanks,
Chao
On 2024/4/22 17:21, Chao Li wrote:

Hi Ard,

Could you take a look at this patch set and give you some suggestions?

On 2024/4/17 18:01, Chao Li wrote:

Hi Gerd,

On 2024/4/17 17:35, Gerd Hoffmann wrote:
On Wed, Apr 17, 2024 at 04=
:12:56PM +0800, Chao Li wrote:

Patch1: Added three PCDs=
 for QemuFwCfgLibMmio
Patch2: Sparate QemuFwCfgLibMmio.c into two files and default as DXE
stage library.
Patch3: Added QemuFwCfgMmiLib PEI version
Patch4: Rename QemuFwCfgLibMmio.inf to QemuFwCfgMmioDxeLib.inf and
enable it in AARCH64 and RISCV64.

Ok, I see, you are using t=
he PCDs because global variables don't work
in PEI.
Yes. :)
Alternative approach would=
 be to create a HOB for that (see
EFI_HOB_PLATFORM_INFO used by X64).  Not sure this is a good idea
though given that we have three different architectures using that code.
Ard, any advise?
I think HOB might be heavy, and the dynamically PCD is also uses the HOB, so this I think this way is more simpler. I'm also looking forward Ard's suggestions. :)
  OvmfPkg: Add three PCD=
s for QemuFwCfgLib
  OvmfPkg: Separate QemuFwCfgLibMmio.c into two files

This patch should be split=
ted into two, one doing the code split without
functional change, and one which switches from global variables to PCDs
(or HOB).
OK, I will split it in V2.
Otherwise this looks good =
to me (and I'd suggest to keep and merge this
as separate patch series).
Sure!
take care,
  Gerd
=20
_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#118179) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--------------yZ8cklcUFyBBpHr0dOxeAca2--