* [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID @ 2021-07-27 16:45 Jeff Brasen 2021-07-27 16:48 ` Ard Biesheuvel 0 siblings, 1 reply; 7+ messages in thread From: Jeff Brasen @ 2021-07-27 16:45 UTC (permalink / raw) To: devel Cc: ardb+tianocore, michael.d.kinney, jordan.l.justen, gaoliming, zhiguang.liu, Jeff Brasen Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, it can be used in a media device path to specify a Linux style initrd that can be loaded by the OS using the LoadFile2 protocol. Move these defines to MdePkg from OvmfPkg as these are relevant to non-OVMF targets as well. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 Signed-off-by: Jeff Brasen <jbrasen@nvidia.com> --- MdePkg/MdePkg.dec | 5 ++++ OvmfPkg/OvmfPkg.dec | 1 - MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ 4 files changed, 36 insertions(+), 18 deletions(-) create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index c5319fdd71ca..a28a2daaffa8 100644 --- a/MdePkg/MdePkg.dec +++ b/MdePkg/MdePkg.dec @@ -818,6 +818,11 @@ [Guids] # gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} + # + # GUID used to provide initrd to linux via LoadFile2 protocol + # + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} + [Guids.IA32, Guids.X64] ## Include/Guid/Cper.h gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C }} diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec index 6ae733f6e39f..3153f5ae4540 100644 --- a/OvmfPkg/OvmfPkg.dec +++ b/OvmfPkg/OvmfPkg.dec @@ -118,7 +118,6 @@ [Guids] gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, 0x4b}} gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, 0x50}} gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, 0x52}} - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, 0xc7}} gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, 0x07}} gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, 0x47}} diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h new file mode 100644 index 000000000000..0e7db8bd8140 --- /dev/null +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h @@ -0,0 +1,31 @@ +/** @file + GUID definition for the Linux Initrd media device path + + Linux distro boot generally relies on an initial ramdisk (initrd) + which is provided by the loader, and which contains additional kernel + modules (for storage and network, for instance), and the initial user + space startup code, i.e., the code which brings up the user space side + of the entire OS. + + In order to provide a standard method to locate this file, + the GUID defined in this file is used to describe the device path + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. + + The kernel EFI Stub will locate and use this instance to load the initrd, + therefore the firmware/loader should install an instance of this to load the + relevant initrd. + + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> + + SPDX-License-Identifier: BSD-2-Clause-Patent +**/ + +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ + +#define LINUX_EFI_INITRD_MEDIA_GUID \ + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} + +extern EFI_GUID gLinuxEfiInitrdMediaGuid; + +#endif diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h deleted file mode 100644 index 83fc3fc79aa6..000000000000 --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h +++ /dev/null @@ -1,17 +0,0 @@ -/** @file - GUID definition for the Linux Initrd media device path - - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> - - SPDX-License-Identifier: BSD-2-Clause-Patent -**/ - -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ - -#define LINUX_EFI_INITRD_MEDIA_GUID \ - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} - -extern EFI_GUID gLinuxEfiInitrdMediaGuid; - -#endif -- 2.25.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-07-27 16:45 [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID Jeff Brasen @ 2021-07-27 16:48 ` Ard Biesheuvel 2021-08-03 18:43 ` Jeff Brasen 0 siblings, 1 reply; 7+ messages in thread From: Ard Biesheuvel @ 2021-07-27 16:48 UTC (permalink / raw) To: Jeff Brasen Cc: edk2-devel-groups-io, Ard Biesheuvel, Michael Kinney, Jordan Justen, Liming Gao (Byosoft address), Zhiguang Liu On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <jbrasen@nvidia.com> wrote: > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > it can be used in a media device path to specify a Linux style initrd > that can be loaded by the OS using the LoadFile2 protocol. > > Move these defines to MdePkg from OvmfPkg as these are relevant to > non-OVMF targets as well. > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > Signed-off-by: Jeff Brasen <jbrasen@nvidia.com> Acked-by: Ard Biesheuvel <ardb@kernel.org> > --- > MdePkg/MdePkg.dec | 5 ++++ > OvmfPkg/OvmfPkg.dec | 1 - > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > 4 files changed, 36 insertions(+), 18 deletions(-) > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > index c5319fdd71ca..a28a2daaffa8 100644 > --- a/MdePkg/MdePkg.dec > +++ b/MdePkg/MdePkg.dec > @@ -818,6 +818,11 @@ [Guids] > # > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > + # > + # GUID used to provide initrd to linux via LoadFile2 protocol > + # > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > [Guids.IA32, Guids.X64] > ## Include/Guid/Cper.h > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C }} > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > index 6ae733f6e39f..3153f5ae4540 100644 > --- a/OvmfPkg/OvmfPkg.dec > +++ b/OvmfPkg/OvmfPkg.dec > @@ -118,7 +118,6 @@ [Guids] > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, 0x4b}} > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, 0x50}} > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, 0x52}} > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, 0xc7}} > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, 0x07}} > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, 0x47}} > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > new file mode 100644 > index 000000000000..0e7db8bd8140 > --- /dev/null > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > @@ -0,0 +1,31 @@ > +/** @file > + GUID definition for the Linux Initrd media device path > + > + Linux distro boot generally relies on an initial ramdisk (initrd) > + which is provided by the loader, and which contains additional kernel > + modules (for storage and network, for instance), and the initial user > + space startup code, i.e., the code which brings up the user space side > + of the entire OS. > + > + In order to provide a standard method to locate this file, > + the GUID defined in this file is used to describe the device path > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > + > + The kernel EFI Stub will locate and use this instance to load the initrd, > + therefore the firmware/loader should install an instance of this to load the > + relevant initrd. > + > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > + > + SPDX-License-Identifier: BSD-2-Clause-Patent > +**/ > + > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > + > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > + > +#endif > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > deleted file mode 100644 > index 83fc3fc79aa6..000000000000 > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > +++ /dev/null > @@ -1,17 +0,0 @@ > -/** @file > - GUID definition for the Linux Initrd media device path > - > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > - > - SPDX-License-Identifier: BSD-2-Clause-Patent > -**/ > - > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > - > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > - > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > - > -#endif > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-07-27 16:48 ` Ard Biesheuvel @ 2021-08-03 18:43 ` Jeff Brasen 2021-08-04 16:51 ` [edk2-devel] " Michael D Kinney 0 siblings, 1 reply; 7+ messages in thread From: Jeff Brasen @ 2021-08-03 18:43 UTC (permalink / raw) To: Ard Biesheuvel Cc: edk2-devel-groups-io, Ard Biesheuvel, Michael Kinney, Jordan Justen, Liming Gao (Byosoft address), Zhiguang Liu [-- Attachment #1: Type: text/plain, Size: 5750 bytes --] Mike, do the comments in this patch work for you? Would be nice to get this in prior to the 8/9 freeze if possible. Thanks, Jeff ________________________________ From: Ard Biesheuvel <ardb@kernel.org> Sent: Tuesday, July 27, 2021 10:48 AM To: Jeff Brasen <jbrasen@nvidia.com> Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Michael Kinney <michael.d.kinney@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <gaoliming@byosoft.com.cn>; Zhiguang Liu <zhiguang.liu@intel.com> Subject: Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID External email: Use caution opening links or attachments On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <jbrasen@nvidia.com> wrote: > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > it can be used in a media device path to specify a Linux style initrd > that can be loaded by the OS using the LoadFile2 protocol. > > Move these defines to MdePkg from OvmfPkg as these are relevant to > non-OVMF targets as well. > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > Signed-off-by: Jeff Brasen <jbrasen@nvidia.com> Acked-by: Ard Biesheuvel <ardb@kernel.org> > --- > MdePkg/MdePkg.dec | 5 ++++ > OvmfPkg/OvmfPkg.dec | 1 - > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > 4 files changed, 36 insertions(+), 18 deletions(-) > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > index c5319fdd71ca..a28a2daaffa8 100644 > --- a/MdePkg/MdePkg.dec > +++ b/MdePkg/MdePkg.dec > @@ -818,6 +818,11 @@ [Guids] > # > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > + # > + # GUID used to provide initrd to linux via LoadFile2 protocol > + # > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > [Guids.IA32, Guids.X64] > ## Include/Guid/Cper.h > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C }} > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > index 6ae733f6e39f..3153f5ae4540 100644 > --- a/OvmfPkg/OvmfPkg.dec > +++ b/OvmfPkg/OvmfPkg.dec > @@ -118,7 +118,6 @@ [Guids] > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, 0x4b}} > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, 0x50}} > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, 0x52}} > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, 0xc7}} > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, 0x07}} > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, 0x47}} > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > new file mode 100644 > index 000000000000..0e7db8bd8140 > --- /dev/null > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > @@ -0,0 +1,31 @@ > +/** @file > + GUID definition for the Linux Initrd media device path > + > + Linux distro boot generally relies on an initial ramdisk (initrd) > + which is provided by the loader, and which contains additional kernel > + modules (for storage and network, for instance), and the initial user > + space startup code, i.e., the code which brings up the user space side > + of the entire OS. > + > + In order to provide a standard method to locate this file, > + the GUID defined in this file is used to describe the device path > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > + > + The kernel EFI Stub will locate and use this instance to load the initrd, > + therefore the firmware/loader should install an instance of this to load the > + relevant initrd. > + > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > + > + SPDX-License-Identifier: BSD-2-Clause-Patent > +**/ > + > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > + > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > + > +#endif > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > deleted file mode 100644 > index 83fc3fc79aa6..000000000000 > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > +++ /dev/null > @@ -1,17 +0,0 @@ > -/** @file > - GUID definition for the Linux Initrd media device path > - > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > - > - SPDX-License-Identifier: BSD-2-Clause-Patent > -**/ > - > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > - > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > - > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > - > -#endif > -- > 2.25.1 > [-- Attachment #2: Type: text/html, Size: 8669 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-08-03 18:43 ` Jeff Brasen @ 2021-08-04 16:51 ` Michael D Kinney 2021-08-04 16:56 ` Ard Biesheuvel 0 siblings, 1 reply; 7+ messages in thread From: Michael D Kinney @ 2021-08-04 16:51 UTC (permalink / raw) To: devel@edk2.groups.io, jbrasen@nvidia.com, Ard Biesheuvel, Kinney, Michael D Cc: Ard Biesheuvel, Justen, Jordan L, Liming Gao (Byosoft address), Liu, Zhiguang Hi Jeff, Yes. This looks good. Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> Mike From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeff Brasen via groups.io Sent: Tuesday, August 3, 2021 11:44 AM To: Ard Biesheuvel <ardb@kernel.org> Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Kinney, Michael D <michael.d.kinney@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com> Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID Mike, do the comments in this patch work for you? Would be nice to get this in prior to the 8/9 freeze if possible. Thanks, Jeff ________________________________________ From: Ard Biesheuvel <mailto:ardb@kernel.org> Sent: Tuesday, July 27, 2021 10:48 AM To: Jeff Brasen <mailto:jbrasen@nvidia.com> Cc: edk2-devel-groups-io <mailto:devel@edk2.groups.io>; Ard Biesheuvel <mailto:ardb+tianocore@kernel.org>; Michael Kinney <mailto:michael.d.kinney@intel.com>; Jordan Justen <mailto:jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <mailto:gaoliming@byosoft.com.cn>; Zhiguang Liu <mailto:zhiguang.liu@intel.com> Subject: Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID External email: Use caution opening links or attachments On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <mailto:jbrasen@nvidia.com> wrote: > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > it can be used in a media device path to specify a Linux style initrd > that can be loaded by the OS using the LoadFile2 protocol. > > Move these defines to MdePkg from OvmfPkg as these are relevant to > non-OVMF targets as well. > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > Signed-off-by: Jeff Brasen <mailto:jbrasen@nvidia.com> Acked-by: Ard Biesheuvel <mailto:ardb@kernel.org> > --- > MdePkg/MdePkg.dec | 5 ++++ > OvmfPkg/OvmfPkg.dec | 1 - > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > 4 files changed, 36 insertions(+), 18 deletions(-) > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > index c5319fdd71ca..a28a2daaffa8 100644 > --- a/MdePkg/MdePkg.dec > +++ b/MdePkg/MdePkg.dec > @@ -818,6 +818,11 @@ [Guids] > # > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > + # > + # GUID used to provide initrd to linux via LoadFile2 protocol > + # > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > [Guids.IA32, Guids.X64] > ## Include/Guid/Cper.h > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C }} > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > index 6ae733f6e39f..3153f5ae4540 100644 > --- a/OvmfPkg/OvmfPkg.dec > +++ b/OvmfPkg/OvmfPkg.dec > @@ -118,7 +118,6 @@ [Guids] > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, 0x4b}} > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, 0x50}} > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, 0x52}} > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, 0xc7}} > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, 0x07}} > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, 0x47}} > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > new file mode 100644 > index 000000000000..0e7db8bd8140 > --- /dev/null > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > @@ -0,0 +1,31 @@ > +/** @file > + GUID definition for the Linux Initrd media device path > + > + Linux distro boot generally relies on an initial ramdisk (initrd) > + which is provided by the loader, and which contains additional kernel > + modules (for storage and network, for instance), and the initial user > + space startup code, i.e., the code which brings up the user space side > + of the entire OS. > + > + In order to provide a standard method to locate this file, > + the GUID defined in this file is used to describe the device path > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > + > + The kernel EFI Stub will locate and use this instance to load the initrd, > + therefore the firmware/loader should install an instance of this to load the > + relevant initrd. > + > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > + > + SPDX-License-Identifier: BSD-2-Clause-Patent > +**/ > + > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > + > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > + > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > + > +#endif > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > deleted file mode 100644 > index 83fc3fc79aa6..000000000000 > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > +++ /dev/null > @@ -1,17 +0,0 @@ > -/** @file > - GUID definition for the Linux Initrd media device path > - > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > - > - SPDX-License-Identifier: BSD-2-Clause-Patent > -**/ > - > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > - > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > - > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > - > -#endif > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-08-04 16:51 ` [edk2-devel] " Michael D Kinney @ 2021-08-04 16:56 ` Ard Biesheuvel 2021-08-04 18:39 ` Michael D Kinney 0 siblings, 1 reply; 7+ messages in thread From: Ard Biesheuvel @ 2021-08-04 16:56 UTC (permalink / raw) To: Kinney, Michael D Cc: devel@edk2.groups.io, jbrasen@nvidia.com, Ard Biesheuvel, Justen, Jordan L, Liming Gao (Byosoft address), Liu, Zhiguang On Wed, 4 Aug 2021 at 18:51, Kinney, Michael D <michael.d.kinney@intel.com> wrote: > > Hi Jeff, > > Yes. This looks good. > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> > Thanks Mike Unless anyone objects, I am going to merge this as a single patch, rather than suggest that adding to MdePkg and removing from OvmfPkg be done as two separate steps, the reason being that that would result in an intermediate state where the GUIDs and .h files exist in two places at the same time. > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeff Brasen via groups.io > Sent: Tuesday, August 3, 2021 11:44 AM > To: Ard Biesheuvel <ardb@kernel.org> > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Kinney, Michael D <michael.d.kinney@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com> > Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > Mike, do the comments in this patch work for you? Would be nice to get this in prior to the 8/9 freeze if possible. > > Thanks, > Jeff > ________________________________________ > From: Ard Biesheuvel <mailto:ardb@kernel.org> > Sent: Tuesday, July 27, 2021 10:48 AM > To: Jeff Brasen <mailto:jbrasen@nvidia.com> > Cc: edk2-devel-groups-io <mailto:devel@edk2.groups.io>; Ard Biesheuvel <mailto:ardb+tianocore@kernel.org>; Michael Kinney <mailto:michael.d.kinney@intel.com>; Jordan Justen <mailto:jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <mailto:gaoliming@byosoft.com.cn>; Zhiguang Liu <mailto:zhiguang.liu@intel.com> > Subject: Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > External email: Use caution opening links or attachments > > > On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <mailto:jbrasen@nvidia.com> wrote: > > > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > > it can be used in a media device path to specify a Linux style initrd > > that can be loaded by the OS using the LoadFile2 protocol. > > > > Move these defines to MdePkg from OvmfPkg as these are relevant to > > non-OVMF targets as well. > > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > > Signed-off-by: Jeff Brasen <mailto:jbrasen@nvidia.com> > > Acked-by: Ard Biesheuvel <mailto:ardb@kernel.org> > > > --- > > MdePkg/MdePkg.dec | 5 ++++ > > OvmfPkg/OvmfPkg.dec | 1 - > > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > > 4 files changed, 36 insertions(+), 18 deletions(-) > > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > > index c5319fdd71ca..a28a2daaffa8 100644 > > --- a/MdePkg/MdePkg.dec > > +++ b/MdePkg/MdePkg.dec > > @@ -818,6 +818,11 @@ [Guids] > > # > > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > > > + # > > + # GUID used to provide initrd to linux via LoadFile2 protocol > > + # > > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > + > > [Guids.IA32, Guids.X64] > > ## Include/Guid/Cper.h > > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C }} > > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > > index 6ae733f6e39f..3153f5ae4540 100644 > > --- a/OvmfPkg/OvmfPkg.dec > > +++ b/OvmfPkg/OvmfPkg.dec > > @@ -118,7 +118,6 @@ [Guids] > > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, 0x4b}} > > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, 0x50}} > > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, 0x52}} > > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, 0xc7}} > > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, 0x07}} > > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, 0x47}} > > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > new file mode 100644 > > index 000000000000..0e7db8bd8140 > > --- /dev/null > > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > @@ -0,0 +1,31 @@ > > +/** @file > > + GUID definition for the Linux Initrd media device path > > + > > + Linux distro boot generally relies on an initial ramdisk (initrd) > > + which is provided by the loader, and which contains additional kernel > > + modules (for storage and network, for instance), and the initial user > > + space startup code, i.e., the code which brings up the user space side > > + of the entire OS. > > + > > + In order to provide a standard method to locate this file, > > + the GUID defined in this file is used to describe the device path > > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > > + > > + The kernel EFI Stub will locate and use this instance to load the initrd, > > + therefore the firmware/loader should install an instance of this to load the > > + relevant initrd. > > + > > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > + > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > +**/ > > + > > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > + > > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > + > > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > + > > +#endif > > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > deleted file mode 100644 > > index 83fc3fc79aa6..000000000000 > > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > +++ /dev/null > > @@ -1,17 +0,0 @@ > > -/** @file > > - GUID definition for the Linux Initrd media device path > > - > > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > - > > - SPDX-License-Identifier: BSD-2-Clause-Patent > > -**/ > > - > > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > - > > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > - > > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > - > > -#endif > > -- > > 2.25.1 > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-08-04 16:56 ` Ard Biesheuvel @ 2021-08-04 18:39 ` Michael D Kinney 2021-08-04 21:01 ` Ard Biesheuvel 0 siblings, 1 reply; 7+ messages in thread From: Michael D Kinney @ 2021-08-04 18:39 UTC (permalink / raw) To: devel@edk2.groups.io, ardb@kernel.org, Kinney, Michael D Cc: jbrasen@nvidia.com, Ard Biesheuvel, Justen, Jordan L, Liming Gao (Byosoft address), Liu, Zhiguang Ard, The intermediate state should not cause any problems. I prefer single patches not modify more than one package. Mike > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ard Biesheuvel > Sent: Wednesday, August 4, 2021 9:56 AM > To: Kinney, Michael D <michael.d.kinney@intel.com> > Cc: devel@edk2.groups.io; jbrasen@nvidia.com; Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L > <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <gaoliming@byosoft.com.cn>; Liu, Zhiguang > <zhiguang.liu@intel.com> > Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > On Wed, 4 Aug 2021 at 18:51, Kinney, Michael D > <michael.d.kinney@intel.com> wrote: > > > > Hi Jeff, > > > > Yes. This looks good. > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> > > > > Thanks Mike > > Unless anyone objects, I am going to merge this as a single patch, > rather than suggest that adding to MdePkg and removing from OvmfPkg be > done as two separate steps, the reason being that that would result in > an intermediate state where the GUIDs and .h files exist in two places > at the same time. > > > > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeff Brasen via groups.io > > Sent: Tuesday, August 3, 2021 11:44 AM > > To: Ard Biesheuvel <ardb@kernel.org> > > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Kinney, Michael D > <michael.d.kinney@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com> > > Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > > > Mike, do the comments in this patch work for you? Would be nice to get this in prior to the 8/9 freeze if possible. > > > > Thanks, > > Jeff > > ________________________________________ > > From: Ard Biesheuvel <mailto:ardb@kernel.org> > > Sent: Tuesday, July 27, 2021 10:48 AM > > To: Jeff Brasen <mailto:jbrasen@nvidia.com> > > Cc: edk2-devel-groups-io <mailto:devel@edk2.groups.io>; Ard Biesheuvel <mailto:ardb+tianocore@kernel.org>; Michael > Kinney <mailto:michael.d.kinney@intel.com>; Jordan Justen <mailto:jordan.l.justen@intel.com>; Liming Gao (Byosoft address) > <mailto:gaoliming@byosoft.com.cn>; Zhiguang Liu <mailto:zhiguang.liu@intel.com> > > Subject: Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > > > External email: Use caution opening links or attachments > > > > > > On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <mailto:jbrasen@nvidia.com> wrote: > > > > > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > > > it can be used in a media device path to specify a Linux style initrd > > > that can be loaded by the OS using the LoadFile2 protocol. > > > > > > Move these defines to MdePkg from OvmfPkg as these are relevant to > > > non-OVMF targets as well. > > > > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > > > Signed-off-by: Jeff Brasen <mailto:jbrasen@nvidia.com> > > > > Acked-by: Ard Biesheuvel <mailto:ardb@kernel.org> > > > > > --- > > > MdePkg/MdePkg.dec | 5 ++++ > > > OvmfPkg/OvmfPkg.dec | 1 - > > > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > > > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > > > 4 files changed, 36 insertions(+), 18 deletions(-) > > > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > > > index c5319fdd71ca..a28a2daaffa8 100644 > > > --- a/MdePkg/MdePkg.dec > > > +++ b/MdePkg/MdePkg.dec > > > @@ -818,6 +818,11 @@ [Guids] > > > # > > > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > > > > > + # > > > + # GUID used to provide initrd to linux via LoadFile2 protocol > > > + # > > > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > + > > > [Guids.IA32, Guids.X64] > > > ## Include/Guid/Cper.h > > > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C > }} > > > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > > > index 6ae733f6e39f..3153f5ae4540 100644 > > > --- a/OvmfPkg/OvmfPkg.dec > > > +++ b/OvmfPkg/OvmfPkg.dec > > > @@ -118,7 +118,6 @@ [Guids] > > > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, > 0x4b}} > > > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, > 0x50}} > > > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, > 0x52}} > > > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, > 0x68}} > > > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, > 0xc7}} > > > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, > 0x07}} > > > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, > 0x47}} > > > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > new file mode 100644 > > > index 000000000000..0e7db8bd8140 > > > --- /dev/null > > > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > @@ -0,0 +1,31 @@ > > > +/** @file > > > + GUID definition for the Linux Initrd media device path > > > + > > > + Linux distro boot generally relies on an initial ramdisk (initrd) > > > + which is provided by the loader, and which contains additional kernel > > > + modules (for storage and network, for instance), and the initial user > > > + space startup code, i.e., the code which brings up the user space side > > > + of the entire OS. > > > + > > > + In order to provide a standard method to locate this file, > > > + the GUID defined in this file is used to describe the device path > > > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > > > + > > > + The kernel EFI Stub will locate and use this instance to load the initrd, > > > + therefore the firmware/loader should install an instance of this to load the > > > + relevant initrd. > > > + > > > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > > + > > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > > +**/ > > > + > > > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > + > > > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > > > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > + > > > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > > + > > > +#endif > > > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > deleted file mode 100644 > > > index 83fc3fc79aa6..000000000000 > > > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > +++ /dev/null > > > @@ -1,17 +0,0 @@ > > > -/** @file > > > - GUID definition for the Linux Initrd media device path > > > - > > > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > > - > > > - SPDX-License-Identifier: BSD-2-Clause-Patent > > > -**/ > > > - > > > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > - > > > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > > > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > - > > > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > > - > > > -#endif > > > -- > > > 2.25.1 > > > > > > > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID 2021-08-04 18:39 ` Michael D Kinney @ 2021-08-04 21:01 ` Ard Biesheuvel 0 siblings, 0 replies; 7+ messages in thread From: Ard Biesheuvel @ 2021-08-04 21:01 UTC (permalink / raw) To: Kinney, Michael D Cc: devel@edk2.groups.io, jbrasen@nvidia.com, Ard Biesheuvel, Justen, Jordan L, Liming Gao (Byosoft address), Liu, Zhiguang On Wed, 4 Aug 2021 at 20:39, Kinney, Michael D <michael.d.kinney@intel.com> wrote: > > Ard, > > The intermediate state should not cause any problems. > > I prefer single patches not modify more than one package. > Fair enough. Jeff: mind splitting this into two patches, one against MdePkg and one against OvmfPkg? You can carry over the reviewed-bys and apply them to both. > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ard Biesheuvel > > Sent: Wednesday, August 4, 2021 9:56 AM > > To: Kinney, Michael D <michael.d.kinney@intel.com> > > Cc: devel@edk2.groups.io; jbrasen@nvidia.com; Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L > > <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) <gaoliming@byosoft.com.cn>; Liu, Zhiguang > > <zhiguang.liu@intel.com> > > Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > > > On Wed, 4 Aug 2021 at 18:51, Kinney, Michael D > > <michael.d.kinney@intel.com> wrote: > > > > > > Hi Jeff, > > > > > > Yes. This looks good. > > > > > > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> > > > > > > > Thanks Mike > > > > Unless anyone objects, I am going to merge this as a single patch, > > rather than suggest that adding to MdePkg and removing from OvmfPkg be > > done as two separate steps, the reason being that that would result in > > an intermediate state where the GUIDs and .h files exist in two places > > at the same time. > > > > > > > > > > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeff Brasen via groups.io > > > Sent: Tuesday, August 3, 2021 11:44 AM > > > To: Ard Biesheuvel <ardb@kernel.org> > > > Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Ard Biesheuvel <ardb+tianocore@kernel.org>; Kinney, Michael D > > <michael.d.kinney@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Liming Gao (Byosoft address) > > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com> > > > Subject: Re: [edk2-devel] [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > > > > > Mike, do the comments in this patch work for you? Would be nice to get this in prior to the 8/9 freeze if possible. > > > > > > Thanks, > > > Jeff > > > ________________________________________ > > > From: Ard Biesheuvel <mailto:ardb@kernel.org> > > > Sent: Tuesday, July 27, 2021 10:48 AM > > > To: Jeff Brasen <mailto:jbrasen@nvidia.com> > > > Cc: edk2-devel-groups-io <mailto:devel@edk2.groups.io>; Ard Biesheuvel <mailto:ardb+tianocore@kernel.org>; Michael > > Kinney <mailto:michael.d.kinney@intel.com>; Jordan Justen <mailto:jordan.l.justen@intel.com>; Liming Gao (Byosoft address) > > <mailto:gaoliming@byosoft.com.cn>; Zhiguang Liu <mailto:zhiguang.liu@intel.com> > > > Subject: Re: [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID > > > > > > External email: Use caution opening links or attachments > > > > > > > > > On Tue, 27 Jul 2021 at 18:45, Jeff Brasen <mailto:jbrasen@nvidia.com> wrote: > > > > > > > > Add LINUX_EFI_INITRD_MEDIA_GUID to our collection of GUID definitions, > > > > it can be used in a media device path to specify a Linux style initrd > > > > that can be loaded by the OS using the LoadFile2 protocol. > > > > > > > > Move these defines to MdePkg from OvmfPkg as these are relevant to > > > > non-OVMF targets as well. > > > > > > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2564 > > > > Signed-off-by: Jeff Brasen <mailto:jbrasen@nvidia.com> > > > > > > Acked-by: Ard Biesheuvel <mailto:ardb@kernel.org> > > > > > > > --- > > > > MdePkg/MdePkg.dec | 5 ++++ > > > > OvmfPkg/OvmfPkg.dec | 1 - > > > > MdePkg/Include/Guid/LinuxEfiInitrdMedia.h | 31 ++++++++++++++++++++++ > > > > OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h | 17 ------------ > > > > 4 files changed, 36 insertions(+), 18 deletions(-) > > > > create mode 100644 MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > delete mode 100644 OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec > > > > index c5319fdd71ca..a28a2daaffa8 100644 > > > > --- a/MdePkg/MdePkg.dec > > > > +++ b/MdePkg/MdePkg.dec > > > > @@ -818,6 +818,11 @@ [Guids] > > > > # > > > > gTianoCustomDecompressGuid = { 0xA31280AD, 0x481E, 0x41B6, { 0x95, 0xE8, 0x12, 0x7F, 0x4C, 0x98, 0x47, 0x79 }} > > > > > > > > + # > > > > + # GUID used to provide initrd to linux via LoadFile2 protocol > > > > + # > > > > + gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > > + > > > > [Guids.IA32, Guids.X64] > > > > ## Include/Guid/Cper.h > > > > gEfiIa32X64ErrorTypeCacheCheckGuid = { 0xA55701F5, 0xE3EF, 0x43de, { 0xAC, 0x72, 0x24, 0x9B, 0x57, 0x3F, 0xAD, 0x2C > > }} > > > > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > > > > index 6ae733f6e39f..3153f5ae4540 100644 > > > > --- a/OvmfPkg/OvmfPkg.dec > > > > +++ b/OvmfPkg/OvmfPkg.dec > > > > @@ -118,7 +118,6 @@ [Guids] > > > > gMicrosoftVendorGuid = {0x77fa9abd, 0x0359, 0x4d32, {0xbd, 0x60, 0x28, 0xf4, 0xe7, 0x8f, 0x78, > > 0x4b}} > > > > gEfiLegacyBiosGuid = {0x2E3044AC, 0x879F, 0x490F, {0x97, 0x60, 0xBB, 0xDF, 0xAF, 0x69, 0x5F, > > 0x50}} > > > > gEfiLegacyDevOrderVariableGuid = {0xa56074db, 0x65fe, 0x45f7, {0xbd, 0x21, 0x2d, 0x2b, 0xdd, 0x8e, 0x96, > > 0x52}} > > > > - gLinuxEfiInitrdMediaGuid = {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, > > 0x68}} > > > > gQemuKernelLoaderFsMediaGuid = {0x1428f772, 0xb64a, 0x441e, {0xb8, 0xc3, 0x9e, 0xbd, 0xd7, 0xf8, 0x93, > > 0xc7}} > > > > gGrubFileGuid = {0xb5ae312c, 0xbc8a, 0x43b1, {0x9c, 0x62, 0xeb, 0xb8, 0x26, 0xdd, 0x5d, > > 0x07}} > > > > gConfidentialComputingSecretGuid = {0xadf956ad, 0xe98c, 0x484c, {0xae, 0x11, 0xb5, 0x1c, 0x7d, 0x33, 0x64, > > 0x47}} > > > > diff --git a/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > new file mode 100644 > > > > index 000000000000..0e7db8bd8140 > > > > --- /dev/null > > > > +++ b/MdePkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > @@ -0,0 +1,31 @@ > > > > +/** @file > > > > + GUID definition for the Linux Initrd media device path > > > > + > > > > + Linux distro boot generally relies on an initial ramdisk (initrd) > > > > + which is provided by the loader, and which contains additional kernel > > > > + modules (for storage and network, for instance), and the initial user > > > > + space startup code, i.e., the code which brings up the user space side > > > > + of the entire OS. > > > > + > > > > + In order to provide a standard method to locate this file, > > > > + the GUID defined in this file is used to describe the device path > > > > + for a LoadFile2 Protocol instance that is responsible for loading the initrd file. > > > > + > > > > + The kernel EFI Stub will locate and use this instance to load the initrd, > > > > + therefore the firmware/loader should install an instance of this to load the > > > > + relevant initrd. > > > > + > > > > + Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > > > + > > > > + SPDX-License-Identifier: BSD-2-Clause-Patent > > > > +**/ > > > > + > > > > +#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > > +#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > > + > > > > +#define LINUX_EFI_INITRD_MEDIA_GUID \ > > > > + {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > > + > > > > +extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > > > + > > > > +#endif > > > > diff --git a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h b/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > deleted file mode 100644 > > > > index 83fc3fc79aa6..000000000000 > > > > --- a/OvmfPkg/Include/Guid/LinuxEfiInitrdMedia.h > > > > +++ /dev/null > > > > @@ -1,17 +0,0 @@ > > > > -/** @file > > > > - GUID definition for the Linux Initrd media device path > > > > - > > > > - Copyright (c) 2020, Arm, Ltd. All rights reserved.<BR> > > > > - > > > > - SPDX-License-Identifier: BSD-2-Clause-Patent > > > > -**/ > > > > - > > > > -#ifndef LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > > -#define LINUX_EFI_INITRD_MEDIA_GUID_H__ > > > > - > > > > -#define LINUX_EFI_INITRD_MEDIA_GUID \ > > > > - {0x5568e427, 0x68fc, 0x4f3d, {0xac, 0x74, 0xca, 0x55, 0x52, 0x31, 0xcc, 0x68}} > > > > - > > > > -extern EFI_GUID gLinuxEfiInitrdMediaGuid; > > > > - > > > > -#endif > > > > -- > > > > 2.25.1 > > > > > > > > > > > > > > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-08-04 21:01 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-07-27 16:45 [PATCH v2 1/1] MdePkg: add definition of LINUX_EFI_INITRD_MEDIA_GUID Jeff Brasen 2021-07-27 16:48 ` Ard Biesheuvel 2021-08-03 18:43 ` Jeff Brasen 2021-08-04 16:51 ` [edk2-devel] " Michael D Kinney 2021-08-04 16:56 ` Ard Biesheuvel 2021-08-04 18:39 ` Michael D Kinney 2021-08-04 21:01 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox