[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/1] modules: Improve error message when module is not fou
From: |
Claudio Fontana |
Subject: |
Re: [PATCH v2 1/1] modules: Improve error message when module is not found |
Date: |
Fri, 23 Jul 2021 13:20:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 |
On 7/23/21 11:52 AM, Gerd Hoffmann wrote:
>>> - g_assert(ops != NULL);
>>> + if (ops == NULL) {
>>> + exit(1);
>>> + }
>>> +
>>
>>
>> Ah, again, why?
>> This change looks wrong to me,
>>
>> the ops code should be present when ops interfaces are initialized:
>> it should be a code level assertion, as it has to do with the proper order
>> of initializations in QEMU,
>>
>> why would we want to do anything else but to assert here?
>>
>> Am I blind to something obvious?
>
> Building tcg accel ops modular moves that from coding error to possible
> user error (user wants use tcg but has qemu-accel-tcg-$arch.rpm not
> installed).
Sorry but without more background I don't buy it.
If ops is null at the time accel_init_interfaces is called,
it means that we are trying to initialize the board (for softmmu)
with an accelerator already selected, and without an accelerator actually
available.
The problem has happened already a long time before we get here.
When we check for viable accelerators, in configure_accelerators,
we should check that the code is actually there, before choosing it as a viable
accelerator.
If we march on and start initializing the machine with an accelerator that is
not available,
of course things will start failing left and right.
If things like:
bool have_tcg = accel_find("tcg");
return true when the code is actually not there, there seems to be a larger
issue to solve.
I think we need to think more broadly about this.
Thanks,
Claudio
>
> The second part of the patch makes qemu print a message on the failed
> module load, so the user would have a chance to figure where the assert
> comes from, but replacing the assert with a more friendly message still
> makes sense to me.
>
> take care,
> Gerd
>
- [PATCH v2 0/1] Improve module accelerator error message, Jose R. Ziviani, 2021/07/22
- [PATCH v2 1/1] modules: Improve error message when module is not found, Jose R. Ziviani, 2021/07/22
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Gerd Hoffmann, 2021/07/23
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Markus Armbruster, 2021/07/23
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Claudio Fontana, 2021/07/23
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Gerd Hoffmann, 2021/07/23
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found,
Claudio Fontana <=
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Claudio Fontana, 2021/07/23
- Re: [PATCH v2 1/1] modules: Improve error message when module is not found, Gerd Hoffmann, 2021/07/23
- modular tcg (was: Re: [PATCH v2 1/1] modules: Improve error message when module is not) found, Gerd Hoffmann, 2021/07/29
- Re: modular tcg, Claudio Fontana, 2021/07/29
- Re: modular tcg, Gerd Hoffmann, 2021/07/29
- Re: modular tcg, Claudio Fontana, 2021/07/29
- Re: modular tcg, Claudio Fontana, 2021/07/29
- Re: modular tcg, Gerd Hoffmann, 2021/07/29
- Re: modular tcg, Claudio Fontana, 2021/07/29
- Re: modular tcg, Philippe Mathieu-Daudé, 2021/07/29