Re: [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

[Paulo Alcantara <pcacjr@zytor.com>]

On 09/09/2017 21:12, Laszlo Ersek wrote:
> Repo:   https://github.com/lersek/edk2.git
> Branch: udf_fixes_cleanups
> 
> Patches #2, #3 and #4 are needed (and enough) for me to build OVMF for
> IA32 and X64 with clang-3.8, after the UDF introduction.
> 
> Patches #1 and #5 are cleanups that I felt fit before patch #2 and after
> patch #4, respectively.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Paulo Alcantara <pcacjr@zytor.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (5):
>    MdeModulePkg/UdfDxe: ASSERT() valid ReadFileInfo Flags for INLINE_DATA
>      req
>    MdeModulePkg/UdfDxe: don't return unset Status if INLINE_DATA req
>      succeeds
>    MdeModulePkg/UdfDxe: replace zero-init of local variables with
>      ZeroMem()
>    MdeModulePkg/PartitionDxe: don't divide 64-bit values with C operators
>    MdeModulePkg/PartitionDxe: remove always false comparison
> 
>   MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c            | 9 +++++++--
>   MdeModulePkg/Universal/Disk/UdfDxe/File.c                 | 6 ++++--
>   MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c | 5 +++++
>   3 files changed, 16 insertions(+), 4 deletions(-)
> 

Looks good to me. Tested it with OVMF X64 and AARCH64.

Reviewed-by: Paulo Alcantara <pcacjr@zytor.com>

Thanks!
Paulo