[LU-4659] link EA inconsistencies

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.6.0, Lustre 2.4.2, Lustre 2.5.1
• Fix Version/s: Lustre 2.6.0, Lustre 2.5.2
• Type: Bug
• Priority: Critical
• Reporter: Henri Doreau (Inactive)
• Assignee: Lai Siyao
• Resolution: Fixed
• Labels: mn4
• Severity: 3

Description

Chained link() and rename() leads to incoherent path/fid mappings.

Here's a simple reproducer:

1. cp /bin/ls test.bin
2. ln test.bin test.bin.lnk
3. cp test.bin.lnk test.bin.new
4. mv test.bin.new test.bin
5. lfs path2fid test.bin
   [0x200000400:0x2:0x0]
6. lfs path2fid test.bin.lnk
   [0x200000400:0x1:0x0]
7. lfs fid2path lustre 0x200000400:0x2:0x0
   test.bin
8. lfs fid2path lustre 0x200000400:0x1:0x0
   test.bin
   test.bin.lnk
9. ls -i *
   144115205255725058 test.bin 144115205255725057 test.bin.lnk

Both inodes have a link count=1.

Comments

[Peter Jones]

Lai

Could you please look into this one?

Thanks

Peter

[Lai Siyao]

The code shows mdd_rename() forgets updating target linkea, I'll commit a patch soon.

[Lai Siyao]

patch is on http://review.whamcloud.com/#/c/9366/

[Lai Siyao]

patch landed.

[Peter Jones]

Patch was reverted due to a build failure. Could you please repush this patch? Thanks!

[Lai Siyao]

patch is updated here: http://review.whamcloud.com/9745

[Jodi Levi (Inactive)]

Patch landed to Master. Please reopen ticket if more work is needed.

[Aurelien Degremont (Inactive)]

I'm not this ticket should be closed. I think this should go to 2.5.2 too.

[Aurelien Degremont (Inactive)]

I'm seeing this ticket flagged as Fixed in 2.5.2 but I cannot find any patch in b2_5 related to this ticket, and any Gerrit for 2.5 is linked here.

Could you confirm this information?

[Peter Jones]

I confirm that, while it is intended to land this patch ahead of the 2.5.2 release, it has not yet been landed to b2_5.

[Bob Glossman (Inactive)]

backport to b2_5:
http://review.whamcloud.com/10409