On 26/02/2024 11.11, Nicholas Piggin wrote:
SPRs annotated with SPR_HARNESS can change between consecutive reads because the test harness code has changed them. Avoid failing the test in this case.Signed-off-by: Nicholas Piggin <npig...@gmail.com> --- powerpc/sprs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/powerpc/sprs.c b/powerpc/sprs.c index 8253ea971..44edd0d7b 100644 --- a/powerpc/sprs.c +++ b/powerpc/sprs.c @@ -563,7 +563,7 @@ int main(int argc, char **argv) if (before[i] >> 32) pass = false; } - if (!(sprs[i].type & SPR_ASYNC) && (before[i] != after[i])) + if (!(sprs[i].type & (SPR_HARNESS|SPR_ASYNC)) && (before[i] != after[i])) pass = false;if (sprs[i].width == 32 && !(before[i] >> 32) && !(after[i] >> 32))
I guess you could also squash this into the previous patch (to avoid problems with bisecting later?) ...
Anyway: Reviewed-by: Thomas Huth <th...@redhat.com>
