[LU-934] external xattr inode support in e2fsprogs

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.2.0
• Affects Version/s: Lustre 2.2.0
• Severity: 3
• Assignee: Jian Yu
• Reporter: Jian Yu

Description

The existing codes have the following issues:

• pass1.c check_ea_in_inode() compares e_value_size to fit in free space of inode, but for xattr in external inode this is wrong.
• e2fsck_lfsck_find_ea() have no code to copy EA from external inode.

Issue Links

• Trackbacks: Changelog 2.2 version 2.2.0 Support for networks: o2iblnd OFED 1.5.4 Server support for kernels: 2.6.32220.4.2.el6 (RHEL6) Client support for unpatched kernels: 2.6.18274.18.1.el5 (RHEL5) 2.6.32220.4.2.el6 (RHEL6) 2.6.32.360....

Comments

[Jian Yu]

The patch has been pushed to e2fsprogs/master-lustre branch. The issue is resolved.

[Build Master (Inactive)]

Integrated in e2fsprogs-master » i686,el5 #159
LU-934 e2fsck: copy EA from external inode, fix size check (Revision f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd)

Result = SUCCESS
alexander_boyko : f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd
Files :

• patches/e2fsprogs-lfsck.patch
• patches/e2fsprogs-large-ea.patch

[Build Master (Inactive)]

Integrated in e2fsprogs-master » i686,el6 #159
LU-934 e2fsck: copy EA from external inode, fix size check (Revision f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd)

Result = SUCCESS
alexander_boyko : f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd
Files :

• patches/e2fsprogs-large-ea.patch
• patches/e2fsprogs-lfsck.patch

[Build Master (Inactive)]

Integrated in e2fsprogs-master » x86_64,el6 #159
LU-934 e2fsck: copy EA from external inode, fix size check (Revision f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd)

Result = SUCCESS
alexander_boyko : f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd
Files :

• patches/e2fsprogs-large-ea.patch
• patches/e2fsprogs-lfsck.patch

[Build Master (Inactive)]

Integrated in e2fsprogs-master » x86_64,el5 #159
LU-934 e2fsck: copy EA from external inode, fix size check (Revision f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd)

Result = SUCCESS
alexander_boyko : f8ba0de1d5d4fb670c3bb287e6d20d1ded8827dd
Files :

• patches/e2fsprogs-lfsck.patch
• patches/e2fsprogs-large-ea.patch

[Jian Yu]

The patch is in: http://review.whamcloud.com/1886

$ git apply external_inode.patch -v
external_inode.patch:19: space before tab in indent.
 		if (entry->e_value_offs > storage_size ||
external_inode.patch:23: space before tab in indent.
 			problem = PR_1_INODE_EA_BAD_VALUE;
external_inode.patch:24: space before tab in indent.
 			goto fix;
external_inode.patch:25: space before tab in indent.
 		}
external_inode.patch:44: trailing whitespace.
@@ -1654,6 +1778,7 @@ static int check_ext_attr(e2fsck_t ctx, 
Checking patch patches/e2fsprogs-large-ea.patch...
Checking patch patches/e2fsprogs-lfsck.patch...
Applied patch patches/e2fsprogs-large-ea.patch cleanly.
Applied patch patches/e2fsprogs-lfsck.patch cleanly.
warning: squelched 38 whitespace errors
warning: 43 lines add whitespace errors.

[Alexander Boyko]

change spaces to tabs