| Message ID | 149745334260.10897.8377331388521502715.stgit@warthog.procyon.org.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Jun 14, 2017 at 04:15:42PM +0100, David Howells wrote: > Make get_mnt_ns() return the namespace it got a ref on for consistency with > other namespace ref getting functions. Is there any point in doing that? I mean, it's not used in your patchset anymore and existing callers are a mixed bag...
diff --git a/fs/mount.h b/fs/mount.h index bf1fda6eed8f..02ef292a77d7 100644 --- a/fs/mount.h +++ b/fs/mount.h @@ -108,9 +108,10 @@ static inline void detach_mounts(struct dentry *dentry) __detach_mounts(dentry); } -static inline void get_mnt_ns(struct mnt_namespace *ns) +static inline struct mnt_namespace *get_mnt_ns(struct mnt_namespace *ns) { atomic_inc(&ns->count); + return ns; } extern seqlock_t mount_lock;
Make get_mnt_ns() return the namespace it got a ref on for consistency with other namespace ref getting functions. Signed-off-by: David Howells <dhowells@redhat.com> --- fs/mount.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)