From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by mx.groups.io with SMTP id smtpd.web10.119.1615906859314192523 for ; Tue, 16 Mar 2021 08:00:59 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=NNi+VQei; spf=pass (domain: kernel.org, ip: 198.145.29.99, mailfrom: ardb@kernel.org) Received: by mail.kernel.org (Postfix) with ESMTPSA id 58A8665090 for ; Tue, 16 Mar 2021 15:00:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1615906858; bh=BsEkWuLBfxu8kppD73WvYdzGpB+Nq+wN/rbZ7HZ1kc0=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=NNi+VQei+UC9Oiel/nWUhUoK9T3fiR+kbGTN7gJUX8fISqXFeT1rXVUxv/7Ka5GZv RfhlDdBJKa88ooITpJMNy1l1nhiEOis8DI9fl6B9deLmCsVFlO7yYjm/xP/zJkwpso /DhIDBSTJ8IZxg/cZUwj1/XHCa34kSFrYdYQKv8vZxljPjarJ9Q6hzxsMo1GIDZ68y +0n4EDnfNoxM8aP41pwtQgN7uy+OnNSLnDTc1E9OGQJ6I9U+YD+bAG5cZf+IQLH3Lg KVoqLaSUt55FRFvAVo45MSUrCiSkjSXzcLTNfuhC+Onxefu82sG5UrIlqaeVHypoUM 8f8FMz6trUm5g== Received: by mail-oi1-f170.google.com with SMTP id d20so38384394oiw.10 for ; Tue, 16 Mar 2021 08:00:58 -0700 (PDT) X-Gm-Message-State: AOAM533Ry2i/YJK4X3esm75jDpadSAbEoshHK7RxM8YK8Jx/awXcnRle T84brw0aOJg1GUl19P3Qh5oLgq2N/i+wwUW/FPM= X-Google-Smtp-Source: ABdhPJxgEn8I+OLwp5QRr2dqOaOiBAW5n2aznYCmE2iQosoD2ecgBVU6JmTy3wlafcz/U7ucqfloVuqB4YWNwBMFoEQ= X-Received: by 2002:aca:538c:: with SMTP id h134mr4086oib.174.1615906857684; Tue, 16 Mar 2021 08:00:57 -0700 (PDT) MIME-Version: 1.0 References: <20210316145428.35616-1-dandan.bi@intel.com> In-Reply-To: <20210316145428.35616-1-dandan.bi@intel.com> From: "Ard Biesheuvel" Date: Tue, 16 Mar 2021 16:00:46 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [edk2-devel] [edk2-platforms] [patch 00/35] Consume RegisterFilterLibNull instance To: devel@edk2.groups.io, dandan.bi@intel.com, Laszlo Ersek , Andrew Fish Cc: Leif Lindholm , Michael D Kinney , Liming Gao Content-Type: text/plain; charset="UTF-8" On Tue, 16 Mar 2021 at 15:56, Dandan Bi 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 > Cc: Michael D Kinney > Cc: Liming Gao > 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? > 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 > > > > > >