diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-03-19 11:44:27 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-03-19 11:48:21 +0000 |
| commit | b9e052ed6b604f0049bcfa968a57f15d6e3d6395 (patch) | |
| tree | 0a71f2b9e65d8427cb61bca8b6786446e229a90b /scripts/qemuimage-tests | |
| parent | 7032d559c04bae09e6b7c39ddeb1bf35acc0584c (diff) | |
| download | openembedded-core-b9e052ed6b604f0049bcfa968a57f15d6e3d6395.tar.gz openembedded-core-b9e052ed6b604f0049bcfa968a57f15d6e3d6395.tar.bz2 openembedded-core-b9e052ed6b604f0049bcfa968a57f15d6e3d6395.zip | |
qemu-testlib: Add python helper and simplify shell
The current code has a race since it greps for *any* qemu process
running, even if it isn't the one we started. This leads to some sanity
tests potentially failing on machines where multiple sets of sanity tests
are running.
To resovle this and some other ugly code issues, add a python script
to accurately walk the process tree and find the qemu process. We can
then replace all the shell functions attempting this which happen to
work in many cases but not all.
Also clean up some of the error handling so its more legible.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/qemuimage-tests')
| -rwxr-xr-x | scripts/qemuimage-tests/sanity/shutdown | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/qemuimage-tests/sanity/shutdown b/scripts/qemuimage-tests/sanity/shutdown index d55c859410..c9e931c4c2 100755 --- a/scripts/qemuimage-tests/sanity/shutdown +++ b/scripts/qemuimage-tests/sanity/shutdown @@ -45,7 +45,7 @@ if [ $RET -eq 0 ]; then # If qemu start up process ends up, it means shutdown completes while [ $i -lt $TIMEOUT ] do - ps -fp $PID > /dev/null + ps -fp $QEMUPID > /dev/null 2> /dev/null if [ $? -ne 0 ]; then RET=0 break |