diff options
| author |   Michael Ellerman <mpe@ellerman.id.au> | 2014-07-23 17:31:33 +1000 |
|---|---|---|
| committer |   Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2014-07-28 14:11:29 +1000 |
| commit | 6873def90016cc1cde03c38e10a80146d980b48a (patch) | |
| tree | cb09eb8cd3d070889e7df20ae259210afe4afba5 /tools/testing/selftests/powerpc/pmu/ebb/Makefile | |
| parent | selftests/powerpc: Fix parse_proc_maps() (diff) | |
| download | linux-dev-6873def90016cc1cde03c38e10a80146d980b48a.tar.xz linux-dev-6873def90016cc1cde03c38e10a80146d980b48a.zip | |
selftests/powerpc: Move core_busy_loop() into asm
There is at least one bug in core_busy_loop(), we use r0, but it's
not in the clobber list. We were getting away with this it seems but
that was luck.
It's also fishy to be touching the stack, even if we do it below the
stack pointer. It seems we get away with it, but looking at the
generated code that may just be luck.
So move it into assembler, do all the stack handling by hand. We create
a stack frame to save the non-volatiles in, so we can muck around with
them.
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to '')
| -rw-r--r-- | tools/testing/selftests/powerpc/pmu/ebb/Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/powerpc/pmu/ebb/Makefile b/tools/testing/selftests/powerpc/pmu/ebb/Makefile index edbba2affc2c..251447eafe49 100644 --- a/tools/testing/selftests/powerpc/pmu/ebb/Makefile +++ b/tools/testing/selftests/powerpc/pmu/ebb/Makefile @@ -17,7 +17,7 @@ PROGS := reg_access_test event_attributes_test cycles_test \ all: $(PROGS) -$(PROGS): ../../harness.c ../event.c ../lib.c ebb.c ebb_handler.S trace.c +$(PROGS): ../../harness.c ../event.c ../lib.c ebb.c ebb_handler.S trace.c busy_loop.S instruction_count_test: ../loop.S |