lkml.org 
[lkml]   [2002]   [Jan]   [5]   [last100]   RSS Feed
Views: [more markup]   [less markup]   [headers]   [forward]  
 
Messages in this thread
Patch in this message
/
DateSat, 5 Jan 2002 04:35:06 -0800
From"Adam J. Richter" <>
SubjectPatch?: linux-2.5.2-pre8/fs/ext3/super.c - variation of fix by Andrew Morton
	Andrwe Morton posted a fix for the compilation error in
linux-2.5.2-pre8/fs/ext3/super.c.  I have changed his test to
use "!kdev_none()" instead of "!kdev_val()".  It will compile
to the same opcodes, but I think this way is slightly more proper.
It may even make a difference some day if the internal representation
of kdev_val or kdev_none changes in a weird way so that
kdev_val(NODEV) returns non-zero, although I think this is unlikely.

-- 
Adam J. Richter     __     ______________   4880 Stevens Creek Blvd, Suite 104
adam@yggdrasil.com     \ /                  San Jose, California 95129-1034
+1 408 261-6630         | g g d r a s i l   United States of America
fax +1 408 261-6631      "Free Software For The Rest Of Us."--- linux-2.5.2-pre8/fs/ext3/super.c	Fri Jan  4 19:40:37 2002
+++ linux/fs/ext3/super.c	Sat Jan  5 04:29:20 2002
@@ -56,10 +56,10 @@
 
 static void make_rdonly(kdev_t dev, int *no_write)
 {
-	if (dev) {
+	if (!kdev_none(dev)) {
 		printk(KERN_WARNING "Turning device %s read-only\n", 
 		       bdevname(dev));
-		*no_write = 0xdead0000 + dev;
+		*no_write = 0xdead0000 + kdev_val(dev);
 	}
 }
 
\
 
 \ /
  Last update: 2005-03-22 11:15    [W:0.133 / U:0.920 seconds]
©2003-2008 Jasper Spaans