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