[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <E1ItqCg-0003W2-Qu@be1.7eggert.dyndns.org>
Date: Sun, 18 Nov 2007 20:53:14 +0100
From: Bodo Eggert <7eggert@....de>
To: Al Boldi <a1426z@...ab.com>, linux-kernel@...r.kernel.org,
hpa@...or.com, akpm@...ux-foundation.org
Subject: Re: [PATCH][retry-2] init: Introduce rootdir bootparm to select which dir to sys_chroot
Al Boldi <a1426z@...ab.com> wrote:
> Second try; this time with a doc-update, and the ability to remount normally.
>
> Tested against 2.6.23.
>
> ---
>
> This patch introduces a rootdir kernel boot parameter, which specifies the
> path to the kernel sys_chroot boot dir.
>
> This is useful for systems that have more than one distribution installed on
> the same fs/partition.
1) This is useful for booting a rescue or test system, too. In those cases,
you might want to have the old root moved somewhere.
(Always "$rootdir/oldroot"? Additional parameter? I'm not sure ...)
2) You use a static buffer, but you don't check for bad return values of
strlcat().
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists