Re: A little question about safe mode

From: Alexander Yerenkow <yerenkow_at_gmail.com>
Date: Fri, 19 Oct 2012 12:03:14 +0300
2012/10/18 Andriy Gapon <avg_at_freebsd.org>:
> on 18/10/2012 18:33 Andriy Gapon said the following:
>> on 18/10/2012 18:20 Andriy Gapon said the following:
>>> My guess is that the easiest way to avoid this ambiguity and the confusion that it
>>> causes (like the problem you described above) is to prepend "/dev/", if it's
>>> missing, right in vfs_mountroot.c before calling kernel_mount().
>>
>> A patch (not tested):
>>
>> --- a/sys/kern/vfs_mountroot.c
>> +++ b/sys/kern/vfs_mountroot.c
>> _at__at_ -676,6 +676,7 _at__at_ static int
>>  parse_mount(char **conf)
>>  {
>>       char errmsg[255];
>> +     char devbuf[MNAMELEN];
>>       struct mntarg *ma;
>>       char *dev, *fs, *opts, *tok;
>>       int delay, error, timeout;
>> _at__at_ -693,6 +694,11 _at__at_ parse_mount(char **conf)
>>       parse_advance(&tok);
>>       dev = tok;
>>
>> +     if (dev[0] != '\0' && strncmp(dev, "/dev/", 5) != 0) {
>> +             snprintf(devbuf, sizeof(devbuf), "/dev/%s", dev);
>> +             dev = devbuf;
>> +     }
>> +
>>       if (root_mount_mddev != -1) {
>>               /* Handle substitution for the md unit number. */
>>               tok = strstr(dev, "md#");
>>
>>
>
> Oops, the patch incorrectly assumes that all "device names" are device names,
> which is incorrect e.g. for ZFS.  Maybe some other filesystems too.
> So don't try it :-)

I think correct patch should fix path and prepend "/dev/" only for
ufs:, for now.

Or at least help should mention this moment.

>
> --
> Andriy Gapon



-- 
Regards,
Alexander Yerenkow
Received on Fri Oct 19 2012 - 07:03:16 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:31 UTC