Re: [Qemu-devel] [PATCH v3 1/5] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests

2014-04-29 Thread Eric Blake
On 04/29/2014 08:06 PM, Fam Zheng wrote: + +if [ "${BASH_VERSINFO[0]}" -ge "4" ] && [ "${BASH_VERSINFO[1]}" -ge "1" ] >>> >>> This condition will be false for 5.0. >>> >> >> Oops, yup. > > Question of ignorance, is there a Bash 5.0 already? Nope; bash 4.3 is barely more than

Re: [Qemu-devel] [PATCH v3 1/5] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests

2014-04-29 Thread Fam Zheng
On Tue, 04/29 10:11, Jeff Cody wrote: > On Tue, Apr 29, 2014 at 03:26:55PM +0200, Kevin Wolf wrote: > > Am 10.04.2014 um 22:47 hat Jeff Cody geschrieben: > > > This creates some common functions for bash language qemu-iotests > > > to control, and communicate with, a running QEMU process. > > > >

Re: [Qemu-devel] [PATCH v3 1/5] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests

2014-04-29 Thread Jeff Cody
On Tue, Apr 29, 2014 at 03:26:55PM +0200, Kevin Wolf wrote: > Am 10.04.2014 um 22:47 hat Jeff Cody geschrieben: > > This creates some common functions for bash language qemu-iotests > > to control, and communicate with, a running QEMU process. > > > > 4 functions are introduced: > > > > 1. _l

Re: [Qemu-devel] [PATCH v3 1/5] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests

2014-04-29 Thread Kevin Wolf
Am 10.04.2014 um 22:47 hat Jeff Cody geschrieben: > This creates some common functions for bash language qemu-iotests > to control, and communicate with, a running QEMU process. > > 4 functions are introduced: > > 1. _launch_qemu() > This launches the QEMU process(es), and sets up the

[Qemu-devel] [PATCH v3 1/5] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests

2014-04-10 Thread Jeff Cody
This creates some common functions for bash language qemu-iotests to control, and communicate with, a running QEMU process. 4 functions are introduced: 1. _launch_qemu() This launches the QEMU process(es), and sets up the file descriptors and fifos for communication. You can