Paulo, Partition driver owner is reviewing your patch, will get back to you when he have feedback.
>From my side, a straightforward issue about device path was raised before. I >think you need propose a formal UDF device path to UEFI group and after it >gets adopted by UEFI then your patch can get approval for integration. Thanks Feng -----Original Message----- From: Paulo Alcantara [mailto:pca...@zytor.com] Sent: Thursday, January 22, 2015 00:56 To: Paulo Alcantara Cc: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [PATCH 0/3] UDF/ECMA-167 file system support On Mon, December 29, 2014 9:25 am, Paulo Alcantara wrote: > Hi, > > This patchset contains a read-only file system driver for UDF/ECMA-167 > volumes and revision 2.00 or higher is the only supported. > > I initially thought to add support to older revisions as well, however > I don't think it's strongly necessary to keep backward compatibility > with them since we already have a problem with restrict sizes on > current flash memories and newer revisions are OK, IMHO. Besides, I > intend - in the near future - to add write support to it, so it will > be become certaintly bigger than it currently is. > > Since I don't have a machine with UEFI at home for testing, all tests > I made were under QEMU + OVMF on Linux, so it would be really > appreciable if someone could test it on a real machine as well. The > tool I used to format my USB stick as UDF was "mkudffs". > > If you want to test with OVMF, make sure you add > "gEfiMdeModulePkgTokenSpaceGuid.PcdUdfFileSystemSupport|TRUE" and > "MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf" to the appropiate DSC > and FDF files. > > Notes > ===== > 1- Once this series is applied, I'll send another patchset > (as requested by Laszlo) that enables it in OVMF and probably in > ArmPlatformPkg/ArmVirtualizationPkg as well. > 2- Due to problems ocurred on lists.sourceforge.net in the last few > days, > please ignore the last patchsets and consider this one. > > git repo: > > git://git.zytor.com/users/pcacjr/edk2.git (branch: for-upstream) > > Thanks, > > Paulo > > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Paulo Alcantara <pca...@zytor.com> > > -- > > Paulo Alcantara (3): > MdeModulePkg/PartitionDxe: Add UDF/ECMA-167 file system support > MdeModulePkg: Initial UDF/ECMA-167 file system support > MdeModulePkg/UdfDxe: Add seek, read and listing support on files > > MdeModulePkg/MdeModulePkg.dec | 6 + > .../Universal/Disk/PartitionDxe/Partition.c | 3 +- > .../Universal/Disk/PartitionDxe/Partition.h | 41 +- > .../Universal/Disk/PartitionDxe/PartitionDxe.inf | 10 +- > .../Universal/Disk/PartitionDxe/PartitionDxe.uni | Bin 3206 -> 3290 > bytes > MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c | 330 +++ > MdeModulePkg/Universal/Disk/UdfDxe/ComponentName.c | 185 ++ > MdeModulePkg/Universal/Disk/UdfDxe/File.c | 908 +++++++ > MdeModulePkg/Universal/Disk/UdfDxe/FileName.c | 195 ++ > .../Universal/Disk/UdfDxe/FileSystemOperations.c | 2595 > ++++++++++++++++++++ > MdeModulePkg/Universal/Disk/UdfDxe/Udf.c | 411 ++++ > MdeModulePkg/Universal/Disk/UdfDxe/Udf.h | 1276 ++++++++++ > MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf | 66 + > MdePkg/Include/IndustryStandard/Udf.h | 79 + > 14 files changed, 6097 insertions(+), 8 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 Ping? Thanks, Paulo -- Paulo Alcantara, C.E.S.A.R Speaking for myself only. ------------------------------------------------------------------------------ New Year. New Location. New Benefits. New Data Center in Ashburn, VA. GigeNET is offering a free month of service with a new server in Ashburn. Choose from 2 high performing configs, both with 100TB of bandwidth. Higher redundancy.Lower latency.Increased capacity.Completely compliant. http://p.sf.net/sfu/gigenet _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ New Year. New Location. New Benefits. New Data Center in Ashburn, VA. GigeNET is offering a free month of service with a new server in Ashburn. Choose from 2 high performing configs, both with 100TB of bandwidth. Higher redundancy.Lower latency.Increased capacity.Completely compliant. http://p.sf.net/sfu/gigenet _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
