Hi Feng,
On Thu, 22 Jan 2015 00:15:44 +0000
"Tian, Feng" <feng.t...@intel.com> wrote:
> 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.
Here is my initial proposal for the Device Path:
UDF Device Path
===============
The UDF Device Path is used to define a logical volume that exists on an
UDF volume. The UDF volume is assumed to contain an UDF file system
(revision 2.00 or higher) and follow ECMA-167 and OSTA UDF format
specifications.
As specified by OSTA UDF specification, there shall be one Logical Volume
recorded per Volume Set, therefore the first Logical Volume found in Main
Volume Descriptor Sequence (MVDS) is supposed to be the bootable entity.
UDF Device Path structure
=========================
Mnemonic Byte Offset Byte Length Description
-------- ----------- ----------- -----------
Type 0 1 Type 4 - Media Device Path.
Sub-Type 1 1 Sub-Type 9 - UDF volume format.
Length 2 2 Length of this structure in bytes.
Length is 16 bytes.
Volume Start 4 8 Starting LSN of the logical volume
on the medium.
UDF volumes use relative logical
sector numbers.
Volume Size 12 8 Size of the logical volume in
units of Blocks, also called
Sectors.
Thanks,
Paulo
> -----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
--
Paulo Alcantara, C.E.S.A.R
Speaking for myself only.
------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
