Agreed. Will rename in V3.
Thanks!
Chasel
> -----Original Message-----
> From: Kubacki, Michael A <michael.a.kubacki@intel.com>
> Sent: Thursday, October 31, 2019 1:50 PM
> To: Chiu, Chasel <chasel.chiu@intel.com>; devel@edk2.groups.io
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Jeremy Soller <jeremy@system76.com>; Lu, Shifei A
> <shifei.a.lu@intel.com>; Zhou, Bowen <bowen.zhou@intel.com>; Oram, Isaac
> W <isaac.w.oram@intel.com>; Wei, David Y <david.y.wei@intel.com>;
> Agyeman, Prince <prince.agyeman@intel.com>
> Subject: RE: [edk2-platforms: PATCH v2 0/6] Add SetCacheLib library class.
>
> Hi Chasel,
>
> I don't think SetCacheLib is quite precise enough. Can you please consider
> an alternative such as SetMtrrLib?
>
> Thanks,
> Michael
>
> > -----Original Message-----
> > From: Chiu, Chasel <chasel.chiu@intel.com>
> > Sent: Wednesday, October 30, 2019 5:30 PM
> > To: devel@edk2.groups.io
> > Cc: Kubacki, Michael A <michael.a.kubacki@intel.com>; Desimone,
> > Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming
> > <liming.gao@intel.com>; Jeremy Soller <jeremy@system76.com>; Lu,
> > Shifei A <shifei.a.lu@intel.com>; Zhou, Bowen <bowen.zhou@intel.com>;
> > Oram, Isaac W <isaac.w.oram@intel.com>; Wei, David Y
> > <david.y.wei@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>
> > Subject: [edk2-platforms: PATCH v2 0/6] Add SetCacheLib library class.
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2314
> >
> > The MTRRs may be redundantly or incorrectly configured.
> > It is recommended to move this functionality to a library class that
> > with a NULL library by default. The board package may provide its own
> > library implementation.
> > MinPlatformPkg should contain the library class header (API) and the
> > NULL library class instance.
> >
> > Cc: Michael Kubacki <michael.a.kubacki@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Cc: Jeremy Soller <jeremy@system76.com>
> > Cc: Shifei A Lu <shifei.a.lu@intel.com>
> > Cc: Xiaohu Zhou <bowen.zhou@intel.com>
> > Cc: Isaac W Oram <isaac.w.oram@intel.com>
> > Cc: Wei David Y <david.y.wei@intel.com>
> > Cc: Agyeman Prince <prince.agyeman@intel.com>
> > Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> >
> > Chasel Chiu (6):
> > MinPlatformPkg: Add SetCacheLib library class.
> > MinPlatformPkg: Add SetCacheLib library class.
> > KabylakeOpenBoardPkg: Add SetCacheLib library class.
> > WhiskeylakeOpenBoardPkg: Add SetCacheLib library class.
> > PurleyOpenBoardPkg/BoardMtOlympus: Add SetCacheLib library class.
> > SimicsOpenBoardPkg/BoardX58Ich10: Add SetCacheLib library class.
> >
> >
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/Override/Platform/Intel
> > /MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.c |
> 640
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------------------------------------------------
> > ------
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLib.c
> > | 325
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > +++++++++++++++++++++++++++++++++++
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLibNull.c
> > | 37 +++++++++++++++++++++++++++++++++++++
> >
> >
> Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPos
> > tMem.c
> | 149 +-----------------------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > -------------
> >
> >
> Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPre
> > Mem.c
> | 164 ++----------------------------------------
> > ----------------------------------------------------------------------
> > ------------------------
> > ----------------------------
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.dsc
> > | 3 ++-
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/OpenBoardPkg.fdf
> > | 2 +-
> >
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/Override/Platform/Intel
> > /MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.inf |
> > 67
> > -------------------------------------------------------------------
> > Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
> > | 1 +
> > Platform/Intel/MinPlatformPkg/Include/Library/SetCacheLib.h
> > | 34 ++++++++++++++++++++++++++++++++++
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLib.inf
> > | 44 ++++++++++++++++++++++++++++++++++++++++++++
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLibNull.inf
> > | 30 ++++++++++++++++++++++++++++++
> >
> >
> Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPos
> > tMem.inf
> | 11 +----------
> >
> >
> Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPre
> > Mem.inf
> | 7 ++-----
> > Platform/Intel/PurleyOpenBoardPkg/BoardMtOlympus/PlatformPkg.dsc
> > | 3 ++-
> > Platform/Intel/SimicsOpenBoardPkg/BoardX58Ich10/OpenBoardPkg.dsc
> > | 1 +
> >
> > Platform/Intel/WhiskeylakeOpenBoardPkg/WhiskeylakeURvp/OpenBoardPk
> > g.dsc
> | 1 +
> > 17 files changed, 484 insertions(+), 1035 deletions(-) delete mode
> > 100644
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/Override/Platform/Intel
> > /MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.c
> > create mode 100644
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLib.c
> > create mode 100644
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLibNull.c
> > delete mode 100644
> > Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/Override/Platform/Intel
> > /MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.inf
> > create mode 100644
> > Platform/Intel/MinPlatformPkg/Include/Library/SetCacheLib.h
> > create mode 100644
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLib.inf
> > create mode 100644
> > Platform/Intel/MinPlatformPkg/Library/SetCacheLib/SetCacheLibNull.inf
> >
> > --
> > 2.13.3.windows.1
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#49750): https://edk2.groups.io/g/devel/message/49750
Mute This Topic: https://groups.io/mt/40049897/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-