jcs.org
qemu with hax to log dma reads & writes
jcs.org/2018/11/12/vfio
qdict qlist: Make most helper macros functions
The macro expansions of qdict_put_TYPE() and qlist_append_TYPE() need
qbool.h, qnull.h, qnum.h and qstring.h to compile. We include qnull.h
and qnum.h in the headers, but not qbool.h and qstring.h. Works,
because we include those wherever the macros get used.
Open-coding these helpers is of dubious value. Turn them into
functions and drop the includes from the headers.
This cleanup makes the number of objects depending on qapi/qmp/qnum.h
from 4551 (out of 4743) to 46 in my "build everything" tree. For
qapi/qmp/qnull.h, the number drops from 4552 to 21.
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20180201111846.21846-10-armbru@redhat.com>
+84
-23
+1
block/qapi.c
+1
block/qapi.c
+1
blockdev.c
+1
blockdev.c
+1
hw/i386/acpi-build.c
+1
hw/i386/acpi-build.c
+1
hw/ppc/spapr_drc.c
+1
hw/ppc/spapr_drc.c
+4
-12
include/qapi/qmp/qdict.h
+4
-12
include/qapi/qmp/qdict.h
···
15
15
16
16
#include "qapi/qmp/qobject.h"
17
17
#include "qapi/qmp/qlist.h"
18
-
#include "qapi/qmp/qnull.h"
19
-
#include "qapi/qmp/qnum.h"
20
18
#include "qemu/queue.h"
21
19
22
20
#define QDICT_BUCKET_MAX 512
···
55
53
#define qdict_put(qdict, key, obj) \
56
54
qdict_put_obj(qdict, key, QOBJECT(obj))
57
55
58
-
/* Helpers for int, bool, null, and string */
59
-
#define qdict_put_int(qdict, key, value) \
60
-
qdict_put(qdict, key, qnum_from_int(value))
61
-
#define qdict_put_bool(qdict, key, value) \
62
-
qdict_put(qdict, key, qbool_from_bool(value))
63
-
#define qdict_put_str(qdict, key, value) \
64
-
qdict_put(qdict, key, qstring_from_str(value))
65
-
#define qdict_put_null(qdict, key) \
66
-
qdict_put(qdict, key, qnull())
56
+
void qdict_put_bool(QDict *qdict, const char *key, bool value);
57
+
void qdict_put_int(QDict *qdict, const char *key, int64_t value);
58
+
void qdict_put_null(QDict *qdict, const char *key);
59
+
void qdict_put_str(QDict *qdict, const char *key, const char *value);
67
60
68
-
/* High level helpers */
69
61
double qdict_get_double(const QDict *qdict, const char *key);
70
62
int64_t qdict_get_int(const QDict *qdict, const char *key);
71
63
bool qdict_get_bool(const QDict *qdict, const char *key);
+4
-11
include/qapi/qmp/qlist.h
+4
-11
include/qapi/qmp/qlist.h
···
14
14
#define QLIST_H
15
15
16
16
#include "qapi/qmp/qobject.h"
17
-
#include "qapi/qmp/qnum.h"
18
-
#include "qapi/qmp/qnull.h"
19
17
#include "qemu/queue.h"
20
18
21
19
typedef struct QListEntry {
···
31
29
#define qlist_append(qlist, obj) \
32
30
qlist_append_obj(qlist, QOBJECT(obj))
33
31
34
-
/* Helpers for int, bool, and string */
35
-
#define qlist_append_int(qlist, value) \
36
-
qlist_append(qlist, qnum_from_int(value))
37
-
#define qlist_append_bool(qlist, value) \
38
-
qlist_append(qlist, qbool_from_bool(value))
39
-
#define qlist_append_str(qlist, value) \
40
-
qlist_append(qlist, qstring_from_str(value))
41
-
#define qlist_append_null(qlist) \
42
-
qlist_append(qlist, qnull())
32
+
void qlist_append_bool(QList *qlist, bool value);
33
+
void qlist_append_int(QList *qlist, int64_t value);
34
+
void qlist_append_null(QList *qlist);
35
+
void qlist_append_str(QList *qlist, const char *value);
43
36
44
37
#define QLIST_FOREACH_ENTRY(qlist, var) \
45
38
for ((var) = ((qlist)->head.tqh_first); \
+1
migration/migration.c
+1
migration/migration.c
+1
monitor.c
+1
monitor.c
+1
qapi/qapi-dealloc-visitor.c
+1
qapi/qapi-dealloc-visitor.c
+2
qapi/qobject-input-visitor.c
+2
qapi/qobject-input-visitor.c
+2
qapi/qobject-output-visitor.c
+2
qapi/qobject-output-visitor.c
+2
qobject/json-parser.c
+2
qobject/json-parser.c
+21
qobject/qdict.c
+21
qobject/qdict.c
···
14
14
#include "qapi/qmp/qnum.h"
15
15
#include "qapi/qmp/qdict.h"
16
16
#include "qapi/qmp/qbool.h"
17
+
#include "qapi/qmp/qnull.h"
17
18
#include "qapi/qmp/qstring.h"
18
19
#include "qapi/qmp/qobject.h"
19
20
#include "qapi/error.h"
···
141
142
QLIST_INSERT_HEAD(&qdict->table[bucket], entry, next);
142
143
qdict->size++;
143
144
}
145
+
}
146
+
147
+
void qdict_put_int(QDict *qdict, const char *key, int64_t value)
148
+
{
149
+
qdict_put(qdict, key, qnum_from_int(value));
150
+
}
151
+
152
+
void qdict_put_bool(QDict *qdict, const char *key, bool value)
153
+
{
154
+
qdict_put(qdict, key, qbool_from_bool(value));
155
+
}
156
+
157
+
void qdict_put_str(QDict *qdict, const char *key, const char *value)
158
+
{
159
+
qdict_put(qdict, key, qstring_from_str(value));
160
+
}
161
+
162
+
void qdict_put_null(QDict *qdict, const char *key)
163
+
{
164
+
qdict_put(qdict, key, qnull());
144
165
}
145
166
146
167
/**
+1
qobject/qjson.c
+1
qobject/qjson.c
+24
qobject/qlist.c
+24
qobject/qlist.c
···
11
11
*/
12
12
13
13
#include "qemu/osdep.h"
14
+
#include "qapi/qmp/qbool.h"
14
15
#include "qapi/qmp/qlist.h"
16
+
#include "qapi/qmp/qnull.h"
17
+
#include "qapi/qmp/qnum.h"
15
18
#include "qapi/qmp/qobject.h"
19
+
#include "qapi/qmp/qstring.h"
16
20
#include "qemu/queue.h"
17
21
#include "qemu-common.h"
18
22
···
62
66
entry->value = value;
63
67
64
68
QTAILQ_INSERT_TAIL(&qlist->head, entry, next);
69
+
}
70
+
71
+
void qlist_append_int(QList *qlist, int64_t value)
72
+
{
73
+
qlist_append(qlist, qnum_from_int(value));
74
+
}
75
+
76
+
void qlist_append_bool(QList *qlist, bool value)
77
+
{
78
+
qlist_append(qlist, qbool_from_bool(value));
79
+
}
80
+
81
+
void qlist_append_str(QList *qlist, const char *value)
82
+
{
83
+
qlist_append(qlist, qstring_from_str(value));
84
+
}
85
+
86
+
void qlist_append_null(QList *qlist)
87
+
{
88
+
qlist_append(qlist, qnull());
65
89
}
66
90
67
91
/**
+1
qobject/qlit.c
+1
qobject/qlit.c
+2
qobject/qobject.c
+2
qobject/qobject.c
+1
qom/object.c
+1
qom/object.c
+1
target/ppc/translate.c
+1
target/ppc/translate.c
+1
tests/check-qdict.c
+1
tests/check-qdict.c
+2
tests/check-qjson.c
+2
tests/check-qjson.c
+2
tests/check-qobject.c
+2
tests/check-qobject.c
+1
tests/test-qmp-commands.c
+1
tests/test-qmp-commands.c
+1
tests/test-qmp-event.c
+1
tests/test-qmp-event.c
+2
tests/test-qobject-input-visitor.c
+2
tests/test-qobject-input-visitor.c
+2
tests/test-qobject-output-visitor.c
+2
tests/test-qobject-output-visitor.c