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