[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/55] qdev: Drop redundant bus realization
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 02/55] qdev: Drop redundant bus realization |
Date: |
Wed, 20 May 2020 16:25:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> writes:
> +Marcel
>
> On 5/19/20 4:54 PM, Markus Armbruster wrote:
>> Realizing a device automatically realizes its buses, in
>> device_set_realized(). Realizing them in realize methods is
>> redundant, unless the methods themselves require them to be realized
>> early. None do. Drop the redundant bus realiziations.
>
> Typo realiziations-> realizations.
Will fix, thanks!
> So you are reverting commits 685f9a3428f & b1af7959a66...
I should've checked where these lines come from.
> I don't understand them, Marcel can you review this patch?
>
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> hw/pci-host/prep.c | 1 -
>> hw/pci-host/versatile.c | 1 -
>> 2 files changed, 2 deletions(-)
>>
>> diff --git a/hw/pci-host/prep.c b/hw/pci-host/prep.c
>> index 1a02e9a670..c821ef889d 100644
>> --- a/hw/pci-host/prep.c
>> +++ b/hw/pci-host/prep.c
>> @@ -268,7 +268,6 @@ static void raven_pcihost_realizefn(DeviceState *d,
>> Error **errp)
>> memory_region_add_subregion(address_space_mem, 0xbffffff0,
>> &s->pci_intack);
>> /* TODO Remove once realize propagates to child devices. */
>> - object_property_set_bool(OBJECT(&s->pci_bus), true, "realized", errp);
>> object_property_set_bool(OBJECT(&s->pci_dev), true, "realized", errp);
>> }
>> diff --git a/hw/pci-host/versatile.c b/hw/pci-host/versatile.c
>> index cfb9a78ea6..28817dbeec 100644
>> --- a/hw/pci-host/versatile.c
>> +++ b/hw/pci-host/versatile.c
>> @@ -458,7 +458,6 @@ static void pci_vpb_realize(DeviceState *dev, Error
>> **errp)
>> }
>> /* TODO Remove once realize propagates to child devices. */
>> - object_property_set_bool(OBJECT(&s->pci_bus), true, "realized", errp);
>> object_property_set_bool(OBJECT(&s->pci_dev), true, "realized", errp);
>> }
>>
>>
- [PATCH 00/55] qdev: Rework how we plug into the parent bus, Markus Armbruster, 2020/05/19
- [PATCH 04/55] qdev: Put qdev_new() to use with Coccinelle, Markus Armbruster, 2020/05/19
- [PATCH 01/55] qdev: Rename qbus_realize() to qbus_init(), Markus Armbruster, 2020/05/19
- [PATCH 02/55] qdev: Drop redundant bus realization, Markus Armbruster, 2020/05/19
- [PATCH 05/55] qdev: Convert to qbus_realize(), qbus_unrealize(), Markus Armbruster, 2020/05/19
- [PATCH 07/55] qdev: Convert to qdev_unrealize() manually, Markus Armbruster, 2020/05/19
- [PATCH 06/55] qdev: Convert to qdev_unrealize() with Coccinelle, Markus Armbruster, 2020/05/19
- [PATCH 13/55] hw/ppc: Eliminate two superfluous QOM casts, Markus Armbruster, 2020/05/19
- [PATCH 19/55] isa: Convert uses of isa_create(), isa_try_create() manually, Markus Armbruster, 2020/05/19
- [PATCH 20/55] isa: isa_create(), isa_try_create() are now unused, drop, Markus Armbruster, 2020/05/19
- [PATCH 09/55] qdev: Convert uses of qdev_create() manually, Markus Armbruster, 2020/05/19