From: Jeff Cody <jcody@redhat.com>
To: Max Reitz <mreitz@redhat.com>
Cc: qemu-block@nongnu.org, Kevin Wolf <kwolf@redhat.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 2/2] iotests: Redirect stderr to stdout in 186
Date: Wed, 26 Jul 2017 18:20:15 -0400 [thread overview]
Message-ID: <20170726222015.GB22129@localhost.localdomain> (raw)
In-Reply-To: <20170725155644.20214-3-mreitz@redhat.com>
On Tue, Jul 25, 2017 at 05:56:44PM +0200, Max Reitz wrote:
> Without redirecting qemu's stderr to stdout, _filter_qemu will not apply
> to warnings. This results in $QEMU_PROG not being replaced by QEMU_PROG
> which is not great if your qemu executable is not called
> qemu-system-x86_64 (e.g. qemu-system-i386).
>
> Signed-off-by: Max Reitz <mreitz@redhat.com>
> ---
> tests/qemu-iotests/186 | 2 +-
> tests/qemu-iotests/186.out | 12 ++++++------
> 2 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/tests/qemu-iotests/186 b/tests/qemu-iotests/186
> index ab83ee4..2b9f618 100755
> --- a/tests/qemu-iotests/186
> +++ b/tests/qemu-iotests/186
> @@ -56,7 +56,7 @@ function do_run_qemu()
> done
> fi
> echo quit
> - ) | $QEMU -S -nodefaults -display none -device virtio-scsi-pci -monitor stdio "$@"
> + ) | $QEMU -S -nodefaults -display none -device virtio-scsi-pci -monitor stdio "$@" 2>&1
> echo
> }
>
> diff --git a/tests/qemu-iotests/186.out b/tests/qemu-iotests/186.out
> index b8bf9a2..c8377fe 100644
> --- a/tests/qemu-iotests/186.out
> +++ b/tests/qemu-iotests/186.out
> @@ -442,28 +442,28 @@ ide0-cd0 (NODE_NAME): null-co:// (null-co, read-only)
> Cache mode: writeback
> (qemu) quit
>
> -qemu-system-x86_64: -drive if=scsi,driver=null-co: warning: bus=0,unit=0 is deprecated with this machine type
> Testing: -drive if=scsi,driver=null-co
> QEMU X.Y.Z monitor - type 'help' for more information
> -(qemu) info block
> +(qemu) QEMU_PROG: -drive if=scsi,driver=null-co: warning: bus=0,unit=0 is deprecated with this machine type
> +info block
> scsi0-hd0 (NODE_NAME): null-co:// (null-co)
> Attached to: /machine/unattached/device[27]/scsi.0/legacy[0]
> Cache mode: writeback
> (qemu) quit
>
> -qemu-system-x86_64: -drive if=scsi,media=cdrom: warning: bus=0,unit=0 is deprecated with this machine type
> Testing: -drive if=scsi,media=cdrom
> QEMU X.Y.Z monitor - type 'help' for more information
> -(qemu) info block
> +(qemu) QEMU_PROG: -drive if=scsi,media=cdrom: warning: bus=0,unit=0 is deprecated with this machine type
> +info block
> scsi0-cd0: [not inserted]
> Attached to: /machine/unattached/device[27]/scsi.0/legacy[0]
> Removable device: not locked, tray closed
> (qemu) quit
>
> -qemu-system-x86_64: -drive if=scsi,driver=null-co,media=cdrom: warning: bus=0,unit=0 is deprecated with this machine type
> Testing: -drive if=scsi,driver=null-co,media=cdrom
> QEMU X.Y.Z monitor - type 'help' for more information
> -(qemu) info block
> +(qemu) QEMU_PROG: -drive if=scsi,driver=null-co,media=cdrom: warning: bus=0,unit=0 is deprecated with this machine type
> +info block
> scsi0-cd0 (NODE_NAME): null-co:// (null-co, read-only)
> Attached to: /machine/unattached/device[27]/scsi.0/legacy[0]
> Removable device: not locked, tray closed
> --
> 2.9.4
>
>
Reviewed-by: Jeff Cody <jcody@redhat.com>
next prev parent reply other threads:[~2017-07-26 22:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-25 15:56 [Qemu-devel] [PATCH 0/2] iotests: Fixes to 156 and 186 Max Reitz
2017-07-25 15:56 ` [Qemu-devel] [PATCH 1/2] iotests: Fix test 156 Max Reitz
2017-07-25 17:26 ` Eric Blake
2017-07-26 22:19 ` Jeff Cody
2017-07-25 15:56 ` [Qemu-devel] [PATCH 2/2] iotests: Redirect stderr to stdout in 186 Max Reitz
2017-07-25 17:50 ` Eric Blake
2017-07-26 22:20 ` Jeff Cody [this message]
2017-07-27 12:24 ` [Qemu-devel] [PATCH 0/2] iotests: Fixes to 156 and 186 Kevin Wolf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170726222015.GB22129@localhost.localdomain \
--to=jcody@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.