diff options
| author | Nicholas Piggin <[email protected]> | 2020-02-26 15:53:02 +1000 |
|---|---|---|
| committer | Michael Ellerman <[email protected]> | 2020-03-18 00:05:54 +1100 |
| commit | 59ed2adf393109c56d383e568f2e57bb5ad6d901 (patch) | |
| tree | 76d5572ad94a8e1c01c3ed40535d74a6d0e987f6 | |
| parent | 993cfecc59e57de237ae27fadc84ed24efa87a4d (diff) | |
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 <[email protected]>
Reviewed-by: Ravi Bangoria <[email protected]>
[mpe: Write change log]
Signed-off-by: Michael Ellerman <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
| -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"); |