From: "Nhi Pham via groups.io" <nhi=os.amperecomputing.com@groups.io>
To: devel@edk2.groups.io, ardb@kernel.org,
Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
Cc: Leif Lindholm <quic_llindhol@quicinc.com>,
Ard Biesheuvel <ardb+tianocore@kernel.org>,
Sami Mujawar <sami.mujawar@arm.com>
Subject: Re: [edk2-devel] [PATCH edk2-platforms 2/2] SbsaQemu: set CPU model in SMBIOS
Date: Mon, 15 Jul 2024 10:24:52 +0700 [thread overview]
Message-ID: <b8c3e0b0-b6a7-4e9c-9b8b-59cbdec120c5@os.amperecomputing.com> (raw)
In-Reply-To: <CAMj1kXFtCUHWEN32Ag8H-CUtb+hEgp2h=t5zzrVKXiQGxomqjw@mail.gmail.com>
On 7/12/2024 5:01 PM, Ard Biesheuvel via groups.io wrote:
> On Fri, 12 Jul 2024 at 11:24, Marcin Juszkiewicz
> <marcin.juszkiewicz@linaro.org> wrote:
>>
>> SbsaQemu platform have several cpu models available. Since start it
>> reported "arm-virt" one in SMBIOS instead of real one.
>>
>> This change replaces it with the real cpu model like Cortex-A57,
>> Neoverse-N2 etc.
>>
>> Requires change in EDK2 to make PcdProcessorVersion dynamic.
>>
>> Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
>
> I am going to make the same remark as I did before in a different
> context: how is it guaranteed that the code that sets the PCD executes
> before the code that consumes it?
We could implement the setting in a 'PlatformInitDxe', which will be
added to the 'APPRIORI DXE' list to ensure it is scheduled to run in the
early DXE phase. This approach will help initialize the neccessary
compoenents early in the DXE phase, providing a stable foundation for
subsequent operations.
>
> Making PCDs dynamic is usually not the solution here, unless they are
> set in the PEI phase and consumed in the DXE phase or after.
>
> The EFI SMBIOS protocol has an 'update string' method that you can
> call from a platform driver. That might be a better fit here, and the
> PCD can remain fixed, and carry a fallback default value.
>
In fact that the 'update string' method could eat up the boot time.
Hence, the SMBIOS string patching is not prefered.
Regards,
Nhi
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119931): https://edk2.groups.io/g/devel/message/119931
Mute This Topic: https://groups.io/mt/107179083/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next prev parent reply other threads:[~2024-07-15 3:25 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-12 9:23 [edk2-devel] [PATCH 0/2] Set CPU model in SMBIOS Marcin Juszkiewicz
2024-07-12 9:23 ` [edk2-devel] [PATCH 1/2] ArmPkg: make PcdProcessorVersion dynamic Marcin Juszkiewicz
2024-07-12 9:23 ` [edk2-devel] [PATCH edk2-platforms 2/2] SbsaQemu: set CPU model in SMBIOS Marcin Juszkiewicz
2024-07-12 10:01 ` Ard Biesheuvel
2024-07-12 10:58 ` Marcin Juszkiewicz
2024-07-15 3:24 ` Nhi Pham via groups.io [this message]
2024-07-15 8:24 ` Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b8c3e0b0-b6a7-4e9c-9b8b-59cbdec120c5@os.amperecomputing.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox