qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186
@ 2017-07-25  8:59 Kevin Wolf
  2017-07-25 11:11 ` Markus Armbruster
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Kevin Wolf @ 2017-07-25  8:59 UTC (permalink / raw)
  To: qemu-block; +Cc: kwolf, armbru, qemu-devel

Commits 70f17a1 ('error: Revert unwanted change of warning messages')
and e1824e5 ('qemu-iotests: Test 'info block'') had a semantic merge
conflict, which results in failure for qemu-iotests case 186. Fix the
reference output to consider the changes of 70f17a1.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
 tests/qemu-iotests/186.out | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/qemu-iotests/186.out b/tests/qemu-iotests/186.out
index b963b12..b8bf9a2 100644
--- a/tests/qemu-iotests/186.out
+++ b/tests/qemu-iotests/186.out
@@ -442,7 +442,7 @@ ide0-cd0 (NODE_NAME): null-co:// (null-co, read-only)
     Cache mode:       writeback
 (qemu) quit
 
-warning: qemu-system-x86_64: -drive if=scsi,driver=null-co: bus=0,unit=0 is deprecated with this machine type
+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
@@ -451,7 +451,7 @@ scsi0-hd0 (NODE_NAME): null-co:// (null-co)
     Cache mode:       writeback
 (qemu) quit
 
-warning: qemu-system-x86_64: -drive if=scsi,media=cdrom: bus=0,unit=0 is deprecated with this machine type
+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
@@ -460,7 +460,7 @@ scsi0-cd0: [not inserted]
     Removable device: not locked, tray closed
 (qemu) quit
 
-warning: qemu-system-x86_64: -drive if=scsi,driver=null-co,media=cdrom: bus=0,unit=0 is deprecated with this machine type
+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
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186
  2017-07-25  8:59 [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186 Kevin Wolf
@ 2017-07-25 11:11 ` Markus Armbruster
  2017-07-25 12:50 ` Stefan Hajnoczi
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Markus Armbruster @ 2017-07-25 11:11 UTC (permalink / raw)
  To: Kevin Wolf; +Cc: qemu-block, qemu-devel

Kevin Wolf <kwolf@redhat.com> writes:

> Commits 70f17a1 ('error: Revert unwanted change of warning messages')
> and e1824e5 ('qemu-iotests: Test 'info block'') had a semantic merge
> conflict, which results in failure for qemu-iotests case 186. Fix the
> reference output to consider the changes of 70f17a1.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>

Reviewed-by: Markus Armbruster <armbru@redhat.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186
  2017-07-25  8:59 [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186 Kevin Wolf
  2017-07-25 11:11 ` Markus Armbruster
@ 2017-07-25 12:50 ` Stefan Hajnoczi
  2017-07-25 14:55 ` [Qemu-devel] [Qemu-block] " Max Reitz
  2017-07-25 15:11 ` Max Reitz
  3 siblings, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2017-07-25 12:50 UTC (permalink / raw)
  To: Kevin Wolf; +Cc: qemu-block, armbru, qemu-devel

[-- Attachment #1: Type: text/plain, Size: 542 bytes --]

On Tue, Jul 25, 2017 at 10:59:36AM +0200, Kevin Wolf wrote:
> Commits 70f17a1 ('error: Revert unwanted change of warning messages')
> and e1824e5 ('qemu-iotests: Test 'info block'') had a semantic merge
> conflict, which results in failure for qemu-iotests case 186. Fix the
> reference output to consider the changes of 70f17a1.
> 
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
>  tests/qemu-iotests/186.out | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [Qemu-block] [PATCH] qemu-iotests: Fix reference output for 186
  2017-07-25  8:59 [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186 Kevin Wolf
  2017-07-25 11:11 ` Markus Armbruster
  2017-07-25 12:50 ` Stefan Hajnoczi
@ 2017-07-25 14:55 ` Max Reitz
  2017-07-25 15:11 ` Max Reitz
  3 siblings, 0 replies; 5+ messages in thread
From: Max Reitz @ 2017-07-25 14:55 UTC (permalink / raw)
  To: Kevin Wolf, qemu-block; +Cc: armbru, qemu-devel

[-- Attachment #1: Type: text/plain, Size: 562 bytes --]

On 2017-07-25 10:59, Kevin Wolf wrote:
> Commits 70f17a1 ('error: Revert unwanted change of warning messages')
> and e1824e5 ('qemu-iotests: Test 'info block'') had a semantic merge
> conflict, which results in failure for qemu-iotests case 186. Fix the
> reference output to consider the changes of 70f17a1.
> 
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
>  tests/qemu-iotests/186.out | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Thanks, applied to my block tree:

https://github.com/XanClic/qemu/commits/block

Max


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 498 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [Qemu-block] [PATCH] qemu-iotests: Fix reference output for 186
  2017-07-25  8:59 [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186 Kevin Wolf
                   ` (2 preceding siblings ...)
  2017-07-25 14:55 ` [Qemu-devel] [Qemu-block] " Max Reitz
@ 2017-07-25 15:11 ` Max Reitz
  3 siblings, 0 replies; 5+ messages in thread
From: Max Reitz @ 2017-07-25 15:11 UTC (permalink / raw)
  To: Kevin Wolf, qemu-block; +Cc: armbru, qemu-devel

[-- Attachment #1: Type: text/plain, Size: 670 bytes --]

On 2017-07-25 10:59, Kevin Wolf wrote:
> Commits 70f17a1 ('error: Revert unwanted change of warning messages')
> and e1824e5 ('qemu-iotests: Test 'info block'') had a semantic merge
> conflict, which results in failure for qemu-iotests case 186. Fix the
> reference output to consider the changes of 70f17a1.
> 
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
>  tests/qemu-iotests/186.out | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

I've noticed that test 186 fails when not using qemu-system-x86_64,
though (just build with --target-list=i386-softmmu, for example). But
this patch doesn't make it worse, so I'll keep it.

Max


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 498 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-07-25 15:11 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-25  8:59 [Qemu-devel] [PATCH] qemu-iotests: Fix reference output for 186 Kevin Wolf
2017-07-25 11:11 ` Markus Armbruster
2017-07-25 12:50 ` Stefan Hajnoczi
2017-07-25 14:55 ` [Qemu-devel] [Qemu-block] " Max Reitz
2017-07-25 15:11 ` Max Reitz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).