[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v5 09/10] fdc-test: Avoid deprecated 'change' comman
From: |
Eric Blake |
Subject: |
[Qemu-devel] [PATCH v5 09/10] fdc-test: Avoid deprecated 'change' command |
Date: |
Thu, 27 Apr 2017 16:58:20 -0500 |
Use the preferred blockdev-change-medium command instead.
Also, use of 'device' is deprecated; adding an explicit id on
the command line lets us use 'id' for both blockdev-change-medium
and eject.
Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: John Snow <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
---
v5: add R-b
v4: use 'id' rather than 'device' [thanks to John and Kevin]
v3: update commit message to point out that we are still using
deprecated 'device'
---
tests/fdc-test.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/fdc-test.c b/tests/fdc-test.c
index 738c6b4..325712e 100644
--- a/tests/fdc-test.c
+++ b/tests/fdc-test.c
@@ -298,8 +298,8 @@ static void test_media_insert(void)
/* Insert media in drive. DSKCHK should not be reset until a step pulse
* is sent. */
- qmp_discard_response("{'execute':'change', 'arguments':{"
- " 'device':'floppy0', 'target': %s, 'arg': 'raw' }}",
+ qmp_discard_response("{'execute':'blockdev-change-medium', 'arguments':{"
+ " 'id':'floppy0', 'filename': %s, 'format': 'raw' }}",
test_image);
dir = inb(FLOPPY_BASE + reg_dir);
@@ -330,7 +330,7 @@ static void test_media_change(void)
/* Eject the floppy and check that DSKCHG is set. Reading it out doesn't
* reset the bit. */
qmp_discard_response("{'execute':'eject', 'arguments':{"
- " 'device':'floppy0' }}");
+ " 'id':'floppy0' }}");
dir = inb(FLOPPY_BASE + reg_dir);
assert_bit_set(dir, DSKCHG);
@@ -564,7 +564,7 @@ int main(int argc, char **argv)
/* Run the tests */
g_test_init(&argc, &argv, NULL);
- qtest_start(NULL);
+ qtest_start("-device floppy,id=floppy0");
qtest_irq_intercept_in(global_qtest, "ioapic");
qtest_add_func("/fdc/cmos", test_cmos);
qtest_add_func("/fdc/no_media_on_start", test_no_media_on_start);
--
2.9.3
- [Qemu-devel] [PATCH v5 00/10] qapi-related cleanups, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 01/10] pci: Use struct instead of QDict to pass back parameters, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 02/10] pci: Reduce scope of error injection, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 03/10] coccinelle: Add script to remove useless QObject casts, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 05/10] qobject: Add helper macros for common scalar insertions, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 04/10] qobject: Drop useless QObject casts, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 08/10] QemuOpts: Simplify qemu_opts_to_qdict(), Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 07/10] block: Simplify bdrv_append_temp_snapshot() logic, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 09/10] fdc-test: Avoid deprecated 'change' command,
Eric Blake <=
- [Qemu-devel] [PATCH v5 10/10] test-qga: Actually test 0xff sync bytes, Eric Blake, 2017/04/27
- [Qemu-devel] [PATCH v5 06/10] qobject: Use simpler QDict/QList scalar insertion macros, Eric Blake, 2017/04/27