[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness |
Date: |
Tue, 28 Jul 2020 10:21:18 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 |
On 28/07/20 09:44, Markus Armbruster wrote:
>> - assert(!DEVICE_GET_CLASS(dev)->bus_type);
>> + } else if (DEVICE_GET_CLASS(dev)->bus_type) {
>> + error_setg(errp, "Unexpected bus '%s' for bus-less device '%s'",
>> + DEVICE_GET_CLASS(dev)->bus_type,
>> + object_get_typename(OBJECT(dev)));
>> + return false;
>> }
>>
>> object_property_set_bool(OBJECT(dev), true, "realized", &err);
> Objection. This turns an abort into something else unless the caller
> passes &error_abort. The caller in your commit message's example does,
> others don't.
>
> Keep the unconditional abort, please. Feel free to print something kind
> right before. I doubt it's all that useful, as I believe whoever gets
> to fix the bug will have to figure out the code anyway, but I could be
> wrong.
>
This was my request, actually. We have an Error**, we should use it in
case this code is reached via device_add.
Paolo
- [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Philippe Mathieu-Daudé, 2020/07/27
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Markus Armbruster, 2020/07/28
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness,
Paolo Bonzini <=
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Markus Armbruster, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Philippe Mathieu-Daudé, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Markus Armbruster, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Philippe Mathieu-Daudé, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Markus Armbruster, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Paolo Bonzini, 2020/07/29
- Re: [PATCH-for-5.2 v4] hw/core/qdev: Increase qdev_realize() kindness, Markus Armbruster, 2020/07/30