mirror of
https://github.com/clearlinux/kvmtool.git
synced 2026-06-16 02:15:47 +00:00
kvmtool/run: append cfg.kernel_cmdline at the end of real_cmdline
This allows the user to always override the paramaters set by lkvm.
Say, currently 'lkvm run -p ro' doesn't work.
To keep the current logic we need to change strstr("root=") to check
cfg.kernel_cmdline, not real_cmdline. And perhaps we can even add a
simple helper add_param(name, val) to make this all more consistent;
it should only append "name=val" to real_cmdline if cfg.kernel_cmdline
doesn't include this paramater.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
This commit is contained in:
committed by
Will Deacon
parent
3695adeb22
commit
8a7163f3db
+6
-7
@@ -566,12 +566,6 @@ static struct kvm *kvm_cmd_run_init(int argc, const char **argv)
|
||||
memset(real_cmdline, 0, sizeof(real_cmdline));
|
||||
kvm__arch_set_cmdline(real_cmdline, kvm->cfg.vnc || kvm->cfg.sdl || kvm->cfg.gtk);
|
||||
|
||||
if (strlen(real_cmdline) > 0)
|
||||
strcat(real_cmdline, " ");
|
||||
|
||||
if (kvm->cfg.kernel_cmdline)
|
||||
strlcat(real_cmdline, kvm->cfg.kernel_cmdline, sizeof(real_cmdline));
|
||||
|
||||
if (!kvm->cfg.guest_name) {
|
||||
if (kvm->cfg.custom_rootfs) {
|
||||
kvm->cfg.guest_name = kvm->cfg.custom_rootfs_name;
|
||||
@@ -607,10 +601,15 @@ static struct kvm *kvm_cmd_run_init(int argc, const char **argv)
|
||||
if (kvm_setup_guest_init(kvm->cfg.custom_rootfs_name))
|
||||
die("Failed to setup init for guest.");
|
||||
}
|
||||
} else if (!strstr(real_cmdline, "root=")) {
|
||||
} else if (!strstr(kvm->cfg.kernel_cmdline, "root=")) {
|
||||
strlcat(real_cmdline, " root=/dev/vda rw ", sizeof(real_cmdline));
|
||||
}
|
||||
|
||||
if (kvm->cfg.kernel_cmdline) {
|
||||
strcat(real_cmdline, " ");
|
||||
strlcat(real_cmdline, kvm->cfg.kernel_cmdline, sizeof(real_cmdline));
|
||||
}
|
||||
|
||||
kvm->cfg.real_cmdline = real_cmdline;
|
||||
|
||||
printf(" # %s run -k %s -m %Lu -c %d --name %s\n", KVM_BINARY_NAME,
|
||||
|
||||
Reference in New Issue
Block a user