possible mountroot regression

[Andriy Gapon]

on 29/08/2011 19:45 Marcel Moolenaar said the following:
> 
> On Aug 29, 2011, at 1:21 AM, Andriy Gapon wrote:
> 
>> on 27/08/2011 18:16 Marcel Moolenaar said the following:
>>>
>>> On Aug 26, 2011, at 2:07 PM, Andriy Gapon wrote:
>>>
>>>>
>>>> It seems that after the introduction of the mountroot scripting language a user
>>>> now has exactly one chance to try to specify a correct root device at the
>>>> mountroot prompt.  I am not sure that that is convenient/enough.
>>>
>>> This is no different from before.
>>
>> Are you sure?
>> I remember trying multiple (incorrect) possibilities at the prompt and not
>> getting the panic.  But I know that sometimes I have cases of "false memories",
>> so _I_ am not sure.
> 
> I'm sure now that we're both not sure :-)
> 
> It's possible the failure mode varied by how the root mount
> failed...


Judging from the code before r214006 it shouldn't have panic-ed upon such a failure:
static int
vfs_mountroot_ask(void)
{
        char name[128];
        char *mountfrom;
        char *options;

        for(;;) {
...
                gets(name, sizeof(name), 1);
                if (name[0] == '\0')
                        return (1);
                if (name[0] == '?') {
                        printf("\nList of GEOM managed disk devices:\n  ");
                        g_dev_print();
                        continue;
                }
                if (!vfs_mountroot_try(name, NULL))
                        return (0);
        }
}


So this "endless" loop was exited only if vfs_mountroot_try() returned success
(error == 0) or if a user entered an empty string.

-- 
Andriy Gapon