Skip to content

Commit

Permalink
exec: Use uintptr_t in cpu_ldst.h
Browse files Browse the repository at this point in the history
This is more descriptive than 'unsigned long'.
No functional change, since these match on all linux+bsd hosts.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20210210000223.884088-5-richard.henderson@linaro.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
  • Loading branch information
rth7680 authored and pm215 committed Feb 11, 2021
1 parent 0feea90 commit 8c1dc74
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions include/exec/cpu_ldst.h
Expand Up @@ -70,22 +70,22 @@ typedef uint64_t abi_ptr;
#endif

/* All direct uses of g2h and h2g need to go away for usermode softmmu. */
#define g2h(x) ((void *)((unsigned long)(abi_ptr)(x) + guest_base))
#define g2h(x) ((void *)((uintptr_t)(abi_ptr)(x) + guest_base))

#if HOST_LONG_BITS <= TARGET_VIRT_ADDR_SPACE_BITS
#define guest_addr_valid(x) (1)
#else
#define guest_addr_valid(x) ((x) <= GUEST_ADDR_MAX)
#endif
#define h2g_valid(x) guest_addr_valid((unsigned long)(x) - guest_base)
#define h2g_valid(x) guest_addr_valid((uintptr_t)(x) - guest_base)

static inline int guest_range_valid(unsigned long start, unsigned long len)
{
return len - 1 <= GUEST_ADDR_MAX && start <= GUEST_ADDR_MAX - len + 1;
}

#define h2g_nocheck(x) ({ \
unsigned long __ret = (unsigned long)(x) - guest_base; \
uintptr_t __ret = (uintptr_t)(x) - guest_base; \
(abi_ptr)__ret; \
})

Expand Down

0 comments on commit 8c1dc74

Please sign in to comment.