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

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to