search

hyperhq / runv

Hypervisor-based Runtime for OCI
Apache License 2.0
828 stars 129 forks source link

add missing close(waitQmp) #425

Closed laijs closed 7 years ago

laijs commented 7 years ago

after vm shutdown, the goroutine was stalled on qemu.(*QemuContext).Close(). fix it by adding missing close(waitQmp).

       0x9f47e4
github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor/qemu.(*QemuContext).Close+0x84
/root/work/gopath/src/github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor/qemu/qemu.go:181
       0x683650
github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor.(*VmContext).Close+0x130
/root/work/gopath/src/github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor/context.go:224
       0x695e93
github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor.stateRunning+0x1b3
/root/work/gopath/src/github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor/vm_states.go:227
       0x68819e
github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor.(*VmContext).loop+0x29e
/root/work/gopath/src/github.com/hyperhq/hyperd/vendor/github.com/hyperhq/runv/hypervisor/hypervisor.go:24

Signed-off-by: Lai Jiangshan jiangshanlai@gmail.com