diff options
author | Michael Roth <mdroth@linux.vnet.ibm.com> | 2013-05-26 22:20:58 -0500 |
---|---|---|
committer | Luiz Capitulino <lcapitulino@redhat.com> | 2013-05-30 09:08:38 -0400 |
commit | a678e26cbe89f7a27cbce794c2c2784571ee9d21 (patch) | |
tree | 2cd61bcd9acd3f775beb2a2034866c0e6cbb8ba7 /include/qapi/visitor.h | |
parent | 87d23f78aa79b72da022afda358bbc8a8509ca70 (diff) | |
download | hqemu-a678e26cbe89f7a27cbce794c2c2784571ee9d21.zip hqemu-a678e26cbe89f7a27cbce794c2c2784571ee9d21.tar.gz |
qapi: pad GenericList value fields to 64 bits
With the introduction of native list types, we now have types such as
int64List where the 'value' field is not a pointer, but the actual
64-bit value.
On 32-bit architectures, this can lead to situations where 'next' field
offset in GenericList does not correspond to the 'next' field in the
types that we cast to GenericList when using the visit_next_list()
interface, causing issues when we attempt to traverse linked list
structures of these types.
To fix this, pad the 'value' field of GenericList and other
schema-defined/native *List types out to 64-bits.
This is less memory-efficient for 32-bit architectures, but allows us to
continue to rely on list-handling interfaces that target GenericList to
simply visitor implementations.
In the future we can improve efficiency by defaulting to using native C
array backends to handle list of non-pointer types, which would be more
memory efficient in itself and allow us to roll back this change.
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'include/qapi/visitor.h')
-rw-r--r-- | include/qapi/visitor.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/include/qapi/visitor.h b/include/qapi/visitor.h index 1fef18c..28c21d8 100644 --- a/include/qapi/visitor.h +++ b/include/qapi/visitor.h @@ -18,7 +18,10 @@ typedef struct GenericList { - void *value; + union { + void *value; + uint64_t padding; + }; struct GenericList *next; } GenericList; |