1 | The following changes since commit 64175afc695c0672876fbbfc31b299c86d562cb4: | 1 | The following changes since commit d88d5a3806d78dcfca648c62dae9d88d3e803bd2: |
---|---|---|---|
2 | 2 | ||
3 | arm_gicv3: Fix ICC_BPR1 reset value when EL3 not implemented (2017-06-07 17:21:44 +0100) | 3 | Merge remote-tracking branch 'remotes/philmd-gitlab/tags/renesas-hw-20200622' into staging (2020-06-23 13:55:52 +0100) |
4 | 4 | ||
5 | are available in the git repository at: | 5 | are available in the Git repository at: |
6 | 6 | ||
7 | git://github.com/codyprime/qemu-kvm-jtc.git tags/block-pull-request | 7 | https://github.com/XanClic/qemu.git tags/pull-block-2020-06-24 |
8 | 8 | ||
9 | for you to fetch changes up to 56faeb9bb6872b3f926b3b3e0452a70beea10af2: | 9 | for you to fetch changes up to 24b861c0386a17ea31eb824310c21118fb7be883: |
10 | 10 | ||
11 | block/gluster.c: Handle qdict_array_entries() failure (2017-06-09 08:41:29 -0400) | 11 | iotests: don't test qcow2.py inside 291 (2020-06-24 10:00:04 +0200) |
12 | 12 | ||
13 | ---------------------------------------------------------------- | 13 | ---------------------------------------------------------------- |
14 | Gluster patch | 14 | Block patches: |
15 | - Two iotest fixes | ||
16 | |||
15 | ---------------------------------------------------------------- | 17 | ---------------------------------------------------------------- |
18 | This is v2, where I dropped Maxim’s LUKS keyslot amendment series and my | ||
19 | iotest patches, because both caused iotest failures on some test | ||
20 | machines. | ||
21 | ---------------------------------------------------------------- | ||
22 | Philippe Mathieu-Daudé (1): | ||
23 | iotests: Fix 051 output after qdev_init_nofail() removal | ||
16 | 24 | ||
17 | Peter Maydell (1): | 25 | Vladimir Sementsov-Ogievskiy (1): |
18 | block/gluster.c: Handle qdict_array_entries() failure | 26 | iotests: don't test qcow2.py inside 291 |
19 | 27 | ||
20 | block/gluster.c | 3 +-- | 28 | tests/qemu-iotests/051.pc.out | 4 ++-- |
21 | 1 file changed, 1 insertion(+), 2 deletions(-) | 29 | tests/qemu-iotests/291 | 4 ---- |
30 | tests/qemu-iotests/291.out | 33 --------------------------------- | ||
31 | 3 files changed, 2 insertions(+), 39 deletions(-) | ||
22 | 32 | ||
23 | -- | 33 | -- |
24 | 2.9.3 | 34 | 2.26.2 |
25 | 35 | ||
26 | 36 | diff view generated by jsdifflib |
New patch | |||
---|---|---|---|
1 | From: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
1 | 2 | ||
3 | Commit 96927c744 replaced qdev_init_nofail() call by | ||
4 | isa_realize_and_unref() which has a different error | ||
5 | message. Update the test output accordingly. | ||
6 | |||
7 | Gitlab CI error after merging b77b5b3dc7: | ||
8 | https://gitlab.com/qemu-project/qemu/-/jobs/597414772#L4375 | ||
9 | |||
10 | Reported-by: Thomas Huth <thuth@redhat.com> | ||
11 | Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> | ||
12 | Message-Id: <20200616154949.6586-1-philmd@redhat.com> | ||
13 | Reviewed-by: Thomas Huth <thuth@redhat.com> | ||
14 | Signed-off-by: Max Reitz <mreitz@redhat.com> | ||
15 | --- | ||
16 | tests/qemu-iotests/051.pc.out | 4 ++-- | ||
17 | 1 file changed, 2 insertions(+), 2 deletions(-) | ||
18 | |||
19 | diff --git a/tests/qemu-iotests/051.pc.out b/tests/qemu-iotests/051.pc.out | ||
20 | index XXXXXXX..XXXXXXX 100644 | ||
21 | --- a/tests/qemu-iotests/051.pc.out | ||
22 | +++ b/tests/qemu-iotests/051.pc.out | ||
23 | @@ -XXX,XX +XXX,XX @@ QEMU X.Y.Z monitor - type 'help' for more information | ||
24 | |||
25 | Testing: -drive if=ide | ||
26 | QEMU X.Y.Z monitor - type 'help' for more information | ||
27 | -(qemu) QEMU_PROG: Initialization of device ide-hd failed: Device needs media, but drive is empty | ||
28 | +(qemu) QEMU_PROG: Device needs media, but drive is empty | ||
29 | |||
30 | Testing: -drive if=virtio | ||
31 | QEMU X.Y.Z monitor - type 'help' for more information | ||
32 | @@ -XXX,XX +XXX,XX @@ QEMU X.Y.Z monitor - type 'help' for more information | ||
33 | |||
34 | Testing: -drive file=TEST_DIR/t.qcow2,if=ide,readonly=on | ||
35 | QEMU X.Y.Z monitor - type 'help' for more information | ||
36 | -(qemu) QEMU_PROG: Initialization of device ide-hd failed: Block node is read-only | ||
37 | +(qemu) QEMU_PROG: Block node is read-only | ||
38 | |||
39 | Testing: -drive file=TEST_DIR/t.qcow2,if=virtio,readonly=on | ||
40 | QEMU X.Y.Z monitor - type 'help' for more information | ||
41 | -- | ||
42 | 2.26.2 | ||
43 | |||
44 | diff view generated by jsdifflib |
1 | From: Peter Maydell <peter.maydell@linaro.org> | 1 | From: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> |
---|---|---|---|
2 | 2 | ||
3 | In qemu_gluster_parse_json(), the call to qdict_array_entries() | 3 | 820c6bee534ec3b added testing of qcow2.py into 291, and it breaks 291 |
4 | could return a negative error code, which we were ignoring | 4 | with external data file. Actually, 291 is bad place for qcow2.py |
5 | because we assigned the result to an unsigned variable. | 5 | testing, better add a separate test. |
6 | Fix this by using the 'int' type instead, which matches the | ||
7 | return type of qdict_array_entries() and also the type | ||
8 | we use for the loop enumeration variable 'i'. | ||
9 | 6 | ||
10 | (Spotted by Coverity, CID 1360960.) | 7 | For now, drop qcow2.py testing from 291 to fix the regression. |
11 | 8 | ||
12 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 9 | Fixes: 820c6bee534ec3b |
10 | Reported-by: Max Reitz <mreitz@redhat.com> | ||
11 | Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> | ||
12 | Message-Id: <20200618154052.8629-1-vsementsov@virtuozzo.com> | ||
13 | Reviewed-by: Eric Blake <eblake@redhat.com> | 13 | Reviewed-by: Eric Blake <eblake@redhat.com> |
14 | Reviewed-by: Jeff Cody <jcody@redhat.com> | 14 | Signed-off-by: Max Reitz <mreitz@redhat.com> |
15 | Message-id: 1496682098-1540-1-git-send-email-peter.maydell@linaro.org | ||
16 | Signed-off-by: Jeff Cody <jcody@redhat.com> | ||
17 | --- | 15 | --- |
18 | block/gluster.c | 3 +-- | 16 | tests/qemu-iotests/291 | 4 ---- |
19 | 1 file changed, 1 insertion(+), 2 deletions(-) | 17 | tests/qemu-iotests/291.out | 33 --------------------------------- |
18 | 2 files changed, 37 deletions(-) | ||
20 | 19 | ||
21 | diff --git a/block/gluster.c b/block/gluster.c | 20 | diff --git a/tests/qemu-iotests/291 b/tests/qemu-iotests/291 |
21 | index XXXXXXX..XXXXXXX 100755 | ||
22 | --- a/tests/qemu-iotests/291 | ||
23 | +++ b/tests/qemu-iotests/291 | ||
24 | @@ -XXX,XX +XXX,XX @@ $QEMU_IO -c 'w 1M 1M' -f $IMGFMT "$TEST_IMG" | _filter_qemu_io | ||
25 | $QEMU_IMG bitmap --disable -f $IMGFMT "$TEST_IMG" b1 | ||
26 | $QEMU_IMG bitmap --enable -f $IMGFMT "$TEST_IMG" b2 | ||
27 | $QEMU_IO -c 'w 2M 1M' -f $IMGFMT "$TEST_IMG" | _filter_qemu_io | ||
28 | -echo "Check resulting qcow2 header extensions:" | ||
29 | -$PYTHON qcow2.py "$TEST_IMG" dump-header-exts | ||
30 | |||
31 | echo | ||
32 | echo "=== Bitmap preservation not possible to non-qcow2 ===" | ||
33 | @@ -XXX,XX +XXX,XX @@ $QEMU_IMG bitmap --merge tmp -f $IMGFMT "$TEST_IMG" b0 | ||
34 | $QEMU_IMG bitmap --remove --image-opts \ | ||
35 | driver=$IMGFMT,file.driver=file,file.filename="$TEST_IMG" tmp | ||
36 | _img_info --format-specific | ||
37 | -echo "Check resulting qcow2 header extensions:" | ||
38 | -$PYTHON qcow2.py "$TEST_IMG" dump-header-exts | ||
39 | |||
40 | echo | ||
41 | echo "=== Check bitmap contents ===" | ||
42 | diff --git a/tests/qemu-iotests/291.out b/tests/qemu-iotests/291.out | ||
22 | index XXXXXXX..XXXXXXX 100644 | 43 | index XXXXXXX..XXXXXXX 100644 |
23 | --- a/block/gluster.c | 44 | --- a/tests/qemu-iotests/291.out |
24 | +++ b/block/gluster.c | 45 | +++ b/tests/qemu-iotests/291.out |
25 | @@ -XXX,XX +XXX,XX @@ static int qemu_gluster_parse_json(BlockdevOptionsGluster *gconf, | 46 | @@ -XXX,XX +XXX,XX @@ wrote 1048576/1048576 bytes at offset 1048576 |
26 | Error *local_err = NULL; | 47 | 1 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) |
27 | char *str = NULL; | 48 | wrote 1048576/1048576 bytes at offset 2097152 |
28 | const char *ptr; | 49 | 1 MiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec) |
29 | - size_t num_servers; | 50 | -Check resulting qcow2 header extensions: |
30 | - int i, type; | 51 | -Header extension: |
31 | + int i, type, num_servers; | 52 | -magic 0xe2792aca (Backing format) |
32 | 53 | -length 5 | |
33 | /* create opts info from runtime_json_opts list */ | 54 | -data 'qcow2' |
34 | opts = qemu_opts_create(&runtime_json_opts, NULL, 0, &error_abort); | 55 | - |
56 | -Header extension: | ||
57 | -magic 0x6803f857 (Feature table) | ||
58 | -length 336 | ||
59 | -data <binary> | ||
60 | - | ||
61 | -Header extension: | ||
62 | -magic 0x23852875 (Bitmaps) | ||
63 | -length 24 | ||
64 | -nb_bitmaps 2 | ||
65 | -reserved32 0 | ||
66 | -bitmap_directory_size 0x40 | ||
67 | -bitmap_directory_offset 0x510000 | ||
68 | - | ||
69 | |||
70 | === Bitmap preservation not possible to non-qcow2 === | ||
71 | |||
72 | @@ -XXX,XX +XXX,XX @@ Format specific information: | ||
73 | granularity: 65536 | ||
74 | refcount bits: 16 | ||
75 | corrupt: false | ||
76 | -Check resulting qcow2 header extensions: | ||
77 | -Header extension: | ||
78 | -magic 0x6803f857 (Feature table) | ||
79 | -length 336 | ||
80 | -data <binary> | ||
81 | - | ||
82 | -Header extension: | ||
83 | -magic 0x23852875 (Bitmaps) | ||
84 | -length 24 | ||
85 | -nb_bitmaps 3 | ||
86 | -reserved32 0 | ||
87 | -bitmap_directory_size 0x60 | ||
88 | -bitmap_directory_offset 0x520000 | ||
89 | - | ||
90 | |||
91 | === Check bitmap contents === | ||
92 | |||
35 | -- | 93 | -- |
36 | 2.9.3 | 94 | 2.26.2 |
37 | 95 | ||
38 | 96 | diff view generated by jsdifflib |