[igt-dev] [PATCH i-g-t] tests: build gem_concurrent_all

[igt-dev] [PATCH i-g-t] tests: build gem_concurrent_all

[Lucas De Marchi]

This was removed from default build in autotools because it has lots of
subtests: a633ad03 ("tests: Separate tests with lots of subtests").
However in meson there's no option to actually build it.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---

Or do we want to hide it under a flag?

 tests/meson.build | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/meson.build b/tests/meson.build
index 029b8043..dd111d1c 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -27,6 +27,7 @@ test_progs = [
 	'gem_create',
 	'gem_cs_prefetch',
 	'gem_cs_tlb',
+	'gem_concurrent_all',
 	'gem_ctx_bad_destroy',
 	'gem_ctx_bad_exec',
 	'gem_ctx_create',
-- 
2.17.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t] tests: build gem_concurrent_all

[Chris Wilson]

Quoting Lucas De Marchi (2018-07-12 21:05:43)
> This was removed from default build in autotools because it has lots of
> subtests: a633ad03 ("tests: Separate tests with lots of subtests").
> However in meson there's no option to actually build it.
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> 
> Or do we want to hide it under a flag?
> 
>  tests/meson.build | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/meson.build b/tests/meson.build
> index 029b8043..dd111d1c 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -27,6 +27,7 @@ test_progs = [
>         'gem_create',
>         'gem_cs_prefetch',
>         'gem_cs_tlb',
> +       'gem_concurrent_all',

Wouldn't alphabetical ordering put it just before gem_concurrent_blit?
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✗ Fi.CI.BAT: failure for tests: build gem_concurrent_all

[Patchwork]

== Series Details ==

Series: tests: build gem_concurrent_all
URL   : https://patchwork.freedesktop.org/series/46435/
State : failure

== Summary ==

= CI Bug Log - changes from CI_DRM_4476 -> IGTPW_1572 =

== Summary - FAILURE ==

  Serious unknown changes coming with IGTPW_1572 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_1572, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/46435/revisions/1/mbox/

== Possible new issues ==

  Here are the unknown changes that may have been introduced in IGTPW_1572:

  === IGT changes ===

    ==== Possible regressions ====

    igt@gem_mmap_gtt@basic-small-copy:
      fi-glk-dsi:         PASS -> INCOMPLETE

    
    ==== Warnings ====

    igt@pm_rpm@basic-rte:
      fi-hsw-peppy:       SKIP -> PASS +1
      fi-hsw-4770r:       SKIP -> PASS +1
      fi-bdw-5557u:       SKIP -> PASS +1
      fi-hsw-4770:        SKIP -> PASS +1

    
== Known issues ==

  Here are the changes found in IGTPW_1572 that come from known issues:

  === IGT changes ===

    ==== Issues hit ====

    igt@drv_module_reload@basic-no-display:
      {fi-skl-iommu}:     NOTRUN -> FAIL (fdo#106066) +2

    igt@prime_vgem@basic-fence-flip:
      fi-ilk-650:         PASS -> FAIL (fdo#104008)

    
    ==== Possible fixes ====

    igt@kms_flip@basic-flip-vs-dpms:
      fi-skl-6700hq:      DMESG-WARN (fdo#105998) -> PASS

    
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  fdo#104008 https://bugs.freedesktop.org/show_bug.cgi?id=104008
  fdo#105998 https://bugs.freedesktop.org/show_bug.cgi?id=105998
  fdo#106066 https://bugs.freedesktop.org/show_bug.cgi?id=106066


== Participating hosts (45 -> 38) ==

  Additional (2): fi-byt-j1900 fi-skl-iommu 
  Missing    (9): fi-ilk-m540 fi-bsw-n3050 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-bwr-2160 fi-ctg-p8600 fi-gdg-551 fi-pnv-d510 


== Build changes ==

    * IGT: IGT_4552 -> IGTPW_1572

  CI_DRM_4476: b818fac0878147c6df45338cb515b9b7bd878b7f @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_1572: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1572/
  IGT_4552: 5175aff31e00e17786ebb97aaaf25ddd38b5e72e @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+++ 571200 lines
--- 0 lines

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1572/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [igt-dev] [PATCH i-g-t] tests: build gem_concurrent_all

[Lucas De Marchi]

On Thu, Jul 12, 2018 at 1:14 PM Chris Wilson <chris@chris-wilson.co.uk> wrote:
>
> Quoting Lucas De Marchi (2018-07-12 21:05:43)
> > This was removed from default build in autotools because it has lots of
> > subtests: a633ad03 ("tests: Separate tests with lots of subtests").
> > However in meson there's no option to actually build it.
> >
> > Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> > ---
> >
> > Or do we want to hide it under a flag?
> >
> >  tests/meson.build | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/meson.build b/tests/meson.build
> > index 029b8043..dd111d1c 100644
> > --- a/tests/meson.build
> > +++ b/tests/meson.build
> > @@ -27,6 +27,7 @@ test_progs = [
> >         'gem_create',
> >         'gem_cs_prefetch',
> >         'gem_cs_tlb',
> > +       'gem_concurrent_all',
>
> Wouldn't alphabetical ordering put it just before gem_concurrent_blit?

right, my bad. Regarding that commit, in meson do we want to hide
building this behind a flag?

Lucas De Marchi

> -Chris
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev



-- 
Lucas De Marchi
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev