[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL for-2.5 00/10] QOM devices patch queue 2015-11-18
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PULL for-2.5 00/10] QOM devices patch queue 2015-11-18 |
Date: |
Thu, 19 Nov 2015 15:09:47 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
Am 19.11.2015 um 12:53 schrieb Daniel P. Berrange:
> On Thu, Nov 19, 2015 at 10:54:15AM +0000, Peter Maydell wrote:
>> On 18 November 2015 at 20:39, Andreas Färber <address@hidden> wrote:
>>> Hello Peter,
>>>
>>> This is my late QOM (devices) patch queue. Please pull.
>>>
>>> Regards,
>>> Andreas
>>>
>>> Cc: Peter Maydell <address@hidden>
>>> Cc: Daniel P. Berrange <address@hidden>
>>> Cc: Pavel Fedin <address@hidden>
>>>
>>> The following changes since commit 74fcbd22d20a2fbc1a47a7b00cce5bf98fd7be5f:
>>>
>>> hw/misc: Add support for ADC controller in Xilinx Zynq 7000 (2015-11-12
>>> 21:30:42 +0000)
>>>
>>> are available in the git repository at:
>>>
>>> git://github.com/afaerber/qemu-cpu.git tags/qom-devices-for-peter
>>>
>>> for you to fetch changes up to c5760450796433621ccdc125866a84b892e2d5b6:
>>>
>>> MAINTAINERS: Add check-qom-{interface,proplist} to QOM (2015-11-18
>>> 21:13:50 +0100)
>>>
>>> ----------------------------------------------------------------
>>> QOM infrastructure fixes and device conversions
>>>
>>> * Fix for properties on objects > 4 GiB
>>> * Performance improvements for QOM property handling
>>> * Assertion cleanups
>>> * MAINTAINERS additions
>>
>> Hi. Unfortunately this doesn't build with our minimum supported
>> glib version:
>>
>> /Users/pm215/src/qemu-for-merges/qom/object.c:921:9: warning: implicit
>> declaration of function 'g_hash_table_contains' is invalid in C99
>> [-Wimplicit-function-declaration]
>> if (g_hash_table_contains(obj->properties, name)) {
>> ^
>>
>> g_hash_table_contains was only introduced in glib 2.32.
>
> Since we never store NULL as a valid value in the hash tble, we
> can simply replace this with g_hash_table_lookup instead
Seems to work fine, too:
diff --git a/qom/object.c b/qom/object.c
index 19e7561..ff34c86 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -918,7 +918,7 @@ object_property_add(Object *obj, const char *name,
const char *type,
return ret;
}
- if (g_hash_table_contains(obj->properties, name)) {
+ if (g_hash_table_lookup(obj->properties, name) != NULL) {
error_setg(errp, "attempt to add duplicate property '%s'"
" to object (type '%s')", name,
object_get_typename(obj));
Will respin.
Thanks,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)
- [Qemu-devel] [PULL 06/10] net: Convert net filter code to use object property iterators, (continued)
- [Qemu-devel] [PULL 06/10] net: Convert net filter code to use object property iterators, Andreas Färber, 2015/11/18
- [Qemu-devel] [PULL 08/10] qom: Replace object property list with GHashTable, Andreas Färber, 2015/11/18
- [Qemu-devel] [PULL 05/10] ppc: Convert spapr code to use object property iterators, Andreas Färber, 2015/11/18
- [Qemu-devel] [PULL 10/10] MAINTAINERS: Add check-qom-{interface, proplist} to QOM, Andreas Färber, 2015/11/18
- [Qemu-devel] [PULL 09/10] qom: Clean up assertions to display values on failure, Andreas Färber, 2015/11/18
- [Qemu-devel] [PULL 07/10] qom: Add a test case for complex property finalization, Andreas Färber, 2015/11/18
- Re: [Qemu-devel] [PULL for-2.5 00/10] QOM devices patch queue 2015-11-18, Peter Maydell, 2015/11/19