diff options
author | Kay Sievers <kay.sievers@vrfy.org> | 2008-05-06 16:31:33 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-05-14 13:37:57 -0400 |
commit | 30f2f0eb4bd2c43d10a8b0d872c6e5ad8f31c9a0 (patch) | |
tree | deb8d2a749cd737d9abead1b4cc8321d00dbaee4 /init | |
parent | 9604006d283fb67dda5ee9e0e15b7cc6c62e1557 (diff) |
block: do_mounts - accept root=<non-existant partition>
Some devices, like md, may create partitions only at first access,
so allow root= to be set to a valid non-existant partition of an
existing disk. This applies only to non-initramfs root mounting.
This fixes a regression from 2.6.24 which did allow this to happen and
broke some users machines :(
Acked-by: Neil Brown <neilb@suse.de>
Tested-by: Joao Luis Meloni Assirati <assirati@nonada.if.usp.br>
Cc: stable <stable@kernel.org>
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'init')
-rw-r--r-- | init/do_mounts.c | 27 |
1 files changed, 26 insertions, 1 deletions
diff --git a/init/do_mounts.c b/init/do_mounts.c index 3885e70e7759..660c1e50c91b 100644 --- a/init/do_mounts.c +++ b/init/do_mounts.c | |||
@@ -76,6 +76,7 @@ dev_t name_to_dev_t(char *name) | |||
76 | char s[32]; | 76 | char s[32]; |
77 | char *p; | 77 | char *p; |
78 | dev_t res = 0; | 78 | dev_t res = 0; |
79 | int part; | ||
79 | 80 | ||
80 | if (strncmp(name, "/dev/", 5) != 0) { | 81 | if (strncmp(name, "/dev/", 5) != 0) { |
81 | unsigned maj, min; | 82 | unsigned maj, min; |
@@ -106,7 +107,31 @@ dev_t name_to_dev_t(char *name) | |||
106 | for (p = s; *p; p++) | 107 | for (p = s; *p; p++) |
107 | if (*p == '/') | 108 | if (*p == '/') |
108 | *p = '!'; | 109 | *p = '!'; |
109 | res = blk_lookup_devt(s); | 110 | res = blk_lookup_devt(s, 0); |
111 | if (res) | ||
112 | goto done; | ||
113 | |||
114 | /* | ||
115 | * try non-existant, but valid partition, which may only exist | ||
116 | * after revalidating the disk, like partitioned md devices | ||
117 | */ | ||
118 | while (p > s && isdigit(p[-1])) | ||
119 | p--; | ||
120 | if (p == s || !*p || *p == '0') | ||
121 | goto fail; | ||
122 | |||
123 | /* try disk name without <part number> */ | ||
124 | part = simple_strtoul(p, NULL, 10); | ||
125 | *p = '\0'; | ||
126 | res = blk_lookup_devt(s, part); | ||
127 | if (res) | ||
128 | goto done; | ||
129 | |||
130 | /* try disk name without p<part number> */ | ||
131 | if (p < s + 2 || !isdigit(p[-2]) || p[-1] != 'p') | ||
132 | goto fail; | ||
133 | p[-1] = '\0'; | ||
134 | res = blk_lookup_devt(s, part); | ||
110 | if (res) | 135 | if (res) |
111 | goto done; | 136 | goto done; |
112 | 137 | ||