From: "Gao, Liming" <liming.gao@intel.com>
To: "Ni, Ruiyu" <ruiyu.ni@intel.com>,
Paulo Alcantara <pcacjr@zytor.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: Laszlo Ersek <lersek@redhat.com>,
"Justen, Jordan L" <jordan.l.justen@intel.com>,
Andrew Fish <afish@apple.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
"Zeng, Star" <star.zeng@intel.com>,
"Dong, Eric" <eric.dong@intel.com>,
"Doran, Mark" <mark.doran@intel.com>,
"Wu, Hao A" <hao.a.wu@intel.com>
Subject: Re: [PATCH v2 0/6] read-only UDF file system support
Date: Mon, 21 Aug 2017 02:37:03 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14D7738EA@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <734D49CCEBEEF84792F5B80ED585239D5B9F3CD8@SHSMSX104.ccr.corp.intel.com>
I suggest to run PatchCheck tool first. It is mentioned in https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process
>-----Original Message-----
>From: Ni, Ruiyu
>Sent: Monday, August 21, 2017 10:29 AM
>To: Paulo Alcantara <pcacjr@zytor.com>; edk2-devel@lists.01.org
>Cc: Laszlo Ersek <lersek@redhat.com>; Justen, Jordan L
><jordan.l.justen@intel.com>; Andrew Fish <afish@apple.com>; Kinney,
>Michael D <michael.d.kinney@intel.com>; Gao, Liming
><liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Dong, Eric
><eric.dong@intel.com>; Doran, Mark <mark.doran@intel.com>; Wu, Hao A
><hao.a.wu@intel.com>
>Subject: RE: [PATCH v2 0/6] read-only UDF file system support
>
>Paulo,
>1. Could you please run the ECC check tool (BaseTools\Source\Python\Ecc\)
> "CRC" might need to be replaced with "Crc".
> I also noticed some TAB key in file content.
>
>2. Your current implementation uses HARD_DRIVE_DEVICE_PATH.
> But with:
> SignatureType = SIGNATURE_TYPE_UDF
> MBRType = MBR_TYPE_PCAT
> Signature = *
> And later UdfDxe driver checks the SignatureType and MBRType.
> I am not sure if it would be better to put the definitions in UEFI Spec,
> since they are referenced by different modules.
> I also noticed you use PARTITION_TYPE_OTHER for PartitionInfo.
> When proposing to UEFI Spec, this also needs to be considered,
> for example, add PARTITION_TYPE_UDF to spec.
>
>3. The driver model part looks good.
>
>Regards,
>Ray
>
>>-----Original Message-----
>>From: Paulo Alcantara [mailto:pcacjr@zytor.com]
>>Sent: Monday, August 21, 2017 2:16 AM
>>To: edk2-devel@lists.01.org
>>Cc: Paulo Alcantara <pcacjr@zytor.com>; Laszlo Ersek <lersek@redhat.com>;
>Justen, Jordan L <jordan.l.justen@intel.com>;
>>Andrew Fish <afish@apple.com>; Kinney, Michael D
><michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>;
>>Zeng, Star <star.zeng@intel.com>; Dong, Eric <eric.dong@intel.com>; Doran,
>Mark <mark.doran@intel.com>; Ni, Ruiyu
>><ruiyu.ni@intel.com>; Wu, Hao A <hao.a.wu@intel.com>
>>Subject: [PATCH v2 0/6] read-only UDF file system support
>>
>>Hi,
>>
>>This series introduces read-only UDF file system support in EDK2. As
>>Laszlo (or Red Hat) seemed to be interested in such support, I'm posting
>>it again after ~3 years.
>>
>>The idea is not replacing the default FAT file system, nor breaking any
>>existing file system support, but extending EDK2 with a new file system
>>that might be useful for some people who are looking for specific file
>>system features that current FAT doesn't support.
>>
>>Originally the driver was written to support UDF file systems as
>>specified by OSTA Universal Disk Format Specification 2.60. However,
>>some Windows 10 Enterprise ISO (UDF bridge) images that I tested
>>supported a revision of 1.02 thus I had to rework the driver a little
>>bit to support such revision as well.
>>
>>v2:
>> - Rework to _partially_ support UDF revisions <2.60.
>> - Use existing CDROM_VOLUME_DESCRIPTOR structure defined in Eltorito.h
>> instead of creating another one (UDF_VOLUME_DESCRIPTOR).
>> - Fixed UdfDxe to correctly follow UEFI driver model.
>> - Use HARDDRIVE_DEVICE_PATH instead of a vendor-defined one.
>> - Detect UDF file systems only in PartitionDxe, and let UdfDxe driver
>> check for specific UDF device path to decide whether or not install
>> SimpleFs protocol.
>> - Place MdePkg changes in a separate patch.
>>
>>Repo: https://github.com/pcacjr/edk2.git
>>Branch: udf-fs
>>
>>Cc: Laszlo Ersek <lersek@redhat.com>
>>Cc: Jordan Justen <jordan.l.justen@intel.com>
>>Cc: Andrew Fish <afish@apple.com>
>>Cc: Michael D Kinney <michael.d.kinney@intel.com>
>>Cc: Liming Gao <liming.gao@intel.com>
>>Cc: Star Zeng <star.zeng@intel.com>
>>Cc: Eric Dong <eric.dong@intel.com>
>>Cc: Mark Doran <mark.doran@intel.com>
>>Cc: Ruiyu Ni <ruiyu.ni@intel.com>
>>Cc: hao.a.wu@intel.com
>>Contributed-under: TianoCore Contribution Agreement 1.1
>>Signed-off-by: Paulo Alcantara <pcacjr@zytor.com>
>>---
>>
>>Paulo Alcantara (6):
>> MdePkg: Add UDF volume structure definitions
>> MdeModulePkg/PartitionDxe: Add UDF file system support
>> MdeModulePkg: Initial UDF/ECMA-167 file system support
>> OvmfPkg: Enable UDF file system support
>> ArmVirtPkg: Enable UDF file system support
>> Nt32Pkg: Enable UDF file system support
>>
>> ArmVirtPkg/ArmVirtQemu.dsc | 3 +-
>> ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 3 +-
>> ArmVirtPkg/ArmVirtQemuKernel.dsc | 3 +-
>> ArmVirtPkg/ArmVirtXen.dsc | 3 +-
>> ArmVirtPkg/ArmVirtXen.fdf | 1 +
>> .../Universal/Disk/PartitionDxe/Partition.c | 9 +-
>> .../Universal/Disk/PartitionDxe/Partition.h | 32 +-
>> .../Universal/Disk/PartitionDxe/PartitionDxe.inf | 3 +-
>> MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c | 307 +++
>> MdeModulePkg/Universal/Disk/UdfDxe/ComponentName.c | 185 ++
>> MdeModulePkg/Universal/Disk/UdfDxe/File.c | 901 ++++++++
>> MdeModulePkg/Universal/Disk/UdfDxe/FileName.c | 195 ++
>> .../Universal/Disk/UdfDxe/FileSystemOperations.c | 2375
>++++++++++++++++++++
>> MdeModulePkg/Universal/Disk/UdfDxe/Udf.c | 415 ++++
>> MdeModulePkg/Universal/Disk/UdfDxe/Udf.h | 1246 ++++++++++
>> MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf | 66 +
>> MdePkg/Include/IndustryStandard/Udf.h | 60 +
>> Nt32Pkg/Nt32Pkg.dsc | 1 +
>> Nt32Pkg/Nt32Pkg.fdf | 1 +
>> OvmfPkg/OvmfPkgIa32.dsc | 1 +
>> OvmfPkg/OvmfPkgIa32.fdf | 1 +
>> OvmfPkg/OvmfPkgIa32X64.dsc | 1 +
>> OvmfPkg/OvmfPkgIa32X64.fdf | 1 +
>> OvmfPkg/OvmfPkgX64.dsc | 1 +
>> OvmfPkg/OvmfPkgX64.fdf | 1 +
>> 25 files changed, 5804 insertions(+), 11 deletions(-)
>> create mode 100644 MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c
>> create mode 100644
>MdeModulePkg/Universal/Disk/UdfDxe/ComponentName.c
>> create mode 100644 MdeModulePkg/Universal/Disk/UdfDxe/File.c
>> create mode 100644 MdeModulePkg/Universal/Disk/UdfDxe/FileName.c
>> create mode 100644
>MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c
>> create mode 100644 MdeModulePkg/Universal/Disk/UdfDxe/Udf.c
>> create mode 100644 MdeModulePkg/Universal/Disk/UdfDxe/Udf.h
>> create mode 100644 MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf
>> create mode 100644 MdePkg/Include/IndustryStandard/Udf.h
>>
>>--
>>2.11.0
next prev parent reply other threads:[~2017-08-21 2:34 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-20 18:15 [PATCH v2 0/6] read-only UDF file system support Paulo Alcantara
2017-08-20 18:15 ` [PATCH v2 1/6] MdePkg: Add UDF volume structure definitions Paulo Alcantara
2017-08-20 18:15 ` [PATCH v2 2/6] MdeModulePkg/PartitionDxe: Add UDF file system support Paulo Alcantara
2017-08-20 18:15 ` [PATCH v2 3/6] MdeModulePkg: Initial UDF/ECMA-167 " Paulo Alcantara
2017-08-20 18:15 ` [PATCH v2 4/6] OvmfPkg: Enable UDF " Paulo Alcantara
2017-08-21 16:14 ` Laszlo Ersek
2017-08-20 18:15 ` [PATCH v2 5/6] ArmVirtPkg: " Paulo Alcantara
2017-08-21 15:35 ` Ard Biesheuvel
2017-08-21 16:11 ` Laszlo Ersek
2017-08-20 18:15 ` [PATCH v2 6/6] Nt32Pkg: " Paulo Alcantara
2017-08-21 2:29 ` [PATCH v2 0/6] read-only " Ni, Ruiyu
2017-08-21 2:37 ` Gao, Liming [this message]
2017-08-22 13:23 ` Paulo Alcantara
2017-08-22 13:14 ` Paulo Alcantara
2017-08-22 17:21 ` Andrew Fish
2017-08-22 17:56 ` Paulo Alcantara
2017-08-22 22:36 ` Andrew Fish
2017-08-22 17:58 ` Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14D7738EA@shsmsx102.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox