Commit 6fbe82a9 authored by Jeff Mahoney's avatar Jeff Mahoney Committed by Linus Torvalds

[PATCH] reiserfs: fix handling of device names with /'s in them

On systems with block devices containing a slash (virtual dasd, cciss,
etc), reiserfs will fail to initialize /proc/fs/reiserfs/<dev> due to it
being interpreted as a subdirectory.  The generic block device code changes
the / to !  for use in the sysfs tree.  This patch uses that convention.

Tested by making dm devices use dm/<number> rather than dm-<number>

[akpm@osdl.org: name variables consistently]
Signed-off-by: default avatarJeff Mahoney <jeffm@suse.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent bca3bffe
...@@ -492,9 +492,17 @@ static void add_file(struct super_block *sb, char *name, ...@@ -492,9 +492,17 @@ static void add_file(struct super_block *sb, char *name,
int reiserfs_proc_info_init(struct super_block *sb) int reiserfs_proc_info_init(struct super_block *sb)
{ {
char b[BDEVNAME_SIZE];
char *s;
/* Some block devices use /'s */
strlcpy(b, reiserfs_bdevname(sb), BDEVNAME_SIZE);
s = strchr(b, '/');
if (s)
*s = '!';
spin_lock_init(&__PINFO(sb).lock); spin_lock_init(&__PINFO(sb).lock);
REISERFS_SB(sb)->procdir = REISERFS_SB(sb)->procdir = proc_mkdir(b, proc_info_root);
proc_mkdir(reiserfs_bdevname(sb), proc_info_root);
if (REISERFS_SB(sb)->procdir) { if (REISERFS_SB(sb)->procdir) {
REISERFS_SB(sb)->procdir->owner = THIS_MODULE; REISERFS_SB(sb)->procdir->owner = THIS_MODULE;
REISERFS_SB(sb)->procdir->data = sb; REISERFS_SB(sb)->procdir->data = sb;
...@@ -508,13 +516,22 @@ int reiserfs_proc_info_init(struct super_block *sb) ...@@ -508,13 +516,22 @@ int reiserfs_proc_info_init(struct super_block *sb)
return 0; return 0;
} }
reiserfs_warning(sb, "reiserfs: cannot create /proc/%s/%s", reiserfs_warning(sb, "reiserfs: cannot create /proc/%s/%s",
proc_info_root_name, reiserfs_bdevname(sb)); proc_info_root_name, b);
return 1; return 1;
} }
int reiserfs_proc_info_done(struct super_block *sb) int reiserfs_proc_info_done(struct super_block *sb)
{ {
struct proc_dir_entry *de = REISERFS_SB(sb)->procdir; struct proc_dir_entry *de = REISERFS_SB(sb)->procdir;
char b[BDEVNAME_SIZE];
char *s;
/* Some block devices use /'s */
strlcpy(b, reiserfs_bdevname(sb), BDEVNAME_SIZE);
s = strchr(b, '/');
if (s)
*s = '!';
if (de) { if (de) {
remove_proc_entry("journal", de); remove_proc_entry("journal", de);
remove_proc_entry("oidmap", de); remove_proc_entry("oidmap", de);
...@@ -528,7 +545,7 @@ int reiserfs_proc_info_done(struct super_block *sb) ...@@ -528,7 +545,7 @@ int reiserfs_proc_info_done(struct super_block *sb)
__PINFO(sb).exiting = 1; __PINFO(sb).exiting = 1;
spin_unlock(&__PINFO(sb).lock); spin_unlock(&__PINFO(sb).lock);
if (proc_info_root) { if (proc_info_root) {
remove_proc_entry(reiserfs_bdevname(sb), proc_info_root); remove_proc_entry(b, proc_info_root);
REISERFS_SB(sb)->procdir = NULL; REISERFS_SB(sb)->procdir = NULL;
} }
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment