[LU-16748] comment in ll_swap_layouts_close() is outdated Created: 17/Apr/23  Updated: 31/May/23  Resolved: 31/May/23

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.15.2
Fix Version/s: Lustre 2.16.0

Type: Improvement Priority: Minor
Reporter: Li Xi Assignee: Li Xi
Resolution: Fixed Votes: 0
Labels: None

Rank (Obsolete): 9223372036854775807

 Description   

mdc_close_layout_swap_pack() does not exist, so the following comment needs to be updated.

    /* Close the file and {swap,merge} layouts between inode & inode2.
     * NB: lease lock handle is released in mdc_close_layout_swap_pack()
     * because we still need it to pack l_remote_handle to MDT. */

And mdt_close_swap_layouts() does not exists any more. So its declaration can be removed from mdt_internal.h.

 Comments   
Comment by Gerrit Updater [ 17/Apr/23 ]

"Li Xi <lixi@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50660
Subject: LU-16748 llite: update comment of ll_swap_layouts_close
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: c3ab577bc25c92c416088bc81c0745dc920bdec0

Comment by Gerrit Updater [ 31/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50660/
Subject: LU-16748 llite: update comment of ll_swap_layouts_close
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cfef0d9d5e708d5343cbd48890395cdcf5a13c3d

Comment by Peter Jones [ 31/May/23 ]

Landed for 2.16

Generated at Sat Feb 10 03:29:40 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.