Skip to content

Commit

Permalink
path: add cleanup helper
Browse files Browse the repository at this point in the history
Add a simple cleanup helper so we can cleanup struct path easily.
No need for any extra machinery. Avoid DEFINE_FREE() as it causes a
local copy of struct path to be used. Just rely on path_put() directly
called from a cleanup helper.

Link: https://lore.kernel.org/r/20240607-vfs-listmount-reverse-v1-2-7877a2bfa5e5@kernel.org
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
  • Loading branch information
Christian Brauner committed Jun 28, 2024
1 parent 1613e60 commit ff2c570
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions include/linux/path.h
Original file line number Diff line number Diff line change
Expand Up @@ -24,4 +24,13 @@ static inline void path_put_init(struct path *path)
*path = (struct path) { };
}

/*
* Cleanup macro for use with __free(path_put). Avoids dereference and
* copying @path unlike DEFINE_FREE(). path_put() will handle the empty
* path correctly just ensure @path is initialized:
*
* struct path path __free(path_put) = {};
*/
#define __free_path_put path_put

#endif /* _LINUX_PATH_H */

0 comments on commit ff2c570

Please sign in to comment.