[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/11] qom: detect bad reentrance during object_
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 02/11] qom: detect bad reentrance during object_class_foreach |
Date: |
Sun, 15 Dec 2013 18:44:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
Am 14.12.2013 07:53, schrieb Peter Crosthwaite:
> On Fri, Dec 13, 2013 at 10:44 PM, Igor Mammedov <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>
>> Signed-off-by: Igor Mammedov <address@hidden>
>> ---
>> v2:
>> * make ver more descriptinve s/enumerating/enumerating_classes/
>> [asked-by: Peter Crosthwaite]
>> ---
>> qom/object.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/qom/object.c b/qom/object.c
>> index 3a43186..4a0fb86 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_classes = false;
>
> This seems to be a change in terminology i.e. s/type/class. Should it
> be enumerating_types or enumerating_type_table?
Too many cooks... While offline, I picked up the original version,
changing to enumerating_types, which Alexey also happened to choose, and
then adopted his drop of "false" assignment.
> Also blank line here between global var decl and fn.
Same thought here, already did in my version. :)
> Otherwise,
>
> Reviewed-by: Peter Crosthwaite <address@hidden>
Thanks, will update while rebasing.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg