lkml.org 
[lkml]   [2002]   [Oct]   [14]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
FromKeith Owens <>
Subject2.4.19 breaks devfs mapping for root=
DateMon, 14 Oct 2002 16:26:07 +1000
A change from 2.4.18 to 2.4.19 has broken the way that devfs maps
root=.  2.4.18 init/main.c::mount_root() has

        devfs_make_root (root_device_name);
        handle = devfs_find_handle (NULL, ROOT_DEVICE_NAME,
                                    MAJOR (ROOT_DEV), MINOR (ROOT_DEV),
                                    DEVFS_SPECIAL_BLK, 1);

where ROOT_DEVICE_NAME maps to the value of root= for non-initrd.  This
allowed devfs to remap an entry such as sda3 to whatever driver was
implementing sda3, even if that driver used a different major number.
The correct major was returned in handle.

2.4.19 init/do_mounts.c::mount_root() has

        devfs_make_root(root_device_name);
        create_dev("/dev/root", ROOT_DEV, root_device_name);

create_dev() has

	handle = devfs_find_handle(NULL, dev ? NULL : devfs_name,
                    MAJOR(dev), MINOR(dev), DEVFS_SPECIAL_BLK, 1);

The difference in 2.4.19 is that if dev is already set from
root_dev_names[] then devfs does NOT get the value of root=, forcing
the use of major from root_dev_names[].  If a driver reimplements one
of the standard device names and uses a different major or minor number
then it no longer works in 2.4.19 because devfs is given incomplete
information.

Quick and dirty workaround

--- 2.4.19/init/do_mounts.c
+++ 2.4.19/init/do_mounts.c
@@ -368,7 +368,7 @@
 	if (!do_devfs)
 		return sys_mknod(name, S_IFBLK|0600, kdev_t_to_nr(dev));
 
-	handle = devfs_find_handle(NULL, dev ? NULL : devfs_name,
+	handle = devfs_find_handle(NULL, devfs_name,
 				MAJOR(dev), MINOR(dev), DEVFS_SPECIAL_BLK, 1);
 	if (!handle)
 		return -1;
But that probably breaks initrd.  What should that code be doing to
cope with both initrd and still allow devfs to remap root=?

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:30    [from the cache]
©2003-2008