diff options
| author | Anthony PERARD <anthony.perard@citrix.com> | 2019-07-04 16:36:05 +0100 |
|---|---|---|
| committer | Anthony PERARD <anthony.perard@citrix.com> | 2019-08-27 14:18:28 +0100 |
| commit | d297e533a5f653336dbc1c5a24ea196391dff9f5 (patch) | |
| tree | cede90cfd71f48c3261d04f6218df9a1b6da8031 /docs/devel/secure-coding-practices.rst | |
| parent | dac03af5d5482ec7ee9c23db467bb7230b33c0d9 (diff) | |
| download | qemu-d297e533a5f653336dbc1c5a24ea196391dff9f5.zip qemu-d297e533a5f653336dbc1c5a24ea196391dff9f5.tar.gz qemu-d297e533a5f653336dbc1c5a24ea196391dff9f5.tar.bz2 | |
xen: Fix ring.h header
The xen_[rw]?mb() macros defined in ring.h can't be used and the fact
that there are gated behind __XEN_INTERFACE_VERSION__ means that it
needs to be defined somewhere. QEMU doesn't implement interfaces with
the Xen hypervisor so defining __XEN_INTERFACE_VERSION__ is pointless.
This leads to:
include/hw/xen/io/ring.h:47:5: error: "__XEN_INTERFACE_VERSION__"
is not defined, evaluates to 0 [-Werror=undef]
Cleanup ring.h. The xen_*mb() macros are already defined in xenctrl.h
which is included in xen_common.h.
Reported-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20190704153605.4140-1-anthony.perard@citrix.com>
[aperard: Adding the comment proposed upstream]
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Diffstat (limited to 'docs/devel/secure-coding-practices.rst')
0 files changed, 0 insertions, 0 deletions