aboutsummaryrefslogtreecommitdiffstats
path: root/fs/sysfs/inode.c
diff options
context:
space:
mode:
authorTejun Heo <tj@kernel.org>2013-09-11 22:29:09 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2013-09-26 18:34:38 -0400
commitcfec0bc835c84d3d3723d4955587f05a94879b26 (patch)
tree18c1239d9113ebc30ca037b1a0025b95ffb7c405 /fs/sysfs/inode.c
parent388975cccaaf11abd47525f664c76891c440481a (diff)
sysfs: @name comes before @ns
Some internal sysfs functions which take explicit namespace argument are weird in that they place the optional @ns in front of @name which is contrary to the established convention. This is confusing and error-prone especially as @ns and @name may be interchanged without causing compilation warning. Swap the positions of @name and @ns in the following internal functions. sysfs_find_dirent() sysfs_rename() sysfs_hash_and_remove() sysfs_name_hash() sysfs_name_compare() create_dir() This patch doesn't introduce any functional changes. Signed-off-by: Tejun Heo <tj@kernel.org> Cc: Eric W. Biederman <ebiederm@xmission.com> Cc: Kay Sievers <kay@vrfy.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs/sysfs/inode.c')
-rw-r--r--fs/sysfs/inode.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/sysfs/inode.c b/fs/sysfs/inode.c
index 963f910c8034..07193d720d92 100644
--- a/fs/sysfs/inode.c
+++ b/fs/sysfs/inode.c
@@ -314,8 +314,8 @@ void sysfs_evict_inode(struct inode *inode)
314 sysfs_put(sd); 314 sysfs_put(sd);
315} 315}
316 316
317int sysfs_hash_and_remove(struct sysfs_dirent *dir_sd, const void *ns, 317int sysfs_hash_and_remove(struct sysfs_dirent *dir_sd, const char *name,
318 const char *name) 318 const void *ns)
319{ 319{
320 struct sysfs_addrm_cxt acxt; 320 struct sysfs_addrm_cxt acxt;
321 struct sysfs_dirent *sd; 321 struct sysfs_dirent *sd;
@@ -328,7 +328,7 @@ int sysfs_hash_and_remove(struct sysfs_dirent *dir_sd, const void *ns,
328 328
329 sysfs_addrm_start(&acxt, dir_sd); 329 sysfs_addrm_start(&acxt, dir_sd);
330 330
331 sd = sysfs_find_dirent(dir_sd, ns, name); 331 sd = sysfs_find_dirent(dir_sd, name, ns);
332 if (sd) 332 if (sd)
333 sysfs_remove_one(&acxt, sd); 333 sysfs_remove_one(&acxt, sd);
334 334