Re: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

["Oram, Isaac W" <isaac.w.oram@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 8861 bytes --]

Eric,

I have envisioned less granular packages for advanced features.  One of the goals for MinPlatform is to improve usability during porting.  The idea is that you would do basic board porting with minimal effort to get your system functional.  Then you would enable additional features by adding collections of features to your baseline.  Then as a last step optimize out unnecessary things.  This is something like a functionality oriented porting approach.  Get all the functionality you need by building up, then optimize out.  I would characterize Intel's traditional reference platforms as having everything and then adding and removing from a starting point that was very feature rich.  The challenge we seemed to face was that it was hard to remove things with the feature rich starting point.

As an example of functionality oriented porting, say that I set up my basic server port by starting with the Purley open board package.  I port it to my motherboard, then I check out my baseline functionality.  Then I add manageability features by including DSC/FDF from the ManageabilityFeaturePkg that add FV to my MinPlatform port.  And I repeat for other sets of features until I get all of the features that I need.  Then I optimize: for size, speed, to reduce complexity, and so on.  It would be best if this optimization were tool assisted to a great degree, e.g a more sophisticated FMMT that lets one cut out extra components.

My concern is that if we allow very specific feature packages, like the UserAuthenticationPkg, we are very much like today.  Yes, you can select any drivers you need and add to your DSC/FDF.  But that is very quickly overwhelming.  There are hundreds of drivers and what they require is often complex to determine.  Thus we tend to copy something else and customize it.  This tends to lead to lots of technical debt and complexity.

I am thinking that we should target something like 10-20 advanced feature packages that produce one or two (if features have pre-memory components) FV with a set of features and simpler dependencies.  We are just in the early stages of defining what this would look like, and our thinking is evolving.  We have identified Manageability and Debug as feature collections.  I think that there is one for adding USB, network, Bluetooth and such peripheral support.  I think setup browser and UI stuff will go somewhere.  We can mine a few reference platforms for data.

Let's discuss this in person and make a proposal for organization for feature packages and the rules for what goes where.  We can add this proposal to your RFC and I think that will help guide the future development of the Platform\Intel contents.

Regards,
Isaac


From: Yao, Jiewen
Sent: Monday, June 3, 2019 9:03 AM
To: Dong, Eric <eric.dong@intel.com>; devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Kubacki, Michael A <michael.a.kubacki@intel.com>; Oram, Isaac W <isaac.w.oram@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Since there is no other module, I think we can use this specific package name to tell people what it is.
It is also good for feature isolation.

Thank you
Yao Jiewen

From: Dong, Eric
Sent: Monday, June 3, 2019 8:53 AM
To: Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Hi Jiewen,

So far, I don't have other modules which need to move to this package.

I think UserAuthenticationPkg is too specific, but if others also agree with this name, I'm ok too.


Hi liming, Isaac & Mike,

Any comments about the new package name?

Thanks,
Eric
From: Yao, Jiewen
Sent: Monday, June 3, 2019 8:35 AM
To: Dong, Eric <eric.dong@intel.com<mailto:eric.dong@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Cc: Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

OK. Do you have any other modules what could be potentially in this package?

I think another option is to name it UserAuthenticationPkg, just like SignedCapsulePkg.

Thank you
Yao Jiewen

From: Dong, Eric
Sent: Monday, June 3, 2019 8:28 AM
To: Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

I think sample means this is an example about how to enable this feature. Code implemented with production quality. Platform can decide whether to use it or not.

Thanks,
Eric
From: Yao, Jiewen
Sent: Friday, May 31, 2019 4:06 PM
To: Dong, Eric <eric.dong@intel.com<mailto:eric.dong@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Would you please clarify what "sample" here really means? Not for production? Or something else?

Thank you
Yao Jiewen


From: Dong, Eric
Sent: Friday, May 31, 2019 4:02 PM
To: Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Hi Jiewen,

I think SecuritySamplePkg used to save 1) sample implementation for security related features, 2) it's platform scope feature.
SecurityPkg used to save 1) common security features, 2) It's not a sample implementation.

This is just my proposal, If you have better one, you can raise here.

Thanks,
Eric
From: Yao, Jiewen
Sent: Friday, May 31, 2019 3:49 PM
To: Dong, Eric <eric.dong@intel.com<mailto:eric.dong@intel.com>>; devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>
Subject: RE: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Would you please clarify the position of SecuritySamplePkg ?

What is the difference between SecurityPkg and SecuritySamplePkg ?

Thank you
Yao Jiewen

From: Dong, Eric
Sent: Friday, May 31, 2019 3:46 PM
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io>; Gao, Liming <liming.gao@intel.com<mailto:liming.gao@intel.com>>; Kubacki, Michael A <michael.a.kubacki@intel.com<mailto:michael.a.kubacki@intel.com>>; Oram, Isaac W <isaac.w.oram@intel.com<mailto:isaac.w.oram@intel.com>>; Kinney, Michael D <michael.d.kinney@intel.com<mailto:michael.d.kinney@intel.com>>; Yao, Jiewen <jiewen.yao@intel.com<mailto:jiewen.yao@intel.com>>
Subject: [RFC][edk2-platform][Add new packages in Platform\Intel directory]

Isaac, Jiewen & Mike,

I plan to add a new driver named UserAuthentication. This driver shows a sample implementation about how to control user enter setup page. I plan to add a new package in Platform\Intel folder to save this driver. New package name is SecuritySamplePkg. Any comments for this RFC?

Thanks,
Eric
_._,_._,_

[-- Attachment #2: Type: text/html, Size: 23695 bytes --]