1. Patchew
  2. EDK2
  3. View series

[edk2-devel] [PATCH v2] BaseTools/VolInfo: Show encapsulation sections

Konstantin Aladyshev posted 1 patch 1 year, 9 months ago
Failed in applying to current master (apply log)
BaseTools/Source/C/VolInfo/VolInfo.c | 6 ++++++
1 file changed, 6 insertions(+)
[edk2-devel] [PATCH v2] BaseTools/VolInfo: Show encapsulation sections
Posted by Konstantin Aladyshev 1 year, 9 months ago 
Currently there is no labels for start and end of the encapsulation
sections. Therefore it is not possible to see where the encapsulation
section ends and another section starts.
Add labels for start and end of encapsulation sections to fix the
issue.

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

diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c b/BaseTools/Source/C/VolInfo/VolInfo.c
index c1f81f2dcb..ce1775f7fd 100644
--- a/BaseTools/Source/C/VolInfo/VolInfo.c
+++ b/BaseTools/Source/C/VolInfo/VolInfo.c
@@ -1903,7 +1903,9 @@ Returns:
         return EFI_SECTION_ERROR;
       }
 
+      printf ("/------------ Encapsulation section start -----------------\\\n");
       Status = ParseSection (UncompressedBuffer, UncompressedLength);
+      printf ("\\------------ Encapsulation section end -------------------/\n");
 
       if (CompressionType == EFI_STANDARD_COMPRESSION) {
         //
@@ -2022,6 +2024,7 @@ Returns:
           return EFI_SECTION_ERROR;
         }
 
+        printf ("/------------ Encapsulation section start -----------------\\\n");
         Status = ParseSection (
                   ToolOutputBuffer,
                   ToolOutputLength
@@ -2030,6 +2033,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of decoded GUIDED section failed", NULL);
           return EFI_SECTION_ERROR;
         }
+        printf ("\\------------ Encapsulation section end -------------------/\n");
 
       //
       // Check for CRC32 sections which we can handle internally if needed.
@@ -2042,6 +2046,7 @@ Returns:
         //
         // CRC32 guided section
         //
+        printf ("/------------ Encapsulation section start -----------------\\\n");
         Status = ParseSection (
                   SectionBuffer + DataOffset,
                   BufferLength - DataOffset
@@ -2050,6 +2055,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of CRC32 GUIDED section failed", NULL);
           return EFI_SECTION_ERROR;
         }
+        printf ("\\------------ Encapsulation section end -------------------/\n");
       } else {
         //
         // We don't know how to parse it now.
-- 
2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91528): https://edk2.groups.io/g/devel/message/91528
Mute This Topic: https://groups.io/mt/92481203/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] 回复: [PATCH v2] BaseTools/VolInfo: Show encapsulation sections
Posted by gaoliming via groups.io 1 year, 7 months ago 
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: Konstantin Aladyshev <aladyshev22@gmail.com>
> 发送时间: 2022年7月19日 20:29
> 收件人: devel@edk2.groups.io
> 抄送: bob.c.feng@intel.com; gaoliming@byosoft.com.cn;
> yuwei.chen@intel.com; Konstantin Aladyshev <aladyshev22@gmail.com>
> 主题: [PATCH v2] BaseTools/VolInfo: Show encapsulation sections
> 
> Currently there is no labels for start and end of the encapsulation
> sections. Therefore it is not possible to see where the encapsulation
> section ends and another section starts.
> Add labels for start and end of encapsulation sections to fix the
> issue.
> 
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> ---
>  BaseTools/Source/C/VolInfo/VolInfo.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c
> b/BaseTools/Source/C/VolInfo/VolInfo.c
> index c1f81f2dcb..ce1775f7fd 100644
> --- a/BaseTools/Source/C/VolInfo/VolInfo.c
> +++ b/BaseTools/Source/C/VolInfo/VolInfo.c
> @@ -1903,7 +1903,9 @@ Returns:
>          return EFI_SECTION_ERROR;
> 
>        }
> 
> 
> 
> +      printf ("/------------ Encapsulation section start
-----------------\\\n");
> 
>        Status = ParseSection (UncompressedBuffer, UncompressedLength);
> 
> +      printf ("\\------------ Encapsulation section end
-------------------/\n");
> 
> 
> 
>        if (CompressionType == EFI_STANDARD_COMPRESSION) {
> 
>          //
> 
> @@ -2022,6 +2024,7 @@ Returns:
>            return EFI_SECTION_ERROR;
> 
>          }
> 
> 
> 
> +        printf ("/------------ Encapsulation section start
> -----------------\\\n");
> 
>          Status = ParseSection (
> 
>                    ToolOutputBuffer,
> 
>                    ToolOutputLength
> 
> @@ -2030,6 +2033,7 @@ Returns:
>            Error (NULL, 0, 0003, "parse of decoded GUIDED section failed",
> NULL);
> 
>            return EFI_SECTION_ERROR;
> 
>          }
> 
> +        printf ("\\------------ Encapsulation section end
> -------------------/\n");
> 
> 
> 
>        //
> 
>        // Check for CRC32 sections which we can handle internally if
> needed.
> 
> @@ -2042,6 +2046,7 @@ Returns:
>          //
> 
>          // CRC32 guided section
> 
>          //
> 
> +        printf ("/------------ Encapsulation section start
> -----------------\\\n");
> 
>          Status = ParseSection (
> 
>                    SectionBuffer + DataOffset,
> 
>                    BufferLength - DataOffset
> 
> @@ -2050,6 +2055,7 @@ Returns:
>            Error (NULL, 0, 0003, "parse of CRC32 GUIDED section failed",
> NULL);
> 
>            return EFI_SECTION_ERROR;
> 
>          }
> 
> +        printf ("\\------------ Encapsulation section end
> -------------------/\n");
> 
>        } else {
> 
>          //
> 
>          // We don't know how to parse it now.
> 
> --
> 2.25.1





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

-----Original Message-----
From: Konstantin Aladyshev <aladyshev22@gmail.com> 
Sent: Tuesday, July 19, 2022 8:29 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 v2] BaseTools/VolInfo: Show encapsulation sections

Currently there is no labels for start and end of the encapsulation sections. Therefore it is not possible to see where the encapsulation section ends and another section starts.
Add labels for start and end of encapsulation sections to fix the issue.

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

diff --git a/BaseTools/Source/C/VolInfo/VolInfo.c b/BaseTools/Source/C/VolInfo/VolInfo.c
index c1f81f2dcb..ce1775f7fd 100644
--- a/BaseTools/Source/C/VolInfo/VolInfo.c
+++ b/BaseTools/Source/C/VolInfo/VolInfo.c
@@ -1903,7 +1903,9 @@ Returns:
         return EFI_SECTION_ERROR;       } +      printf ("/------------ Encapsulation section start -----------------\\\n");       Status = ParseSection (UncompressedBuffer, UncompressedLength);+      printf ("\\------------ Encapsulation section end -------------------/\n");        if (CompressionType == EFI_STANDARD_COMPRESSION) {         //@@ -2022,6 +2024,7 @@ Returns:
           return EFI_SECTION_ERROR;         } +        printf ("/------------ Encapsulation section start -----------------\\\n");         Status = ParseSection (                   ToolOutputBuffer,                   ToolOutputLength@@ -2030,6 +2033,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of decoded GUIDED section failed", NULL);           return EFI_SECTION_ERROR;         }+        printf ("\\------------ Encapsulation section end -------------------/\n");        //       // Check for CRC32 sections which we can handle internally if needed.@@ -2042,6 +2046,7 @@ Returns:
         //         // CRC32 guided section         //+        printf ("/------------ Encapsulation section start -----------------\\\n");         Status = ParseSection (                   SectionBuffer + DataOffset,                   BufferLength - DataOffset@@ -2050,6 +2055,7 @@ Returns:
           Error (NULL, 0, 0003, "parse of CRC32 GUIDED section failed", NULL);           return EFI_SECTION_ERROR;         }+        printf ("\\------------ Encapsulation section end -------------------/\n");       } else {         //         // We don't know how to parse it now.-- 
2.25.1



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

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.