Re: [PATCH] intel-gpu-tools: add igt_core init func calls to some tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 20 Jun 2014 11:28, <tim.gore@xxxxxxxxx> wrote:
>
> From: Tim Gore <tim.gore@xxxxxxxxx>
>
> igt-core.h/c provides some macros and initialisation
> functions to support the tests but some of the single
> tests do not use these. Modifying these tests to use
> the igt_simple_main macro and igt_simple_init function
> is the first step towards a consistent command line
> across all tests.


This also helps add support for other common functionality such as
setting the debug level.

Patch has been merged, thanks.


>
> Signed-off-by: Tim Gore <tim.gore@xxxxxxxxx>
> ---
>  tests/gem_bad_address.c      | 4 +---
>  tests/gem_bad_batch.c        | 4 +---
>  tests/gem_bad_blit.c         | 4 +---
>  tests/gem_hang.c             | 2 ++
>  tests/gem_non_secure_batch.c | 4 +---
>  tests/gem_stress.c           | 2 ++
>  6 files changed, 8 insertions(+), 12 deletions(-)
>
> diff --git a/tests/gem_bad_address.c b/tests/gem_bad_address.c
> index f8fda90..e7a9587 100644
> --- a/tests/gem_bad_address.c
> +++ b/tests/gem_bad_address.c
> @@ -60,7 +60,7 @@ bad_store(void)
>         intel_batchbuffer_flush(batch);
>  }
>
> -int main(int argc, char **argv)
> +igt_simple_main
>  {
>         int fd;
>
> @@ -76,6 +76,4 @@ int main(int argc, char **argv)
>         drm_intel_bufmgr_destroy(bufmgr);
>
>         close(fd);
> -
> -       return 0;
>  }
> diff --git a/tests/gem_bad_batch.c b/tests/gem_bad_batch.c
> index 33b3241..7f92a93 100644
> --- a/tests/gem_bad_batch.c
> +++ b/tests/gem_bad_batch.c
> @@ -56,7 +56,7 @@ bad_batch(void)
>         intel_batchbuffer_flush(batch);
>  }
>
> -int main(int argc, char **argv)
> +igt_simple_main
>  {
>         int fd;
>
> @@ -72,6 +72,4 @@ int main(int argc, char **argv)
>         drm_intel_bufmgr_destroy(bufmgr);
>
>         close(fd);
> -
> -       return 0;
>  }
> diff --git a/tests/gem_bad_blit.c b/tests/gem_bad_blit.c
> index 9c03117..71a9f78 100644
> --- a/tests/gem_bad_blit.c
> +++ b/tests/gem_bad_blit.c
> @@ -95,7 +95,7 @@ bad_blit(drm_intel_bo *src_bo, uint32_t devid)
>         intel_batchbuffer_flush(batch);
>  }
>
> -int main(int argc, char **argv)
> +igt_simple_main
>  {
>         drm_intel_bo *src;
>         int fd;
> @@ -114,6 +114,4 @@ int main(int argc, char **argv)
>         drm_intel_bufmgr_destroy(bufmgr);
>
>         close(fd);
> -
> -       return 0;
>  }
> diff --git a/tests/gem_hang.c b/tests/gem_hang.c
> index 8ebf606..6248244 100644
> --- a/tests/gem_hang.c
> +++ b/tests/gem_hang.c
> @@ -72,6 +72,8 @@ int main(int argc, char **argv)
>  {
>         int fd;
>
> +       igt_simple_init();
> +
>         igt_assert_f(argc == 2,
>                      "usage: %s <disabled pipe number>\n",
>                      argv[0]);
> diff --git a/tests/gem_non_secure_batch.c
b/tests/gem_non_secure_batch.c
> index 9acfda4..01101e9 100644
> --- a/tests/gem_non_secure_batch.c
> +++ b/tests/gem_non_secure_batch.c
> @@ -77,7 +77,7 @@ mi_lri_loop(void)
>         }
>  }
>
> -int main(int argc, char **argv)
> +igt_simple_main
>  {
>         int fd;
>         int devid;
> @@ -108,6 +108,4 @@ int main(int argc, char **argv)
>         drm_intel_bufmgr_destroy(bufmgr);
>
>         close(fd);
> -
> -       return 0;
>  }
> diff --git a/tests/gem_stress.c b/tests/gem_stress.c
> index d46c643..2ccb6fc 100644
> --- a/tests/gem_stress.c
> +++ b/tests/gem_stress.c
> @@ -865,6 +865,8 @@ int main(int argc, char **argv)
>         int i, j;
>         unsigned *current_permutation, *tmp_permutation;
>
> +       igt_simple_init();
> +
>         drm_fd = drm_open_any();
>         devid = intel_get_drm_devid(drm_fd);
>
> --
> 1.9.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux