[LU-16082] old-style Lustre EA inodes support is broken in newer kernel

• Type: Bug
• Resolution: Fixed
• Priority: Critical
• Fix Version/s: Lustre 2.16.0, Lustre 2.15.2
• Affects Version/s: Lustre 2.14.0, Lustre 2.15.0
• Severity: 3
• Assignee: Alexander Zarochentsev
• Reporter: Alexander Zarochentsev

Description

the system upgraded from Lustre/rhel7.6/kernel-3.10 experiences the following problems:

files unaccessible with "Structure needs cleaning" error , kernel log at MDS is full of

[Tue Aug 2 11:15:32 2022] LDISKFS-fs warning (device md1): ldiskfs_xattr_inode_get:497: inode #4290430381: comm mdt04_005: EA inode hash validation failed
[Tue Aug 2 11:15:32 2022] LDISKFS-fs warning (device md1): ldiskfs_xattr_inode_get:497: inode #4290430355: comm mdt04_032: EA inode hash validation failed
[Tue Aug 2 11:15:32 2022] LDISKFS-fs warning (device md1): ldiskfs_xattr_inode_get:497: inode #4290430357: comm mdt04_013: EA inode hash validation failed
[Tue Aug 2 11:15:32 2022] LDISKFS-fs warning (device md1): ldiskfs_xattr_inode_get:497: inode #4290430383: comm mdt07_024: EA inode hash validation failed

Issue Links

• is related to: LU-15171 corrupted in-inode xattr
• is related to: LU-11643 create disk images for Lustre 2.10 and 2.12 for ldiskfs

Comments

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48611/
Subject: LU-16082 ldiskfs: old-style EA inode fix for el8.5/el8.6
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 818e2cc867f948693bec9c5d64314c7546dc858d

[Gerrit Updater]

"Jian Yu <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48611
Subject: LU-16082 ldiskfs: old-style EA inode fix for el8.5/el8.6
Project: fs/lustre-release
Branch: b2_15
Current Patch Set: 1
Commit: c4dcdeff3f5514fc68e71f6a7b911298932b13c5

[Alexander Zarochentsev]

> Does the patch that just merged replace the earlier ones pushed or are they all needed?
each patch adds more rhel8 targets.

[Peter Jones]

Does the patch that just merged replace the earlier ones pushed or are they all needed?

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48496/
Subject: LU-16082 ldiskfs: old-style EA inode fix for el8.5/el8.6
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: ba9845274c8ea5c55f57b7fa0e839f18d76031ea

[Gerrit Updater]

"Alexander Zarochentsev <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48498
Subject: LU-16082 ldiskfs: more targets for old ea_inode fix
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b387b80f9dcffbdc484283b62dec0792c6c452fb

[Gerrit Updater]

"Andreas Dilger <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48496
Subject: LU-16082 ldiskfs: old-style EA inode fix for el8.5/el8.6
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 235395d7b39adcc2c21006a44b27788ebf83e631

[Gerrit Updater]

"Jian Yu <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48412
Subject: LU-16082 ldiskfs: old-style EA inode handling fix
Project: fs/lustre-release
Branch: b2_15
Current Patch Set: 1
Commit: 31a73bf8c3371c44628e584d7e2234440dcc13f9

[Gerrit Updater]

"Oleg Drokin <[e-mail]>" merged in patch https://review.whamcloud.com/48174/
Subject: LU-16082 ldiskfs: old-style EA inode handling fix
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 76c3fa96dc30f21e95d80f9119972d7358975258

[Alexander Zarochentsev]

Test failure with the patch https://review.whamcloud.com/48350 :

== check Large EA ==
/tmp/t32/mnt/lustre/large_xattr_test_dir/large_xattr_file: user.fooattr: Structure needs cleaning
 conf-sanity test_32g: @@@@@@ FAIL: Large EA cannot be read 
  Trace dump:
  = ./../tests/test-framework.sh:6505:error_noexit()
  = conf-sanity.sh:2541:t32_test()
  = conf-sanity.sh:2980:test_32g()
  = ./../tests/test-framework.sh:6858:run_one()
  = ./../tests/test-framework.sh:6908:run_one_logged()
  = ./../tests/test-framework.sh:6730:run_test()
  = conf-sanity.sh:2985:main()
Dumping lctl log to /tmp/test_logs/1661958175/conf-sanity.test_32g.*.1661958356.log
Dumping logs only on local client.
== cleanup with rc=0 ==

[Gerrit Updater]

"Alexander Zarochentsev <[e-mail]>" uploaded a new patch: https://review.whamcloud.com/48350
Subject: LU-16082 ldiskfs: Large EA upgrade test
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5711c926acc327d3d77645485df555a0a42fd390