[igt-dev] [PATCH i-g-t] i915/gem_softpin: Added test description for test case.

[Michał Winiarski]

Quoting sai.gowtham.ch at intel.com (2020-07-21 07:07:07)
> From: sai gowtham <sai.gowtham.ch at intel.com>
> 
> Added test description for test and to all the subtests that are
> available.
> 
> Signed-off-by: sai gowtham <sai.gowtham.ch at intel.com>
> ---
>  tests/i915/gem_softpin.c | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
> 
> diff --git a/tests/i915/gem_softpin.c b/tests/i915/gem_softpin.c
> index 202abdd8..2e2bf048 100644
> --- a/tests/i915/gem_softpin.c
> +++ b/tests/i915/gem_softpin.c
> @@ -29,6 +29,12 @@
>  #include "i915/gem.h"
>  #include "igt.h"
>  
> +IGT_TEST_DESCRIPTION("These tests exercise the userptr ioctl to create shared buffers"
> +               " between CPU and GPU. They contain error and normal usage scenarios."
> +               " They also contain a couple of stress tests which copy buffers between"
> +               " CPU and GPU. These tests rely on the softpin in order to pin buffers"
> +               " to a certain VA");
> +

Do any of the testcases here use userptr?

-Michał

>  #define EXEC_OBJECT_PINNED     (1<<4)
>  #define EXEC_OBJECT_SUPPORTS_48B_ADDRESS (1<<3)
>  
> @@ -557,30 +563,55 @@ igt_main
>                 igt_require(gem_can_store_dword(fd, 0));
>         }
>  
> +       igt_describe("This test does negative testing like passing invalid offset values,"
> +                       " invalid alignment values, passing beyond bond of aperture value,"
> +                       " and gen8 canonical addressing, the intension of this test to"
> +                       " check the behaviour with invalid inputs");
>         igt_subtest("invalid")
>                 test_invalid(fd);
> +
> +       igt_describe("This test is to verify if the concept of softpin works, where we"
> +                       " are trying pass different offset values to the execbuf and"
> +                       " check the behaviour");
>         igt_subtest("softpin")
>                 test_softpin(fd);
> +
> +       igt_describe("This test checks the behaviour by runing on all possible"
> +                       " page alligned overlaps");
>         igt_subtest("overlap")
>                 test_overlap(fd);
> +
> +       igt_describe("This test swaps the offset values of two objects and checks"
> +                       " the behaviour");
>         igt_subtest("reverse")
>                 test_reverse(fd);
>  
> +       igt_describe("This test explicitly tests the no relocation support");
>         igt_subtest("noreloc")
>                 test_noreloc(fd, NOSLEEP, 0);
> +
> +       igt_describe("Tests the no relocation support with interruptible");
>         igt_subtest("noreloc-interruptible")
>                 test_noreloc(fd, NOSLEEP, INTERRUPTIBLE);
> +
> +       igt_describe("Check norelocations hold versus suspend/resume");
>         igt_subtest("noreloc-S3")
>                 test_noreloc(fd, SUSPEND, 0);
> +
> +       igt_describe("Check norelocations hold versus suspend/resume");
>         igt_subtest("noreloc-S4")
>                 test_noreloc(fd, HIBERNATE, 0);
>  
>         for (int signal = 0; signal <= 1; signal++) {
> +               igt_describe("This test checks the behaviour of softpin with busy batch");
>                 igt_subtest_f("evict-active%s", signal ? "-interruptible" : "")
>                         test_evict_active(fd, signal);
> +
> +               igt_describe("Snoop test by forcibly injecting signals");
>                 igt_subtest_f("evict-snoop%s", signal ? "-interruptible" : "")
>                         test_evict_snoop(fd, signal);
>         }
> +       igt_describe("Checks behaviour of softpin with hung batch");
>         igt_subtest("evict-hang")
>                 test_evict_hang(fd);
>  
> -- 
> 2.25.0
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev