From: "Laszlo Ersek" <lersek@redhat.com>
To: Ard Biesheuvel <ardb@kernel.org>,
devel@edk2.groups.io, dandan.bi@intel.com,
Andrew Fish <afish@apple.com>
Cc: Leif Lindholm <leif@nuviainc.com>,
Michael D Kinney <michael.d.kinney@intel.com>,
Liming Gao <gaoliming@byosoft.com.cn>
Subject: Re: [edk2-devel] [edk2-platforms] [patch 00/35] Consume RegisterFilterLibNull instance
Date: Tue, 16 Mar 2021 17:23:39 +0100 [thread overview]
Message-ID: <37431ead-7d7e-74a2-2f94-b8cc82307902@redhat.com> (raw)
In-Reply-To: <CAMj1kXGgP10-Af70j6wOVYzfthqm9xduuvjV5p8oGvcnZwdATg@mail.gmail.com>
On 03/16/21 16:00, Ard Biesheuvel wrote:
> On Tue, 16 Mar 2021 at 15:56, Dandan Bi <dandan.bi@intel.com> wrote:
>>
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>> RFC: https://edk2.groups.io/g/devel/message/72530
>>
>>
>> Add RegisterFilterLibNull in dsc files in edk2-platforms repo,
>> which will be consumed by IoLib and BaseLib.
>>
>> This is the following update in edk2-platforms repo for the change in edk2,
>> which will add RegisterFilterLib dependency for IoLib and BaseLib to filter/trace
>> port IO/MMIO/MSR access.
>> https://edk2.groups.io/g/devel/message/72754
>>
>> Cc: Leif Lindholm <leif@nuviainc.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>>
>
> It is a bit disappointing that we have to update every platform in
> existence again to apply a change to a core module.
>
> Is there really not a better way to provide a 'default' resolution for
> a library class? Maybe a change to the .DEC format, so that the file
> which defines the library class can provide a resolution that is used
> if none is provided by the .DSC file?
I vaguely remember this feature was raised earlier (maybe implemented
via cross-Pkg DSC !include files). I believe some patches were posted
even. I think I asked for a libclass-by-libclass conversion for existent
platform DSCs (so I could verify that the new DSC include file would
give OVMF and ArmVirt the same resolutions as before). I don't recall
updates afterwards.
Thanks
Laszlo
>
>
>
>> Dandan Bi (35):
>> Drivers/ASIX: Consume RegisterFilterLibNull instance
>> Drivers/DisplayLink: Consume RegisterFilterLibNull instance
>> Drivers/OptionRomPkg: Consume RegisterFilterLibNull instance
>> Features/Debugging: Consume RegisterFilterLibNull instance
>> Features/Network: Consume RegisterFilterLibNull instance
>> Features/OutOfBandManagement: Consume RegisterFilterLibNull instance
>> Features/PowerManagement: Consume RegisterFilterLibNull instance
>> Features/SystemInformation: Consume RegisterFilterLibNull instance
>> Features/UserInterface: Consume RegisterFilterLibNull instance
>> Platform/AMD: Consume RegisterFilterLibNull instance
>> Platform/ARM: Consume RegisterFilterLibNull instance
>> Platform/BeagleBoard: Consume RegisterFilterLibNull instance
>> Platform/BoardModulePkg: Consume RegisterFilterLibNull instance
>> Platform/MinPlatformPkg: Consume RegisterFilterLibNull instance
>> Platform/QuarkPlatformPkg: Consume RegisterFilterLibNull instance
>> Platform/Vlv2TbltDevicePkg: Consume RegisterFilterLibNull instance
>> Platform/LeMaker: Consume RegisterFilterLibNull instance
>> Platform/Qemu: Consume RegisterFilterLibNull instance
>> Platform/RaspberryPi: Consume RegisterFilterLibNull instance
>> Platform/RISC-V: Consume RegisterFilterLibNull instance
>> Platform/SiFive: Consume RegisterFilterLibNull instance
>> Platform/Socionext: Consume RegisterFilterLibNull instance
>> Platform/SoftIron: Consume RegisterFilterLibNull instance
>> Silicon/Hisilicon: Consume RegisterFilterLibNull instance
>> Silicon/CoffeelakeSiliconPkg: Consume RegisterFilterLibNull instance
>> Silicon/IntelSiliconPkg: Consume RegisterFilterLibNull instance
>> Silicon/KabylakeSiliconPkg: Consume RegisterFilterLibNull instance
>> Silicon/QuarkSocPkg: Consume RegisterFilterLibNull instance
>> Silicon/TigerlakeSiliconPkg: Consume RegisterFilterLibNull instance
>> Silicon/Marvell: Consume RegisterFilterLibNull instance
>> Silicon/NXP: Consume RegisterFilterLibNull instance
>> Silicon/Openmoko: Consume RegisterFilterLibNull instance
>> Silicon/RISC_V: Consume RegisterFilterLibNull instance
>> Silicon/Synopsys/DesignWare: Consume RegisterFilterLibNull instance
>> Silicon/TexasInstruments: Consume RegisterFilterLibNull instance
>>
>> Drivers/ASIX/Asix.dsc | 1 +
>> Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkPkg.dsc | 1 +
>> Drivers/OptionRomPkg/OptionRomPkg.dsc | 3 ++-
>> .../Debugging/AcpiDebugFeaturePkg/Include/AcpiDebugFeature.dsc | 3 ++-
>> .../Debugging/BeepDebugFeaturePkg/Include/BeepDebugFeature.dsc | 3 ++-
>> .../PostCodeDebugFeaturePkg/Include/PostCodeDebugFeature.dsc | 3 ++-
>> .../Debugging/Usb3DebugFeaturePkg/Include/Usb3DebugFeature.dsc | 3 ++-
>> .../Intel/Network/NetworkFeaturePkg/Include/NetworkFeature.dsc | 3 ++-
>> .../OutOfBandManagement/IpmiFeaturePkg/Include/IpmiFeature.dsc | 3 ++-
>> .../OutOfBandManagement/SpcrFeaturePkg/Include/SpcrFeature.dsc | 3 ++-
>> .../Intel/PowerManagement/S3FeaturePkg/Include/S3Feature.dsc | 3 ++-
>> .../SmbiosFeaturePkg/Include/SmbiosFeature.dsc | 3 ++-
>> .../Intel/UserInterface/LogoFeaturePkg/Include/LogoFeature.dsc | 3 ++-
>> .../UserAuthFeaturePkg/Include/UserAuthFeature.dsc | 3 ++-
>> .../Include/VirtualKeyboardFeature.dsc | 3 ++-
>> Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 +
>> Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc | 1 +
>> Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc | 1 +
>> Platform/BeagleBoard/BeagleBoardPkg/BeagleBoardPkg.dsc | 3 ++-
>> Platform/Intel/BoardModulePkg/BoardModulePkg.dsc | 3 ++-
>> Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 3 ++-
>> Platform/Intel/QuarkPlatformPkg/Quark.dsc | 1 +
>> Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc | 1 +
>> Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 3 ++-
>> Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 3 ++-
>> Platform/LeMaker/CelloBoard/CelloBoard.dsc | 1 +
>> Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 1 +
>> Platform/RISC-V/PlatformPkg/RiscVPlatformPkg.dsc | 1 +
>> Platform/RaspberryPi/RPi3/RPi3.dsc | 3 ++-
>> Platform/RaspberryPi/RPi4/RPi4.dsc | 3 ++-
>> Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc | 1 +
>> .../U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
>> Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc | 1 +
>> Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 1 +
>> Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 1 +
>> Silicon/Hisilicon/Hisilicon.dsc.inc | 1 +
>> Silicon/Intel/CoffeelakeSiliconPkg/CoffeelakeSiliconPkg.dsc | 1 +
>> Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 3 ++-
>> Silicon/Intel/KabylakeSiliconPkg/KabylakeSiliconPkg.dsc | 3 ++-
>> Silicon/Intel/QuarkSocPkg/QuarkSocPkg.dsc | 3 ++-
>> Silicon/Intel/TigerlakeSiliconPkg/TigerlakeSiliconPkg.dsc | 1 +
>> Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 1 +
>> Silicon/NXP/NxpQoriqLs.dsc.inc | 1 +
>> Silicon/Openmoko/Openmoko.dsc | 1 +
>> Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc | 1 +
>> Silicon/Synopsys/DesignWare/DesignWare.dsc | 1 +
>> Silicon/TexasInstruments/Omap35xxPkg/Omap35xxPkg.dsc | 1 +
>> 47 files changed, 70 insertions(+), 23 deletions(-)
>>
>> --
>> 2.18.0.windows.1
>>
>>
>>
>>
>>
>>
>
next prev parent reply other threads:[~2021-03-16 16:23 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-16 14:53 [edk2-platforms] [patch 00/35] Consume RegisterFilterLibNull instance Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 01/35] Drivers/ASIX: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 02/35] Drivers/DisplayLink: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 03/35] Drivers/OptionRomPkg: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 04/35] Features/Debugging: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 05/35] Features/Network: " Dandan Bi
2021-03-16 14:53 ` [edk2-platforms] [patch 06/35] Features/OutOfBandManagement: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 07/35] Features/PowerManagement: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 08/35] Features/SystemInformation: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 09/35] Features/UserInterface: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 10/35] Platform/AMD: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 11/35] Platform/ARM: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 12/35] Platform/BeagleBoard: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 13/35] Platform/BoardModulePkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 14/35] Platform/MinPlatformPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 15/35] Platform/QuarkPlatformPkg: " Dandan Bi
2021-03-16 15:04 ` Steele, Kelly
2021-03-16 14:54 ` [edk2-platforms] [patch 16/35] Platform/Vlv2TbltDevicePkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 17/35] Platform/LeMaker: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 18/35] Platform/Qemu: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 19/35] Platform/RaspberryPi: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 20/35] Platform/RISC-V: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 21/35] Platform/SiFive: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 22/35] Platform/Socionext: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 23/35] Platform/SoftIron: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 24/35] Silicon/Hisilicon: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 25/35] Silicon/CoffeelakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 26/35] Silicon/IntelSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 27/35] Silicon/KabylakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 28/35] Silicon/QuarkSocPkg: " Dandan Bi
2021-03-16 15:05 ` Steele, Kelly
2021-03-16 14:54 ` [edk2-platforms] [patch 29/35] Silicon/TigerlakeSiliconPkg: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 30/35] Silicon/Marvell: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 31/35] Silicon/NXP: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 32/35] Silicon/Openmoko: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 33/35] Silicon/RISC_V: " Dandan Bi
2021-03-17 8:14 ` Abner Chang
2021-03-16 14:54 ` [edk2-platforms] [patch 34/35] Silicon/Synopsys/DesignWare: " Dandan Bi
2021-03-16 14:54 ` [edk2-platforms] [patch 35/35] Silicon/TexasInstruments: " Dandan Bi
2021-03-16 15:00 ` [edk2-devel] [edk2-platforms] [patch 00/35] " Ard Biesheuvel
2021-03-16 16:23 ` Laszlo Ersek [this message]
2021-03-17 3:04 ` 回复: " gaoliming
2021-03-17 15:05 ` Dandan Bi
2021-03-17 17:08 ` Laszlo Ersek
2021-03-18 4:43 ` 回复: " gaoliming
2021-03-18 13:25 ` Laszlo Ersek
2021-03-18 13:42 ` Ni, Ray
2021-03-19 1:20 ` 回复: " gaoliming
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=37431ead-7d7e-74a2-2f94-b8cc82307902@redhat.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