Hi Ruiyu,
On 9/18/2017 1:52 AM, Ni, Ruiyu wrote:
Paulo,
Could you please paste a "map -r" output on a CDROM which
contains Eltorito volume?
I want to confirm that the result is expected.
With UdfDxe driver disabled in OVMF:
>Mapping table
> FS0: Alias(s):CD0c65535a1:;BLK2:
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/CDROM(0x1)
> BLK0: Alias(s):
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)
> BLK1: Alias(s):
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/CDROM(0x0)
> BLK3: Alias(s):
>
PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/VenMedia(C5BD4D42-1A76-4996-8956-73CDA326CD0A)
With UdfDxe driver enabled in OVMF:
> Mapping table
> FS0: Alias(s):CD0c65535a1:;BLK2:
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/CDROM(0x1)
> FS1: Alias(s):F0c65535a:;BLK3:
>
PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/VenMedia(C5BD4D42-1A76-4996-8956-73CDA326CD0A)
> BLK0: Alias(s):
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)
> BLK1: Alias(s):
> PciRoot(0x0)/Pci(0x1F,0x2)/Sata(0x2,0xFFFF,0x0)/CDROM(0x0)
Thanks!
Paulo
Thanks/Ray
-----Original Message-----
From: Paulo Alcantara [mailto:[email protected]]
Sent: Sunday, September 17, 2017 9:13 PM
To: [email protected]
Cc: Paulo Alcantara <[email protected]>; Kinney, Michael D
<[email protected]>; Gao, Liming <[email protected]>; Laszlo
Ersek <[email protected]>; Ni, Ruiyu <[email protected]>; Zeng, Star
<[email protected]>; Yao, Jiewen <[email protected]>
Subject: [PATCH v2 0/3] UDF partition driver fix
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=707
Hi,
This patchset fixes a bug in Partition driver that UDF partitions occupied the
entire disk space instead of using LVD space only.
BTW, I've only tested it under OVMF and built it with GCC only. That would
be appreciable if someone could build with other toolchains and see if this
doesn't break.
I used a Windows 10 ISO image with UdfDxe disabled and enabled. The `map
-r` output seemed OK. No breakage when booting an OS off an ElTorito
partition from an UDF bridge disk.
v1->v2:
- Followed Laszlo's suggestions to submit a proper patchset. Thanks!
- As I'm still waiting for Ruiyu and Star to test this fix, I took
advantage of it and did some code cleanups :-)
Repo: https://github.com/pcacjr/edk2.git
Branch: udf-partition-fix-v2
Cc: Michael D Kinney <[email protected]>
Cc: Liming Gao <[email protected]>
Cc: Laszlo Ersek <[email protected]>
Cc: Ruiyu Ni <[email protected]>
Cc: Star Zeng <[email protected]>
Cc: Jiewen Yao <[email protected]>
Contributed-under: TianoCore Contribution Agreement 1.1
Reported-by: Ruiyu Ni <[email protected]>
Signed-off-by: Paulo Alcantara <[email protected]>
---
Paulo Alcantara (3):
MdePkg: Add UDF volume structure definitions
MdeModulePkg/PartitionDxe: Fix creation of UDF logical partition
MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes
MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c | 323
+++++++++++-
MdeModulePkg/Universal/Disk/UdfDxe/File.c | 13 +-
MdeModulePkg/Universal/Disk/UdfDxe/FileSystemOperations.c | 515
++++++++------------
MdeModulePkg/Universal/Disk/UdfDxe/Udf.c | 7 -
MdeModulePkg/Universal/Disk/UdfDxe/Udf.h | 88 +---
MdePkg/Include/IndustryStandard/Udf.h | 63 +++
6 files changed, 566 insertions(+), 443 deletions(-)
--
2.11.0
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel
