Re: [PATCH v14 18/22] accel: introduce AccelCPUClass extending CPUClass

[Alex Bennée]

Richard Henderson <richard.henderson@linaro.org> writes:

> On 1/30/21 12:53 AM, Claudio Fontana wrote:
>> To summarize:
>> 
>> 1) accel->cpu_realizefn extends the current cpu target-specific realize 
>> functions with accelerator-specific code,
>>    which currently does not make use of errp at all (thus, the temptation to 
>> remove errp from the interface until it is actually needed by a target).
>
> No, arm does use errp in realizefn already.
> It's just that the void return value is hinky.

Sounds like fixing the void return would be part of a larger qdev
clean-up rather than this particular series. If I recall there have been
various phases of clean-ups and refactoring of the error code paths in
the past.

>
>
> r~


-- 
Alex Bennée