On 18/11/19 11:58, Thomas Huth wrote:
>> +static void configure_accelerators(void)
>> +{
>> + qemu_opts_foreach(qemu_find_opts("icount"),
>> + do_configure_icount, NULL, &error_fatal);
>> +
>> + qemu_opts_foreach(qemu_find_opts("accel"),
>> + do_configure_accelerator, NULL, &error_fatal);
>> +}
>
> vl.c is already quite overcrowded ... maybe you could add the new code
> to accel/accel.c instead? Just my 0.02 €.
I liked the idea of keeping all command line parsing in vl.c, especially
because all the ugliness for backwards compatibility can then be
confined there.
Paolo