diff options
| author | Christopher Clark <christopher.w.clark@gmail.com> | 2020-06-29 18:54:54 -0700 | 
|---|---|---|
| committer | Bruce Ashfield <bruce.ashfield@gmail.com> | 2020-07-06 16:25:56 -0400 | 
| commit | 5bf7fae58146c6009679f11f68e96a847bb52976 (patch) | |
| tree | ba9a9390b51eca0d4884ce1e99ccaeb14b552999 /recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch | |
| parent | 90f663b738f59a2bfca778d7d77356cb3ee135d9 (diff) | |
| download | meta-virtualization-5bf7fae58146c6009679f11f68e96a847bb52976.tar.gz | |
xen: add patch to fix build on ARM64 with gcc 10.1.0
The newer gcc toolchain won't link due to a missing builtin:
__sync_fetch_and_add. Xen is built with -fno-builtin, so add a
patch to implement the one required function.
Signed-off-by: Christopher Clark <christopher.w.clark@gmail.com>
Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>
Diffstat (limited to 'recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch')
| -rw-r--r-- | recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch | 50 | 
1 files changed, 50 insertions, 0 deletions
diff --git a/recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch b/recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch new file mode 100644 index 00000000..6e957a50 --- /dev/null +++ b/recipes-extended/xen/files/xen-arm64-implement-atomic-fetch-add.patch  | |||
| @@ -0,0 +1,50 @@ | |||
| 1 | From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Christopher Clark <christopher.w.clark@gmail.com> | ||
| 3 | Date: Fri, 26 June 2020 16:34:00 -0800 | ||
| 4 | Subject: [PATCH] xen: implement atomic op to fix arm64 compilation | ||
| 5 | |||
| 6 | Xen's ARM implementation of arch_fetch_and_add since f9cc3cd9 | ||
| 7 | uses a builtin, despite the build being performed with -fno-builtin. | ||
| 8 | With gcc 10.1.0, this now breaks prelinking spinlock.c, so | ||
| 9 | implement the one atomic operation that is required with logic | ||
| 10 | derived from Linux's atomic_ll_sc.h: ATOMIC_FETCH_OP and comparison with | ||
| 11 | the binary produced with and without the patch with gcc 9.2.0. | ||
| 12 | |||
| 13 | Signed-off-by: Christopher Clark <christopher.w.clark@gmail.com> | ||
| 14 | diff --git a/xen/include/asm-arm/system.h b/xen/include/asm-arm/system.h | ||
| 15 | index e5d062667d..c46dd3ac71 100644 | ||
| 16 | --- a/xen/include/asm-arm/system.h | ||
| 17 | +++ b/xen/include/asm-arm/system.h | ||
| 18 | @@ -55,7 +55,32 @@ static inline int local_abort_is_enabled(void) | ||
| 19 | return !(flags & PSR_ABT_MASK); | ||
| 20 | } | ||
| 21 | |||
| 22 | +#ifdef CONFIG_ARM_64 | ||
| 23 | + | ||
| 24 | +/* see atomic_ll_sc.h: ATOMIC_FETCH_OP(name, mb, acq, rel, cl, op, asm_op, constraint) */ | ||
| 25 | +static inline int arch_fetch_and_add(unsigned int *ptr, unsigned long i) | ||
| 26 | +{ | ||
| 27 | + int register lptr asm("x0"); | ||
| 28 | + int register result asm("w1"); | ||
| 29 | + int register newval asm("w2"); | ||
| 30 | + int register status asm("w3"); | ||
| 31 | + | ||
| 32 | + asm volatile( | ||
| 33 | + " mov %[lptr], %[ptr]\n" | ||
| 34 | + "1: ldxr %w[result], [%[lptr]]\n" | ||
| 35 | + " add %w[newval], %w[result], %w[i]\n" | ||
| 36 | + " stlxr %w[status], %w[newval], [%[lptr]]\n" | ||
| 37 | + " cbnz %w[status], 1b\n" | ||
| 38 | + " dmb ish\n" | ||
| 39 | + : [result] "=&r" (result), [lptr] "=&r" (lptr), [newval] "=&r" (newval), [status] "=&r" (status), [i] "+r" (i), "+Q" (*ptr) | ||
| 40 | + : [ptr] "r" (ptr), "r" (i) | ||
| 41 | + : "memory"); | ||
| 42 | + | ||
| 43 | + return result; | ||
| 44 | +} | ||
| 45 | +#else | ||
| 46 | #define arch_fetch_and_add(x, v) __sync_fetch_and_add(x, v) | ||
| 47 | +#endif | ||
| 48 | |||
| 49 | extern struct vcpu *__context_switch(struct vcpu *prev, struct vcpu *next); | ||
| 50 | |||
