diff options
author | Thomas Huth <thuth@redhat.com> | 2019-09-23 14:00:29 +0200 |
---|---|---|
committer | Thomas Huth <thuth@redhat.com> | 2019-10-01 11:54:17 +0200 |
commit | 3d5e90a50bd4ffa199166e01a365f5c5995534ae (patch) | |
tree | 1227116108c843742480fe1e7a6d65ff5b881feb /tests | |
parent | e423455c4f23a1a828901c78fe6d03b7dde79319 (diff) | |
download | qemu-3d5e90a50bd4ffa199166e01a365f5c5995534ae.zip qemu-3d5e90a50bd4ffa199166e01a365f5c5995534ae.tar.gz qemu-3d5e90a50bd4ffa199166e01a365f5c5995534ae.tar.bz2 |
Disallow colons in the parameter of "-accel"
Everybody who used something like "-machine accel=kvm:tcg" in the past
might be tempted to specify a similar list with the -accel parameter,
too, for example "-accel kvm:tcg". However, this is not how this
options is thought to be used, since each "-accel" should only take care
of one specific accelerator.
In the long run, we really should rework the "-accel" code completely,
so that it does not set "-machine accel=..." anymore internally, but
is completely independent from "-machine". For the short run, let's
make sure that users cannot use "-accel xyz:tcg", so that we avoid
that we have to deal with such cases in the wild later.
Message-Id: <20190930123505.11607-1-thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/cdrom-test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/cdrom-test.c b/tests/cdrom-test.c index 05611da..34e9974 100644 --- a/tests/cdrom-test.c +++ b/tests/cdrom-test.c @@ -120,7 +120,7 @@ static void test_cdboot(gconstpointer data) { QTestState *qts; - qts = qtest_initf("-accel kvm:tcg -no-shutdown %s%s", (const char *)data, + qts = qtest_initf("-M accel=kvm:tcg -no-shutdown %s%s", (const char *)data, isoimage); boot_sector_test(qts); qtest_quit(qts); |