summaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorDmitry Rozhkov <dmitry.rozhkov@linux.intel.com>2017-02-21 17:18:08 +0200
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-09-04 17:38:09 +0100
commit0255195de483587659aa28aeefcec379526b088e (patch)
tree0f4f8b6d89d6c4f28e8f2ba3bdf65769e2015aae /scripts
parent2a70e84643381eca0e7bf7928d4a3d56f9651128 (diff)
downloadpoky-0255195de483587659aa28aeefcec379526b088e.tar.gz
scripts/runqemu: avoid overridden user input for bootparams
Currently runqemu hardcodes the "ip=" kernel boot parameter when configuring QEMU to use tap or slirp networking. This makes the guest system to have a network interface pre-configured by kernel and causes systemd to fail renaming the interface to whatever pleases it: Feb 21 10:10:20 intel-corei7-64 systemd-udevd[201]: Error changing net interface name 'eth0' to 'enp0s3': Device or resource busy, Always append user input for kernel boot params after the ones added by the script. This way user input has priority over runqemu's default params. (From OE-Core rev: 3f68b5c8d24b52aed5bb3ed970dd8f779b65b1b3) (From OE-Core rev: 6050b4ac146722d6714589225ad58ccc26c9ca32) Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> Signed-off-by: Armin Kuster <akuster808@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/runqemu7
1 files changed, 5 insertions, 2 deletions
diff --git a/scripts/runqemu b/scripts/runqemu
index 1df6875111..6748cb258a 100755
--- a/scripts/runqemu
+++ b/scripts/runqemu
@@ -167,6 +167,7 @@ class BaseConfig(object):
167 self.kernel = '' 167 self.kernel = ''
168 self.kernel_cmdline = '' 168 self.kernel_cmdline = ''
169 self.kernel_cmdline_script = '' 169 self.kernel_cmdline_script = ''
170 self.bootparams = ''
170 self.dtb = '' 171 self.dtb = ''
171 self.fstype = '' 172 self.fstype = ''
172 self.kvm_enabled = False 173 self.kvm_enabled = False
@@ -368,7 +369,7 @@ class BaseConfig(object):
368 elif arg.startswith('qemuparams='): 369 elif arg.startswith('qemuparams='):
369 self.qemu_opt_script += ' %s' % arg[len('qemuparams='):] 370 self.qemu_opt_script += ' %s' % arg[len('qemuparams='):]
370 elif arg.startswith('bootparams='): 371 elif arg.startswith('bootparams='):
371 self.kernel_cmdline_script += ' %s' % arg[len('bootparams='):] 372 self.bootparams = arg[len('bootparams='):]
372 elif os.path.exists(arg) or (re.search(':', arg) and re.search('/', arg)): 373 elif os.path.exists(arg) or (re.search(':', arg) and re.search('/', arg)):
373 self.check_arg_path(os.path.abspath(arg)) 374 self.check_arg_path(os.path.abspath(arg))
374 elif re.search('-image-', arg): 375 elif re.search('-image-', arg):
@@ -954,7 +955,9 @@ class BaseConfig(object):
954 955
955 def start_qemu(self): 956 def start_qemu(self):
956 if self.kernel: 957 if self.kernel:
957 kernel_opts = "-kernel %s -append '%s %s %s'" % (self.kernel, self.kernel_cmdline, self.kernel_cmdline_script, self.get('QB_KERNEL_CMDLINE_APPEND')) 958 kernel_opts = "-kernel %s -append '%s %s %s %s'" % (self.kernel, self.kernel_cmdline,
959 self.kernel_cmdline_script, self.get('QB_KERNEL_CMDLINE_APPEND'),
960 self.bootparams)
958 if self.dtb: 961 if self.dtb:
959 kernel_opts += " -dtb %s" % self.dtb 962 kernel_opts += " -dtb %s" % self.dtb
960 else: 963 else: