1. Patchew
  2. EDK2
  3. [edk2] [PATCH 0/4] {Mde, IntelFramework, Network, UefiCpu}Pkg: roundup for BZs 1074, 1090, 1092
  4. View patch

[edk2] [PATCH 2/4] IntelFrameworkPkg/FrameworkUefiLib: don't special-case EFI_FILE_MODE_CREATE

Laszlo Ersek posted 4 patches 4 years, 7 months ago
[edk2] [PATCH 2/4] IntelFrameworkPkg/FrameworkUefiLib: don't special-case EFI_FILE_MODE_CREATE
Posted by Laszlo Ersek 4 years, 7 months ago 
Synchronize EfiOpenFileByDevicePath() with the MdePkg/UefiLib instance, as
described in the previous patch.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Suggested-by: Ruiyu Ni <ruiyu.ni@intel.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1074
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
---

Notes:
    build-tested with:
    
    build -a IA32 -a X64 -b NOOPT -t GCC48 \
      -p IntelFrameworkPkg/IntelFrameworkPkg.dsc \
      -m IntelFrameworkPkg/Library/FrameworkUefiLib/FrameworkUefiLib.inf

 IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c | 40 +++-----------------
 1 file changed, 6 insertions(+), 34 deletions(-)

diff --git a/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c b/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
index b283d775b470..45bf4b17c5c3 100644
--- a/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
+++ b/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
@@ -1702,12 +1702,7 @@ EfiLocateProtocolBuffer (
 
   On the remaining device path, the longest initial sequence of
   FILEPATH_DEVICE_PATH nodes is node-wise traversed with
-  EFI_FILE_PROTOCOL.Open(). For the pathname fragment specified by each
-  traversed FILEPATH_DEVICE_PATH node, EfiOpenFileByDevicePath() first masks
-  EFI_FILE_MODE_CREATE out of OpenMode, and passes 0 for Attributes. If
-  EFI_FILE_PROTOCOL.Open() fails, and OpenMode includes EFI_FILE_MODE_CREATE,
-  then the operation is retried with the caller's OpenMode and Attributes
-  unmodified.
+  EFI_FILE_PROTOCOL.Open().
 
   (As a consequence, if OpenMode includes EFI_FILE_MODE_CREATE, and Attributes
   includes EFI_FILE_DIRECTORY, and each FILEPATH_DEVICE_PATH specifies a single
@@ -1739,18 +1734,10 @@ EfiLocateProtocolBuffer (
                            the last node in FilePath.
 
   @param[in] OpenMode      The OpenMode parameter to pass to
-                           EFI_FILE_PROTOCOL.Open(). For each
-                           FILEPATH_DEVICE_PATH node in FilePath,
-                           EfiOpenFileByDevicePath() first opens the specified
-                           pathname fragment with EFI_FILE_MODE_CREATE masked
-                           out of OpenMode and with Attributes set to 0, and
-                           only retries the operation with EFI_FILE_MODE_CREATE
-                           unmasked and Attributes propagated if the first open
-                           attempt fails.
+                           EFI_FILE_PROTOCOL.Open().
 
   @param[in] Attributes    The Attributes parameter to pass to
-                           EFI_FILE_PROTOCOL.Open(), when EFI_FILE_MODE_CREATE
-                           is propagated unmasked in OpenMode.
+                           EFI_FILE_PROTOCOL.Open().
 
   @retval EFI_SUCCESS            The file or directory has been opened or
                                  created.
@@ -1861,31 +1848,16 @@ EfiOpenFileByDevicePath (
     }
 
     //
-    // Open the next pathname fragment with EFI_FILE_MODE_CREATE masked out and
-    // with Attributes set to 0.
+    // Open or create the file corresponding to the next pathname fragment.
     //
     Status = LastFile->Open (
                          LastFile,
                          &NextFile,
                          PathName,
-                         OpenMode & ~(UINT64)EFI_FILE_MODE_CREATE,
-                         0
+                         OpenMode,
+                         Attributes
                          );
 
-    //
-    // Retry with EFI_FILE_MODE_CREATE and the original Attributes if the first
-    // attempt failed, and the caller specified EFI_FILE_MODE_CREATE.
-    //
-    if (EFI_ERROR (Status) && (OpenMode & EFI_FILE_MODE_CREATE) != 0) {
-      Status = LastFile->Open (
-                           LastFile,
-                           &NextFile,
-                           PathName,
-                           OpenMode,
-                           Attributes
-                           );
-    }
-
     //
     // Release any AlignedPathName on both error and success paths; PathName is
     // no longer needed.
-- 
2.14.1.3.gb7cf6e02401b


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 2/4] IntelFrameworkPkg/FrameworkUefiLib: don't special-case EFI_FILE_MODE_CREATE
Posted by Gao, Liming 4 years, 7 months ago 
Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Friday, August 17, 2018 7:36 AM
> To: edk2-devel-01 <edk2-devel@lists.01.org>
> Cc: Gao, Liming <liming.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>
> Subject: [PATCH 2/4] IntelFrameworkPkg/FrameworkUefiLib: don't special-case EFI_FILE_MODE_CREATE
> 
> Synchronize EfiOpenFileByDevicePath() with the MdePkg/UefiLib instance, as
> described in the previous patch.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Suggested-by: Ruiyu Ni <ruiyu.ni@intel.com>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1074
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Laszlo Ersek <lersek@redhat.com>
> ---
> 
> Notes:
>     build-tested with:
> 
>     build -a IA32 -a X64 -b NOOPT -t GCC48 \
>       -p IntelFrameworkPkg/IntelFrameworkPkg.dsc \
>       -m IntelFrameworkPkg/Library/FrameworkUefiLib/FrameworkUefiLib.inf
> 
>  IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c | 40 +++-----------------
>  1 file changed, 6 insertions(+), 34 deletions(-)
> 
> diff --git a/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c b/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
> index b283d775b470..45bf4b17c5c3 100644
> --- a/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
> +++ b/IntelFrameworkPkg/Library/FrameworkUefiLib/UefiLib.c
> @@ -1702,12 +1702,7 @@ EfiLocateProtocolBuffer (
> 
>    On the remaining device path, the longest initial sequence of
>    FILEPATH_DEVICE_PATH nodes is node-wise traversed with
> -  EFI_FILE_PROTOCOL.Open(). For the pathname fragment specified by each
> -  traversed FILEPATH_DEVICE_PATH node, EfiOpenFileByDevicePath() first masks
> -  EFI_FILE_MODE_CREATE out of OpenMode, and passes 0 for Attributes. If
> -  EFI_FILE_PROTOCOL.Open() fails, and OpenMode includes EFI_FILE_MODE_CREATE,
> -  then the operation is retried with the caller's OpenMode and Attributes
> -  unmodified.
> +  EFI_FILE_PROTOCOL.Open().
> 
>    (As a consequence, if OpenMode includes EFI_FILE_MODE_CREATE, and Attributes
>    includes EFI_FILE_DIRECTORY, and each FILEPATH_DEVICE_PATH specifies a single
> @@ -1739,18 +1734,10 @@ EfiLocateProtocolBuffer (
>                             the last node in FilePath.
> 
>    @param[in] OpenMode      The OpenMode parameter to pass to
> -                           EFI_FILE_PROTOCOL.Open(). For each
> -                           FILEPATH_DEVICE_PATH node in FilePath,
> -                           EfiOpenFileByDevicePath() first opens the specified
> -                           pathname fragment with EFI_FILE_MODE_CREATE masked
> -                           out of OpenMode and with Attributes set to 0, and
> -                           only retries the operation with EFI_FILE_MODE_CREATE
> -                           unmasked and Attributes propagated if the first open
> -                           attempt fails.
> +                           EFI_FILE_PROTOCOL.Open().
> 
>    @param[in] Attributes    The Attributes parameter to pass to
> -                           EFI_FILE_PROTOCOL.Open(), when EFI_FILE_MODE_CREATE
> -                           is propagated unmasked in OpenMode.
> +                           EFI_FILE_PROTOCOL.Open().
> 
>    @retval EFI_SUCCESS            The file or directory has been opened or
>                                   created.
> @@ -1861,31 +1848,16 @@ EfiOpenFileByDevicePath (
>      }
> 
>      //
> -    // Open the next pathname fragment with EFI_FILE_MODE_CREATE masked out and
> -    // with Attributes set to 0.
> +    // Open or create the file corresponding to the next pathname fragment.
>      //
>      Status = LastFile->Open (
>                           LastFile,
>                           &NextFile,
>                           PathName,
> -                         OpenMode & ~(UINT64)EFI_FILE_MODE_CREATE,
> -                         0
> +                         OpenMode,
> +                         Attributes
>                           );
> 
> -    //
> -    // Retry with EFI_FILE_MODE_CREATE and the original Attributes if the first
> -    // attempt failed, and the caller specified EFI_FILE_MODE_CREATE.
> -    //
> -    if (EFI_ERROR (Status) && (OpenMode & EFI_FILE_MODE_CREATE) != 0) {
> -      Status = LastFile->Open (
> -                           LastFile,
> -                           &NextFile,
> -                           PathName,
> -                           OpenMode,
> -                           Attributes
> -                           );
> -    }
> -
>      //
>      // Release any AlignedPathName on both error and success paths; PathName is
>      // no longer needed.
> --
> 2.14.1.3.gb7cf6e02401b
> 

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Patchew 2.1-devel-8c64711

© 2016 - 2023 Red Hat, Inc.