On 7/1/21 1:27 AM, Jose R. Ziviani wrote: > Hello! > > I'm sending this as RFC because it's based on a patch still on > review[1], so I'd like to see if it makes sense. > > Tt will improve the error message when an accelerator module could > not be loaded. Instead of the current assert error, a formated > message will be displayed. > > [1] https://patchwork.kernel.org/project/qemu-devel/list/?series=506379 > > Jose R. Ziviani (3): > modules: Add CONFIG_TCG_MODULAR in config_host > modules: Implement module_is_loaded function > qom: Improve error message in module_object_class_by_name() > > include/qemu/module.h | 3 +++ > meson.build | 3 +++ > qom/object.c | 30 ++++++++++++++++++++++++++++++ > util/module.c | 28 +++++++++++++++++++++------- > 4 files changed, 57 insertions(+), 7 deletions(-) >
Open question to all, why don't we have/add the ability to configure CONFIG_XXX=m for all potentially modular pieces? It should be possible to say, I want to build the storage plugins as modules, TCG I would like it built-in, and KVM as a module, or any combination of these. The most useful combination I see for virtualization use of qemu is with TCG as a module (M), KVM as built-in (Y), and various other optional pieces as modules (M). Should this not be the vision? To me it looks that way. Thanks, Claudio