Please don't reply to lustre-devel. Instead, comment in Bugzilla by using the
following link:
https://bugzilla.lustre.org/show_bug.cgi?id=3244
What |Removed |Added
----------------------------------------------------------------------------
Attachment #9464|review?([EMAIL PROTECTED]|review+
Flag|m) |
(From update of attachment 9464)
This looks good, but please note that the EXT2_LINK_MAX part is already in the
1.39-WIP patch in e2fsprogs-cfs3, which means that it is also in the upstream
code. The 1.39-WIP patch also holds the #define for RO_COMPAT_DIR_NLINK. The
one thing that is missing is the fix for bug 6472, where e2fsck shouldn't
report an error for a directory with i_nlink = 1 (this can happen if the
directory is > 65000 subdirs and they are then removed but the parent directory
is not).
Can you please make a test case for e2fsprogs (Girish knows how to do this if
you need help). The test case should contain an empty directory with i_nlink =
1, and a directory with > 65000 subdirectories.
_______________________________________________
Lustre-devel mailing list
[email protected]
https://mail.clusterfs.com/mailman/listinfo/lustre-devel
