I have been using the new Ext4Pkg and been pretty successful and it is solving 
a use case we had.

Had a couple updates to propose

1. Changed the implementation of the binding protocol to both check if the 
driver is already bound
to the partition as well as added a really quick check to validate the magic 
value in supported.
This improves performance when you have a large number of non-ext4 partitions 
on the system.

2. As we are planning on using this for boot support we want to support unclean 
filesystem states in
case the user doesn't reset cleanly. I added a check if the recovery journal is 
present and if so treat
the filesystem as read-only (I know the driver is only RO at this point, but 
figured if you added write
support prior to recovery journal support we would want that). With this 
everything seems to work great.
I can add this under a FeaturePcd if desired as well.

Change log

v3 - Removed goto flow on binding supported failures
     Minor code review comments
v2 - Minor code review comments
v1 - Initial revision

Jeff Brasen (2):
  Ext4Pkg: Improve Ext4IsBindingSupported() behavior
  Ext4Pkg: Support uncleanly unmounted filesystems

 Features/Ext4Pkg/Ext4Dxe/Ext4Dxe.h    | 14 +++++++
 Features/Ext4Pkg/Ext4Dxe/Ext4Dxe.c    | 54 +++++++++++++++++++++------
 Features/Ext4Pkg/Ext4Dxe/Superblock.c | 46 +++++++++++++++++++++--
 3 files changed, 99 insertions(+), 15 deletions(-)

-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#80510): https://edk2.groups.io/g/devel/message/80510
Mute This Topic: https://groups.io/mt/85521567/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to