Commit 256f631f authored by Stefano Stabellini's avatar Stefano Stabellini

xen/arm: Introduce xen_ulong_t for unsigned long

All the original Xen headers have xen_ulong_t as unsigned long type, however
when they have been imported in Linux, xen_ulong_t has been replaced with
unsigned long. That might work for x86 and ia64 but it does not for arm.
Bring back xen_ulong_t and let each architecture define xen_ulong_t as they
see fit.

Also explicitly size pointers (__DEFINE_GUEST_HANDLE) to 64 bit.


Changes in v3:

- remove the incorrect changes to multicall_entry;
- remove the change to apic_physbase.
Signed-off-by: default avatarStefano Stabellini <stefano.stabellini@eu.citrix.com>
Acked-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
parent 2e01f166
...@@ -9,8 +9,11 @@ ...@@ -9,8 +9,11 @@
#include <linux/types.h> #include <linux/types.h>
#define uint64_aligned_t uint64_t __attribute__((aligned(8)))
#define __DEFINE_GUEST_HANDLE(name, type) \ #define __DEFINE_GUEST_HANDLE(name, type) \
typedef type * __guest_handle_ ## name typedef struct { union { type *p; uint64_aligned_t q; }; } \
__guest_handle_ ## name
#define DEFINE_GUEST_HANDLE_STRUCT(name) \ #define DEFINE_GUEST_HANDLE_STRUCT(name) \
__DEFINE_GUEST_HANDLE(name, struct name) __DEFINE_GUEST_HANDLE(name, struct name)
...@@ -21,13 +24,14 @@ ...@@ -21,13 +24,14 @@
do { \ do { \
if (sizeof(hnd) == 8) \ if (sizeof(hnd) == 8) \
*(uint64_t *)&(hnd) = 0; \ *(uint64_t *)&(hnd) = 0; \
(hnd) = val; \ (hnd).p = val; \
} while (0) } while (0)
#ifndef __ASSEMBLY__ #ifndef __ASSEMBLY__
/* Explicitly size integers that represent pfns in the interface with /* Explicitly size integers that represent pfns in the interface with
* Xen so that we can have one ABI that works for 32 and 64 bit guests. */ * Xen so that we can have one ABI that works for 32 and 64 bit guests. */
typedef uint64_t xen_pfn_t; typedef uint64_t xen_pfn_t;
typedef uint64_t xen_ulong_t;
/* Guest handles for primitive C types. */ /* Guest handles for primitive C types. */
__DEFINE_GUEST_HANDLE(uchar, unsigned char); __DEFINE_GUEST_HANDLE(uchar, unsigned char);
__DEFINE_GUEST_HANDLE(uint, unsigned int); __DEFINE_GUEST_HANDLE(uint, unsigned int);
......
...@@ -71,6 +71,7 @@ ...@@ -71,6 +71,7 @@
* with Xen so that we could have one ABI that works for 32 and 64 bit * with Xen so that we could have one ABI that works for 32 and 64 bit
* guests. */ * guests. */
typedef unsigned long xen_pfn_t; typedef unsigned long xen_pfn_t;
typedef unsigned long xen_ulong_t;
/* Guest handles for primitive C types. */ /* Guest handles for primitive C types. */
__DEFINE_GUEST_HANDLE(uchar, unsigned char); __DEFINE_GUEST_HANDLE(uchar, unsigned char);
__DEFINE_GUEST_HANDLE(uint, unsigned int); __DEFINE_GUEST_HANDLE(uint, unsigned int);
......
...@@ -51,6 +51,7 @@ ...@@ -51,6 +51,7 @@
* with Xen so that on ARM we can have one ABI that works for 32 and 64 * with Xen so that on ARM we can have one ABI that works for 32 and 64
* bit guests. */ * bit guests. */
typedef unsigned long xen_pfn_t; typedef unsigned long xen_pfn_t;
typedef unsigned long xen_ulong_t;
/* Guest handles for primitive C types. */ /* Guest handles for primitive C types. */
__DEFINE_GUEST_HANDLE(uchar, unsigned char); __DEFINE_GUEST_HANDLE(uchar, unsigned char);
__DEFINE_GUEST_HANDLE(uint, unsigned int); __DEFINE_GUEST_HANDLE(uint, unsigned int);
......
...@@ -34,7 +34,7 @@ struct xen_memory_reservation { ...@@ -34,7 +34,7 @@ struct xen_memory_reservation {
GUEST_HANDLE(xen_pfn_t) extent_start; GUEST_HANDLE(xen_pfn_t) extent_start;
/* Number of extents, and size/alignment of each (2^extent_order pages). */ /* Number of extents, and size/alignment of each (2^extent_order pages). */
unsigned long nr_extents; xen_ulong_t nr_extents;
unsigned int extent_order; unsigned int extent_order;
/* /*
...@@ -92,7 +92,7 @@ struct xen_memory_exchange { ...@@ -92,7 +92,7 @@ struct xen_memory_exchange {
* command will be non-zero. * command will be non-zero.
* 5. THIS FIELD MUST BE INITIALISED TO ZERO BY THE CALLER! * 5. THIS FIELD MUST BE INITIALISED TO ZERO BY THE CALLER!
*/ */
unsigned long nr_exchanged; xen_ulong_t nr_exchanged;
}; };
DEFINE_GUEST_HANDLE_STRUCT(xen_memory_exchange); DEFINE_GUEST_HANDLE_STRUCT(xen_memory_exchange);
...@@ -148,8 +148,8 @@ DEFINE_GUEST_HANDLE_STRUCT(xen_machphys_mfn_list); ...@@ -148,8 +148,8 @@ DEFINE_GUEST_HANDLE_STRUCT(xen_machphys_mfn_list);
*/ */
#define XENMEM_machphys_mapping 12 #define XENMEM_machphys_mapping 12
struct xen_machphys_mapping { struct xen_machphys_mapping {
unsigned long v_start, v_end; /* Start and end virtual addresses. */ xen_ulong_t v_start, v_end; /* Start and end virtual addresses. */
unsigned long max_mfn; /* Maximum MFN that can be looked up. */ xen_ulong_t max_mfn; /* Maximum MFN that can be looked up. */
}; };
DEFINE_GUEST_HANDLE_STRUCT(xen_machphys_mapping_t); DEFINE_GUEST_HANDLE_STRUCT(xen_machphys_mapping_t);
...@@ -172,7 +172,7 @@ struct xen_add_to_physmap { ...@@ -172,7 +172,7 @@ struct xen_add_to_physmap {
unsigned int space; unsigned int space;
/* Index into source mapping space. */ /* Index into source mapping space. */
unsigned long idx; xen_ulong_t idx;
/* GPFN where the source mapping page should appear. */ /* GPFN where the source mapping page should appear. */
xen_pfn_t gpfn; xen_pfn_t gpfn;
...@@ -189,7 +189,7 @@ struct xen_translate_gpfn_list { ...@@ -189,7 +189,7 @@ struct xen_translate_gpfn_list {
domid_t domid; domid_t domid;
/* Length of list. */ /* Length of list. */
unsigned long nr_gpfns; xen_ulong_t nr_gpfns;
/* List of GPFNs to translate. */ /* List of GPFNs to translate. */
GUEST_HANDLE(ulong) gpfn_list; GUEST_HANDLE(ulong) gpfn_list;
......
...@@ -56,7 +56,7 @@ struct physdev_eoi { ...@@ -56,7 +56,7 @@ struct physdev_eoi {
#define PHYSDEVOP_pirq_eoi_gmfn_v2 28 #define PHYSDEVOP_pirq_eoi_gmfn_v2 28
struct physdev_pirq_eoi_gmfn { struct physdev_pirq_eoi_gmfn {
/* IN */ /* IN */
unsigned long gmfn; xen_ulong_t gmfn;
}; };
/* /*
......
...@@ -45,7 +45,7 @@ struct xen_changeset_info { ...@@ -45,7 +45,7 @@ struct xen_changeset_info {
#define XENVER_platform_parameters 5 #define XENVER_platform_parameters 5
struct xen_platform_parameters { struct xen_platform_parameters {
unsigned long virt_start; xen_ulong_t virt_start;
}; };
#define XENVER_get_features 6 #define XENVER_get_features 6
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment