Hi,
On Thu, August 21, 2014 4:33 am, Tian, Feng wrote:
> Yes, it's a tradeoff:-).
>
> Putting it to MdeModulePkg will need carefully review its
> modularity/interface/layer design. Putting it to a new package could
> enable this feature quickly.
>
> That's why I am asking UDF device path related question if Paulo want to
> put into MdeModulePkg:-). I saw the code is using a self-defined UDF
> device path and my personal idea is it's not suggested. And another
> concern is about if we can refer to PartitionDxe and EnhancedFatDxe to
> split to two layers. Ni, Ruiyu and I could provide consultant on this if
> Paulo needed. I never used IRC before... Just login but don't know how to
> use it.
I would like to put into MdeModulePkg -- IMHO, it seems to be now the
correct place after getting some context on that.
I'd also probably need to get some help of you on the question you came up
with "UDF Device Path". Does that mean to not use vendor-defined Device
Paths anymore, instead add an unique Device Path for UDF?
If you guys happen to join in #edk2 on OFTC, please, just ping me (pcacjr)
and I pong you back :-). Due to different timezones, I might take a bit
longer than you'd expect to answer, but it's OK.
>
> For the issue " for some disk image formats, if the current PartitionDxe
> driver starts before his UdfDxe driver, then it may prevent UdfDxe from
> working.", I guess it might be because the DVD Rom is a UDF bridge disc,
> which means the disc contains both UDF and ISO9660(Eltorito) file system.
> So if PartitionDxe driver runs before UdfDxe, the BlockIo protocol will be
> opened by PartitionDxe driver exclusively and cause UdfDxe fail.
Yes. That's a real issue. I think we could solve such issue by splitting
UdfDxe into two layers (like Feng Tian mentioned), so PartitionDxe would
check for UDF partitions and add EFI_BLOCK_IO_PRTOCOL's that layer on
them, and UdfDxe would handle filesystem-specific code
(EFI_SIMPLE_FILE_SYSTEM_PROTOCOL and EFI_FILE_PROTOCOL). Does that sound
correct to you? am I missing something here?
Thanks,
-pcacjr
--
Paulo Alcantara, C.E.S.A.R
Speaking for myself only.
------------------------------------------------------------------------------
Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
