public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
@ 2019-10-22 19:06 Michael D Kinney
  2019-10-22 22:57 ` Laszlo Ersek
  2019-10-24  0:06 ` [edk2-devel] " Dong, Eric
  0 siblings, 2 replies; 12+ messages in thread
From: Michael D Kinney @ 2019-10-22 19:06 UTC (permalink / raw)
  To: devel; +Cc: Sean Brogan, Eric Dong, Ray Ni, Laszlo Ersek

From: Sean Brogan <sean.brogan@microsoft.com>

https://bugzilla.tianocore.org/show_bug.cgi?id=2255

Update UefiCpuPkg.dsc to guarantee all libraries and
modules are always built.  Add the following components.

* UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
* UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf

ResetVector.inf is a binary INF, so no source builds are
triggered from adding this line.  However, a build with
this component does verify the contents of the INF file.

Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
---
 UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/UefiCpuPkg/UefiCpuPkg.dsc b/UefiCpuPkg/UefiCpuPkg.dsc
index e7dfe30eda..d28cb5cccb 100644
--- a/UefiCpuPkg/UefiCpuPkg.dsc
+++ b/UefiCpuPkg/UefiCpuPkg.dsc
@@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
   UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
   UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
   UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf
+  UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptionHandlerLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExceptionHandlerLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf
@@ -146,6 +147,7 @@ [Components.IA32, Components.X64]
       SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmCpuFeaturesLibStm.inf
   }
   UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
+  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
 
 [BuildOptions]
   *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
-- 
2.21.0.windows.1


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 19:06 [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc Michael D Kinney
@ 2019-10-22 22:57 ` Laszlo Ersek
  2019-10-22 23:13   ` Michael D Kinney
  2019-10-24  0:06 ` [edk2-devel] " Dong, Eric
  1 sibling, 1 reply; 12+ messages in thread
From: Laszlo Ersek @ 2019-10-22 22:57 UTC (permalink / raw)
  To: Michael D Kinney, devel; +Cc: Sean Brogan, Eric Dong, Ray Ni

On 10/22/19 21:06, Michael D Kinney wrote:
> From: Sean Brogan <sean.brogan@microsoft.com>
> 
> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> 
> Update UefiCpuPkg.dsc to guarantee all libraries and
> modules are always built.  Add the following components.
> 
> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> * UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
> 
> ResetVector.inf is a binary INF, so no source builds are
> triggered from adding this line.  However, a build with
> this component does verify the contents of the INF file.
> 
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> ---
>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc b/UefiCpuPkg/UefiCpuPkg.dsc
> index e7dfe30eda..d28cb5cccb 100644
> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> @@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
>    UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
>    UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf
> +  UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
>    UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptionHandlerLib.inf
>    UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExceptionHandlerLib.inf
>    UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf
> @@ -146,6 +147,7 @@ [Components.IA32, Components.X64]
>        SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmCpuFeaturesLibStm.inf
>    }
>    UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>  
>  [BuildOptions]
>    *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
> 

Can we retire

  UefiCpuPkg/ResetVector/Vtf0/Bin/

instead (in a separate patch)?

Because, according to

  git log --stat master -- UefiCpuPkg/ResetVector/Vtf0/Bin/

the most recent commit in which we updated the *.raw files has been
f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for finding SEC image of
type TE.", 2011-09-28).

I doubt anyone consumes this (binary) module, since Jordan's commits
- 5a1f324d946c ("UefiCpuPkg: Support building VTF0 ResetVector during
the EDK II build", 2014-08-18)
- eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page tables in NASM code",
2014-08-18).

We've by now dropped all top-level *BinPkg package directories too.

If we remove the Bin subdir, then we should probably remove:
- Build.py
- Tools/
too.

For the other addition (CpuCommonFeaturesLib): if that gets its own
patch (after the removal of UefiCpuPkg/ResetVector/Vtf0/Bin/), then
please add

Reviewed-by: Laszlo Ersek <lersek@redhat.com>

to it.

Thanks
Laszlo


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 22:57 ` Laszlo Ersek
@ 2019-10-22 23:13   ` Michael D Kinney
  2019-10-22 23:27     ` Laszlo Ersek
  0 siblings, 1 reply; 12+ messages in thread
From: Michael D Kinney @ 2019-10-22 23:13 UTC (permalink / raw)
  To: Laszlo Ersek, devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Dong, Eric, Ni, Ray

Laszlo,

I see 2 usages of this module in edk2-platforms:

edk2-platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13):  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf

edk2-platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fdf(16):INF  RuleOverride=RESET_VECTOR USE = IA32 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf

Mike

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Tuesday, October 22, 2019 3:57 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> Subject: Re: [Patch] UefiCpuPkg: Add missing components
> to UefiCpuPkg.dsc
> 
> On 10/22/19 21:06, Michael D Kinney wrote:
> > From: Sean Brogan <sean.brogan@microsoft.com>
> >
> > https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> >
> > Update UefiCpuPkg.dsc to guarantee all libraries and
> modules are
> > always built.  Add the following components.
> >
> > * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > *
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> sLib.inf
> >
> > ResetVector.inf is a binary INF, so no source builds
> are triggered
> > from adding this line.  However, a build with this
> component does
> > verify the contents of the INF file.
> >
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Signed-off-by: Michael D Kinney
> <michael.d.kinney@intel.com>
> > ---
> >  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> b/UefiCpuPkg/UefiCpuPkg.dsc
> > index e7dfe30eda..d28cb5cccb 100644
> > --- a/UefiCpuPkg/UefiCpuPkg.dsc
> > +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> > @@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
> >
> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> >    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> >
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> .inf
> > +
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> sLib.inf
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> nHandlerLib.inf
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> tionHandlerLib.inf
> >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> nHandlerLib.in
> > f @@ -146,6 +147,7 @@ [Components.IA32,
> Components.X64]
> >
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> mmCpuFeaturesLibStm.inf
> >    }
> >
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> > +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >
> >  [BuildOptions]
> >    *_*_*_CC_FLAGS = -D
> DISABLE_NEW_DEPRECATED_INTERFACES
> >
> 
> Can we retire
> 
>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> 
> instead (in a separate patch)?
> 
> Because, according to
> 
>   git log --stat master --
> UefiCpuPkg/ResetVector/Vtf0/Bin/
> 
> the most recent commit in which we updated the *.raw
> files has been
> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for finding
> SEC image of type TE.", 2011-09-28).
> 
> I doubt anyone consumes this (binary) module, since
> Jordan's commits
> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
> ResetVector during the EDK II build", 2014-08-18)
> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page tables
> in NASM code", 2014-08-18).
> 
> We've by now dropped all top-level *BinPkg package
> directories too.
> 
> If we remove the Bin subdir, then we should probably
> remove:
> - Build.py
> - Tools/
> too.
> 
> For the other addition (CpuCommonFeaturesLib): if that
> gets its own patch (after the removal of
> UefiCpuPkg/ResetVector/Vtf0/Bin/), then please add
> 
> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> 
> to it.
> 
> Thanks
> Laszlo


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 23:13   ` Michael D Kinney
@ 2019-10-22 23:27     ` Laszlo Ersek
  2019-10-22 23:38       ` Michael D Kinney
  0 siblings, 1 reply; 12+ messages in thread
From: Laszlo Ersek @ 2019-10-22 23:27 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io
  Cc: Sean Brogan, Dong, Eric, Ni, Ray, Wei David Y, Agyeman Prince

(+David, +Prince)

On 10/23/19 01:13, Kinney, Michael D wrote:
> Laszlo,
> 
> I see 2 usages of this module in edk2-platforms:
> 
> edk2-platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13):  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> 
> edk2-platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fdf(16):INF  RuleOverride=RESET_VECTOR USE = IA32 UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf

Can we update those platforms to build the reset vector from source?

Embedding a binary blob seems wrong in its own right...

Anyway, if Ray and/or Eric are OK with the present patch, I won't try to
block it.

Thanks,
Laszlo

>> -----Original Message-----
>> From: Laszlo Ersek <lersek@redhat.com>
>> Sent: Tuesday, October 22, 2019 3:57 PM
>> To: Kinney, Michael D <michael.d.kinney@intel.com>;
>> devel@edk2.groups.io
>> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
>> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
>> Subject: Re: [Patch] UefiCpuPkg: Add missing components
>> to UefiCpuPkg.dsc
>>
>> On 10/22/19 21:06, Michael D Kinney wrote:
>>> From: Sean Brogan <sean.brogan@microsoft.com>
>>>
>>> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
>>>
>>> Update UefiCpuPkg.dsc to guarantee all libraries and
>> modules are
>>> always built.  Add the following components.
>>>
>>> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>> *
>> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
>> sLib.inf
>>>
>>> ResetVector.inf is a binary INF, so no source builds
>> are triggered
>>> from adding this line.  However, a build with this
>> component does
>>> verify the contents of the INF file.
>>>
>>> Cc: Eric Dong <eric.dong@intel.com>
>>> Cc: Ray Ni <ray.ni@intel.com>
>>> Cc: Laszlo Ersek <lersek@redhat.com>
>>> Signed-off-by: Michael D Kinney
>> <michael.d.kinney@intel.com>
>>> ---
>>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
>> b/UefiCpuPkg/UefiCpuPkg.dsc
>>> index e7dfe30eda..d28cb5cccb 100644
>>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
>>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
>>> @@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
>>>
>> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
>>>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
>>>
>> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
>> .inf
>>> +
>> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
>> sLib.inf
>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
>> nHandlerLib.inf
>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
>> tionHandlerLib.inf
>>>
>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
>> nHandlerLib.in
>>> f @@ -146,6 +147,7 @@ [Components.IA32,
>> Components.X64]
>>>
>> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
>> mmCpuFeaturesLibStm.inf
>>>    }
>>>
>> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
>>> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>>
>>>  [BuildOptions]
>>>    *_*_*_CC_FLAGS = -D
>> DISABLE_NEW_DEPRECATED_INTERFACES
>>>
>>
>> Can we retire
>>
>>   UefiCpuPkg/ResetVector/Vtf0/Bin/
>>
>> instead (in a separate patch)?
>>
>> Because, according to
>>
>>   git log --stat master --
>> UefiCpuPkg/ResetVector/Vtf0/Bin/
>>
>> the most recent commit in which we updated the *.raw
>> files has been
>> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for finding
>> SEC image of type TE.", 2011-09-28).
>>
>> I doubt anyone consumes this (binary) module, since
>> Jordan's commits
>> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
>> ResetVector during the EDK II build", 2014-08-18)
>> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page tables
>> in NASM code", 2014-08-18).
>>
>> We've by now dropped all top-level *BinPkg package
>> directories too.
>>
>> If we remove the Bin subdir, then we should probably
>> remove:
>> - Build.py
>> - Tools/
>> too.
>>
>> For the other addition (CpuCommonFeaturesLib): if that
>> gets its own patch (after the removal of
>> UefiCpuPkg/ResetVector/Vtf0/Bin/), then please add
>>
>> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
>>
>> to it.
>>
>> Thanks
>> Laszlo
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 23:27     ` Laszlo Ersek
@ 2019-10-22 23:38       ` Michael D Kinney
  2019-10-23  1:28         ` Dong, Eric
  2019-10-24 11:03         ` Laszlo Ersek
  0 siblings, 2 replies; 12+ messages in thread
From: Michael D Kinney @ 2019-10-22 23:38 UTC (permalink / raw)
  To: Laszlo Ersek, devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Dong, Eric, Ni, Ray, Wei, David Y, Agyeman, Prince

Laszlo,

Can we move forward with this patch and enter a new
BZ to remove the binary module?

Mike

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Tuesday, October 22, 2019 4:27 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Wei,
> David Y <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> Subject: Re: [Patch] UefiCpuPkg: Add missing components
> to UefiCpuPkg.dsc
> 
> (+David, +Prince)
> 
> On 10/23/19 01:13, Kinney, Michael D wrote:
> > Laszlo,
> >
> > I see 2 usages of this module in edk2-platforms:
> >
> > edk2-
> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
> ):
> > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >
> > edk2-
> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
> f(16):INF
> > RuleOverride=RESET_VECTOR USE = IA32
> > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> 
> Can we update those platforms to build the reset vector
> from source?
> 
> Embedding a binary blob seems wrong in its own right...
> 
> Anyway, if Ray and/or Eric are OK with the present
> patch, I won't try to block it.
> 
> Thanks,
> Laszlo
> 
> >> -----Original Message-----
> >> From: Laszlo Ersek <lersek@redhat.com>
> >> Sent: Tuesday, October 22, 2019 3:57 PM
> >> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> >> devel@edk2.groups.io
> >> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> Eric
> >> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> >> Subject: Re: [Patch] UefiCpuPkg: Add missing
> components to
> >> UefiCpuPkg.dsc
> >>
> >> On 10/22/19 21:06, Michael D Kinney wrote:
> >>> From: Sean Brogan <sean.brogan@microsoft.com>
> >>>
> >>> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> >>>
> >>> Update UefiCpuPkg.dsc to guarantee all libraries and
> >> modules are
> >>> always built.  Add the following components.
> >>>
> >>> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >>> *
> >>
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> >> sLib.inf
> >>>
> >>> ResetVector.inf is a binary INF, so no source builds
> >> are triggered
> >>> from adding this line.  However, a build with this
> >> component does
> >>> verify the contents of the INF file.
> >>>
> >>> Cc: Eric Dong <eric.dong@intel.com>
> >>> Cc: Ray Ni <ray.ni@intel.com>
> >>> Cc: Laszlo Ersek <lersek@redhat.com>
> >>> Signed-off-by: Michael D Kinney
> >> <michael.d.kinney@intel.com>
> >>> ---
> >>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> >>>  1 file changed, 2 insertions(+)
> >>>
> >>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> >> b/UefiCpuPkg/UefiCpuPkg.dsc
> >>> index e7dfe30eda..d28cb5cccb 100644
> >>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> >>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> >>> @@ -121,6 +121,7 @@ [Components.IA32,
> Components.X64]
> >>>
> >> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> >>>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> >>>
> >>
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> >> .inf
> >>> +
> >>
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> >> sLib.inf
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> >> nHandlerLib.inf
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> >> tionHandlerLib.inf
> >>>
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> >> nHandlerLib.in
> >>> f @@ -146,6 +147,7 @@ [Components.IA32,
> >> Components.X64]
> >>>
> >>
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> >> mmCpuFeaturesLibStm.inf
> >>>    }
> >>>
> >>
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> >>> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >>>
> >>>  [BuildOptions]
> >>>    *_*_*_CC_FLAGS = -D
> >> DISABLE_NEW_DEPRECATED_INTERFACES
> >>>
> >>
> >> Can we retire
> >>
> >>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> >>
> >> instead (in a separate patch)?
> >>
> >> Because, according to
> >>
> >>   git log --stat master --
> >> UefiCpuPkg/ResetVector/Vtf0/Bin/
> >>
> >> the most recent commit in which we updated the *.raw
> files has been
> >> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
> finding SEC image of
> >> type TE.", 2011-09-28).
> >>
> >> I doubt anyone consumes this (binary) module, since
> Jordan's commits
> >> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
> ResetVector during
> >> the EDK II build", 2014-08-18)
> >> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
> tables in NASM
> >> code", 2014-08-18).
> >>
> >> We've by now dropped all top-level *BinPkg package
> directories too.
> >>
> >> If we remove the Bin subdir, then we should probably
> >> remove:
> >> - Build.py
> >> - Tools/
> >> too.
> >>
> >> For the other addition (CpuCommonFeaturesLib): if
> that gets its own
> >> patch (after the removal of
> UefiCpuPkg/ResetVector/Vtf0/Bin/), then
> >> please add
> >>
> >> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> >>
> >> to it.
> >>
> >> Thanks
> >> Laszlo
> >


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 23:38       ` Michael D Kinney
@ 2019-10-23  1:28         ` Dong, Eric
  2019-10-23 15:15           ` Michael D Kinney
  2019-10-24 11:03         ` Laszlo Ersek
  1 sibling, 1 reply; 12+ messages in thread
From: Dong, Eric @ 2019-10-23  1:28 UTC (permalink / raw)
  To: Kinney, Michael D, Laszlo Ersek, devel@edk2.groups.io
  Cc: Sean Brogan, Ni, Ray, Wei, David Y, Agyeman, Prince

Hi Mike and Laszlo,

I agree with Mike's proposal. We can submit new bugz to remove the binary.

Thanks,
Eric

-----Original Message-----
From: Kinney, Michael D 
Sent: Wednesday, October 23, 2019 7:39 AM
To: Laszlo Ersek <lersek@redhat.com>; devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Wei, David Y <david.y.wei@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>
Subject: RE: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc

Laszlo,

Can we move forward with this patch and enter a new BZ to remove the binary module?

Mike

> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com>
> Sent: Tuesday, October 22, 2019 4:27 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; 
> devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric 
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Wei, David Y 
> <david.y.wei@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>
> Subject: Re: [Patch] UefiCpuPkg: Add missing components to 
> UefiCpuPkg.dsc
> 
> (+David, +Prince)
> 
> On 10/23/19 01:13, Kinney, Michael D wrote:
> > Laszlo,
> >
> > I see 2 usages of this module in edk2-platforms:
> >
> > edk2-
> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
> ):
> > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >
> > edk2-
> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
> f(16):INF
> > RuleOverride=RESET_VECTOR USE = IA32 
> > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> 
> Can we update those platforms to build the reset vector from source?
> 
> Embedding a binary blob seems wrong in its own right...
> 
> Anyway, if Ray and/or Eric are OK with the present patch, I won't try 
> to block it.
> 
> Thanks,
> Laszlo
> 
> >> -----Original Message-----
> >> From: Laszlo Ersek <lersek@redhat.com>
> >> Sent: Tuesday, October 22, 2019 3:57 PM
> >> To: Kinney, Michael D <michael.d.kinney@intel.com>; 
> >> devel@edk2.groups.io
> >> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> Eric
> >> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> >> Subject: Re: [Patch] UefiCpuPkg: Add missing
> components to
> >> UefiCpuPkg.dsc
> >>
> >> On 10/22/19 21:06, Michael D Kinney wrote:
> >>> From: Sean Brogan <sean.brogan@microsoft.com>
> >>>
> >>> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> >>>
> >>> Update UefiCpuPkg.dsc to guarantee all libraries and
> >> modules are
> >>> always built.  Add the following components.
> >>>
> >>> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >>> *
> >>
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> >> sLib.inf
> >>>
> >>> ResetVector.inf is a binary INF, so no source builds
> >> are triggered
> >>> from adding this line.  However, a build with this
> >> component does
> >>> verify the contents of the INF file.
> >>>
> >>> Cc: Eric Dong <eric.dong@intel.com>
> >>> Cc: Ray Ni <ray.ni@intel.com>
> >>> Cc: Laszlo Ersek <lersek@redhat.com>
> >>> Signed-off-by: Michael D Kinney
> >> <michael.d.kinney@intel.com>
> >>> ---
> >>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> >>>  1 file changed, 2 insertions(+)
> >>>
> >>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> >> b/UefiCpuPkg/UefiCpuPkg.dsc
> >>> index e7dfe30eda..d28cb5cccb 100644
> >>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> >>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> >>> @@ -121,6 +121,7 @@ [Components.IA32,
> Components.X64]
> >>>
> >> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> >>>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> >>>
> >>
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> >> .inf
> >>> +
> >>
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> >> sLib.inf
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> >> nHandlerLib.inf
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> >> tionHandlerLib.inf
> >>>
> >>>
> >>
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> >> nHandlerLib.in
> >>> f @@ -146,6 +147,7 @@ [Components.IA32,
> >> Components.X64]
> >>>
> >>
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> >> mmCpuFeaturesLibStm.inf
> >>>    }
> >>>
> >>
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> >>> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >>>
> >>>  [BuildOptions]
> >>>    *_*_*_CC_FLAGS = -D
> >> DISABLE_NEW_DEPRECATED_INTERFACES
> >>>
> >>
> >> Can we retire
> >>
> >>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> >>
> >> instead (in a separate patch)?
> >>
> >> Because, according to
> >>
> >>   git log --stat master --
> >> UefiCpuPkg/ResetVector/Vtf0/Bin/
> >>
> >> the most recent commit in which we updated the *.raw
> files has been
> >> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
> finding SEC image of
> >> type TE.", 2011-09-28).
> >>
> >> I doubt anyone consumes this (binary) module, since
> Jordan's commits
> >> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
> ResetVector during
> >> the EDK II build", 2014-08-18)
> >> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
> tables in NASM
> >> code", 2014-08-18).
> >>
> >> We've by now dropped all top-level *BinPkg package
> directories too.
> >>
> >> If we remove the Bin subdir, then we should probably
> >> remove:
> >> - Build.py
> >> - Tools/
> >> too.
> >>
> >> For the other addition (CpuCommonFeaturesLib): if
> that gets its own
> >> patch (after the removal of
> UefiCpuPkg/ResetVector/Vtf0/Bin/), then
> >> please add
> >>
> >> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> >>
> >> to it.
> >>
> >> Thanks
> >> Laszlo
> >


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-23  1:28         ` Dong, Eric
@ 2019-10-23 15:15           ` Michael D Kinney
  2019-10-23 16:06             ` Michael D Kinney
  0 siblings, 1 reply; 12+ messages in thread
From: Michael D Kinney @ 2019-10-23 15:15 UTC (permalink / raw)
  To: Dong, Eric, Laszlo Ersek, devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Ni, Ray, Wei, David Y, Agyeman, Prince

Hi Eric,

I will enter a new BZ to remove binary.

Can you please review the current patch?

Thanks,

Mike

> -----Original Message-----
> From: Dong, Eric <eric.dong@intel.com>
> Sent: Tuesday, October 22, 2019 6:29 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> Laszlo Ersek <lersek@redhat.com>; devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray
> <ray.ni@intel.com>; Wei, David Y
> <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> Subject: RE: [Patch] UefiCpuPkg: Add missing components
> to UefiCpuPkg.dsc
> 
> Hi Mike and Laszlo,
> 
> I agree with Mike's proposal. We can submit new bugz to
> remove the binary.
> 
> Thanks,
> Eric
> 
> -----Original Message-----
> From: Kinney, Michael D
> Sent: Wednesday, October 23, 2019 7:39 AM
> To: Laszlo Ersek <lersek@redhat.com>;
> devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Wei,
> David Y <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> Subject: RE: [Patch] UefiCpuPkg: Add missing components
> to UefiCpuPkg.dsc
> 
> Laszlo,
> 
> Can we move forward with this patch and enter a new BZ
> to remove the binary module?
> 
> Mike
> 
> > -----Original Message-----
> > From: Laszlo Ersek <lersek@redhat.com>
> > Sent: Tuesday, October 22, 2019 4:27 PM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> > devel@edk2.groups.io
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> Eric
> > <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>;
> Wei, David Y
> > <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> > Subject: Re: [Patch] UefiCpuPkg: Add missing
> components to
> > UefiCpuPkg.dsc
> >
> > (+David, +Prince)
> >
> > On 10/23/19 01:13, Kinney, Michael D wrote:
> > > Laszlo,
> > >
> > > I see 2 usages of this module in edk2-platforms:
> > >
> > > edk2-
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
> > ):
> > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > >
> > > edk2-
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
> > f(16):INF
> > > RuleOverride=RESET_VECTOR USE = IA32
> > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> >
> > Can we update those platforms to build the reset
> vector from source?
> >
> > Embedding a binary blob seems wrong in its own
> right...
> >
> > Anyway, if Ray and/or Eric are OK with the present
> patch, I won't try
> > to block it.
> >
> > Thanks,
> > Laszlo
> >
> > >> -----Original Message-----
> > >> From: Laszlo Ersek <lersek@redhat.com>
> > >> Sent: Tuesday, October 22, 2019 3:57 PM
> > >> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> > >> devel@edk2.groups.io
> > >> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> > Eric
> > >> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> > >> Subject: Re: [Patch] UefiCpuPkg: Add missing
> > components to
> > >> UefiCpuPkg.dsc
> > >>
> > >> On 10/22/19 21:06, Michael D Kinney wrote:
> > >>> From: Sean Brogan <sean.brogan@microsoft.com>
> > >>>
> > >>>
> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> > >>>
> > >>> Update UefiCpuPkg.dsc to guarantee all libraries
> and
> > >> modules are
> > >>> always built.  Add the following components.
> > >>>
> > >>> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > >>> *
> > >>
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > >> sLib.inf
> > >>>
> > >>> ResetVector.inf is a binary INF, so no source
> builds
> > >> are triggered
> > >>> from adding this line.  However, a build with this
> > >> component does
> > >>> verify the contents of the INF file.
> > >>>
> > >>> Cc: Eric Dong <eric.dong@intel.com>
> > >>> Cc: Ray Ni <ray.ni@intel.com>
> > >>> Cc: Laszlo Ersek <lersek@redhat.com>
> > >>> Signed-off-by: Michael D Kinney
> > >> <michael.d.kinney@intel.com>
> > >>> ---
> > >>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> > >>>  1 file changed, 2 insertions(+)
> > >>>
> > >>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> > >> b/UefiCpuPkg/UefiCpuPkg.dsc
> > >>> index e7dfe30eda..d28cb5cccb 100644
> > >>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> > >>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> > >>> @@ -121,6 +121,7 @@ [Components.IA32,
> > Components.X64]
> > >>>
> > >>
> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> > >>>
> UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> > >>>
> > >>
> >
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> > >> .inf
> > >>> +
> > >>
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > >> sLib.inf
> > >>>
> > >>
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> > >> nHandlerLib.inf
> > >>>
> > >>
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> > >> tionHandlerLib.inf
> > >>>
> > >>>
> > >>
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> > >> nHandlerLib.in
> > >>> f @@ -146,6 +147,7 @@ [Components.IA32,
> > >> Components.X64]
> > >>>
> > >>
> >
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> > >> mmCpuFeaturesLibStm.inf
> > >>>    }
> > >>>
> > >>
> >
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> > >>> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > >>>
> > >>>  [BuildOptions]
> > >>>    *_*_*_CC_FLAGS = -D
> > >> DISABLE_NEW_DEPRECATED_INTERFACES
> > >>>
> > >>
> > >> Can we retire
> > >>
> > >>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> > >>
> > >> instead (in a separate patch)?
> > >>
> > >> Because, according to
> > >>
> > >>   git log --stat master --
> > >> UefiCpuPkg/ResetVector/Vtf0/Bin/
> > >>
> > >> the most recent commit in which we updated the
> *.raw
> > files has been
> > >> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
> > finding SEC image of
> > >> type TE.", 2011-09-28).
> > >>
> > >> I doubt anyone consumes this (binary) module, since
> > Jordan's commits
> > >> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
> > ResetVector during
> > >> the EDK II build", 2014-08-18)
> > >> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
> > tables in NASM
> > >> code", 2014-08-18).
> > >>
> > >> We've by now dropped all top-level *BinPkg package
> > directories too.
> > >>
> > >> If we remove the Bin subdir, then we should
> probably
> > >> remove:
> > >> - Build.py
> > >> - Tools/
> > >> too.
> > >>
> > >> For the other addition (CpuCommonFeaturesLib): if
> > that gets its own
> > >> patch (after the removal of
> > UefiCpuPkg/ResetVector/Vtf0/Bin/), then
> > >> please add
> > >>
> > >> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> > >>
> > >> to it.
> > >>
> > >> Thanks
> > >> Laszlo
> > >


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-23 15:15           ` Michael D Kinney
@ 2019-10-23 16:06             ` Michael D Kinney
  2019-10-23 16:08               ` Agyeman, Prince
  0 siblings, 1 reply; 12+ messages in thread
From: Michael D Kinney @ 2019-10-23 16:06 UTC (permalink / raw)
  To: Dong, Eric, Laszlo Ersek, devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Ni, Ray, Wei, David Y, Agyeman, Prince

I have enter 2 bugs.  One for SimicsX58SktPkg and one for UefiCpuPkg.

https://bugzilla.tianocore.org/show_bug.cgi?id=2307

https://bugzilla.tianocore.org/show_bug.cgi?id=2306


Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Wednesday, October 23, 2019 8:15 AM
> To: Dong, Eric <eric.dong@intel.com>; Laszlo Ersek
> <lersek@redhat.com>; devel@edk2.groups.io; Kinney,
> Michael D <michael.d.kinney@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray
> <ray.ni@intel.com>; Wei, David Y
> <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> Subject: RE: [Patch] UefiCpuPkg: Add missing components
> to UefiCpuPkg.dsc
> 
> Hi Eric,
> 
> I will enter a new BZ to remove binary.
> 
> Can you please review the current patch?
> 
> Thanks,
> 
> Mike
> 
> > -----Original Message-----
> > From: Dong, Eric <eric.dong@intel.com>
> > Sent: Tuesday, October 22, 2019 6:29 PM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> Laszlo Ersek
> > <lersek@redhat.com>; devel@edk2.groups.io
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray
> > <ray.ni@intel.com>; Wei, David Y
> <david.y.wei@intel.com>; Agyeman,
> > Prince <prince.agyeman@intel.com>
> > Subject: RE: [Patch] UefiCpuPkg: Add missing
> components to
> > UefiCpuPkg.dsc
> >
> > Hi Mike and Laszlo,
> >
> > I agree with Mike's proposal. We can submit new bugz
> to remove the
> > binary.
> >
> > Thanks,
> > Eric
> >
> > -----Original Message-----
> > From: Kinney, Michael D
> > Sent: Wednesday, October 23, 2019 7:39 AM
> > To: Laszlo Ersek <lersek@redhat.com>;
> > devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> Eric
> > <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>;
> Wei, David Y
> > <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> > Subject: RE: [Patch] UefiCpuPkg: Add missing
> components to
> > UefiCpuPkg.dsc
> >
> > Laszlo,
> >
> > Can we move forward with this patch and enter a new BZ
> to remove the
> > binary module?
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Laszlo Ersek <lersek@redhat.com>
> > > Sent: Tuesday, October 22, 2019 4:27 PM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> > > devel@edk2.groups.io
> > > Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> > Eric
> > > <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>;
> > Wei, David Y
> > > <david.y.wei@intel.com>; Agyeman, Prince
> > <prince.agyeman@intel.com>
> > > Subject: Re: [Patch] UefiCpuPkg: Add missing
> > components to
> > > UefiCpuPkg.dsc
> > >
> > > (+David, +Prince)
> > >
> > > On 10/23/19 01:13, Kinney, Michael D wrote:
> > > > Laszlo,
> > > >
> > > > I see 2 usages of this module in edk2-platforms:
> > > >
> > > > edk2-
> > >
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
> > > ):
> > > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >
> > > > edk2-
> > >
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
> > > f(16):INF
> > > > RuleOverride=RESET_VECTOR USE = IA32
> > > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > >
> > > Can we update those platforms to build the reset
> > vector from source?
> > >
> > > Embedding a binary blob seems wrong in its own
> > right...
> > >
> > > Anyway, if Ray and/or Eric are OK with the present
> > patch, I won't try
> > > to block it.
> > >
> > > Thanks,
> > > Laszlo
> > >
> > > >> -----Original Message-----
> > > >> From: Laszlo Ersek <lersek@redhat.com>
> > > >> Sent: Tuesday, October 22, 2019 3:57 PM
> > > >> To: Kinney, Michael D
> <michael.d.kinney@intel.com>;
> > > >> devel@edk2.groups.io
> > > >> Cc: Sean Brogan <sean.brogan@microsoft.com>;
> Dong,
> > > Eric
> > > >> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > >> Subject: Re: [Patch] UefiCpuPkg: Add missing
> > > components to
> > > >> UefiCpuPkg.dsc
> > > >>
> > > >> On 10/22/19 21:06, Michael D Kinney wrote:
> > > >>> From: Sean Brogan <sean.brogan@microsoft.com>
> > > >>>
> > > >>>
> > https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> > > >>>
> > > >>> Update UefiCpuPkg.dsc to guarantee all libraries
> > and
> > > >> modules are
> > > >>> always built.  Add the following components.
> > > >>>
> > > >>> *
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >>> *
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > > >> sLib.inf
> > > >>>
> > > >>> ResetVector.inf is a binary INF, so no source
> > builds
> > > >> are triggered
> > > >>> from adding this line.  However, a build with
> this
> > > >> component does
> > > >>> verify the contents of the INF file.
> > > >>>
> > > >>> Cc: Eric Dong <eric.dong@intel.com>
> > > >>> Cc: Ray Ni <ray.ni@intel.com>
> > > >>> Cc: Laszlo Ersek <lersek@redhat.com>
> > > >>> Signed-off-by: Michael D Kinney
> > > >> <michael.d.kinney@intel.com>
> > > >>> ---
> > > >>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> > > >>>  1 file changed, 2 insertions(+)
> > > >>>
> > > >>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> > > >> b/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> index e7dfe30eda..d28cb5cccb 100644
> > > >>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> @@ -121,6 +121,7 @@ [Components.IA32,
> > > Components.X64]
> > > >>>
> > > >>
> > UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> > > >>>
> > UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> > > >> .inf
> > > >>> +
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > > >> sLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> > > >> nHandlerLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> > > >> tionHandlerLib.inf
> > > >>>
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> > > >> nHandlerLib.in
> > > >>> f @@ -146,6 +147,7 @@ [Components.IA32,
> > > >> Components.X64]
> > > >>>
> > > >>
> > >
> >
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> > > >> mmCpuFeaturesLibStm.inf
> > > >>>    }
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> > > >>> +
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >>>
> > > >>>  [BuildOptions]
> > > >>>    *_*_*_CC_FLAGS = -D
> > > >> DISABLE_NEW_DEPRECATED_INTERFACES
> > > >>>
> > > >>
> > > >> Can we retire
> > > >>
> > > >>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> > > >>
> > > >> instead (in a separate patch)?
> > > >>
> > > >> Because, according to
> > > >>
> > > >>   git log --stat master --
> > > >> UefiCpuPkg/ResetVector/Vtf0/Bin/
> > > >>
> > > >> the most recent commit in which we updated the
> > *.raw
> > > files has been
> > > >> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
> > > finding SEC image of
> > > >> type TE.", 2011-09-28).
> > > >>
> > > >> I doubt anyone consumes this (binary) module,
> since
> > > Jordan's commits
> > > >> - 5a1f324d946c ("UefiCpuPkg: Support building
> VTF0
> > > ResetVector during
> > > >> the EDK II build", 2014-08-18)
> > > >> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
> > > tables in NASM
> > > >> code", 2014-08-18).
> > > >>
> > > >> We've by now dropped all top-level *BinPkg
> package
> > > directories too.
> > > >>
> > > >> If we remove the Bin subdir, then we should
> > probably
> > > >> remove:
> > > >> - Build.py
> > > >> - Tools/
> > > >> too.
> > > >>
> > > >> For the other addition (CpuCommonFeaturesLib): if
> > > that gets its own
> > > >> patch (after the removal of
> > > UefiCpuPkg/ResetVector/Vtf0/Bin/), then
> > > >> please add
> > > >>
> > > >> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> > > >>
> > > >> to it.
> > > >>
> > > >> Thanks
> > > >> Laszlo
> > > >


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-23 16:06             ` Michael D Kinney
@ 2019-10-23 16:08               ` Agyeman, Prince
  0 siblings, 0 replies; 12+ messages in thread
From: Agyeman, Prince @ 2019-10-23 16:08 UTC (permalink / raw)
  To: Kinney, Michael D, Dong, Eric, Laszlo Ersek, devel@edk2.groups.io
  Cc: Sean Brogan, Ni, Ray, Wei, David Y

Thanks Mike !

Prince

-----Original Message-----
From: Kinney, Michael D <michael.d.kinney@intel.com> 
Sent: Wednesday, October 23, 2019 9:07 AM
To: Dong, Eric <eric.dong@intel.com>; Laszlo Ersek <lersek@redhat.com>; devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray <ray.ni@intel.com>; Wei, David Y <david.y.wei@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>
Subject: RE: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc

I have enter 2 bugs.  One for SimicsX58SktPkg and one for UefiCpuPkg.

https://bugzilla.tianocore.org/show_bug.cgi?id=2307

https://bugzilla.tianocore.org/show_bug.cgi?id=2306


Mike

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Wednesday, October 23, 2019 8:15 AM
> To: Dong, Eric <eric.dong@intel.com>; Laszlo Ersek 
> <lersek@redhat.com>; devel@edk2.groups.io; Kinney, Michael D 
> <michael.d.kinney@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray 
> <ray.ni@intel.com>; Wei, David Y <david.y.wei@intel.com>; Agyeman, 
> Prince <prince.agyeman@intel.com>
> Subject: RE: [Patch] UefiCpuPkg: Add missing components to 
> UefiCpuPkg.dsc
> 
> Hi Eric,
> 
> I will enter a new BZ to remove binary.
> 
> Can you please review the current patch?
> 
> Thanks,
> 
> Mike
> 
> > -----Original Message-----
> > From: Dong, Eric <eric.dong@intel.com>
> > Sent: Tuesday, October 22, 2019 6:29 PM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> Laszlo Ersek
> > <lersek@redhat.com>; devel@edk2.groups.io
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray 
> > <ray.ni@intel.com>; Wei, David Y
> <david.y.wei@intel.com>; Agyeman,
> > Prince <prince.agyeman@intel.com>
> > Subject: RE: [Patch] UefiCpuPkg: Add missing
> components to
> > UefiCpuPkg.dsc
> >
> > Hi Mike and Laszlo,
> >
> > I agree with Mike's proposal. We can submit new bugz
> to remove the
> > binary.
> >
> > Thanks,
> > Eric
> >
> > -----Original Message-----
> > From: Kinney, Michael D
> > Sent: Wednesday, October 23, 2019 7:39 AM
> > To: Laszlo Ersek <lersek@redhat.com>; devel@edk2.groups.io; Kinney, 
> > Michael D
> <michael.d.kinney@intel.com>
> > Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> Eric
> > <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>;
> Wei, David Y
> > <david.y.wei@intel.com>; Agyeman, Prince
> <prince.agyeman@intel.com>
> > Subject: RE: [Patch] UefiCpuPkg: Add missing
> components to
> > UefiCpuPkg.dsc
> >
> > Laszlo,
> >
> > Can we move forward with this patch and enter a new BZ
> to remove the
> > binary module?
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Laszlo Ersek <lersek@redhat.com>
> > > Sent: Tuesday, October 22, 2019 4:27 PM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>; 
> > > devel@edk2.groups.io
> > > Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
> > Eric
> > > <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>;
> > Wei, David Y
> > > <david.y.wei@intel.com>; Agyeman, Prince
> > <prince.agyeman@intel.com>
> > > Subject: Re: [Patch] UefiCpuPkg: Add missing
> > components to
> > > UefiCpuPkg.dsc
> > >
> > > (+David, +Prince)
> > >
> > > On 10/23/19 01:13, Kinney, Michael D wrote:
> > > > Laszlo,
> > > >
> > > > I see 2 usages of this module in edk2-platforms:
> > > >
> > > > edk2-
> > >
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
> > > ):
> > > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >
> > > > edk2-
> > >
> >
> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
> > > f(16):INF
> > > > RuleOverride=RESET_VECTOR USE = IA32 
> > > > UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > >
> > > Can we update those platforms to build the reset
> > vector from source?
> > >
> > > Embedding a binary blob seems wrong in its own
> > right...
> > >
> > > Anyway, if Ray and/or Eric are OK with the present
> > patch, I won't try
> > > to block it.
> > >
> > > Thanks,
> > > Laszlo
> > >
> > > >> -----Original Message-----
> > > >> From: Laszlo Ersek <lersek@redhat.com>
> > > >> Sent: Tuesday, October 22, 2019 3:57 PM
> > > >> To: Kinney, Michael D
> <michael.d.kinney@intel.com>;
> > > >> devel@edk2.groups.io
> > > >> Cc: Sean Brogan <sean.brogan@microsoft.com>;
> Dong,
> > > Eric
> > > >> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
> > > >> Subject: Re: [Patch] UefiCpuPkg: Add missing
> > > components to
> > > >> UefiCpuPkg.dsc
> > > >>
> > > >> On 10/22/19 21:06, Michael D Kinney wrote:
> > > >>> From: Sean Brogan <sean.brogan@microsoft.com>
> > > >>>
> > > >>>
> > https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> > > >>>
> > > >>> Update UefiCpuPkg.dsc to guarantee all libraries
> > and
> > > >> modules are
> > > >>> always built.  Add the following components.
> > > >>>
> > > >>> *
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >>> *
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > > >> sLib.inf
> > > >>>
> > > >>> ResetVector.inf is a binary INF, so no source
> > builds
> > > >> are triggered
> > > >>> from adding this line.  However, a build with
> this
> > > >> component does
> > > >>> verify the contents of the INF file.
> > > >>>
> > > >>> Cc: Eric Dong <eric.dong@intel.com>
> > > >>> Cc: Ray Ni <ray.ni@intel.com>
> > > >>> Cc: Laszlo Ersek <lersek@redhat.com>
> > > >>> Signed-off-by: Michael D Kinney
> > > >> <michael.d.kinney@intel.com>
> > > >>> ---
> > > >>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
> > > >>>  1 file changed, 2 insertions(+)
> > > >>>
> > > >>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
> > > >> b/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> index e7dfe30eda..d28cb5cccb 100644
> > > >>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> > > >>> @@ -121,6 +121,7 @@ [Components.IA32,
> > > Components.X64]
> > > >>>
> > > >>
> > UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
> > > >>>
> > UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
> > > >> .inf
> > > >>> +
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
> > > >> sLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
> > > >> nHandlerLib.inf
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
> > > >> tionHandlerLib.inf
> > > >>>
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
> > > >> nHandlerLib.in
> > > >>> f @@ -146,6 +147,7 @@ [Components.IA32,
> > > >> Components.X64]
> > > >>>
> > > >>
> > >
> >
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
> > > >> mmCpuFeaturesLibStm.inf
> > > >>>    }
> > > >>>
> > > >>
> > >
> >
> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> > > >>> +
> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> > > >>>
> > > >>>  [BuildOptions]
> > > >>>    *_*_*_CC_FLAGS = -D
> > > >> DISABLE_NEW_DEPRECATED_INTERFACES
> > > >>>
> > > >>
> > > >> Can we retire
> > > >>
> > > >>   UefiCpuPkg/ResetVector/Vtf0/Bin/
> > > >>
> > > >> instead (in a separate patch)?
> > > >>
> > > >> Because, according to
> > > >>
> > > >>   git log --stat master --
> > > >> UefiCpuPkg/ResetVector/Vtf0/Bin/
> > > >>
> > > >> the most recent commit in which we updated the
> > *.raw
> > > files has been
> > > >> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
> > > finding SEC image of
> > > >> type TE.", 2011-09-28).
> > > >>
> > > >> I doubt anyone consumes this (binary) module,
> since
> > > Jordan's commits
> > > >> - 5a1f324d946c ("UefiCpuPkg: Support building
> VTF0
> > > ResetVector during
> > > >> the EDK II build", 2014-08-18)
> > > >> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
> > > tables in NASM
> > > >> code", 2014-08-18).
> > > >>
> > > >> We've by now dropped all top-level *BinPkg
> package
> > > directories too.
> > > >>
> > > >> If we remove the Bin subdir, then we should
> > probably
> > > >> remove:
> > > >> - Build.py
> > > >> - Tools/
> > > >> too.
> > > >>
> > > >> For the other addition (CpuCommonFeaturesLib): if
> > > that gets its own
> > > >> patch (after the removal of
> > > UefiCpuPkg/ResetVector/Vtf0/Bin/), then
> > > >> please add
> > > >>
> > > >> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> > > >>
> > > >> to it.
> > > >>
> > > >> Thanks
> > > >> Laszlo
> > > >


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [edk2-devel] [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 19:06 [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc Michael D Kinney
  2019-10-22 22:57 ` Laszlo Ersek
@ 2019-10-24  0:06 ` Dong, Eric
  2019-10-24  0:21   ` Ni, Ray
  1 sibling, 1 reply; 12+ messages in thread
From: Dong, Eric @ 2019-10-24  0:06 UTC (permalink / raw)
  To: devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Ni, Ray, Laszlo Ersek

Reviewed-by: Eric Dong <eric.dong@intel.com>

-----Original Message-----
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Michael D Kinney
Sent: Wednesday, October 23, 2019 3:07 AM
To: devel@edk2.groups.io
Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Laszlo Ersek <lersek@redhat.com>
Subject: [edk2-devel] [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc

From: Sean Brogan <sean.brogan@microsoft.com>

https://bugzilla.tianocore.org/show_bug.cgi?id=2255

Update UefiCpuPkg.dsc to guarantee all libraries and modules are always built.  Add the following components.

* UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
* UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf

ResetVector.inf is a binary INF, so no source builds are triggered from adding this line.  However, a build with this component does verify the contents of the INF file.

Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
---
 UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/UefiCpuPkg/UefiCpuPkg.dsc b/UefiCpuPkg/UefiCpuPkg.dsc index e7dfe30eda..d28cb5cccb 100644
--- a/UefiCpuPkg/UefiCpuPkg.dsc
+++ b/UefiCpuPkg/UefiCpuPkg.dsc
@@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
   UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
   UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
   UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf
+  UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptionHandlerLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExceptionHandlerLib.inf
   UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf
@@ -146,6 +147,7 @@ [Components.IA32, Components.X64]
       SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmCpuFeaturesLibStm.inf
   }
   UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
+  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
 
 [BuildOptions]
   *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
--
2.21.0.windows.1





^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [edk2-devel] [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-24  0:06 ` [edk2-devel] " Dong, Eric
@ 2019-10-24  0:21   ` Ni, Ray
  0 siblings, 0 replies; 12+ messages in thread
From: Ni, Ray @ 2019-10-24  0:21 UTC (permalink / raw)
  To: Dong, Eric, devel@edk2.groups.io, Kinney, Michael D
  Cc: Sean Brogan, Laszlo Ersek

Reviewed-by: Ray Ni <ray.ni@intel.com>

> -----Original Message-----
> From: Dong, Eric <eric.dong@intel.com>
> Sent: Thursday, October 24, 2019 8:07 AM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Ni, Ray <ray.ni@intel.com>;
> Laszlo Ersek <lersek@redhat.com>
> Subject: RE: [edk2-devel] [Patch] UefiCpuPkg: Add missing components to
> UefiCpuPkg.dsc
> 
> Reviewed-by: Eric Dong <eric.dong@intel.com>
> 
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Michael D Kinney
> Sent: Wednesday, October 23, 2019 3:07 AM
> To: devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Laszlo Ersek
> <lersek@redhat.com>
> Subject: [edk2-devel] [Patch] UefiCpuPkg: Add missing components to
> UefiCpuPkg.dsc
> 
> From: Sean Brogan <sean.brogan@microsoft.com>
> 
> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
> 
> Update UefiCpuPkg.dsc to guarantee all libraries and modules are always built.
> Add the following components.
> 
> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> * UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
> 
> ResetVector.inf is a binary INF, so no source builds are triggered from adding
> this line.  However, a build with this component does verify the contents of the
> INF file.
> 
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> ---
>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc b/UefiCpuPkg/UefiCpuPkg.dsc index
> e7dfe30eda..d28cb5cccb 100644
> --- a/UefiCpuPkg/UefiCpuPkg.dsc
> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
> @@ -121,6 +121,7 @@ [Components.IA32, Components.X64]
>    UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
>    UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf
> +  UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeaturesLib.inf
>    UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptionHandlerLib.inf
> 
> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExceptionHandlerLib.inf
>    UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf
> @@ -146,6 +147,7 @@ [Components.IA32, Components.X64]
> 
> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/SmmCpuFeatures
> LibStm.inf
>    }
>    UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
> 
>  [BuildOptions]
>    *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
> --
> 2.21.0.windows.1
> 
> 
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc
  2019-10-22 23:38       ` Michael D Kinney
  2019-10-23  1:28         ` Dong, Eric
@ 2019-10-24 11:03         ` Laszlo Ersek
  1 sibling, 0 replies; 12+ messages in thread
From: Laszlo Ersek @ 2019-10-24 11:03 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io
  Cc: Sean Brogan, Dong, Eric, Ni, Ray, Wei, David Y, Agyeman, Prince

On 10/23/19 01:38, Kinney, Michael D wrote:
> Laszlo,
> 
> Can we move forward with this patch and enter a new
> BZ to remove the binary module?

Sure, thank you!
Laszlo

>> -----Original Message-----
>> From: Laszlo Ersek <lersek@redhat.com>
>> Sent: Tuesday, October 22, 2019 4:27 PM
>> To: Kinney, Michael D <michael.d.kinney@intel.com>;
>> devel@edk2.groups.io
>> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong, Eric
>> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Wei,
>> David Y <david.y.wei@intel.com>; Agyeman, Prince
>> <prince.agyeman@intel.com>
>> Subject: Re: [Patch] UefiCpuPkg: Add missing components
>> to UefiCpuPkg.dsc
>>
>> (+David, +Prince)
>>
>> On 10/23/19 01:13, Kinney, Michael D wrote:
>>> Laszlo,
>>>
>>> I see 2 usages of this module in edk2-platforms:
>>>
>>> edk2-
>> platforms\Silicon\Intel\SimicsX58SktPkg\SktPkgPei.dsc(13
>> ):
>>> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>>
>>> edk2-
>> platforms\Silicon\Intel\SimicsX58SktPkg\SktSecInclude.fd
>> f(16):INF
>>> RuleOverride=RESET_VECTOR USE = IA32
>>> UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>
>> Can we update those platforms to build the reset vector
>> from source?
>>
>> Embedding a binary blob seems wrong in its own right...
>>
>> Anyway, if Ray and/or Eric are OK with the present
>> patch, I won't try to block it.
>>
>> Thanks,
>> Laszlo
>>
>>>> -----Original Message-----
>>>> From: Laszlo Ersek <lersek@redhat.com>
>>>> Sent: Tuesday, October 22, 2019 3:57 PM
>>>> To: Kinney, Michael D <michael.d.kinney@intel.com>;
>>>> devel@edk2.groups.io
>>>> Cc: Sean Brogan <sean.brogan@microsoft.com>; Dong,
>> Eric
>>>> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>
>>>> Subject: Re: [Patch] UefiCpuPkg: Add missing
>> components to
>>>> UefiCpuPkg.dsc
>>>>
>>>> On 10/22/19 21:06, Michael D Kinney wrote:
>>>>> From: Sean Brogan <sean.brogan@microsoft.com>
>>>>>
>>>>> https://bugzilla.tianocore.org/show_bug.cgi?id=2255
>>>>>
>>>>> Update UefiCpuPkg.dsc to guarantee all libraries and
>>>> modules are
>>>>> always built.  Add the following components.
>>>>>
>>>>> * UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>>>> *
>>>>
>> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
>>>> sLib.inf
>>>>>
>>>>> ResetVector.inf is a binary INF, so no source builds
>>>> are triggered
>>>>> from adding this line.  However, a build with this
>>>> component does
>>>>> verify the contents of the INF file.
>>>>>
>>>>> Cc: Eric Dong <eric.dong@intel.com>
>>>>> Cc: Ray Ni <ray.ni@intel.com>
>>>>> Cc: Laszlo Ersek <lersek@redhat.com>
>>>>> Signed-off-by: Michael D Kinney
>>>> <michael.d.kinney@intel.com>
>>>>> ---
>>>>>  UefiCpuPkg/UefiCpuPkg.dsc | 2 ++
>>>>>  1 file changed, 2 insertions(+)
>>>>>
>>>>> diff --git a/UefiCpuPkg/UefiCpuPkg.dsc
>>>> b/UefiCpuPkg/UefiCpuPkg.dsc
>>>>> index e7dfe30eda..d28cb5cccb 100644
>>>>> --- a/UefiCpuPkg/UefiCpuPkg.dsc
>>>>> +++ b/UefiCpuPkg/UefiCpuPkg.dsc
>>>>> @@ -121,6 +121,7 @@ [Components.IA32,
>> Components.X64]
>>>>>
>>>> UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf
>>>>>    UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf
>>>>>
>>>>
>> UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib
>>>> .inf
>>>>> +
>>>>
>> UefiCpuPkg/Library/CpuCommonFeaturesLib/CpuCommonFeature
>>>> sLib.inf
>>>>>
>>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/DxeCpuExceptio
>>>> nHandlerLib.inf
>>>>>
>>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/SecPeiCpuExcep
>>>> tionHandlerLib.inf
>>>>>
>>>>>
>>>>
>> UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptio
>>>> nHandlerLib.in
>>>>> f @@ -146,6 +147,7 @@ [Components.IA32,
>>>> Components.X64]
>>>>>
>>>>
>> SmmCpuFeaturesLib|UefiCpuPkg/Library/SmmCpuFeaturesLib/S
>>>> mmCpuFeaturesLibStm.inf
>>>>>    }
>>>>>
>>>>
>> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf
>>>>> +  UefiCpuPkg/ResetVector/Vtf0/Bin/ResetVector.inf
>>>>>
>>>>>  [BuildOptions]
>>>>>    *_*_*_CC_FLAGS = -D
>>>> DISABLE_NEW_DEPRECATED_INTERFACES
>>>>>
>>>>
>>>> Can we retire
>>>>
>>>>   UefiCpuPkg/ResetVector/Vtf0/Bin/
>>>>
>>>> instead (in a separate patch)?
>>>>
>>>> Because, according to
>>>>
>>>>   git log --stat master --
>>>> UefiCpuPkg/ResetVector/Vtf0/Bin/
>>>>
>>>> the most recent commit in which we updated the *.raw
>> files has been
>>>> f7bb98019a71 ("UefiCpuPkg VTF0: Fix support for
>> finding SEC image of
>>>> type TE.", 2011-09-28).
>>>>
>>>> I doubt anyone consumes this (binary) module, since
>> Jordan's commits
>>>> - 5a1f324d946c ("UefiCpuPkg: Support building VTF0
>> ResetVector during
>>>> the EDK II build", 2014-08-18)
>>>> - eee1d2ca9078 ("UefiCpuPkg VTF0 X64: Build page
>> tables in NASM
>>>> code", 2014-08-18).
>>>>
>>>> We've by now dropped all top-level *BinPkg package
>> directories too.
>>>>
>>>> If we remove the Bin subdir, then we should probably
>>>> remove:
>>>> - Build.py
>>>> - Tools/
>>>> too.
>>>>
>>>> For the other addition (CpuCommonFeaturesLib): if
>> that gets its own
>>>> patch (after the removal of
>> UefiCpuPkg/ResetVector/Vtf0/Bin/), then
>>>> please add
>>>>
>>>> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
>>>>
>>>> to it.
>>>>
>>>> Thanks
>>>> Laszlo
>>>
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2019-10-24 11:03 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-22 19:06 [Patch] UefiCpuPkg: Add missing components to UefiCpuPkg.dsc Michael D Kinney
2019-10-22 22:57 ` Laszlo Ersek
2019-10-22 23:13   ` Michael D Kinney
2019-10-22 23:27     ` Laszlo Ersek
2019-10-22 23:38       ` Michael D Kinney
2019-10-23  1:28         ` Dong, Eric
2019-10-23 15:15           ` Michael D Kinney
2019-10-23 16:06             ` Michael D Kinney
2019-10-23 16:08               ` Agyeman, Prince
2019-10-24 11:03         ` Laszlo Ersek
2019-10-24  0:06 ` [edk2-devel] " Dong, Eric
2019-10-24  0:21   ` Ni, Ray

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox