Re: [edk2] [PATCH v2 3/3] MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes

2017-09-20 Thread Ni, Ruiyu
OK I understood the reason now. I tried to understand the relationship between AVDP, LVD, and PD, but failed. The code below in Partition driver seems to say the LVD and PD are close to each other. Is it always true? If it is, I agree with the implementation. // // Cover the main VDS

Re: [edk2] [PATCH v2 3/3] MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes

2017-09-18 Thread Paulo Alcantara
On 9/17/2017 10:00 PM, Ni, Ruiyu wrote: Paulo, With the change in partition driver, I suppose UdfDxe driver only needs to take care of area covered by the partition descriptor. But why StartMainVolumeDescriptorSequence() still reads LVD, TD, and PD? We still need the LVD again for a few

Re: [edk2] [PATCH v2 3/3] MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes

2017-09-17 Thread Ni, Ruiyu
Paulo, With the change in partition driver, I suppose UdfDxe driver only needs to take care of area covered by the partition descriptor. But why StartMainVolumeDescriptorSequence() still reads LVD, TD, and PD? I thought the UdfDxe driver's logic can be simplified a lot. There should be no

[edk2] [PATCH v2 3/3] MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes

2017-09-17 Thread Paulo Alcantara
This patch reworks the driver to support Partition driver changes. Cc: Eric Dong Cc: Paulo Alcantara Cc: Ruiyu Ni Cc: Star Zeng Cc: Laszlo Ersek Contributed-under: TianoCore Contribution