Re: [PATCH] Please get this committed, _NOW_ (preferrably 5 years ago), thanks.

Andreas Mohr
2009-11-08T14:45:46Z

Hi,

On Sun, Nov 08, 2009 at 10:25:06PM +0800, Américo Wang wrote:
> On Sun, Nov 08, 2009 at 01:00:08PM +0100, Andreas Mohr wrote:
> >--- linux-2.6.32-rc6/init/main.c.orig 2009-11-08 11:09:51.000000000 +0100
> >+++ linux-2.6.32-rc6/init/main.c 2009-11-08 12:40:11.000000000 +0100
> >@@ -846,7 +846,47 @@ static noinline int init_post(void)
> > run_init_process("/bin/init");
> > run_init_process("/bin/sh");
> >
> >- panic("No init found. Try passing init= option to kernel.");
> >+ panic("No init found. Try passing init= option to kernel. "
> >+ y "See Linux " __FILE__ " for guidance.");
>
>
> I would like to put those guidelines in a doc file, instead
> of being a FAT comment in the source code.

Heh, somehow I fully expected this objection to come up ;)
(and you're probably right after all)

So, where to put it?
Documentation/initrd.txt is related, however I'd create something
like Documentation/initialization/...
Or perhaps Documentation/bootup/init.txt ?

> And, you're explaining the _user space_ reasons that causes
> this problem, not related with kernel at all.

That's usually within the responsibility of the one passing control,
not the one gaining control.
Just like with children, where it's firmly the parents' job to take care
of getting their children safely to kindergarden, not the kindergarden's ;)

Plus in my case it _was_ a kernel issue (problematic/non-working serial
console device). And except for the linker issues of missing libraries,
many issues (USB functionality etc.) were kernel-related, too.

Thanks,

Andreas Mohr