[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qemu-iotests 069 and 111 are failing on NetBSD
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] qemu-iotests 069 and 111 are failing on NetBSD |
Date: |
Wed, 24 Jul 2019 18:29:08 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 24/07/19 11:34, Thomas Huth wrote:
> In case somebody is interested, two of the "auto" iotests are failing
> on NetBSD due to non-matching output:
>
> TEST iotest-qcow2: 069 [fail]
> --- /var/tmp/qemu-test.1BMupF/tests/qemu-iotests/069.out 2019-07-24
> 09:19:22.000000000 +0000
> +++ /var/tmp/qemu-test.1BMupF/tests/qemu-iotests/069.out.bad 2019-07-24
> 09:21:34.000000000 +0000
> @@ -4,5 +4,5 @@
>
> Formatting 'TEST_DIR/t.IMGFMT.base', fmt=IMGFMT size=131072
> Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=131072
> backing_file=TEST_DIR/t.IMGFMT.base
> -qemu-io: can't open device TEST_DIR/t.IMGFMT: Could not open backing file:
> Could not open 'TEST_DIR/t.IMGFMT.base': No such file or directory
> +qemu-io: can't open device TEST_DIR/t.IMGFMT: Could not open backing file:
> TEST_DIR/t.IMGFMT.base: stat failed: No such file or directory
> *** done
>
> and:
>
> TEST iotest-qcow2: 111 [fail]
> --- /var/tmp/qemu-test.1BMupF/tests/qemu-iotests/111.out 2019-07-24
> 09:19:22.000000000 +0000
> +++ /var/tmp/qemu-test.1BMupF/tests/qemu-iotests/111.out.bad 2019-07-24
> 09:21:40.000000000 +0000
> @@ -1,4 +1,4 @@
> QA output created by 111
> -qemu-img: TEST_DIR/t.IMGFMT: Could not open 'TEST_DIR/t.IMGFMT.inexistent':
> No such file or directory
> +qemu-img: TEST_DIR/t.IMGFMT: TEST_DIR/t.IMGFMT.inexistent: stat failed: No
> such file or directory
> Could not open backing image to determine size.
> *** done
>
> It's currently not a problem yet since we're not running the
> iotests on NetBSD yet (since our netbsd VM image does not have
> bash and gsed installed yet), but if somebody has some spare
> minutes, it would be great if this could be fixed so that we
> can enable the iotests on NetBSD, too, one day...
Is this (slightly ridiculous but effective) patch enough?
diff --git a/block/file-posix.c b/block/file-posix.c
index 73a001ceb7..ce847f4d62 100644
--- a/block/file-posix.c
+++ b/block/file-posix.c
@@ -217,7 +217,7 @@ static int raw_normalize_devicepath(const char **filename,
Error **errp)
fname = *filename;
dp = strrchr(fname, '/');
if (lstat(fname, &sb) < 0) {
- error_setg_errno(errp, errno, "%s: stat failed", fname);
+ error_setg_errno(errp, errno, "Could not open: '%s'", fname);
return -errno;
}
Paolo