Re: [PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-07-03 Thread Paolo Bonzini
On 03/07/20 07:34, Thomas Huth wrote: > On 18/06/2020 09.40, Thomas Huth wrote: >> The new -accel option does not accept colons in the parameters anymore >> (since it does not convert the parameters to -machine accel=... >> parameters >> anymore). Thus we can now remove the check for colons in

Re: [PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-07-03 Thread Claudio Fontana
On 7/3/20 7:34 AM, Thomas Huth wrote: > On 18/06/2020 09.40, Thomas Huth wrote: >> The new -accel option does not accept colons in the parameters anymore >> (since it does not convert the parameters to -machine accel=... parameters >> anymore). Thus we can now remove the check for colons in

Re: [PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-07-02 Thread Thomas Huth
On 18/06/2020 09.40, Thomas Huth wrote: The new -accel option does not accept colons in the parameters anymore (since it does not convert the parameters to -machine accel=... parameters anymore). Thus we can now remove the check for colons in -accel: $ qemu-system-x86_64 -accel kvm:tcg

Re: [PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-06-18 Thread Thomas Huth
Hi Peter, could you maybe apply the Clang patches on the list directly to the repository as a build fix? ... these warning mails from Patchew are currently rather annoying... :-( Thomas On 18/06/2020 10.03, no-re...@patchew.org wrote: [...] > /tmp/qemu-test/src/fpu/softfloat.c:3365:13:

Re: [PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-06-18 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200618074001.13642-1-th...@redhat.com/ Hi, This series failed the asan build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/bin/bash export

[PATCH] softmmu/vl: Remove the check for colons in -accel parameters

2020-06-18 Thread Thomas Huth
The new -accel option does not accept colons in the parameters anymore (since it does not convert the parameters to -machine accel=... parameters anymore). Thus we can now remove the check for colons in -accel: $ qemu-system-x86_64 -accel kvm:tcg qemu-system-x86_64: -accel kvm:tcg: invalid