mirror of
https://gitlab.uni-freiburg.de/opensourcevdi/spice-common
synced 2025-12-26 14:18:36 +00:00
74e50b57ae
Instead of assuming that the system can safely do unaligned access
to memory use packed structures to allow the compiler generate
best code possible.
A packed structure tells the compiler to not leave padding inside it
and that the structure can be unaligned so any field can be unaligned
having to generate proper access code based on architecture.
For instance ARM7 can use unaligned access but not for 64 bit
numbers (currently these accesses are emulated by Linux kernel
with obvious performance consequences).
This changes the current methods from:
#ifdef WORDS_BIGENDIAN
#define read_uint32(ptr) ((uint32_t)SPICE_BYTESWAP32(*((uint32_t *)(ptr))))
#define write_uint32(ptr, val) *(uint32_t *)(ptr) = SPICE_BYTESWAP32((uint32_t)val)
#else
#define read_uint32(ptr) (*((uint32_t *)(ptr)))
#define write_uint32(ptr, val) (*((uint32_t *)(ptr))) = val
#endif
to:
#include <spice/start-packed.h>
typedef struct SPICE_ATTR_PACKED {
uint32_t v;
} uint32_unaligned_t;
#include <spice/end-packed.h>
#ifdef WORDS_BIGENDIAN
#define read_uint32(ptr) ((uint32_t)SPICE_BYTESWAP32(((uint32_unaligned_t *)(ptr))->v))
#define write_uint32(ptr, val) ((uint32_unaligned_t *)(ptr))->v = SPICE_BYTESWAP32((uint32_t)val)
#else
#define read_uint32(ptr) (((uint32_unaligned_t *)(ptr))->v)
#define write_uint32(ptr, val) (((uint32_unaligned_t *)(ptr))->v) = val
#endif
Signed-off-by: Frediano Ziglio <fziglio@redhat.com>
Acked-by: Christophe Fergeau <cfergeau@redhat.com>
|
||
|---|---|---|
| .. | ||
| backtrace.c | ||
| backtrace.h | ||
| bitops.h | ||
| canvas_base.c | ||
| canvas_base.h | ||
| canvas_utils.c | ||
| canvas_utils.h | ||
| client_demarshallers.h | ||
| client_marshallers.h | ||
| draw.h | ||
| gdi_canvas.c | ||
| gdi_canvas.h | ||
| lines.c | ||
| lines.h | ||
| log.c | ||
| log.h | ||
| lz_common.h | ||
| lz_compress_tmpl.c | ||
| lz_config.h | ||
| lz_decompress_tmpl.c | ||
| lz.c | ||
| lz.h | ||
| macros.h | ||
| Makefile.am | ||
| marshaller.c | ||
| marshaller.h | ||
| mem.c | ||
| mem.h | ||
| messages.h | ||
| pixman_utils.c | ||
| pixman_utils.h | ||
| quic_config.h | ||
| quic_family_tmpl.c | ||
| quic_rgb_tmpl.c | ||
| quic_tmpl.c | ||
| quic.c | ||
| quic.h | ||
| rect.h | ||
| region.c | ||
| region.h | ||
| ring.h | ||
| rop3.c | ||
| rop3.h | ||
| snd_codec.c | ||
| snd_codec.h | ||
| spice_common.h | ||
| ssl_verify.c | ||
| ssl_verify.h | ||
| sw_canvas.c | ||
| sw_canvas.h | ||
| verify.h | ||