diff options
| author | Nicholas Piggin <npiggin@gmail.com> | 2020-02-26 15:53:02 +1000 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2020-03-18 00:05:54 +1100 |
| commit | 59ed2adf393109c56d383e568f2e57bb5ad6d901 (patch) | |
| tree | 76d5572ad94a8e1c01c3ed40535d74a6d0e987f6 | |
| parent | 993cfecc59e57de237ae27fadc84ed24efa87a4d (diff) | |
| download | linux-59ed2adf393109c56d383e568f2e57bb5ad6d901.tar.bz2 | |
powerpc/lib: Fix emulate_step() std test
We should be checking that the instruction was stepped *and* that the
target register has the right value.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Reviewed-by: Ravi Bangoria <ravi.bangoria@linux.ibm.com>
[mpe: Write change log]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20200226055302.1577954-1-npiggin@gmail.com
| -rw-r--r-- | arch/powerpc/lib/test_emulate_step.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/lib/test_emulate_step.c b/arch/powerpc/lib/test_emulate_step.c index 42347067739c..00d70253cb5b 100644 --- a/arch/powerpc/lib/test_emulate_step.c +++ b/arch/powerpc/lib/test_emulate_step.c @@ -160,7 +160,7 @@ static void __init test_std(void) /* std r5, 0(r3) */ stepped = emulate_step(®s, TEST_STD(5, 3, 0)); - if (stepped == 1 || regs.gpr[5] == a) + if (stepped == 1 && regs.gpr[5] == a) show_result("std", "PASS"); else show_result("std", "FAIL"); |