[PATCH] kexec-tools: do not copy 1st kernel root= param in fs2dt.c
Dave Young
dyoung at redhat.com
Thu Oct 8 22:57:22 PDT 2015
Jan Stodola <jstodola at redhat.com> reported ppc64 root= is always added in kexec
kernel cmdline. But sometimes we need boot without root= for example we use
kexec to boot into installation initramfs image like below:
kexec --load vmlinuz --initrd=initrd.img --command-line=\
"inst.repo=http://<server>/<path>/Server/ppc64le/os/"
While creating dtb, in case there's no root= in user provided cmdline params
kexec-tools will find the original root= param used in 1st kernel and pass it
to 2nd kernel. This caused that user have no way to remove root= cmdline.
Dropping that part of code so that one can get chance to kexec into 2nd kernel
without root= param. One can still provide root= in --command-line=""
Signed-off-by: Dave Young <dyoung at redhat.com>
---
kexec/fs2dt.c | 31 +------------------------------
1 file changed, 1 insertion(+), 30 deletions(-)
--- kexec-tools.orig/kexec/fs2dt.c
+++ kexec-tools/kexec/fs2dt.c
@@ -579,8 +579,7 @@ static void putnode(void)
reserve(initrd_base, initrd_size);
}
- /* Add cmdline to the second kernel. Check to see if the new
- * cmdline has a root=. If not, use the old root= cmdline. */
+ /* Add cmdline to the second kernel. */
if (!strcmp(basename,"chosen/")) {
size_t result;
size_t cmd_len = 0;
@@ -594,36 +593,8 @@ static void putnode(void)
param = strstr(local_cmdline, "crashkernel=");
if (param)
crash_param = 1;
- /* does the new cmdline have a root= ? ... */
- param = strstr(local_cmdline, "root=");
}
- /* ... if not, grab root= from the old command line */
- if (!param) {
- FILE *fp;
- char *last_cmdline = NULL;
- char *old_param;
-
- strcpy(filename, pathname);
- strcat(filename, "bootargs");
- fp = fopen(filename, "r");
- if (fp) {
- if (getline(&last_cmdline, &cmd_len, fp) == -1)
- die("unable to read %s\n", filename);
-
- param = strstr(last_cmdline, "root=");
- if (param) {
- old_param = strtok(param, " ");
- if (cmd_len != 0)
- strcat(local_cmdline, " ");
- strcat(local_cmdline, old_param);
- }
- }
- if (last_cmdline)
- free(last_cmdline);
- }
- strcat(local_cmdline, " ");
- cmd_len = strlen(local_cmdline);
cmd_len = cmd_len + 1;
/* add new bootargs */
More information about the kexec
mailing list