Hi,
> --- a/accel/accel-softmmu.c
> +++ b/accel/accel-softmmu.c
> @@ -79,7 +79,10 @@ void accel_init_ops_interfaces(AccelClass *ac)
> * all accelerators need to define ops, providing at least a mandatory
> * non-NULL create_vcpu_thread operation.
> */
> - g_assert(ops != NULL);
> + if (ops == NULL) {
Error message here?
Also split accel bits into a separate patch?
> g_hash_table_remove(loaded_modules, module_name);
> + fprintf(stderr, "%s module is missing, install the "
> + "package or config the library path "
> + "correctly.\n", module_name);
This should be error_report(), or maybe warn_report() as this
isn't a fatal error in all cases. Otherwise looks good to me.
take care,
Gerd
