* [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. @ 2018-01-23 20:03 Supreeth Venkatesh 2018-01-24 0:34 ` Marvin H?user 0 siblings, 1 reply; 7+ messages in thread From: Supreeth Venkatesh @ 2018-01-23 20:03 UTC (permalink / raw) To: edk2-devel; +Cc: michael.d.kinney, liming.gao, Supreeth Venkatesh, Achin Gupta As per PI specification v1.6, The following new file types are added: EFI_FV_FILETYPE_MM_STANDALONE EFI_FV_FILETYPE_MM_CORE_STANDALONE The following new section type is added: EFI_SECTION_MM_DEPEX This patch adds the management mode FV file type and depex. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Achin Gupta <achin.gupta@arm.com> Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> --- MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h b/MdePkg/Include/Pi/PiFirmwareFile.h index b982c9eda3..6086d1bb2f 100644 --- a/MdePkg/Include/Pi/PiFirmwareFile.h +++ b/MdePkg/Include/Pi/PiFirmwareFile.h @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 #define EFI_FV_FILETYPE_APPLICATION 0x09 #define EFI_FV_FILETYPE_SMM 0x0A +#define EFI_FV_FILETYPE_MM 0x0A #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C #define EFI_FV_FILETYPE_SMM_CORE 0x0D +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E #define EFI_FV_FILETYPE_OEM_MIN 0xc0 #define EFI_FV_FILETYPE_OEM_MAX 0xdf #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; #define EFI_SECTION_RAW 0x19 #define EFI_SECTION_PEI_DEPEX 0x1B #define EFI_SECTION_SMM_DEPEX 0x1C +#define EFI_SECTION_MM_DEPEX 0x1C /// /// Common section header. -- 2.14.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-23 20:03 [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex Supreeth Venkatesh @ 2018-01-24 0:34 ` Marvin H?user 2018-01-25 17:45 ` Supreeth Venkatesh 0 siblings, 1 reply; 7+ messages in thread From: Marvin H?user @ 2018-01-24 0:34 UTC (permalink / raw) To: edk2-devel@lists.01.org Cc: Supreeth Venkatesh, michael.d.kinney@intel.com, liming.gao@intel.com Good day, I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" (0x0C). Furthermore, may I suggest changing the *_SMM_* definitions to be defined via the *_MM_* definitions? Best regards, Marvin. > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Supreeth Venkatesh > Sent: Tuesday, January 23, 2018 9:03 PM > To: edk2-devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV > file type and depex. > > As per PI specification v1.6, > The following new file types are added: > EFI_FV_FILETYPE_MM_STANDALONE > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > The following new section type is added: > EFI_SECTION_MM_DEPEX > > This patch adds the management mode FV file type and depex. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > --- > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > b/MdePkg/Include/Pi/PiFirmwareFile.h > index b982c9eda3..6086d1bb2f 100644 > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > #define EFI_FV_FILETYPE_APPLICATION 0x09 > #define EFI_FV_FILETYPE_SMM 0x0A > +#define EFI_FV_FILETYPE_MM 0x0A > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > #define EFI_SECTION_RAW 0x19 > #define EFI_SECTION_PEI_DEPEX 0x1B > #define EFI_SECTION_SMM_DEPEX 0x1C > +#define EFI_SECTION_MM_DEPEX 0x1C > > /// > /// Common section header. > -- > 2.14.1 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-24 0:34 ` Marvin H?user @ 2018-01-25 17:45 ` Supreeth Venkatesh 2018-01-25 17:52 ` Tim Lewis 0 siblings, 1 reply; 7+ messages in thread From: Supreeth Venkatesh @ 2018-01-25 17:45 UTC (permalink / raw) To: Marvin H?user, edk2-devel@lists.01.org Cc: michael.d.kinney@intel.com, liming.gao@intel.com Marvin, Thanks for your comments. As per PI v1.6 specification, EFI_FV_FILETYPE_MM_CORE value is 0x0D (MM Foundation that support MM Traditional Mode.) This is traditional MM mode, which ARM is not supporting at this point. We are more interested in MM_CORE_STANDALONE mode. However, I have no issues in adding this in the patch, but would prefer if this is added when MM traditional mode is supported in ARM. Further, w.r.t defining *_SMM_* definitions via the *_MM_* definitions. I don't want to break backwards compatibility with existing SMM traditional mode implementations. I will be happy to let folks who have migrated to _MM_ definitions from _SMM_ definitions to send the patch across. Thanks, Supreeth -----Original Message----- From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] Sent: Tuesday, January 23, 2018 6:34 PM To: edk2-devel@lists.01.org Cc: Supreeth Venkatesh <Supreeth.Venkatesh@arm.com>; michael.d.kinney@intel.com; liming.gao@intel.com Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. Good day, I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" (0x0C). Furthermore, may I suggest changing the *_SMM_* definitions to be defined via the *_MM_* definitions? Best regards, Marvin. > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Supreeth Venkatesh > Sent: Tuesday, January 23, 2018 9:03 PM > To: edk2-devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV > file type and depex. > > As per PI specification v1.6, > The following new file types are added: > EFI_FV_FILETYPE_MM_STANDALONE > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > The following new section type is added: > EFI_SECTION_MM_DEPEX > > This patch adds the management mode FV file type and depex. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > --- > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > b/MdePkg/Include/Pi/PiFirmwareFile.h > index b982c9eda3..6086d1bb2f 100644 > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > #define EFI_FV_FILETYPE_APPLICATION 0x09 > #define EFI_FV_FILETYPE_SMM 0x0A > +#define EFI_FV_FILETYPE_MM 0x0A > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > #define EFI_SECTION_RAW 0x19 > #define EFI_SECTION_PEI_DEPEX 0x1B > #define EFI_SECTION_SMM_DEPEX 0x1C > +#define EFI_SECTION_MM_DEPEX 0x1C > > /// > /// Common section header. > -- > 2.14.1 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-25 17:45 ` Supreeth Venkatesh @ 2018-01-25 17:52 ` Tim Lewis 2018-01-25 18:31 ` Marvin H?user 0 siblings, 1 reply; 7+ messages in thread From: Tim Lewis @ 2018-01-25 17:52 UTC (permalink / raw) To: 'Supreeth Venkatesh', 'Marvin H?user', edk2-devel Cc: michael.d.kinney, liming.gao Supreeth -- Doesn't Appendix A of the PI 1.6 define this as: #define EFI_FV_FILETYPE_SMM EFI_FV_FILETYPE_MM #define EFI_FV_FILETYPE_SMM_CORE EFI_FV_FILETYPE_MM_CORE Thanks, Tim -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Supreeth Venkatesh Sent: Thursday, January 25, 2018 9:45 AM To: Marvin H?user <Marvin.Haeuser@outlook.com>; edk2-devel@lists.01.org Cc: michael.d.kinney@intel.com; liming.gao@intel.com Subject: Re: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. Marvin, Thanks for your comments. As per PI v1.6 specification, EFI_FV_FILETYPE_MM_CORE value is 0x0D (MM Foundation that support MM Traditional Mode.) This is traditional MM mode, which ARM is not supporting at this point. We are more interested in MM_CORE_STANDALONE mode. However, I have no issues in adding this in the patch, but would prefer if this is added when MM traditional mode is supported in ARM. Further, w.r.t defining *_SMM_* definitions via the *_MM_* definitions. I don't want to break backwards compatibility with existing SMM traditional mode implementations. I will be happy to let folks who have migrated to _MM_ definitions from _SMM_ definitions to send the patch across. Thanks, Supreeth -----Original Message----- From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] Sent: Tuesday, January 23, 2018 6:34 PM To: edk2-devel@lists.01.org Cc: Supreeth Venkatesh <Supreeth.Venkatesh@arm.com>; michael.d.kinney@intel.com; liming.gao@intel.com Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. Good day, I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" (0x0C). Furthermore, may I suggest changing the *_SMM_* definitions to be defined via the *_MM_* definitions? Best regards, Marvin. > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Supreeth Venkatesh > Sent: Tuesday, January 23, 2018 9:03 PM > To: edk2-devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV > file type and depex. > > As per PI specification v1.6, > The following new file types are added: > EFI_FV_FILETYPE_MM_STANDALONE > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > The following new section type is added: > EFI_SECTION_MM_DEPEX > > This patch adds the management mode FV file type and depex. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > --- > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > b/MdePkg/Include/Pi/PiFirmwareFile.h > index b982c9eda3..6086d1bb2f 100644 > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > #define EFI_FV_FILETYPE_APPLICATION 0x09 > #define EFI_FV_FILETYPE_SMM 0x0A > +#define EFI_FV_FILETYPE_MM 0x0A > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > #define EFI_SECTION_RAW 0x19 > #define EFI_SECTION_PEI_DEPEX 0x1B > #define EFI_SECTION_SMM_DEPEX 0x1C > +#define EFI_SECTION_MM_DEPEX 0x1C > > /// > /// Common section header. > -- > 2.14.1 > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-25 17:52 ` Tim Lewis @ 2018-01-25 18:31 ` Marvin H?user 2018-01-25 21:20 ` Supreeth Venkatesh 0 siblings, 1 reply; 7+ messages in thread From: Marvin H?user @ 2018-01-25 18:31 UTC (permalink / raw) To: edk2-devel@lists.01.org, Tim Lewis, Supreeth.Venkatesh@arm.com Cc: michael.d.kinney@intel.com, liming.gao@intel.com Hey Tim and Supreeth, Sorry, 0x0C was a typo, 0x0D is the correct one. The values of the SMM and MM constants are identical, this is just a naming update plus the introduction of the new value, MM Standalone Core, so backwards-compatibility is given. Regarding the traditional MM Core, I just assumed it was an oversight as MdePkg is generic. If it wasn't, sorry. Thanks, Marvin > -----Original Message----- > From: Tim Lewis [mailto:tim.lewis@insyde.com] > Sent: Thursday, January 25, 2018 6:53 PM > To: 'Supreeth Venkatesh' <Supreeth.Venkatesh@arm.com>; 'Marvin H?user' > <Marvin.Haeuser@outlook.com>; edk2-devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV file type and depex. > > Supreeth -- > > Doesn't Appendix A of the PI 1.6 define this as: > > #define EFI_FV_FILETYPE_SMM EFI_FV_FILETYPE_MM #define > EFI_FV_FILETYPE_SMM_CORE EFI_FV_FILETYPE_MM_CORE > > Thanks, > > Tim > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Supreeth Venkatesh > Sent: Thursday, January 25, 2018 9:45 AM > To: Marvin H?user <Marvin.Haeuser@outlook.com>; edk2- > devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: Re: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management > mode FV file type and depex. > > Marvin, > > Thanks for your comments. > As per PI v1.6 specification, > EFI_FV_FILETYPE_MM_CORE value is 0x0D (MM Foundation that support > MM Traditional Mode.) > > This is traditional MM mode, which ARM is not supporting at this point. We > are more interested in MM_CORE_STANDALONE mode. > However, I have no issues in adding this in the patch, but would prefer if this > is added when MM traditional mode is supported in ARM. > > Further, > w.r.t defining *_SMM_* definitions via the *_MM_* definitions. I don't > want to break backwards compatibility with existing SMM traditional mode > implementations. > I will be happy to let folks who have migrated to _MM_ definitions from > _SMM_ definitions to send the patch across. > > Thanks, > Supreeth > > -----Original Message----- > From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] > Sent: Tuesday, January 23, 2018 6:34 PM > To: edk2-devel@lists.01.org > Cc: Supreeth Venkatesh <Supreeth.Venkatesh@arm.com>; > michael.d.kinney@intel.com; liming.gao@intel.com > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV file type and depex. > > Good day, > > I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" > (0x0C). > Furthermore, may I suggest changing the *_SMM_* definitions to be > defined via the *_MM_* definitions? > > Best regards, > Marvin. > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Supreeth Venkatesh > > Sent: Tuesday, January 23, 2018 9:03 PM > > To: edk2-devel@lists.01.org > > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV > > file type and depex. > > > > As per PI specification v1.6, > > The following new file types are added: > > EFI_FV_FILETYPE_MM_STANDALONE > > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > > > The following new section type is added: > > EFI_SECTION_MM_DEPEX > > > > This patch adds the management mode FV file type and depex. > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > > --- > > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > > b/MdePkg/Include/Pi/PiFirmwareFile.h > > index b982c9eda3..6086d1bb2f 100644 > > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > > #define EFI_FV_FILETYPE_APPLICATION 0x09 > > #define EFI_FV_FILETYPE_SMM 0x0A > > +#define EFI_FV_FILETYPE_MM 0x0A > > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > > #define EFI_SECTION_RAW 0x19 > > #define EFI_SECTION_PEI_DEPEX 0x1B > > #define EFI_SECTION_SMM_DEPEX 0x1C > > +#define EFI_SECTION_MM_DEPEX 0x1C > > > > /// > > /// Common section header. > > -- > > 2.14.1 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended recipient, > please notify the sender immediately and do not disclose the contents to any > other person, use it for any purpose, or store or copy the information in any > medium. Thank you. > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-25 18:31 ` Marvin H?user @ 2018-01-25 21:20 ` Supreeth Venkatesh 2018-01-29 14:55 ` Gao, Liming 0 siblings, 1 reply; 7+ messages in thread From: Supreeth Venkatesh @ 2018-01-25 21:20 UTC (permalink / raw) To: Marvin H?user, edk2-devel@lists.01.org, tim.lewis@insyde.com Cc: michael.d.kinney@intel.com, liming.gao@intel.com Marvin/Tim, Thanks. Apologies, if I didn't mention that this patch was just addition of new values EFI_FV_FILETYPE_MM_STANDALONE EFI_FV_FILETYPE_MM_CORE_STANDALONE and EFI_SECTION_MM_DEPEX No other changes. In existing code, EFI_FV_FILETYPE_SMM and EFI_FV_FILETYPE_MM are same already. Supreeth -----Original Message----- From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] Sent: Thursday, January 25, 2018 12:32 PM To: edk2-devel@lists.01.org; tim.lewis@insyde.com; Supreeth Venkatesh <Supreeth.Venkatesh@arm.com> Cc: michael.d.kinney@intel.com; liming.gao@intel.com Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. Hey Tim and Supreeth, Sorry, 0x0C was a typo, 0x0D is the correct one. The values of the SMM and MM constants are identical, this is just a naming update plus the introduction of the new value, MM Standalone Core, so backwards-compatibility is given. Regarding the traditional MM Core, I just assumed it was an oversight as MdePkg is generic. If it wasn't, sorry. Thanks, Marvin > -----Original Message----- > From: Tim Lewis [mailto:tim.lewis@insyde.com] > Sent: Thursday, January 25, 2018 6:53 PM > To: 'Supreeth Venkatesh' <Supreeth.Venkatesh@arm.com>; 'Marvin H?user' > <Marvin.Haeuser@outlook.com>; edk2-devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV file type and depex. > > Supreeth -- > > Doesn't Appendix A of the PI 1.6 define this as: > > #define EFI_FV_FILETYPE_SMM EFI_FV_FILETYPE_MM #define > EFI_FV_FILETYPE_SMM_CORE EFI_FV_FILETYPE_MM_CORE > > Thanks, > > Tim > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Supreeth Venkatesh > Sent: Thursday, January 25, 2018 9:45 AM > To: Marvin H?user <Marvin.Haeuser@outlook.com>; edk2- > devel@lists.01.org > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: Re: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV file type and depex. > > Marvin, > > Thanks for your comments. > As per PI v1.6 specification, > EFI_FV_FILETYPE_MM_CORE value is 0x0D (MM Foundation that support MM > Traditional Mode.) > > This is traditional MM mode, which ARM is not supporting at this > point. We are more interested in MM_CORE_STANDALONE mode. > However, I have no issues in adding this in the patch, but would > prefer if this is added when MM traditional mode is supported in ARM. > > Further, > w.r.t defining *_SMM_* definitions via the *_MM_* definitions. I don't > want to break backwards compatibility with existing SMM traditional > mode implementations. > I will be happy to let folks who have migrated to _MM_ definitions > from _SMM_ definitions to send the patch across. > > Thanks, > Supreeth > > -----Original Message----- > From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] > Sent: Tuesday, January 23, 2018 6:34 PM > To: edk2-devel@lists.01.org > Cc: Supreeth Venkatesh <Supreeth.Venkatesh@arm.com>; > michael.d.kinney@intel.com; liming.gao@intel.com > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV file type and depex. > > Good day, > > I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" > (0x0C). > Furthermore, may I suggest changing the *_SMM_* definitions to be > defined via the *_MM_* definitions? > > Best regards, > Marvin. > > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf > > Of Supreeth Venkatesh > > Sent: Tuesday, January 23, 2018 9:03 PM > > To: edk2-devel@lists.01.org > > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > FV > > file type and depex. > > > > As per PI specification v1.6, > > The following new file types are added: > > EFI_FV_FILETYPE_MM_STANDALONE > > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > > > The following new section type is added: > > EFI_SECTION_MM_DEPEX > > > > This patch adds the management mode FV file type and depex. > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > > --- > > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > > b/MdePkg/Include/Pi/PiFirmwareFile.h > > index b982c9eda3..6086d1bb2f 100644 > > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > > #define EFI_FV_FILETYPE_APPLICATION 0x09 > > #define EFI_FV_FILETYPE_SMM 0x0A > > +#define EFI_FV_FILETYPE_MM 0x0A > > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > > #define EFI_SECTION_RAW 0x19 > > #define EFI_SECTION_PEI_DEPEX 0x1B > > #define EFI_SECTION_SMM_DEPEX 0x1C > > +#define EFI_SECTION_MM_DEPEX 0x1C > > > > /// > > /// Common section header. > > -- > > 2.14.1 > > > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, please notify the sender immediately and do not disclose > the contents to any other person, use it for any purpose, or store or > copy the information in any medium. Thank you. > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. 2018-01-25 21:20 ` Supreeth Venkatesh @ 2018-01-29 14:55 ` Gao, Liming 0 siblings, 0 replies; 7+ messages in thread From: Gao, Liming @ 2018-01-29 14:55 UTC (permalink / raw) To: Supreeth Venkatesh, Marvin H?user, edk2-devel@lists.01.org, tim.lewis@insyde.com Cc: Kinney, Michael D Supreeth: I suggest to add all MM type and section definition in MdePkg. They are not many. After they are added, SMM definition can be defined to MM value. Thanks Liming > -----Original Message----- > From: Supreeth Venkatesh [mailto:Supreeth.Venkatesh@arm.com] > Sent: Friday, January 26, 2018 5:21 AM > To: Marvin H?user <Marvin.Haeuser@outlook.com>; edk2-devel@lists.01.org; tim.lewis@insyde.com > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com> > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. > > Marvin/Tim, > > Thanks. > Apologies, if I didn't mention that this patch was just addition of new values > EFI_FV_FILETYPE_MM_STANDALONE > EFI_FV_FILETYPE_MM_CORE_STANDALONE > and > EFI_SECTION_MM_DEPEX > > No other changes. > In existing code, EFI_FV_FILETYPE_SMM and EFI_FV_FILETYPE_MM are same already. > > Supreeth > -----Original Message----- > From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] > Sent: Thursday, January 25, 2018 12:32 PM > To: edk2-devel@lists.01.org; tim.lewis@insyde.com; Supreeth Venkatesh <Supreeth.Venkatesh@arm.com> > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex. > > Hey Tim and Supreeth, > > Sorry, 0x0C was a typo, 0x0D is the correct one. > The values of the SMM and MM constants are identical, this is just a naming update plus the introduction of the new value, MM > Standalone Core, so backwards-compatibility is given. > Regarding the traditional MM Core, I just assumed it was an oversight as MdePkg is generic. If it wasn't, sorry. > > Thanks, > Marvin > > > -----Original Message----- > > From: Tim Lewis [mailto:tim.lewis@insyde.com] > > Sent: Thursday, January 25, 2018 6:53 PM > > To: 'Supreeth Venkatesh' <Supreeth.Venkatesh@arm.com>; 'Marvin H?user' > > <Marvin.Haeuser@outlook.com>; edk2-devel@lists.01.org > > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > > FV file type and depex. > > > > Supreeth -- > > > > Doesn't Appendix A of the PI 1.6 define this as: > > > > #define EFI_FV_FILETYPE_SMM EFI_FV_FILETYPE_MM #define > > EFI_FV_FILETYPE_SMM_CORE EFI_FV_FILETYPE_MM_CORE > > > > Thanks, > > > > Tim > > -----Original Message----- > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > > Supreeth Venkatesh > > Sent: Thursday, January 25, 2018 9:45 AM > > To: Marvin H?user <Marvin.Haeuser@outlook.com>; edk2- > > devel@lists.01.org > > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > > Subject: Re: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > > FV file type and depex. > > > > Marvin, > > > > Thanks for your comments. > > As per PI v1.6 specification, > > EFI_FV_FILETYPE_MM_CORE value is 0x0D (MM Foundation that support MM > > Traditional Mode.) > > > > This is traditional MM mode, which ARM is not supporting at this > > point. We are more interested in MM_CORE_STANDALONE mode. > > However, I have no issues in adding this in the patch, but would > > prefer if this is added when MM traditional mode is supported in ARM. > > > > Further, > > w.r.t defining *_SMM_* definitions via the *_MM_* definitions. I don't > > want to break backwards compatibility with existing SMM traditional > > mode implementations. > > I will be happy to let folks who have migrated to _MM_ definitions > > from _SMM_ definitions to send the patch across. > > > > Thanks, > > Supreeth > > > > -----Original Message----- > > From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com] > > Sent: Tuesday, January 23, 2018 6:34 PM > > To: edk2-devel@lists.01.org > > Cc: Supreeth Venkatesh <Supreeth.Venkatesh@arm.com>; > > michael.d.kinney@intel.com; liming.gao@intel.com > > Subject: RE: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > > FV file type and depex. > > > > Good day, > > > > I noticed this patch lacks the definition of "EFI_FV_FILETYPE_MM_CORE" > > (0x0C). > > Furthermore, may I suggest changing the *_SMM_* definitions to be > > defined via the *_MM_* definitions? > > > > Best regards, > > Marvin. > > > > > -----Original Message----- > > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf > > > Of Supreeth Venkatesh > > > Sent: Tuesday, January 23, 2018 9:03 PM > > > To: edk2-devel@lists.01.org > > > Cc: michael.d.kinney@intel.com; liming.gao@intel.com > > > Subject: [edk2] [PATCH v1 1/1] MdePkg/Include: Add management mode > > FV > > > file type and depex. > > > > > > As per PI specification v1.6, > > > The following new file types are added: > > > EFI_FV_FILETYPE_MM_STANDALONE > > > EFI_FV_FILETYPE_MM_CORE_STANDALONE > > > > > > The following new section type is added: > > > EFI_SECTION_MM_DEPEX > > > > > > This patch adds the management mode FV file type and depex. > > > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > > Signed-off-by: Achin Gupta <achin.gupta@arm.com> > > > Signed-off-by: Supreeth Venkatesh <supreeth.venkatesh@arm.com> > > > Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com> > > > --- > > > MdePkg/Include/Pi/PiFirmwareFile.h | 4 ++++ > > > 1 file changed, 4 insertions(+) > > > > > > diff --git a/MdePkg/Include/Pi/PiFirmwareFile.h > > > b/MdePkg/Include/Pi/PiFirmwareFile.h > > > index b982c9eda3..6086d1bb2f 100644 > > > --- a/MdePkg/Include/Pi/PiFirmwareFile.h > > > +++ b/MdePkg/Include/Pi/PiFirmwareFile.h > > > @@ -72,9 +72,12 @@ typedef UINT8 EFI_FFS_FILE_STATE; #define > > > EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER 0x08 > > > #define EFI_FV_FILETYPE_APPLICATION 0x09 > > > #define EFI_FV_FILETYPE_SMM 0x0A > > > +#define EFI_FV_FILETYPE_MM 0x0A > > > #define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B > > > #define EFI_FV_FILETYPE_COMBINED_SMM_DXE 0x0C > > > +#define EFI_FV_FILETYPE_COMBINED_MM_DXE 0x0C > > > #define EFI_FV_FILETYPE_SMM_CORE 0x0D > > > +#define EFI_FV_FILETYPE_MM_STANDALONE 0x0E > > > #define EFI_FV_FILETYPE_OEM_MIN 0xc0 > > > #define EFI_FV_FILETYPE_OEM_MAX 0xdf > > > #define EFI_FV_FILETYPE_DEBUG_MIN 0xe0 > > > @@ -218,6 +221,7 @@ typedef UINT8 EFI_SECTION_TYPE; > > > #define EFI_SECTION_RAW 0x19 > > > #define EFI_SECTION_PEI_DEPEX 0x1B > > > #define EFI_SECTION_SMM_DEPEX 0x1C > > > +#define EFI_SECTION_MM_DEPEX 0x1C > > > > > > /// > > > /// Common section header. > > > -- > > > 2.14.1 > > > > > > _______________________________________________ > > > edk2-devel mailing list > > > edk2-devel@lists.01.org > > > https://lists.01.org/mailman/listinfo/edk2-devel > > IMPORTANT NOTICE: The contents of this email and any attachments are > > confidential and may also be privileged. If you are not the intended > > recipient, please notify the sender immediately and do not disclose > > the contents to any other person, use it for any purpose, or store or > > copy the information in any medium. Thank you. > > _______________________________________________ > > edk2-devel mailing list > > edk2-devel@lists.01.org > > https://lists.01.org/mailman/listinfo/edk2-devel > > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the > intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, > or store or copy the information in any medium. Thank you. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-01-29 14:50 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-01-23 20:03 [PATCH v1 1/1] MdePkg/Include: Add management mode FV file type and depex Supreeth Venkatesh 2018-01-24 0:34 ` Marvin H?user 2018-01-25 17:45 ` Supreeth Venkatesh 2018-01-25 17:52 ` Tim Lewis 2018-01-25 18:31 ` Marvin H?user 2018-01-25 21:20 ` Supreeth Venkatesh 2018-01-29 14:55 ` Gao, Liming
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox