On Fri Sep 29 17:37:33 2023 +0000, Jinoh Kang wrote:
This open-codes part of the access sharing check logic (which is normally doned by `check_sharing()`). This stretches the scope of `set_fd_name` (as evidenced by the indent). We should factor out actual sharing validation logic out of `check_sharing()` and use that. This will make `set_fd_name()` shorter, hopefully improving readability.
Fair enough. I moved this out to a helper: `get_inode_open_sharing`.