diff options
| author |   Mathieu Desnoyers <mathieu.desnoyers@efficios.com> | 2022-01-24 12:12:48 -0500 |
|---|---|---|
| committer |   Peter Zijlstra <peterz@infradead.org> | 2022-02-02 13:11:36 +0100 |
| commit | 26dc8a6d8e11552f3b797b5aafe01071ca32d692 (patch) | |
| tree | 305500a16e20756407834b7ce8bab398b4233c39 /tools/testing/selftests/rseq/param_test.c | |
| parent | selftests/rseq: Fix ppc32 missing instruction selection "u" and "x" for load/store (diff) | |
| download | wireguard-linux-26dc8a6d8e11552f3b797b5aafe01071ca32d692.tar.xz wireguard-linux-26dc8a6d8e11552f3b797b5aafe01071ca32d692.zip | |
selftests/rseq: Fix ppc32 offsets by using long rather than off_t
The semantic of off_t is for file offsets. We mean to use it as an
offset from a pointer. We really expect it to fit in a single register,
and not use a 64-bit type on 32-bit architectures.
Fix runtime issues on ppc32 where the offset is always 0 due to
inconsistency between the argument type (off_t -> 64-bit) and type
expected by the inline assembler (32-bit).
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20220124171253.22072-11-mathieu.desnoyers@efficios.com
Diffstat (limited to 'tools/testing/selftests/rseq/param_test.c')
| -rw-r--r-- | tools/testing/selftests/rseq/param_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/rseq/param_test.c b/tools/testing/selftests/rseq/param_test.c index cc2cfc1da938..335c290b39e7 100644 --- a/tools/testing/selftests/rseq/param_test.c +++ b/tools/testing/selftests/rseq/param_test.c @@ -549,7 +549,7 @@ struct percpu_list_node *this_cpu_list_pop(struct percpu_list *list, for (;;) { struct percpu_list_node *head; intptr_t *targetptr, expectnot, *load; - off_t offset; + long offset; int ret; cpu = rseq_cpu_start(); |