[PATCH] die kdevname(), die!

[PATCH] die kdevname(), die!

[Christoph Hellwig]

Remove the three remaining uses of kdevname():

fs/locks.c:
	use MAJOR/MINOR directly as userspace is stupid, this part of the
	patch is from the maintainer of that piece of code, Matthew Wilcox.
fs/intermezzo/sysctl.c:
	printk removed as suggested by you, surrunding code is far from
	beeing compilable anyway.
fs/partitions/check.c:
	replace with __bdevname

and the function plus it's two prototypes in the headers.


--- 1.14/fs/libfs.c	Tue Dec 31 20:18:35 2002
+++ edited/fs/libfs.c	Sat Mar  1 11:23:49 2003
@@ -332,14 +332,3 @@
 	set_page_dirty(page);
 	return 0;
 }
-
-/*
- * Print device name (in decimal, hexadecimal or symbolic)
- * Note: returns pointer to static data!
- */
-const char * kdevname(kdev_t dev)
-{
-	static char buffer[32];
-	sprintf(buffer, "%02x:%02x", major(dev), minor(dev));
-	return buffer;
-}
--- 1.37/fs/locks.c	Thu Feb 13 12:25:01 2003
+++ edited/fs/locks.c	Sat Mar  1 11:22:36 2003
@@ -1785,19 +1785,20 @@
 			       ? (fl->fl_type & F_UNLCK) ? "UNLCK" : "READ "
 			       : (fl->fl_type & F_WRLCK) ? "WRITE" : "READ ");
 	}
-#if WE_CAN_BREAK_LSLK_NOW
 	if (inode) {
+#if WE_CAN_BREAK_LSLK_NOW
 		out += sprintf(out, "%d %s:%ld ", fl->fl_pid,
 				inode->i_sb->s_id, inode->i_ino);
+#else
+		/* userspace relies on this representation of dev_t ;-( */
+		out += sprintf(out, "%d %02x:%02x:%ld ", fl->fl_pid,
+				MAJOR(inode->i_sb->s_dev),
+				MINOR(inode->i_sb->s_dev), inode->i_ino);
+#endif
 	} else {
 		out += sprintf(out, "%d <none>:0 ", fl->fl_pid);
 	}
-#else
-	/* kdevname is a broken interface.  but we expose it to userspace */
-	out += sprintf(out, "%d %s:%ld ", fl->fl_pid,
-			inode ? kdevname(to_kdev_t(inode->i_sb->s_dev)) : "<none>",
-			inode ? inode->i_ino : 0);
-#endif
+
 	if (IS_POSIX(fl)) {
 		if (fl->fl_end == OFFSET_MAX)
 			out += sprintf(out, "%Ld EOF\n", fl->fl_start);
--- 1.7/fs/intermezzo/sysctl.c	Fri Dec 13 20:27:19 2002
+++ edited/fs/intermezzo/sysctl.c	Sat Mar  1 11:22:55 2003
@@ -175,9 +175,6 @@
 		if (errorval < 0) {
 			if (newval == 0)
 				set_device_ro(-errorval, 0);
-			else
-				CERROR("device %s already read only\n",
-				       kdevname(-errorval));
 		} else {
 			if (newval < 0)
 				set_device_ro(-newval, 1);
--- 1.95/fs/partitions/check.c	Tue Feb 11 02:22:51 2003
+++ edited/fs/partitions/check.c	Sat Mar  1 11:23:32 2003
@@ -544,7 +544,6 @@
 	}
 
 	dname = kmalloc(sizeof(*dname), GFP_KERNEL);
-
 	if (!dname)
 		return nomem;
 	/*
@@ -558,7 +557,7 @@
 		put_disk(hd);
 	}
 	if (!dname->name) {
-		sprintf(dname->namebuf, "[dev %s]", kdevname(to_kdev_t(dev)));
+		sprintf(dname->namebuf, "[dev %s]", __bdevname(dev));
 		dname->name = dname->namebuf;
 	}
 
--- 1.221/include/linux/fs.h	Tue Feb 25 11:21:08 2003
+++ edited/include/linux/fs.h	Sat Mar  1 11:24:06 2003
@@ -1069,7 +1069,6 @@
 extern void close_bdev_excl(struct block_device *, int);
 
 extern const char * cdevname(kdev_t);
-extern const char * kdevname(kdev_t);
 extern void init_special_inode(struct inode *, umode_t, dev_t);
 
 /* Invalid inode operations -- fs/bad_inode.c */
--- 1.7/include/linux/kdev_t.h	Fri Nov  1 07:28:19 2002
+++ edited/include/linux/kdev_t.h	Sat Mar  1 11:24:02 2003
@@ -101,8 +101,6 @@
 #define NODEV		(mk_kdev(0,0))
 #define B_FREE		(mk_kdev(0xff,0xff))
 
-extern const char * kdevname(kdev_t);	/* note: returns pointer to static data! */
-
 static inline int kdev_same(kdev_t dev1, kdev_t dev2)
 {
 	return dev1.value == dev2.value;
--- 1.184/kernel/ksyms.c	Tue Feb 18 21:58:45 2003
+++ edited/kernel/ksyms.c	Sat Mar  1 11:23:54 2003
@@ -517,7 +517,6 @@
 EXPORT_SYMBOL(vsprintf);
 EXPORT_SYMBOL(vsnprintf);
 EXPORT_SYMBOL(vsscanf);
-EXPORT_SYMBOL(kdevname);
 EXPORT_SYMBOL(__bdevname);
 EXPORT_SYMBOL(cdevname);
 EXPORT_SYMBOL(simple_strtoull);
	

Re: [PATCH] die kdevname(), die!

[Linus Torvalds]

I really think that replacing "kdevname()" with "__bdevname()" is a step 
in the wrong direction (or at least sideways).

		Linus

Re: [PATCH] die kdevname(), die!

[Christoph Hellwig]

On Sat, Mar 01, 2003 at 10:10:23AM -0800, Linus Torvalds wrote:
> 
> I really think that replacing "kdevname()" with "__bdevname()" is a step 
> in the wrong direction (or at least sideways).

kdevname() is the really wrong interface because it doesn't know whether
we have a character or block device.  __bdevname is a bit better because
it's block-device specific.  It's not perfect yet, though.

Re: [PATCH] die kdevname(), die!

[Christoph Hellwig]

On Sat, Mar 01, 2003 at 07:14:41PM +0100, Christoph Hellwig wrote:
> On Sat, Mar 01, 2003 at 10:10:23AM -0800, Linus Torvalds wrote:
> > 
> > I really think that replacing "kdevname()" with "__bdevname()" is a step 
> > in the wrong direction (or at least sideways).
> 
> kdevname() is the really wrong interface because it doesn't know whether
> we have a character or block device.  __bdevname is a bit better because
> it's block-device specific.  It's not perfect yet, though.

And the actual important information is missing in this mail (I'm a bit
fast in hitting the send button today :)).  The proper replacement for
kdevname on blockdevices would be partitions_name - but in this specific
case we can't use it because it's the piece of code that constructs the
name that is stored for use by partition_name if we don't have any
better information.