From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46320) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQGiT-0008Ue-2e for qemu-devel@nongnu.org; Wed, 19 Mar 2014 09:39:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WQGiM-0005IZ-Lc for qemu-devel@nongnu.org; Wed, 19 Mar 2014 09:39:32 -0400 Received: from lputeaux-656-01-25-125.w80-12.abo.wanadoo.fr ([80.12.84.125]:56317 helo=paradis.irqsave.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQGiM-0005IP-D0 for qemu-devel@nongnu.org; Wed, 19 Mar 2014 09:39:26 -0400 Date: Wed, 19 Mar 2014 14:39:25 +0100 From: =?iso-8859-1?Q?Beno=EEt?= Canet Message-ID: <20140319133925.GA3019@irqsave.net> References: <5e57ec8dce227b3095dd476e893137f2b14c0d81.1395105370.git.jcody@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <5e57ec8dce227b3095dd476e893137f2b14c0d81.1395105370.git.jcody@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 1/4] block: qemu-iotests - add common.qemu, for bash-controlled qemu tests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jeff Cody Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com The Monday 17 Mar 2014 =E0 21:24:37 (-0400), Jeff Cody wrote : > This creates some common functions for bash language qemu-iotests > to control, and communicate with, a running QEMU process. >=20 > 4 functions are introduced: >=20 > 1. _launch_qemu() > This launches the QEMU process(es), and sets up the file > descriptors and fifos for communication. You can choose to > launch each QEMU process listening for either QMP or HMP > monitor. You can call this function multiple times, and > save the handle returned from each. >=20 > Commands 2 and 3 use the handle received from _launch_qemu(), to talk > to the appropriate process. >=20 > 2. _send_qemu_cmd() > Sends a command string, specified by $2, to QEMU. If $2 is > non-NULL, will wait for it as the required resulting. Failure "will wait for it as the required resulting" I don't understand this part= of the sentence, probably because I am not a native speaker. > to receive $3 will cause the test to fail. >=20 > 3. _timed_wait_for() > Waits for a response, for up to a default of 10 seconds. If > $2 is not seen in that time (anywhere in the response), then > the test fails. Primarily used by _send_qemu_cmd, but could > be useful standalone, as well. >=20 > 4. _cleanup_qemu() > Kills the running QEMU processes, and removes the fifos. >=20 > Signed-off-by: Jeff Cody > --- > tests/qemu-iotests/common.qemu | 164 +++++++++++++++++++++++++++++++++= ++++++++ > 1 file changed, 164 insertions(+) > create mode 100644 tests/qemu-iotests/common.qemu >=20 > diff --git a/tests/qemu-iotests/common.qemu b/tests/qemu-iotests/common= .qemu > new file mode 100644 > index 0000000..8068395 > --- /dev/null > +++ b/tests/qemu-iotests/common.qemu > @@ -0,0 +1,164 @@ > +#!/bin/bash > +# > +# This allows for launching of multiple QEMU instances, with independe= nt > +# communication possible to each instance. > +# > +# Each instance can choose, at launch, to use either the QMP or the > +# HMP (monitor) interface. > +# > +# All instances are cleaned up via _cleanup_qemu, including killing th= e > +# running qemu instance. > +# > +# Copyright (C) 2014 Red Hat, Inc. > +# > +# This program is free software; you can redistribute it and/or modify > +# it under the terms of the GNU General Public License as published by > +# the Free Software Foundation; either version 2 of the License, or > +# (at your option) any later version. > +# > +# This program is distributed in the hope that it will be useful, > +# but WITHOUT ANY WARRANTY; without even the implied warranty of > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > +# GNU General Public License for more details. > +# > +# You should have received a copy of the GNU General Public License > +# along with this program. If not, see = . > +# > + > +QEMU_COMM_TIMEOUT=3D10 > + > +QEMU_FIFO_IN=3D"${TEST_DIR}/qmp-in-$$" > +QEMU_FIFO_OUT=3D"${TEST_DIR}/qmp-out-$$" > + > +QEMU_PID=3D > +_QEMU_HANDLE=3D0 > +QEMU_HANDLE=3D0 > + > +# If bash version is >=3D 4.1, these will be overwritten and dynamic > +# file descriptor values assigned. > +_out_fd=3D3 > +_in_fd=3D4 > + > +# Wait for expected QMP response from QEMU. Will time out > +# after 10 seconds, which counts as failure. > +# > +# Override QEMU_COMM_TIMEOUT for a timeout different than the > +# default 10 seconds > +# > +# $1: The handle to use > +# $2+ All remaining arguments comprise the string to search for > +# in the response. > +# > +# If $silent is set to anything but an empty string, then > +# response is not echoed out. > +function _timed_wait_for() > +{ > + local h=3D${1} > + shift > + while read -t ${QEMU_COMM_TIMEOUT} resp <&${QEMU_OUT[$h]} > + do > + if [ -z "${silent}" ]; then > + echo "${resp}" | _filter_testdir | _filter_qemu > + fi > + grep -q "${*}" < <(echo ${resp}) > + if [ $? -eq 0 ]; then > + return > + fi > + done > + echo "Timeout waiting for ${*} on handle ${h}" > + exit 1 # Timeout means the test failed > +} > + > + > +# Sends QMP or HMP command to QEMU, and waits for the expected respons= e > +# > +# $1: QEMU handle to use > +# $2: String of the QMP command to send > +# ${@: -1} (Last string passed) > +# String that the QEMU response should contain. If $2 is a= null > +# string, do not wait for a response > +function _send_qemu_cmd() > +{ > + local h=3D${1} > + shift > + # This array element extraction is done to accomodate pathnames wi= th spaces > + echo "${@: 1:${#@}-1}" >&${QEMU_IN[${h}]} > + shift > + > + if [ -n "${1}" ] > + then > + _timed_wait_for ${h} "${@: -1}" You have done shift before this. Aren't ${*} the remaining strings to wai= t for ? > + fi > +} > + > + > +# Launch a QEMU process. > +# > +# Input parameters: > +# $qemu_comm_method: set this variable to 'monitor' (case insensitive) > +# to use the QEMU HMP monitor for communication. > +# Otherwise, the default of QMP is used. > +# Returns: > +# $QEMU_HANDLE: set to a handle value to communicate with this QEMU in= stance. > +# > +function _launch_qemu() > +{ > + local comm=3D > + local fifo_out=3D > + local fifo_in=3D > + > + if (shopt -s nocasematch; [[ "${qemu_comm_method}" =3D=3D "monitor= " ]]) > + then > + comm=3D"-monitor stdio -qmp none" > + else > + local qemu_comm_method=3D"qmp" > + comm=3D"-monitor none -qmp stdio" > + fi > + > + fifo_out=3D${QEMU_FIFO_OUT}_${_QEMU_HANDLE} > + fifo_in=3D${QEMU_FIFO_IN}_${_QEMU_HANDLE} > + mkfifo "${fifo_out}" > + mkfifo "${fifo_in}" > + > + "${QEMU}" -nographic -serial none ${comm} "${@}" 2>&1 \ > + >"${fifo_out}" \ > + <"${fifo_in}" & > + QEMU_PID[${_QEMU_HANDLE}]=3D$! > + > + if [ "${BASH_VERSINFO[0]}" -ge "4" ] && [ "${BASH_VERSINFO[1]}" -g= e "1" ] > + then > + # bash >=3D 4.1 required for automatic fd > + exec {_out_fd}<"${fifo_out}" > + exec {_in_fd}>"${fifo_in}" Isn't it ${_out_fd} and ${_in_fd} ? > + else > + let _out_fd++ > + let _in_fd++ > + eval "exec ${_out_fd}<'${fifo_out}'" > + eval "exec ${_in_fd}>'${fifo_in}'" > + fi > + > + QEMU_OUT[${_QEMU_HANDLE}]=3D${_out_fd} > + QEMU_IN[${_QEMU_HANDLE}]=3D${_in_fd} > + > + if [ "${qemu_comm_method}" =3D=3D "qmp" ] > + then > + # Don't print response, since it has version information in it > + silent=3Dyes _timed_wait_for ${_QEMU_HANDLE} "capabilities" > + fi > + QEMU_HANDLE=3D${_QEMU_HANDLE} > + let _QEMU_HANDLE++ > +} > + > + > +# Silenty kills the QEMU process > +function _cleanup_qemu() > +{ > + # QEMU_PID[], QEMU_IN[], QEMU_OUT[] all use same indices > + for i in "${!QEMU_OUT[@]}" > + do > + kill -KILL ${QEMU_PID[$i]} > + wait ${QEMU_PID[$i]} 2>/dev/null # silent kill > + rm -f "${QEMU_FIFO_IN}_${i}" "${QEMU_FIFO_OUT}_${i}" > + done > +} > + > --=20 > 1.8.3.1 >=20