Message ID | 1412261496-24455-4-git-send-email-armbru@redhat.com |
---|---|
State | New |
Headers | show |
On 10/02/2014 08:51 AM, Markus Armbruster wrote: > The first argument should be a PCI address, which pci-addr=auto isn't. > Doesn't really matter, as drive_add ignores its first argument when > its second argument has if=none. Clean it up anyway. > > Signed-off-by: Markus Armbruster <armbru@redhat.com> > --- > tests/drive_del-test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Eric Blake <eblake@redhat.com>
diff --git a/tests/drive_del-test.c b/tests/drive_del-test.c index 786f026..67309fd 100644 --- a/tests/drive_del-test.c +++ b/tests/drive_del-test.c @@ -83,7 +83,7 @@ static void test_after_failed_device_add(void) */ response = qmp("{'execute': 'human-monitor-command'," " 'arguments': {" - " 'command-line': 'drive_add pci-addr=auto if=none,id=drive0'" + " 'command-line': 'drive_add 0 if=none,id=drive0'" "}}"); g_assert(response); g_assert_cmpstr(qdict_get_try_str(response, "return"), ==, "OK\r\n");
The first argument should be a PCI address, which pci-addr=auto isn't. Doesn't really matter, as drive_add ignores its first argument when its second argument has if=none. Clean it up anyway. Signed-off-by: Markus Armbruster <armbru@redhat.com> --- tests/drive_del-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)