diff options
| author | Alejandro Hernandez <alejandro.hernandez@linux.intel.com> | 2015-08-06 18:44:22 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-08-09 00:13:56 -0700 |
| commit | 0658434ee3c97bc27b9aa7bde682309af5c2f3c3 (patch) | |
| tree | 7a2db317e0c1e129b2bd6b5f3dd4bf1a111ad8fe | |
| parent | 58197f345b203f55de202f8c91129e3d381359ed (diff) | |
| download | poky-0658434ee3c97bc27b9aa7bde682309af5c2f3c3.tar.gz | |
qemurunner: Improves checking for server and target IPs on qemus parameters
Fixes OS hanging infinitely waiting for qemus process to release bitbake.lock
(From OE-Core rev: d168bf34c553dbe5de7511e158cd83869d7a88bc)
Signed-off-by: Alejandro Hernandez <alejandro.hernandez@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/lib/oeqa/utils/qemurunner.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py index 1cf8f7668c..02ac89c142 100644 --- a/meta/lib/oeqa/utils/qemurunner.py +++ b/meta/lib/oeqa/utils/qemurunner.py | |||
| @@ -120,14 +120,17 @@ class QemuRunner: | |||
| 120 | cmdline = '' | 120 | cmdline = '' |
| 121 | with open('/proc/%s/cmdline' % self.qemupid) as p: | 121 | with open('/proc/%s/cmdline' % self.qemupid) as p: |
| 122 | cmdline = p.read() | 122 | cmdline = p.read() |
| 123 | ips = re.findall("((?:[0-9]{1,3}\.){3}[0-9]{1,3})", cmdline.split("ip=")[1]) | 123 | try: |
| 124 | if not ips or len(ips) != 3: | 124 | ips = re.findall("((?:[0-9]{1,3}\.){3}[0-9]{1,3})", cmdline.split("ip=")[1]) |
| 125 | if not ips or len(ips) != 3: | ||
| 126 | raise ValueError | ||
| 127 | else: | ||
| 128 | self.ip = ips[0] | ||
| 129 | self.server_ip = ips[1] | ||
| 130 | except IndexError, ValueError: | ||
| 125 | logger.info("Couldn't get ip from qemu process arguments! Here is the qemu command line used: %s" % cmdline) | 131 | logger.info("Couldn't get ip from qemu process arguments! Here is the qemu command line used: %s" % cmdline) |
| 126 | self.stop() | 132 | self.stop() |
| 127 | return False | 133 | return False |
| 128 | else: | ||
| 129 | self.ip = ips[0] | ||
| 130 | self.server_ip = ips[1] | ||
| 131 | logger.info("Target IP: %s" % self.ip) | 134 | logger.info("Target IP: %s" % self.ip) |
| 132 | logger.info("Server IP: %s" % self.server_ip) | 135 | logger.info("Server IP: %s" % self.server_ip) |
| 133 | logger.info("Waiting at most %d seconds for login banner" % self.boottime) | 136 | logger.info("Waiting at most %d seconds for login banner" % self.boottime) |
