"Aneesh Kumar K.V" <aneesh.ku...@linux.vnet.ibm.com> writes:
> Michael Ellerman <m...@ellerman.id.au> writes: > >> It's helpful for automated testing if the test returns error codes back >> to the calling program. >> >> Signed-off-by: Michael Ellerman <m...@ellerman.id.au> > > > Reviewed-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com> > >> --- >> tools/testing/selftests/powerpc/mm/subpage_prot.c | 10 ++++++---- >> 1 file changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/tools/testing/selftests/powerpc/mm/subpage_prot.c >> b/tools/testing/selftests/powerpc/mm/subpage_prot.c >> index 440180ff8089..7ccdc96b977d 100644 >> --- a/tools/testing/selftests/powerpc/mm/subpage_prot.c >> +++ b/tools/testing/selftests/powerpc/mm/subpage_prot.c >> @@ -207,14 +207,16 @@ int test_file(void) >> >> int main(int argc, char *argv[]) >> { >> - test_harness(test_anon, "subpage_prot_anon"); >> + int rc; >> + >> + rc = test_harness(test_anon, "subpage_prot_anon"); >> + if (rc) >> + return rc; >> >> if (argc > 1) >> file_name = argv[1]; >> else >> file_name = "tempfile"; >> >> - test_harness(test_file, "subpage_prot_file"); >> - >> - return 0; >> + return test_harness(test_file, "subpage_prot_file"); >> } >> -- >> 2.5.0 While doing this fix the error print ? 520175565: (4520111850): Failed at 0x0x3fffad4b0000 (p=13,sp=0,w=0), want=fault, got=pass ! diff --git a/tools/testing/selftests/powerpc/mm/subpage_prot.c b/tools/testing/selftests/powerpc/mm/subpage_prot.c index 440180ff8089..5929866aa444 100644 --- a/tools/testing/selftests/powerpc/mm/subpage_prot.c +++ b/tools/testing/selftests/powerpc/mm/subpage_prot.c @@ -73,7 +73,7 @@ static inline void check_faulted(void *addr, long page, long subpage, int write) want_fault |= (subpage == ((page + 1) % 16)); if (faulted != want_fault) { - printf("Failed at 0x%p (p=%ld,sp=%ld,w=%d), want=%s, got=%s !\n", + printf("Failed at %p (p=%ld,sp=%ld,w=%d), want=%s, got=%s !\n", addr, page, subpage, write, want_fault ? "fault" : "pass", faulted ? "fault" : "pass"); _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev