On Jul 11, 2017, at 9:37 PM, Tahsin Erdogan <tahsin@xxxxxxxxxx> wrote: > > Reviewed-by: tahsin@xxxxxxxxxx > > On Tue, Jul 11, 2017 at 8:56 PM, Emoly Liu <emoly.liu@xxxxxxxxx> wrote: >> For Lustre, if ea_inode fails in hash validation but passes parent >> inode and generation checks, it won't be added to the cache as well >> as the error "-EFSCORRUPTED" should be cleared, otherwise it will >> cause "Structure needs cleaning" when running getfattr command. >> >> Signed-off-by: Emoly Liu <emoly.liu@xxxxxxxxx> Reviewed-by: Andreas Dilger <adilger@xxxxxxxxx> This could also include: Fixes: dec214d00e0d78a08b947d7dccdfdb84407a9f4d unless this fix is merged into the original patch. >> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-9723 >> --- >> fs/ext4/xattr.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c >> index cff4f41..de217a0 100644 >> --- a/fs/ext4/xattr.c >> +++ b/fs/ext4/xattr.c >> @@ -451,6 +451,7 @@ static int ext4_xattr_inode_iget(struct inode *parent, unsigned long ea_ino, >> } >> /* Do not add ea_inode to the cache. */ >> ea_inode_cache = NULL; >> + err = 0; >> } else if (err) >> goto out; Cheers, Andreas
Attachment:
signature.asc
Description: Message signed with OpenPGP
