[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_cl
From: |
Alexey Kardashevskiy |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach |
Date: |
Fri, 20 Dec 2013 22:43:05 +1100 |
User-agent: |
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 12/20/2013 10:29 PM, Andreas Färber wrote:
> Am 19.12.2013 12:38, schrieb Alexey Kardashevskiy:
>> On 12/16/2013 08:23 AM, Andreas Färber wrote:
>>> Am 04.12.2013 06:51, schrieb Peter Crosthwaite:
>>>> On Wed, Dec 4, 2013 at 1:42 AM, Paolo Bonzini <address@hidden> wrote:
>>>>> From: Hervé Poussineau <address@hidden>
>>>>>
>>>>> We should not modify the type hash table while it is being iterated on.
>>>>> Assert that it does not happen.
>>>>>
>>>>> Signed-off-by: Hervé Poussineau <address@hidden>
>>>>> Signed-off-by: Paolo Bonzini <address@hidden>
>>>>> ---
>>>>> qom/object.c | 4 ++++
>>>>> 1 file changed, 4 insertions(+)
>>>>>
>>>>> diff --git a/qom/object.c b/qom/object.c
>>>>> index 3a43186..1dee9f0 100644
>>>>> --- a/qom/object.c
>>>>> +++ b/qom/object.c
>>>>> @@ -78,8 +78,10 @@ static GHashTable *type_table_get(void)
>>>>> return type_table;
>>>>> }
>>>>>
>>>>> +static bool enumerating = false;
>>>>
>>>> Global variable could probably use a more descriptive name.
>>>
>>> I renamed it to enumerating_types and dropped the assignment as
>>> suggested elsewhere by Alexey (a reply here would've been nice!).
>>
>> Whose reply? To what? :)
>
> A reply of yours to Peter C.'s change request, stating that you have
> addressed it - and in this case in which series. :)
You lost me here. I got these patches from Paolo Bonzini and I was told to
repost them together with my other patches, that's it :)
>>> I also took the liberty of inserted a white line to make the function
>>> better readable.
>>
>> I do not mind, this was not my stuff :)
>>
>> What I wonder about is what is going to happen to the rest of what I
>> posted? Should I wait till this qom-next gets merged to upstream and repost
>> my patches for Alex Graf again? Thanks.
>
> I do intend to post a pull before Chistmas, reviewing+applying the last
> no_user patch from Markus was one key ingredient for that.
>
> Not sure which rest exactly you're referring to? I need to review your
> latest QemuOpts proposal among others. Actually I have a whole A4 page
> written down with series I'm wading through. ;)
I posted the "[PATCH v4 0/8] spapr: bootindex support" series which
included the 2 patches you mentioned in this series, from the rest - 3 are
for spapr and 3 are platform independent.
--
Alexey
- [Qemu-devel] [PATCH 0/2] qom: fix registration of QOM interfaces, Paolo Bonzini, 2013/12/03
- [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Paolo Bonzini, 2013/12/03
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Peter Crosthwaite, 2013/12/04
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Andreas Färber, 2013/12/15
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Alexey Kardashevskiy, 2013/12/19
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Andreas Färber, 2013/12/20
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach,
Alexey Kardashevskiy <=
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Alexey Kardashevskiy, 2013/12/24
- Re: [Qemu-devel] [PATCH 2/2] qom: detect bad reentrance during object_class_foreach, Andreas Färber, 2013/12/24
[Qemu-devel] [PATCH 1/2] qom: do not register interface "types" in the type table, Paolo Bonzini, 2013/12/03
Re: [Qemu-devel] [PATCH 0/2] qom: fix registration of QOM interfaces, Andreas Färber, 2013/12/15