[edk2-devel] [PATCH] BaseTools/GenSec: Support EFI_SECTION_FREEFORM_SUBTYPE_GUID sections

Konstantin Aladyshev posted 1 patch 1 year, 8 months ago
Failed in applying to current master (apply log)
BaseTools/Source/C/GenSec/GenSec.c | 171 ++++++++++++++++++++++++++++-
1 file changed, 169 insertions(+), 2 deletions(-)
[edk2-devel] [PATCH] BaseTools/GenSec: Support EFI_SECTION_FREEFORM_SUBTYPE_GUID sections
Posted by Konstantin Aladyshev 1 year, 8 months ago
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
 BaseTools/Source/C/GenSec/GenSec.c | 171 ++++++++++++++++++++++++++++-
 1 file changed, 169 insertions(+), 2 deletions(-)

diff --git a/BaseTools/Source/C/GenSec/GenSec.c b/BaseTools/Source/C/GenSec/GenSec.c
index 35a1ac64ea..d86cc197cc 100644
--- a/BaseTools/Source/C/GenSec/GenSec.c
+++ b/BaseTools/Source/C/GenSec/GenSec.c
@@ -988,6 +988,155 @@ Returns:
   return EFI_SUCCESS;
 }
 
+EFI_STATUS
+GenSectionSubtypeGuidSection (
+  CHAR8    **InputFileName,
+  UINT32   *InputFileAlign,
+  UINT32   InputFileNum,
+  EFI_GUID *SubTypeGuid,
+  UINT8    **OutFileBuffer
+  )
+/*++
+
+Routine Description:
+
+  Generate a section of type EFI_SECTION_FREEFORM_SUBTYPE_GUID
+  The function won't validate the input file contents.
+  The utility will add section header to the file.
+
+Arguments:
+
+  InputFileName - Name of the input file.
+
+  InputFileAlign - Alignment required by the input file data.
+
+  InputFileNum - Number of input files. Should be 1 for this section.
+
+  SubTypeGuid - Specify vendor guid value.
+
+  OutFileBuffer   - Buffer pointer to Output file contents
+
+Returns:
+
+  EFI_SUCCESS on successful return
+  EFI_INVALID_PARAMETER if InputFileNum is less than 1
+  EFI_ABORTED if unable to open input file.
+  EFI_OUT_OF_RESOURCES  No resource to complete the operation.
+
+--*/
+{
+  UINT32                TotalLength;
+  UINT32                InputLength;
+  UINT32                Offset;
+  UINT8                 *FileBuffer;
+  EFI_STATUS            Status;
+  EFI_FREEFORM_SUBTYPE_GUID_SECTION  *SubtypeGuidSect;
+  EFI_FREEFORM_SUBTYPE_GUID_SECTION2  *SubtypeGuidSect2;
+
+
+  InputLength = 0;
+  Offset      = 0;
+  FileBuffer  = NULL;
+  TotalLength = 0;
+
+  if (InputFileNum > 1) {
+    Error (NULL, 0, 2000, "Invalid parameter", "more than one input file specified");
+    return STATUS_ERROR;
+  } else if (InputFileNum < 1) {
+    Error (NULL, 0, 2000, "Invalid parameter", "no input file specified");
+    return STATUS_ERROR;
+  }
+
+  //
+  // read all input file contents into a buffer
+  // first get the size of all file contents
+  //
+  Status = GetSectionContents (
+            InputFileName,
+            InputFileAlign,
+            InputFileNum,
+            FileBuffer,
+            &InputLength
+            );
+
+  if (Status == EFI_BUFFER_TOO_SMALL) {
+    Offset = sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION);
+    if (InputLength + Offset >= MAX_SECTION_SIZE) {
+      Offset = sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION2);
+    }
+    TotalLength = InputLength + Offset;
+
+    FileBuffer = (UINT8 *) malloc (InputLength + Offset);
+    if (FileBuffer == NULL) {
+      Error (NULL, 0, 4001, "Resource", "memory cannot be allocated");
+      return EFI_OUT_OF_RESOURCES;
+    }
+    //
+    // read all input file contents into a buffer
+    //
+    Status = GetSectionContents (
+              InputFileName,
+              InputFileAlign,
+              InputFileNum,
+              FileBuffer + Offset,
+              &InputLength
+              );
+  }
+
+  if (EFI_ERROR (Status)) {
+    if (FileBuffer != NULL) {
+      free (FileBuffer);
+    }
+    Error (NULL, 0, 0001, "Error opening file for reading", InputFileName[0]);
+    return Status;
+  }
+
+  if (InputLength == 0) {
+    if (FileBuffer != NULL) {
+      free (FileBuffer);
+    }
+    Error (NULL, 0, 2000, "Invalid parameter", "the size of input file %s can't be zero", InputFileName);
+    return EFI_NOT_FOUND;
+  }
+
+  //
+  // InputLength != 0, but FileBuffer == NULL means out of resources.
+  //
+  if (FileBuffer == NULL) {
+    Error (NULL, 0, 4001, "Resource", "memory cannot be allocated");
+    return EFI_OUT_OF_RESOURCES;
+  }
+
+  //
+  // Now data is in FileBuffer + Offset
+  //
+  if (TotalLength >= MAX_SECTION_SIZE) {
+    SubtypeGuidSect2 = (EFI_FREEFORM_SUBTYPE_GUID_SECTION2 *) FileBuffer;
+    SubtypeGuidSect2->CommonHeader.Type     = EFI_SECTION_GUID_DEFINED;
+    SubtypeGuidSect2->CommonHeader.Size[0]  = (UINT8) 0xff;
+    SubtypeGuidSect2->CommonHeader.Size[1]  = (UINT8) 0xff;
+    SubtypeGuidSect2->CommonHeader.Size[2]  = (UINT8) 0xff;
+    SubtypeGuidSect2->CommonHeader.ExtendedSize = InputLength + sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION2);
+    memcpy (&(SubtypeGuidSect2->SubTypeGuid), SubTypeGuid, sizeof (EFI_GUID));
+  } else {
+    SubtypeGuidSect = (EFI_FREEFORM_SUBTYPE_GUID_SECTION *) FileBuffer;
+    SubtypeGuidSect->CommonHeader.Type     = EFI_SECTION_FREEFORM_SUBTYPE_GUID;
+    SubtypeGuidSect->CommonHeader.Size[0]  = (UINT8) (TotalLength & 0xff);
+    SubtypeGuidSect->CommonHeader.Size[1]  = (UINT8) ((TotalLength & 0xff00) >> 8);
+    SubtypeGuidSect->CommonHeader.Size[2]  = (UINT8) ((TotalLength & 0xff0000) >> 16);
+    memcpy (&(SubtypeGuidSect->SubTypeGuid), SubTypeGuid, sizeof (EFI_GUID));
+  }
+
+  VerboseMsg ("the size of the created section file is %u bytes", (unsigned) TotalLength);
+
+  //
+  // Set OutFileBuffer
+  //
+  *OutFileBuffer = FileBuffer;
+
+  return EFI_SUCCESS;
+}
+
 EFI_STATUS
 FfsRebaseImageRead (
     IN      VOID    *FileHandle,
@@ -1591,14 +1740,22 @@ Returns:
   }
 
   //
-  // GuidValue is only required by Guided section.
+  // GuidValue is only required by Guided section and SubtypeGuid section.
   //
-  if ((SectType != EFI_SECTION_GUID_DEFINED) &&
+  if ((SectType != EFI_SECTION_GUID_DEFINED) && (SectType != EFI_SECTION_FREEFORM_SUBTYPE_GUID) &&
     (SectionName != NULL) &&
     (CompareGuid (&VendorGuid, &mZeroGuid) != 0)) {
     fprintf (stdout, "Warning: the input guid value is not required for this section type %s\n", SectionName);
   }
 
+  //
+  // Check whether there is GUID for the SubtypeGuid section
+  //
+  if ((SectType == EFI_SECTION_FREEFORM_SUBTYPE_GUID) && (CompareGuid (&VendorGuid, &mZeroGuid) == 0)) {
+    Error (NULL, 0, 1001, "Missing options", "GUID");
+    goto Finish;
+  }
+
   //
   // Check whether there is input file
   //
@@ -1667,6 +1824,16 @@ Returns:
               );
     break;
 
+  case EFI_SECTION_FREEFORM_SUBTYPE_GUID:
+    Status = GenSectionSubtypeGuidSection (
+              InputFileName,
+              InputFileAlign,
+              InputFileNum,
+              &VendorGuid,
+              &OutFileBuffer
+              );
+    break;
+
   case EFI_SECTION_VERSION:
     Index           = sizeof (EFI_COMMON_SECTION_HEADER);
     //
-- 
2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91584): https://edk2.groups.io/g/devel/message/91584
Mute This Topic: https://groups.io/mt/92505133/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] BaseTools/GenSec: Support EFI_SECTION_FREEFORM_SUBTYPE_GUID sections
Posted by Bob Feng 1 year, 7 months ago
Reviewed-by: Bob Feng <bob.c.feng@intel.com>

-----Original Message-----
From: Konstantin Aladyshev <aladyshev22@gmail.com> 
Sent: Wednesday, July 20, 2022 10:08 PM
To: devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>; Konstantin Aladyshev <aladyshev22@gmail.com>
Subject: [PATCH] BaseTools/GenSec: Support EFI_SECTION_FREEFORM_SUBTYPE_GUID sections

Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
 BaseTools/Source/C/GenSec/GenSec.c | 171 ++++++++++++++++++++++++++++-
 1 file changed, 169 insertions(+), 2 deletions(-)

diff --git a/BaseTools/Source/C/GenSec/GenSec.c b/BaseTools/Source/C/GenSec/GenSec.c
index 35a1ac64ea..d86cc197cc 100644
--- a/BaseTools/Source/C/GenSec/GenSec.c
+++ b/BaseTools/Source/C/GenSec/GenSec.c
@@ -988,6 +988,155 @@ Returns:
   return EFI_SUCCESS;

 }

 

+EFI_STATUS

+GenSectionSubtypeGuidSection (

+  CHAR8    **InputFileName,

+  UINT32   *InputFileAlign,

+  UINT32   InputFileNum,

+  EFI_GUID *SubTypeGuid,

+  UINT8    **OutFileBuffer

+  )

+/*++

+

+Routine Description:

+

+  Generate a section of type EFI_SECTION_FREEFORM_SUBTYPE_GUID

+  The function won't validate the input file contents.

+  The utility will add section header to the file.

+

+Arguments:

+

+  InputFileName - Name of the input file.

+

+  InputFileAlign - Alignment required by the input file data.

+

+  InputFileNum - Number of input files. Should be 1 for this section.

+

+  SubTypeGuid - Specify vendor guid value.

+

+  OutFileBuffer   - Buffer pointer to Output file contents

+

+Returns:

+

+  EFI_SUCCESS on successful return

+  EFI_INVALID_PARAMETER if InputFileNum is less than 1

+  EFI_ABORTED if unable to open input file.

+  EFI_OUT_OF_RESOURCES  No resource to complete the operation.

+

+--*/

+{

+  UINT32                TotalLength;

+  UINT32                InputLength;

+  UINT32                Offset;

+  UINT8                 *FileBuffer;

+  EFI_STATUS            Status;

+  EFI_FREEFORM_SUBTYPE_GUID_SECTION  *SubtypeGuidSect;

+  EFI_FREEFORM_SUBTYPE_GUID_SECTION2  *SubtypeGuidSect2;

+

+

+  InputLength = 0;

+  Offset      = 0;

+  FileBuffer  = NULL;

+  TotalLength = 0;

+

+  if (InputFileNum > 1) {

+    Error (NULL, 0, 2000, "Invalid parameter", "more than one input file specified");

+    return STATUS_ERROR;

+  } else if (InputFileNum < 1) {

+    Error (NULL, 0, 2000, "Invalid parameter", "no input file specified");

+    return STATUS_ERROR;

+  }

+

+  //

+  // read all input file contents into a buffer

+  // first get the size of all file contents

+  //

+  Status = GetSectionContents (

+            InputFileName,

+            InputFileAlign,

+            InputFileNum,

+            FileBuffer,

+            &InputLength

+            );

+

+  if (Status == EFI_BUFFER_TOO_SMALL) {

+    Offset = sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION);

+    if (InputLength + Offset >= MAX_SECTION_SIZE) {

+      Offset = sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION2);

+    }

+    TotalLength = InputLength + Offset;

+

+    FileBuffer = (UINT8 *) malloc (InputLength + Offset);

+    if (FileBuffer == NULL) {

+      Error (NULL, 0, 4001, "Resource", "memory cannot be allocated");

+      return EFI_OUT_OF_RESOURCES;

+    }

+    //

+    // read all input file contents into a buffer

+    //

+    Status = GetSectionContents (

+              InputFileName,

+              InputFileAlign,

+              InputFileNum,

+              FileBuffer + Offset,

+              &InputLength

+              );

+  }

+

+  if (EFI_ERROR (Status)) {

+    if (FileBuffer != NULL) {

+      free (FileBuffer);

+    }

+    Error (NULL, 0, 0001, "Error opening file for reading", InputFileName[0]);

+    return Status;

+  }

+

+  if (InputLength == 0) {

+    if (FileBuffer != NULL) {

+      free (FileBuffer);

+    }

+    Error (NULL, 0, 2000, "Invalid parameter", "the size of input file %s can't be zero", InputFileName);

+    return EFI_NOT_FOUND;

+  }

+

+  //

+  // InputLength != 0, but FileBuffer == NULL means out of resources.

+  //

+  if (FileBuffer == NULL) {

+    Error (NULL, 0, 4001, "Resource", "memory cannot be allocated");

+    return EFI_OUT_OF_RESOURCES;

+  }

+

+  //

+  // Now data is in FileBuffer + Offset

+  //

+  if (TotalLength >= MAX_SECTION_SIZE) {

+    SubtypeGuidSect2 = (EFI_FREEFORM_SUBTYPE_GUID_SECTION2 *) FileBuffer;

+    SubtypeGuidSect2->CommonHeader.Type     = EFI_SECTION_GUID_DEFINED;

+    SubtypeGuidSect2->CommonHeader.Size[0]  = (UINT8) 0xff;

+    SubtypeGuidSect2->CommonHeader.Size[1]  = (UINT8) 0xff;

+    SubtypeGuidSect2->CommonHeader.Size[2]  = (UINT8) 0xff;

+    SubtypeGuidSect2->CommonHeader.ExtendedSize = InputLength + sizeof (EFI_FREEFORM_SUBTYPE_GUID_SECTION2);

+    memcpy (&(SubtypeGuidSect2->SubTypeGuid), SubTypeGuid, sizeof (EFI_GUID));

+  } else {

+    SubtypeGuidSect = (EFI_FREEFORM_SUBTYPE_GUID_SECTION *) FileBuffer;

+    SubtypeGuidSect->CommonHeader.Type     = EFI_SECTION_FREEFORM_SUBTYPE_GUID;

+    SubtypeGuidSect->CommonHeader.Size[0]  = (UINT8) (TotalLength & 0xff);

+    SubtypeGuidSect->CommonHeader.Size[1]  = (UINT8) ((TotalLength & 0xff00) >> 8);

+    SubtypeGuidSect->CommonHeader.Size[2]  = (UINT8) ((TotalLength & 0xff0000) >> 16);

+    memcpy (&(SubtypeGuidSect->SubTypeGuid), SubTypeGuid, sizeof (EFI_GUID));

+  }

+

+  VerboseMsg ("the size of the created section file is %u bytes", (unsigned) TotalLength);

+

+  //

+  // Set OutFileBuffer

+  //

+  *OutFileBuffer = FileBuffer;

+

+  return EFI_SUCCESS;

+}

+

 EFI_STATUS

 FfsRebaseImageRead (

     IN      VOID    *FileHandle,

@@ -1591,14 +1740,22 @@ Returns:
   }

 

   //

-  // GuidValue is only required by Guided section.

+  // GuidValue is only required by Guided section and SubtypeGuid section.

   //

-  if ((SectType != EFI_SECTION_GUID_DEFINED) &&

+  if ((SectType != EFI_SECTION_GUID_DEFINED) && (SectType != EFI_SECTION_FREEFORM_SUBTYPE_GUID) &&

     (SectionName != NULL) &&

     (CompareGuid (&VendorGuid, &mZeroGuid) != 0)) {

     fprintf (stdout, "Warning: the input guid value is not required for this section type %s\n", SectionName);

   }

 

+  //

+  // Check whether there is GUID for the SubtypeGuid section

+  //

+  if ((SectType == EFI_SECTION_FREEFORM_SUBTYPE_GUID) && (CompareGuid (&VendorGuid, &mZeroGuid) == 0)) {

+    Error (NULL, 0, 1001, "Missing options", "GUID");

+    goto Finish;

+  }

+

   //

   // Check whether there is input file

   //

@@ -1667,6 +1824,16 @@ Returns:
               );

     break;

 

+  case EFI_SECTION_FREEFORM_SUBTYPE_GUID:

+    Status = GenSectionSubtypeGuidSection (

+              InputFileName,

+              InputFileAlign,

+              InputFileNum,

+              &VendorGuid,

+              &OutFileBuffer

+              );

+    break;

+

   case EFI_SECTION_VERSION:

     Index           = sizeof (EFI_COMMON_SECTION_HEADER);

     //

-- 
2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92117): https://edk2.groups.io/g/devel/message/92117
Mute This Topic: https://groups.io/mt/92505133/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-