diff options
| author | Mathieu Desnoyers <mathieu.desnoyers@efficios.com> | 2022-01-24 20:12:48 +0300 |
|---|---|---|
| committer | Peter Zijlstra <peterz@infradead.org> | 2022-02-02 15:11:36 +0300 |
| commit | 26dc8a6d8e11552f3b797b5aafe01071ca32d692 (patch) | |
| tree | 305500a16e20756407834b7ce8bab398b4233c39 /tools/testing/selftests/rseq/basic_percpu_ops_test.c | |
| parent | de6b52a21420a18dc8a36438d581efd1313d5fe3 (diff) | |
| download | linux-26dc8a6d8e11552f3b797b5aafe01071ca32d692.tar.xz | |
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/basic_percpu_ops_test.c')
| -rw-r--r-- | tools/testing/selftests/rseq/basic_percpu_ops_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/rseq/basic_percpu_ops_test.c b/tools/testing/selftests/rseq/basic_percpu_ops_test.c index b953a52ff706..517756afc2a4 100644 --- a/tools/testing/selftests/rseq/basic_percpu_ops_test.c +++ b/tools/testing/selftests/rseq/basic_percpu_ops_test.c @@ -167,7 +167,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; cpu = rseq_cpu_start(); |