[LU-15833] Don't use realpath in llapi_search_fsname. Created: 09/May/22 Updated: 28/Mar/23 Resolved: 12/Sep/22 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.16.0, Lustre 2.15.3 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Etienne Aujames | Assignee: | Etienne Aujames |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||
| Rank (Obsolete): | 9223372036854775807 | ||||
| Description |
|
llapi_search_fsname() uses to determine the mountpoint of a path and then return its fsname. realpath() executes lstat() for each component of the path: this means that the client must have a LDLM read lock on every directory in the path. We can use st_dev or llapi_get_name to reduce the number of stats. |
| Comments |
| Comment by Gerrit Updater [ 09/May/22 ] |
|
"Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/47258 |
| Comment by Peter Jones [ 12/Sep/22 ] |
|
Landed for 2.16 |
| Comment by Gerrit Updater [ 21/Dec/22 ] |
|
"Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49482 |
| Comment by Gerrit Updater [ 17/Jan/23 ] |
|
"Jian Yu <yujian@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49649 |
| Comment by Gerrit Updater [ 28/Mar/23 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49649/ |