Re: [edk2] [staging/UEFI_Redfish][PATCH v1] Announce to create "UEFI_Redfish" branch in edk2-staging.

2019-01-20 Thread Wu, Jiaxin
Good suggestion. Thanks Leif. Already resubmit as version 2.
 

> -Original Message-
> From: Leif Lindholm [mailto:leif.lindh...@linaro.org]
> Sent: Friday, January 18, 2019 9:19 PM
> To: Wu, Jiaxin 
> Cc: edk2-devel@lists.01.org; Wang, Fan ; Ye, Ting
> ; Li, Ruth ; Kinney, Michael D
> ; Fu, Siyuan 
> Subject: Re: [edk2] [staging/UEFI_Redfish][PATCH v1] Announce to create
> "UEFI_Redfish" branch in edk2-staging.
> 
> Hi Jiaxin,
> 
> I am happy to see the creation of this branch. However, Could you
> possibly resubmit this as a diff adding a Readme.md rather than
> modifying it?
> 
> The diff against edk2/Readme.md is not really relevant, and confuses review.
> 
> (For example, in your branch, *delete* the existing Readme.md in a
> separate commit, and then in the commit next *add* the one for the
> branch. Only the *add* patch needs to be reviewed.)
> 
> Best Regards,
> 
> Leif
> 
> On Fri, Jan 18, 2019 at 05:42:40PM +0800, Jiaxin Wu wrote:
> > UEFI_Redfish branch is to develop the UEFI Redfish feature. The code base
> > of development is based on the release of edk2-stable201811 tag. Please
> > refer to the patch of Readme.md to get the detailed feature introduction.
> >
> > Note: The branch will be created by the end of Jan 28th if no objection.
> >
> > Cc: Rothman Michael A 
> > Cc: Kinney Michael D 
> > Cc: Li Ruth 
> > Cc: Ye Ting 
> > Cc: Fu Siyuan 
> > Cc: Wang Fan 
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Wu Jiaxin 
> > ---
> >  Readme.md | 114 
> --
> >  1 file changed, 85 insertions(+), 29 deletions(-)
> >
> > diff --git a/Readme.md b/Readme.md
> > index 1ef0780ee0..b9b5ab38e2 100644
> > --- a/Readme.md
> > +++ b/Readme.md
> > @@ -1,29 +1,85 @@
> > -# EDK II Project
> > -
> > -A modern, feature-rich, cross-platform firmware development environment
> > -for the UEFI and PI specifications from www.uefi.org.
> > -
> > -Contributions to the EDK II open source project are covered by the
> > -[TianoCore Contribution Agreement 1.1](Contributions.txt)
> > -
> > -The majority of the content in the EDK II open source project uses a
> > -[BSD 2-Clause License](License.txt).  The EDK II open source project 
> > contains
> > -the following components that are covered by additional licenses:
> > -* [AppPkg/Applications/Python/Python-
> 2.7.2/Tools/pybench](AppPkg/Applications/Python/Python-
> 2.7.2/Tools/pybench/LICENSE)
> > -* [AppPkg/Applications/Python/Python-
> 2.7.2](AppPkg/Applications/Python/Python-2.7.2/LICENSE)
> > -* [AppPkg/Applications/Python/Python-
> 2.7.10](AppPkg/Applications/Python/Python-2.7.10/LICENSE)
> > -*
> [BaseTools/Source/C/BrotliCompress](BaseTools/Source/C/BrotliCompress/LIC
> ENSE)
> > -*
> [MdeModulePkg/Library/BrotliCustomDecompressLib](MdeModulePkg/Library/
> BrotliCustomDecompressLib/LICENSE)
> > -* [OvmfPkg](OvmfPkg/License.txt)
> > -*
> [CryptoPkg/Library/OpensslLib/openssl](CryptoPkg/Library/OpensslLib/openssl/
> LICENSE)
> > -
> > -The EDK II Project is composed of packages.  The maintainers for each
> package
> > -are listed in [Maintainers.txt](Maintainers.txt).
> > -
> > -# Resources
> > -* [TianoCore](http://www.tianocore.org)
> > -* [EDK II](https://github.com/tianocore/tianocore.github.io/wiki/EDK-II)
> > -* [Getting Started with EDK
> II](https://github.com/tianocore/tianocore.github.io/wiki/Getting-Started-with-
> EDK-II)
> > -* [Mailing
> Lists](https://github.com/tianocore/tianocore.github.io/wiki/Mailing-Lists)
> > -* [TianoCore Bugzilla](https://bugzilla.tianocore.org)
> > -* [How To
> Contribute](https://github.com/tianocore/tianocore.github.io/wiki/How-To-
> Contribute)
> > +This branch is used to develop the **UEFI Redfish Feature**. The code base
> of development is based on the release of **edk2-stable201811** tag.
> > +
> > +The branch owner:
> > +Fu Siyuan , Ye Ting , Wang Fan
> , Wu Jiaxin 
> > +
> > +## Introduction
> > +UEFI Redfish is an efficient and secure solution for end users to remote
> control and configure UEFI pre-OS environment by leveraging the RESTful API.
> It's simple for end users to access the data from UEFI firmware defined in 
> JSON
> format.
> > +
> > +One of the design goals for UEFI Redfish solution is to provide a scalable
> implementation which allow users to easily add/remove/modify each
> independent Redfish configure features (RedfishBiosDxe & RedfishBootInfoDxe).
> This is done by e

Re: [edk2] [staging/UEFI_Redfish][PATCH v1] Announce to create "UEFI_Redfish" branch in edk2-staging.

2019-01-18 Thread Leif Lindholm
Hi Jiaxin,

I am happy to see the creation of this branch. However, Could you
possibly resubmit this as a diff adding a Readme.md rather than
modifying it?

The diff against edk2/Readme.md is not really relevant, and confuses review.

(For example, in your branch, *delete* the existing Readme.md in a
separate commit, and then in the commit next *add* the one for the
branch. Only the *add* patch needs to be reviewed.)

Best Regards,

Leif

On Fri, Jan 18, 2019 at 05:42:40PM +0800, Jiaxin Wu wrote:
> UEFI_Redfish branch is to develop the UEFI Redfish feature. The code base
> of development is based on the release of edk2-stable201811 tag. Please
> refer to the patch of Readme.md to get the detailed feature introduction.
> 
> Note: The branch will be created by the end of Jan 28th if no objection.
> 
> Cc: Rothman Michael A 
> Cc: Kinney Michael D 
> Cc: Li Ruth 
> Cc: Ye Ting 
> Cc: Fu Siyuan 
> Cc: Wang Fan 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Wu Jiaxin 
> ---
>  Readme.md | 114 --
>  1 file changed, 85 insertions(+), 29 deletions(-)
> 
> diff --git a/Readme.md b/Readme.md
> index 1ef0780ee0..b9b5ab38e2 100644
> --- a/Readme.md
> +++ b/Readme.md
> @@ -1,29 +1,85 @@
> -# EDK II Project
> -
> -A modern, feature-rich, cross-platform firmware development environment
> -for the UEFI and PI specifications from www.uefi.org.
> -
> -Contributions to the EDK II open source project are covered by the
> -[TianoCore Contribution Agreement 1.1](Contributions.txt)
> -
> -The majority of the content in the EDK II open source project uses a
> -[BSD 2-Clause License](License.txt).  The EDK II open source project contains
> -the following components that are covered by additional licenses:
> -* 
> [AppPkg/Applications/Python/Python-2.7.2/Tools/pybench](AppPkg/Applications/Python/Python-2.7.2/Tools/pybench/LICENSE)
> -* 
> [AppPkg/Applications/Python/Python-2.7.2](AppPkg/Applications/Python/Python-2.7.2/LICENSE)
> -* 
> [AppPkg/Applications/Python/Python-2.7.10](AppPkg/Applications/Python/Python-2.7.10/LICENSE)
> -* 
> [BaseTools/Source/C/BrotliCompress](BaseTools/Source/C/BrotliCompress/LICENSE)
> -* 
> [MdeModulePkg/Library/BrotliCustomDecompressLib](MdeModulePkg/Library/BrotliCustomDecompressLib/LICENSE)
> -* [OvmfPkg](OvmfPkg/License.txt)
> -* 
> [CryptoPkg/Library/OpensslLib/openssl](CryptoPkg/Library/OpensslLib/openssl/LICENSE)
> -
> -The EDK II Project is composed of packages.  The maintainers for each package
> -are listed in [Maintainers.txt](Maintainers.txt).
> -
> -# Resources
> -* [TianoCore](http://www.tianocore.org)
> -* [EDK II](https://github.com/tianocore/tianocore.github.io/wiki/EDK-II)
> -* [Getting Started with EDK 
> II](https://github.com/tianocore/tianocore.github.io/wiki/Getting-Started-with-EDK-II)
> -* [Mailing 
> Lists](https://github.com/tianocore/tianocore.github.io/wiki/Mailing-Lists)
> -* [TianoCore Bugzilla](https://bugzilla.tianocore.org)
> -* [How To 
> Contribute](https://github.com/tianocore/tianocore.github.io/wiki/How-To-Contribute)
> +This branch is used to develop the **UEFI Redfish Feature**. The code base 
> of development is based on the release of **edk2-stable201811** tag.
> +
> +The branch owner:
> +Fu Siyuan , Ye Ting , Wang Fan 
> , Wu Jiaxin 
> +
> +## Introduction
> +UEFI Redfish is an efficient and secure solution for end users to remote 
> control and configure UEFI pre-OS environment by leveraging the RESTful API.  
> It's simple for end users to access the data from UEFI firmware defined in 
> JSON format.
> +
> +One of the design goals for UEFI Redfish solution is to provide a scalable 
> implementation which allow users to easily add/remove/modify each independent 
> Redfish configure features (RedfishBiosDxe & RedfishBootInfoDxe). This is 
> done by extracting the generic logic to a single UEFI driver model driver 
> (RedfishConfigDxe), and several library instances (DxeRedfishLib & 
> BaseJsonLib).
> +
> + Supported Features
> +  * Protocols
> +* EFI RestEx Service Binding Protocol
> +* EFI RestEx Protocol
> +* Redfish ConfigHandler Protocol
> +* Redfish Credential Protocol
> +
> +  * Configuration Items via UEFI Redfish
> +* [ISCSI Boot Keywords](http://www.uefi.org/confignamespace).
> +* HII Opcodes/Questions marked with REST_SYTLE flag or in REST_SYTLE 
> formset.
> +* BootOrder/BootNext variables.
> +
> +  * Redfish Schemas
> +* 
> [AttributeRegistry](https://redfish.dmtf.org/schemas/v1/AttributeRegistry.v1_1_0.json)
> +* 
> [ComputerSystemCollection](https://redfish.dmtf.org/schemas/ComputerSystemCollection.json)
> +* 
> [ComputerSystem](https://redfish.dmtf.org/schemas/v1/ComputerSystem.v1_5_0.json)
> +* [Bios](https://redfish.dmtf.org/schemas/v1/Bios.v1_0_2.json)
> +* 
> [BootOptionCollection](https://redfish.dmtf.org/schemas/BootOptionCollection.json)
> +* 

[edk2] [staging/UEFI_Redfish][PATCH v1] Announce to create "UEFI_Redfish" branch in edk2-staging.

2019-01-18 Thread Jiaxin Wu
UEFI_Redfish branch is to develop the UEFI Redfish feature. The code base
of development is based on the release of edk2-stable201811 tag. Please
refer to the patch of Readme.md to get the detailed feature introduction.

Note: The branch will be created by the end of Jan 28th if no objection.

Cc: Rothman Michael A 
Cc: Kinney Michael D 
Cc: Li Ruth 
Cc: Ye Ting 
Cc: Fu Siyuan 
Cc: Wang Fan 
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Wu Jiaxin 
---
 Readme.md | 114 --
 1 file changed, 85 insertions(+), 29 deletions(-)

diff --git a/Readme.md b/Readme.md
index 1ef0780ee0..b9b5ab38e2 100644
--- a/Readme.md
+++ b/Readme.md
@@ -1,29 +1,85 @@
-# EDK II Project
-
-A modern, feature-rich, cross-platform firmware development environment
-for the UEFI and PI specifications from www.uefi.org.
-
-Contributions to the EDK II open source project are covered by the
-[TianoCore Contribution Agreement 1.1](Contributions.txt)
-
-The majority of the content in the EDK II open source project uses a
-[BSD 2-Clause License](License.txt).  The EDK II open source project contains
-the following components that are covered by additional licenses:
-* 
[AppPkg/Applications/Python/Python-2.7.2/Tools/pybench](AppPkg/Applications/Python/Python-2.7.2/Tools/pybench/LICENSE)
-* 
[AppPkg/Applications/Python/Python-2.7.2](AppPkg/Applications/Python/Python-2.7.2/LICENSE)
-* 
[AppPkg/Applications/Python/Python-2.7.10](AppPkg/Applications/Python/Python-2.7.10/LICENSE)
-* 
[BaseTools/Source/C/BrotliCompress](BaseTools/Source/C/BrotliCompress/LICENSE)
-* 
[MdeModulePkg/Library/BrotliCustomDecompressLib](MdeModulePkg/Library/BrotliCustomDecompressLib/LICENSE)
-* [OvmfPkg](OvmfPkg/License.txt)
-* 
[CryptoPkg/Library/OpensslLib/openssl](CryptoPkg/Library/OpensslLib/openssl/LICENSE)
-
-The EDK II Project is composed of packages.  The maintainers for each package
-are listed in [Maintainers.txt](Maintainers.txt).
-
-# Resources
-* [TianoCore](http://www.tianocore.org)
-* [EDK II](https://github.com/tianocore/tianocore.github.io/wiki/EDK-II)
-* [Getting Started with EDK 
II](https://github.com/tianocore/tianocore.github.io/wiki/Getting-Started-with-EDK-II)
-* [Mailing 
Lists](https://github.com/tianocore/tianocore.github.io/wiki/Mailing-Lists)
-* [TianoCore Bugzilla](https://bugzilla.tianocore.org)
-* [How To 
Contribute](https://github.com/tianocore/tianocore.github.io/wiki/How-To-Contribute)
+This branch is used to develop the **UEFI Redfish Feature**. The code base of 
development is based on the release of **edk2-stable201811** tag.
+
+The branch owner:
+Fu Siyuan , Ye Ting , Wang Fan 
, Wu Jiaxin 
+
+## Introduction
+UEFI Redfish is an efficient and secure solution for end users to remote 
control and configure UEFI pre-OS environment by leveraging the RESTful API.  
It's simple for end users to access the data from UEFI firmware defined in JSON 
format.
+
+One of the design goals for UEFI Redfish solution is to provide a scalable 
implementation which allow users to easily add/remove/modify each independent 
Redfish configure features (RedfishBiosDxe & RedfishBootInfoDxe). This is done 
by extracting the generic logic to a single UEFI driver model driver 
(RedfishConfigDxe), and several library instances (DxeRedfishLib & BaseJsonLib).
+
+ Supported Features
+  * Protocols
+* EFI RestEx Service Binding Protocol
+* EFI RestEx Protocol
+* Redfish ConfigHandler Protocol
+* Redfish Credential Protocol
+
+  * Configuration Items via UEFI Redfish
+* [ISCSI Boot Keywords](http://www.uefi.org/confignamespace).
+* HII Opcodes/Questions marked with REST_SYTLE flag or in REST_SYTLE 
formset.
+* BootOrder/BootNext variables.
+
+  * Redfish Schemas
+* 
[AttributeRegistry](https://redfish.dmtf.org/schemas/v1/AttributeRegistry.v1_1_0.json)
+* 
[ComputerSystemCollection](https://redfish.dmtf.org/schemas/ComputerSystemCollection.json)
+* 
[ComputerSystem](https://redfish.dmtf.org/schemas/v1/ComputerSystem.v1_5_0.json)
+* [Bios](https://redfish.dmtf.org/schemas/v1/Bios.v1_0_2.json)
+* 
[BootOptionCollection](https://redfish.dmtf.org/schemas/BootOptionCollection.json)
+* [BootOption](https://redfish.dmtf.org/schemas/BootOption.v1_0_0.json)
+
+If any additional Redfish Schema or a new version of above Schemas are 
required to be supported, please send the email to edk2-devel mailing list by 
following [edk2-satging process](https://github.com/tianocore/edk2-staging).
+
+ Related Modules
+  The following modules are related to UEFI Redfish solution, **RedfishPkg** 
is the new package to support UEFI Redfish solution:
+  * **RedfishPkg\RestExDxe\RestExDxe.inf** - UEFI driver to enable 
standardized RESTful access to resources from UEFI environment.
+
+  * **RedfishPkg\Library\DxeRedfishLib** - Library to 
Create/Read/Update/Delete (CRUD) resources and provide basic query abilities by 
using