summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrediano Ziglio <fziglio@redhat.com>2019-05-13 10:41:03 +0100
committerFrediano Ziglio <fziglio@redhat.com>2019-07-21 09:47:20 +0100
commit2d34651f4094ae145843799ce14c0958b4d0e384 (patch)
treeb154b96fcc8c0212309ee7c62c24a05639e03f93
parentf436204eadc89e911b523c26c09b2ab13def1f53 (diff)
qxl_dev: Move QXLReleaseInfoExt out of start/end-packed.h
This structure is not declares as SPICE_ATTR_PACKED resulting it as aligned and no packed using GCC (no MingW). This structure is only packed under MingW or Microsoft compilers. This structure is not technically a definition for QXL device but is used only for spice-server QXL interface. As said above this change could affect only MingW/Microsoft that is only Windows ABI of spice-server (very recently added) so following lines are really not much important. Note that Windows does not support architectures with CPUs not supporting unaligned access. Under Win32 this change don't affect size so nothing much can change. This structure is currently passed as value to release_resource callback. Under Win64 this structure is passed like a "const QXLReleaseInfoExt *" so even if the structure is 16 bytes instead of 12 this is read the same way. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Snir Sheriber <ssheribe@redhat.com>
-rw-r--r--spice/qxl_dev.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/spice/qxl_dev.h b/spice/qxl_dev.h
index 659f930..c844d7b 100644
--- a/spice/qxl_dev.h
+++ b/spice/qxl_dev.h
@@ -280,11 +280,6 @@ typedef union SPICE_ATTR_PACKED QXLReleaseInfo {
uint64_t next; // out
} QXLReleaseInfo;
-typedef struct QXLReleaseInfoExt {
- QXLReleaseInfo *info;
- uint32_t group_id;
-} QXLReleaseInfoExt;
-
typedef struct SPICE_ATTR_PACKED QXLDataChunk {
uint32_t data_size;
QXLPHYSICAL prev_chunk;
@@ -779,4 +774,9 @@ typedef struct SPICE_ATTR_PACKED QXLMonitorsConfig {
#include <spice/end-packed.h>
+typedef struct QXLReleaseInfoExt {
+ QXLReleaseInfo *info;
+ uint32_t group_id;
+} QXLReleaseInfoExt;
+
#endif /* _H_QXL_DEV */