From: Claudio Fontana <cfontana@suse.de>
To: "Jose R. Ziviani" <jziviani@suse.de>
Cc: pbonzini@redhat.com, kraxel@redhat.com, berrange@redhat.com,
qemu-devel@nongnu.org, ehabkost@redhat.com
Subject: Re: [RFC 3/3] qom: Improve error message in module_object_class_by_name()
Date: Tue, 20 Jul 2021 08:40:54 +0200 [thread overview]
Message-ID: <afe32121-8bfa-9438-2a23-1411c73b520d@suse.de> (raw)
In-Reply-To: <YPYmLEuJ165xy5aR@pizza>
On 7/20/21 3:26 AM, Jose R. Ziviani wrote:
> On Mon, Jul 19, 2021 at 05:29:49PM +0200, Claudio Fontana wrote:
>> On 7/1/21 1:27 AM, Jose R. Ziviani wrote:
>>> module_object_class_by_name() calls module_load_qom_one if the object
>>> is provided by a dynamically linked library. Such library might not be
>>> available at this moment - for instance, it can be a package not yet
>>> installed. Thus, instead of assert error messages, this patch outputs
>>> more friendly messages.
>>>
>>> Current error messages:
>>> $ ./qemu-system-x86_64 -machine q35 -accel tcg -kernel /boot/vmlinuz
>>> ...
>>> ERROR:../accel/accel-softmmu.c:82:accel_init_ops_interfaces: assertion failed: (ops != NULL)
>>> Bail out! ERROR:../accel/accel-softmmu.c:82:accel_init_ops_interfaces: assertion failed: (ops != NULL)
>>> [1] 31964 IOT instruction (core dumped) ./qemu-system-x86_64 ...
>>>
>>> New error message:
>>> $ ./qemu-system-x86_64 -machine q35 -accel tcg -kernel /boot/vmlinuz
>>> accel-tcg-x86_64 module is missing, install the package or config the library path correctly.
>>>
>>> $ make check
>>> ...
>>> Running test qtest-x86_64/test-filter-mirror
>>> Running test qtest-x86_64/endianness-test
>>> accel-qtest-x86_64 module is missing, install the package or config the library path correctly.
>>> accel-qtest-x86_64 module is missing, install the package or config the library path correctly.
>>> accel-qtest-x86_64 module is missing, install the package or config the library path correctly.
>>> accel-qtest-x86_64 module is missing, install the package or config the library path correctly.
>>> accel-qtest-x86_64 module is missing, install the package or config the library path correctly.
>>> accel-tcg-x86_64 module is missing, install the package or config the library path correctly.
>>> ...
>>>
>>> Signed-off-by: Jose R. Ziviani <jziviani@suse.de>
>>> ---
>>> qom/object.c | 30 ++++++++++++++++++++++++++++++
>>> 1 file changed, 30 insertions(+)
>>>
>>> diff --git a/qom/object.c b/qom/object.c
>>> index 6a01d56546..2d40245af9 100644
>>> --- a/qom/object.c
>>> +++ b/qom/object.c
>>> @@ -1024,6 +1024,24 @@ ObjectClass *object_class_by_name(const char *typename)
>>> return type->class;
>>> }
>>>
>>> +char *get_accel_module_name(const char *ac_name);
>>> +
>>> +char *get_accel_module_name(const char *ac_name)
>>> +{
>>> + size_t len = strlen(ac_name);
>>> + char *module_name = NULL;
>>> +
>>> + if (strncmp(ac_name, "tcg-accel-ops", len) == 0) {
>>> +#ifdef CONFIG_TCG_MODULAR
>>> + module_name = g_strdup_printf("%s%s", "accel-tcg-", "x86_64");
>>> +#endif
>>> + } else if (strncmp(ac_name, "qtest-accel-ops", len) == 0) {
>>> + module_name = g_strdup_printf("%s%s", "accel-qtest-", "x86_64");
>>> + }
>>> +
>>> + return module_name;
>>> +}
>>> +
>>> ObjectClass *module_object_class_by_name(const char *typename)
>>> {
>>> ObjectClass *oc;
>>> @@ -1031,8 +1049,20 @@ ObjectClass *module_object_class_by_name(const char *typename)
>>> oc = object_class_by_name(typename);
>>> #ifdef CONFIG_MODULES
>>> if (!oc) {
>>> + char *module_name;
>>> module_load_qom_one(typename);
>>> oc = object_class_by_name(typename);
>>> + module_name = get_accel_module_name(typename);
>>> + if (module_name) {
>>> + if (!module_is_loaded(module_name)) {
>>> + fprintf(stderr, "%s module is missing, install the "
>>> + "package or config the library path "
>>> + "correctly.\n", module_name);
>>> + g_free(module_name);
>>> + exit(1);
>>> + }
>>> + g_free(module_name);
>>> + }
>>> }
>>> #endif
>>> return oc;
>>>
>>
>> Hi Jose,
>>
>> this inserts accel logic into module_object_class_by_name,
>> maybe some other place would be better to insert this check,
>> like when trying to select an accelerator?
>
>
> Hello Claudio,
>
> Yes, I think that 'get_accel_module_name()' may be a more generic
> 'get_module_name()' to handle any module failure (not only
> accelerators).
>
> I'll improve it and send a v2. Thank you for reviewing it.
>
I also wonder, and @Gerd , should "tcg_enabled()", tcg_allowed etc as a mechanism be rethought,
in light of modular TCG?
Probably would be easier to do if we had a general mechanism for CONFIG_TCG=m CONFIG_KVM=m ... which can separately be set as modules.
Thanks,
Claudio
next prev parent reply other threads:[~2021-07-20 6:41 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-30 23:27 [RFC 0/3] Improve module accelerator error message Jose R. Ziviani
2021-06-30 23:27 ` [RFC 1/3] modules: Add CONFIG_TCG_MODULAR in config_host Jose R. Ziviani
2021-06-30 23:27 ` [RFC 2/3] modules: Implement module_is_loaded function Jose R. Ziviani
2021-06-30 23:27 ` [RFC 3/3] qom: Improve error message in module_object_class_by_name() Jose R. Ziviani
2021-07-19 15:29 ` Claudio Fontana
2021-07-20 1:26 ` Jose R. Ziviani
2021-07-20 6:40 ` Claudio Fontana [this message]
2021-07-21 9:54 ` Gerd Hoffmann
2021-07-21 9:57 ` Daniel P. Berrangé
2021-07-21 13:36 ` Jose R. Ziviani
2021-07-05 8:18 ` QEMU modules improvements objective (Was: Re: [RFC 0/3] Improve module accelerator error message) Claudio Fontana
2021-07-21 10:35 ` Gerd Hoffmann
2021-07-21 10:47 ` Claudio Fontana
2021-07-21 10:50 ` Claudio Fontana
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=afe32121-8bfa-9438-2a23-1411c73b520d@suse.de \
--to=cfontana@suse.de \
--cc=berrange@redhat.com \
--cc=ehabkost@redhat.com \
--cc=jziviani@suse.de \
--cc=kraxel@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).