diff options
Diffstat (limited to 'results/classifier/105/graphic/391879')
| -rw-r--r-- | results/classifier/105/graphic/391879 | 54 |
1 files changed, 54 insertions, 0 deletions
diff --git a/results/classifier/105/graphic/391879 b/results/classifier/105/graphic/391879 new file mode 100644 index 000000000..17e9e81ed --- /dev/null +++ b/results/classifier/105/graphic/391879 @@ -0,0 +1,54 @@ +graphic: 0.924 +semantic: 0.919 +instruction: 0.910 +KVM: 0.907 +other: 0.876 +device: 0.833 +mistranslation: 0.832 +network: 0.791 +socket: 0.689 +vnc: 0.682 +boot: 0.590 +assembly: 0.473 + +migrate exec ignores exit status + +Binary package hint: kvm + +Using + + migrate "exec:cat > foo; false" + +in the monitor results in the state of the VM being written to foo, as expected, and the VM then being stopped. This is surprising, as I think it stands to reason that in case of a failed migrate-exec process, which is what a non-zero exit status implies to me, the VM should continue. + +== Version information + +$ lsb_release -rd +Description: Ubuntu 9.04 +Release: 9.04 + +$ apt-cache policy kvm +kvm: + Installed: 1:84+dfsg-0ubuntu11 + Candidate: 1:84+dfsg-0ubuntu11 + Version table: + *** 1:84+dfsg-0ubuntu11 0 + 500 http://gb.archive.ubuntu.com jaunty/main Packages + 100 /var/lib/dpkg/status + +Well, I have reproduced this behavior in Lucid's qemu-kvm 0.12.3, so the report is still accurate. + +I don't have a strong opinion on the desired behavior, though I can certainly see the bug reporter's point. + +This bug is filed against the upstream QEMU project, so we'll defer to Upstream's decision on this feature. Thanks for the report. + +This is a bug and has been reported upstream, it is unlikely to be fixed at the distribution level and therefore anyone interested in working on this bug should contribute a patch to the upstream project. This will then filter down to Ubuntu when it is merged mainline. Marking "Won't Fix" against the Ubuntu package. + +Thanks for reporting this bug. + +The attached patch works for me with the posted test case. + +Patch has been included here: +http://git.qemu.org/?p=qemu.git;a=commitdiff;h=41ef56e61153d7bd27d34a63463 +So I assume this should be working now. + |