On 2025-01-24 13:41:17 [-1000], Tejun Heo wrote:
> Hello,
>
> On Fri, Jan 24, 2025 at 06:46:14PM +0100, Sebastian Andrzej Siewior wrote:
> > Using RCU lifetime rules to access kernfs_node::name can avoid the
> > trouble kernfs_rename_lock in kernfs_name() and kernfs_path_from_node()
> > if the fs was created with KERNFS_ROOT_INVARIANT_PARENT.
>
> Maybe explain why we want to do this?
>
> > +static inline const char *kernfs_rcu_get_name(const struct kernfs_node *kn)
> > +{
> > + return rcu_dereference_check(kn->name, kernfs_root_is_locked(kn));
> > +}
>
> Can you drop "get" from the accessors? Other accessors don't have it and it
> gets confusing with reference counting operations.
Sure.
> Thanks.
Sebastian
