public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps
@ 2019-05-29 10:51 Petri Latvala
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required Petri Latvala
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Petri Latvala @ 2019-05-29 10:51 UTC (permalink / raw)
  To: igt-dev; +Cc: Petri Latvala

Listing it once should be quite enough.

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Simon Ser <simon.ser@intel.com>
---
 lib/meson.build | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/meson.build b/lib/meson.build
index cdb450e1..ae1a9120 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -69,7 +69,6 @@ lib_deps = [
 	libkmod,
 	libprocps,
 	libudev,
-	libdw,
 	math,
 	pciaccess,
 	pixman,
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required
  2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
@ 2019-05-29 10:51 ` Petri Latvala
  2019-05-29 10:54   ` Ser, Simon
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed Petri Latvala
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 10+ messages in thread
From: Petri Latvala @ 2019-05-29 10:51 UTC (permalink / raw)
  To: igt-dev; +Cc: Petri Latvala

libpciaccess and libkmod have been mandatory already, but not marked
as required.

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Simon Ser <simon.ser@intel.com>
---
 meson.build | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meson.build b/meson.build
index 6268c58d..336ce4e6 100644
--- a/meson.build
+++ b/meson.build
@@ -114,8 +114,8 @@ endif
 
 build_info += 'With libdrm: ' + ','.join(libdrm_info)
 
-pciaccess = dependency('pciaccess', version : '>=0.10')
-libkmod = dependency('libkmod')
+pciaccess = dependency('pciaccess', version : '>=0.10', required : true)
+libkmod = dependency('libkmod', required : true)
 libprocps = dependency('libprocps', required : true)
 
 libunwind = dependency('libunwind', required : get_option('with_libunwind'))
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed
  2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required Petri Latvala
@ 2019-05-29 10:51 ` Petri Latvala
  2019-05-29 11:07   ` Petri Latvala
  2019-05-31 10:31   ` Ser, Simon
  2019-05-29 10:55 ` [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Ser, Simon
                   ` (2 subsequent siblings)
  4 siblings, 2 replies; 10+ messages in thread
From: Petri Latvala @ 2019-05-29 10:51 UTC (permalink / raw)
  To: igt-dev; +Cc: Petri Latvala

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Simon Ser <simon.ser@intel.com>
---

Pipeline success without the first two extra patches:
https://gitlab.freedesktop.org/adrinael/igt-gpu-tools/pipelines/39284

Review-trap:

"
But Petri, surely it's better to make the normal build image depend
on the minimal one and just install the extra packages on top. This is
how you do it:

$howyoudoit
"



.gitlab-ci.yml            | 34 ++++++++++++++++++++++++++++++++++
 Dockerfile.debian-minimal | 24 ++++++++++++++++++++++++
 2 files changed, 58 insertions(+)
 create mode 100644 Dockerfile.debian-minimal

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 771143a9..e29429e0 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -68,6 +68,16 @@ build:tests-debian-meson:
     paths:
       - meson-test-list.txt
 
+build:tests-debian-minimal:
+  image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal:latest
+  stage: build
+  script:
+    - meson -Dbuild_tests=disabled -Dwith_libdrm= build
+    - ninja -C build
+  artifacts:
+    paths:
+      - build
+
 build:tests-debian-meson-armhf:
   image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-armhf:latest
   stage: build
@@ -117,6 +127,13 @@ test:ninja-test-clang:
   stage: test
   script: ninja -C build test
 
+test:ninja-test-minimal:
+  image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal:latest
+  dependencies:
+    - build:tests-debian-minimal
+  stage: test
+  script: ninja -C build test
+
 test:ninja-test-arm64:
   image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-arm64:latest
   dependencies:
@@ -187,6 +204,23 @@ containers:igt-debian:
     - docker build -t $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian -f Dockerfile.debian .
     - docker push $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian
 
+containers:igt-debian-minimal:
+  stage: containers
+  image: docker:stable
+  only:
+    changes:
+      - Dockerfile.debian-minimal
+      - .gitlab-ci.yml
+  services:
+    - docker:dind
+  variables:
+    DOCKER_HOST: tcp://docker:2375
+    DOCKER_DRIVER: overlay2
+  script:
+    - docker login -u gitlab-ci-token -p $CI_JOB_TOKEN $CI_REGISTRY
+    - docker build -t $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal -f Dockerfile.debian-minimal .
+    - docker push $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal
+
 containers:igt-debian-armhf:
   stage: containers
   image: docker:stable
diff --git a/Dockerfile.debian-minimal b/Dockerfile.debian-minimal
new file mode 100644
index 00000000..bbe70bed
--- /dev/null
+++ b/Dockerfile.debian-minimal
@@ -0,0 +1,24 @@
+FROM debian:stretch-backports
+
+RUN apt-get update
+RUN apt-get install -y \
+			gcc \
+			flex \
+			bison \
+			pkg-config \
+			libpciaccess-dev \
+			libkmod-dev \
+			libprocps-dev \
+			libdw-dev \
+			zlib1g-dev \
+			liblzma-dev \
+			libcairo-dev \
+			libpixman-1-dev \
+			libudev-dev \
+			libxrandr-dev \
+			libxv-dev \
+			x11proto-dri2-dev
+
+RUN apt-get install -t stretch-backports -y \
+					meson \
+					libdrm-dev
-- 
2.19.1

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

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required Petri Latvala
@ 2019-05-29 10:54   ` Ser, Simon
  2019-05-29 10:57     ` Petri Latvala
  0 siblings, 1 reply; 10+ messages in thread
From: Ser, Simon @ 2019-05-29 10:54 UTC (permalink / raw)
  To: igt-dev@lists.freedesktop.org, Latvala, Petri

On Wed, 2019-05-29 at 13:51 +0300, Petri Latvala wrote:
> libpciaccess and libkmod have been mandatory already, but not marked
> as required.

I thought mandatory deps were the default?

In other words, I believe this:

   libkmod = dependency('libkmod')

Makes libkmod a mandatory dep.

> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> Cc: Simon Ser <simon.ser@intel.com>
> ---
>  meson.build | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index 6268c58d..336ce4e6 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -114,8 +114,8 @@ endif
>  
>  build_info += 'With libdrm: ' + ','.join(libdrm_info)
>  
> -pciaccess = dependency('pciaccess', version : '>=0.10')
> -libkmod = dependency('libkmod')
> +pciaccess = dependency('pciaccess', version : '>=0.10', required : true)
> +libkmod = dependency('libkmod', required : true)
>  libprocps = dependency('libprocps', required : true)
>  
>  libunwind = dependency('libunwind', required : get_option('with_libunwind'))
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps
  2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required Petri Latvala
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed Petri Latvala
@ 2019-05-29 10:55 ` Ser, Simon
  2019-05-29 12:34 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/3] " Patchwork
  2019-05-29 16:37 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Ser, Simon @ 2019-05-29 10:55 UTC (permalink / raw)
  To: igt-dev@lists.freedesktop.org, Latvala, Petri

On Wed, 2019-05-29 at 13:51 +0300, Petri Latvala wrote:
> Listing it once should be quite enough.
> 
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> Cc: Simon Ser <simon.ser@intel.com>

Reviewed-by: Simon Ser <simon.ser@intel.com>

> ---
>  lib/meson.build | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/lib/meson.build b/lib/meson.build
> index cdb450e1..ae1a9120 100644
> --- a/lib/meson.build
> +++ b/lib/meson.build
> @@ -69,7 +69,6 @@ lib_deps = [
>  	libkmod,
>  	libprocps,
>  	libudev,
> -	libdw,
>  	math,
>  	pciaccess,
>  	pixman,
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required
  2019-05-29 10:54   ` Ser, Simon
@ 2019-05-29 10:57     ` Petri Latvala
  0 siblings, 0 replies; 10+ messages in thread
From: Petri Latvala @ 2019-05-29 10:57 UTC (permalink / raw)
  To: Ser, Simon; +Cc: igt-dev@lists.freedesktop.org

On Wed, May 29, 2019 at 01:54:54PM +0300, Ser, Simon wrote:
> On Wed, 2019-05-29 at 13:51 +0300, Petri Latvala wrote:
> > libpciaccess and libkmod have been mandatory already, but not marked
> > as required.
> 
> I thought mandatory deps were the default?
> 
> In other words, I believe this:
> 
>    libkmod = dependency('libkmod')
> 
> Makes libkmod a mandatory dep.

You're right, I got the default backwards!


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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed Petri Latvala
@ 2019-05-29 11:07   ` Petri Latvala
  2019-05-31 10:31   ` Ser, Simon
  1 sibling, 0 replies; 10+ messages in thread
From: Petri Latvala @ 2019-05-29 11:07 UTC (permalink / raw)
  To: igt-dev

On Wed, May 29, 2019 at 01:51:50PM +0300, Petri Latvala wrote:
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> Cc: Simon Ser <simon.ser@intel.com>


Naturally I forgot to add a better commit message to this:

The meson command line has -Dbuild_tests=disabled -Dwith_libdrm=
because those aren't trivial to make automatic. Plain libdrm is also
required, but the _vendor ones are optional so for now those are just
explicitly disabled. Otherwise this tests that the automated machinery
is able to figure out what can be built.



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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/3] meson: Remove redundant libdw from lib_deps
  2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
                   ` (2 preceding siblings ...)
  2019-05-29 10:55 ` [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Ser, Simon
@ 2019-05-29 12:34 ` Patchwork
  2019-05-29 16:37 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2019-05-29 12:34 UTC (permalink / raw)
  To: Petri Latvala; +Cc: igt-dev

== Series Details ==

Series: series starting with [i-g-t,1/3] meson: Remove redundant libdw from lib_deps
URL   : https://patchwork.freedesktop.org/series/61309/
State : success

== Summary ==

CI Bug Log - changes from IGT_5024 -> IGTPW_3071
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

Known issues
------------

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

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_basic@userptr:
    - fi-kbl-8809g:       [PASS][1] -> [DMESG-WARN][2] ([fdo#108965])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/fi-kbl-8809g/igt@amdgpu/amd_basic@userptr.html

  * igt@i915_selftest@live_contexts:
    - fi-bdw-gvtdvm:      [PASS][3] -> [DMESG-FAIL][4] ([fdo#110235])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/fi-bdw-gvtdvm/igt@i915_selftest@live_contexts.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/fi-bdw-gvtdvm/igt@i915_selftest@live_contexts.html

  
#### Warnings ####

  * igt@i915_pm_rpm@basic-pci-d3-state:
    - fi-pnv-d510:        [INCOMPLETE][5] ([fdo#110740]) -> [SKIP][6] ([fdo#109271])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/fi-pnv-d510/igt@i915_pm_rpm@basic-pci-d3-state.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/fi-pnv-d510/igt@i915_pm_rpm@basic-pci-d3-state.html

  
  [fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#110235]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
  [fdo#110740]: https://bugs.freedesktop.org/show_bug.cgi?id=110740


Participating hosts (46 -> 40)
------------------------------

  Additional (1): fi-skl-guc 
  Missing    (7): fi-kbl-soraka fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-cfl-8109u fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * IGT: IGT_5024 -> IGTPW_3071

  CI_DRM_6160: 43905c26d4fd15cba890e62f271befc0eca8de4c @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3071: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/
  IGT_5024: f414756be2ac57e194919973da7b86644ba61241 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [i-g-t,1/3] meson: Remove redundant libdw from lib_deps
  2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
                   ` (3 preceding siblings ...)
  2019-05-29 12:34 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/3] " Patchwork
@ 2019-05-29 16:37 ` Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2019-05-29 16:37 UTC (permalink / raw)
  To: Petri Latvala; +Cc: igt-dev

== Series Details ==

Series: series starting with [i-g-t,1/3] meson: Remove redundant libdw from lib_deps
URL   : https://patchwork.freedesktop.org/series/61309/
State : failure

== Summary ==

CI Bug Log - changes from IGT_5024_full -> IGTPW_3071_full
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_3071_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_3071_full, 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/61309/revisions/1/mbox/

Possible new issues
-------------------

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

### IGT changes ###

#### Possible regressions ####

  * igt@kms_cursor_legacy@cursor-vs-flip-atomic:
    - shard-glk:          [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-glk1/igt@kms_cursor_legacy@cursor-vs-flip-atomic.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-glk3/igt@kms_cursor_legacy@cursor-vs-flip-atomic.html

  
Known issues
------------

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_tiled_swapping@non-threaded:
    - shard-glk:          [PASS][3] -> [DMESG-WARN][4] ([fdo#108686])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-glk9/igt@gem_tiled_swapping@non-threaded.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-glk3/igt@gem_tiled_swapping@non-threaded.html

  * igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen:
    - shard-kbl:          [PASS][5] -> [FAIL][6] ([fdo#103232])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-kbl3/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-kbl7/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
    - shard-apl:          [PASS][7] -> [FAIL][8] ([fdo#103232])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl4/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl3/igt@kms_cursor_crc@pipe-c-cursor-64x64-onscreen.html

  * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic:
    - shard-hsw:          [PASS][9] -> [SKIP][10] ([fdo#109271]) +8 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-hsw8/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-hsw1/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html

  * igt@kms_universal_plane@universal-plane-pipe-a-functional:
    - shard-apl:          [PASS][11] -> [FAIL][12] ([fdo#110037])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl6/igt@kms_universal_plane@universal-plane-pipe-a-functional.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl5/igt@kms_universal_plane@universal-plane-pipe-a-functional.html
    - shard-kbl:          [PASS][13] -> [FAIL][14] ([fdo#110037])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-kbl1/igt@kms_universal_plane@universal-plane-pipe-a-functional.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-kbl6/igt@kms_universal_plane@universal-plane-pipe-a-functional.html

  * igt@kms_vblank@pipe-a-ts-continuation-suspend:
    - shard-apl:          [PASS][15] -> [DMESG-WARN][16] ([fdo#108566]) +6 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl8/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl7/igt@kms_vblank@pipe-a-ts-continuation-suspend.html

  * igt@kms_vblank@pipe-b-wait-forked-busy-hang:
    - shard-glk:          [PASS][17] -> [INCOMPLETE][18] ([fdo#103359] / [k.org#198133])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-glk8/igt@kms_vblank@pipe-b-wait-forked-busy-hang.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-glk8/igt@kms_vblank@pipe-b-wait-forked-busy-hang.html

  * igt@prime_busy@wait-hang-blt:
    - shard-apl:          [PASS][19] -> [INCOMPLETE][20] ([fdo#103927])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl1/igt@prime_busy@wait-hang-blt.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl1/igt@prime_busy@wait-hang-blt.html

  
#### Possible fixes ####

  * igt@gem_ctx_switch@basic-all-heavy:
    - shard-glk:          [INCOMPLETE][21] ([fdo#103359] / [k.org#198133]) -> [PASS][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-glk9/igt@gem_ctx_switch@basic-all-heavy.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-glk5/igt@gem_ctx_switch@basic-all-heavy.html

  * igt@gem_eio@in-flight-suspend:
    - shard-apl:          [FAIL][23] ([fdo#110667]) -> [PASS][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl3/igt@gem_eio@in-flight-suspend.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl5/igt@gem_eio@in-flight-suspend.html

  * igt@i915_pm_rc6_residency@rc6-accuracy:
    - shard-snb:          [SKIP][25] ([fdo#109271]) -> [PASS][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-snb2/igt@i915_pm_rc6_residency@rc6-accuracy.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-snb4/igt@i915_pm_rc6_residency@rc6-accuracy.html

  * igt@i915_suspend@fence-restore-tiled2untiled:
    - shard-apl:          [DMESG-WARN][27] ([fdo#108566]) -> [PASS][28] +3 similar issues
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-apl6/igt@i915_suspend@fence-restore-tiled2untiled.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-apl8/igt@i915_suspend@fence-restore-tiled2untiled.html

  * igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic:
    - shard-hsw:          [SKIP][29] ([fdo#109271]) -> [PASS][30] +21 similar issues
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-hsw1/igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-hsw5/igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic.html

  * igt@kms_flip@flip-vs-suspend:
    - shard-snb:          [DMESG-WARN][31] ([fdo#102365]) -> [PASS][32]
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-snb5/igt@kms_flip@flip-vs-suspend.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-snb7/igt@kms_flip@flip-vs-suspend.html

  * igt@kms_flip@plain-flip-fb-recreate:
    - shard-glk:          [FAIL][33] ([fdo#100368]) -> [PASS][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5024/shard-glk8/igt@kms_flip@plain-flip-fb-recreate.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/shard-glk9/igt@kms_flip@plain-flip-fb-recreate.html

  
  [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
  [fdo#102365]: https://bugs.freedesktop.org/show_bug.cgi?id=102365
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
  [fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#110037]: https://bugs.freedesktop.org/show_bug.cgi?id=110037
  [fdo#110667]: https://bugs.freedesktop.org/show_bug.cgi?id=110667
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (6 -> 5)
------------------------------

  Missing    (1): shard-skl 


Build changes
-------------

  * IGT: IGT_5024 -> IGTPW_3071

  CI_DRM_6160: 43905c26d4fd15cba890e62f271befc0eca8de4c @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3071: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3071/
  IGT_5024: f414756be2ac57e194919973da7b86644ba61241 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed
  2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed Petri Latvala
  2019-05-29 11:07   ` Petri Latvala
@ 2019-05-31 10:31   ` Ser, Simon
  1 sibling, 0 replies; 10+ messages in thread
From: Ser, Simon @ 2019-05-31 10:31 UTC (permalink / raw)
  To: igt-dev@lists.freedesktop.org, Latvala, Petri

On Wed, 2019-05-29 at 13:51 +0300, Petri Latvala wrote:
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
> Cc: Simon Ser <simon.ser@intel.com>
> ---
> 
> Pipeline success without the first two extra patches:
> https://gitlab.freedesktop.org/adrinael/igt-gpu-tools/pipelines/39284
> 
> Review-trap:
> 
> "
> But Petri, surely it's better to make the normal build image depend
> on the minimal one and just install the extra packages on top. This is
> how you do it:
> 
> $howyoudoit
> "
> 

But Petri, surely it's better to make the normal build image depend
on the minimal one and just install the extra packages on top. This is
how you do it:

https://gitlab.freedesktop.org/emersion/igt-gpu-tools/commit/f5df872c71ff8a8ec7a6d306323dfe61b0bfbd8a

(We can probably remove some stuff from Dockerfile.debian after this
change)

> 
> .gitlab-ci.yml            | 34 ++++++++++++++++++++++++++++++++++
>  Dockerfile.debian-minimal | 24 ++++++++++++++++++++++++
>  2 files changed, 58 insertions(+)
>  create mode 100644 Dockerfile.debian-minimal
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 771143a9..e29429e0 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -68,6 +68,16 @@ build:tests-debian-meson:
>      paths:
>        - meson-test-list.txt
>  
> +build:tests-debian-minimal:
> +  image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal:latest
> +  stage: build
> +  script:
> +    - meson -Dbuild_tests=disabled -Dwith_libdrm= build
> +    - ninja -C build
> +  artifacts:
> +    paths:
> +      - build
> +
>  build:tests-debian-meson-armhf:
>    image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-armhf:latest
>    stage: build
> @@ -117,6 +127,13 @@ test:ninja-test-clang:
>    stage: test
>    script: ninja -C build test
>  
> +test:ninja-test-minimal:
> +  image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal:latest
> +  dependencies:
> +    - build:tests-debian-minimal
> +  stage: test
> +  script: ninja -C build test
> +
>  test:ninja-test-arm64:
>    image: $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-arm64:latest
>    dependencies:
> @@ -187,6 +204,23 @@ containers:igt-debian:
>      - docker build -t $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian -f Dockerfile.debian .
>      - docker push $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian
>  
> +containers:igt-debian-minimal:
> +  stage: containers
> +  image: docker:stable
> +  only:
> +    changes:
> +      - Dockerfile.debian-minimal
> +      - .gitlab-ci.yml
> +  services:
> +    - docker:dind
> +  variables:
> +    DOCKER_HOST: tcp://docker:2375
> +    DOCKER_DRIVER: overlay2
> +  script:
> +    - docker login -u gitlab-ci-token -p $CI_JOB_TOKEN $CI_REGISTRY
> +    - docker build -t $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal -f Dockerfile.debian-minimal .
> +    - docker push $CI_REGISTRY/$CI_PROJECT_PATH/igt-debian-minimal
> +
>  containers:igt-debian-armhf:
>    stage: containers
>    image: docker:stable
> diff --git a/Dockerfile.debian-minimal b/Dockerfile.debian-minimal
> new file mode 100644
> index 00000000..bbe70bed
> --- /dev/null
> +++ b/Dockerfile.debian-minimal
> @@ -0,0 +1,24 @@
> +FROM debian:stretch-backports
> +
> +RUN apt-get update
> +RUN apt-get install -y \
> +			gcc \
> +			flex \
> +			bison \
> +			pkg-config \
> +			libpciaccess-dev \
> +			libkmod-dev \
> +			libprocps-dev \
> +			libdw-dev \
> +			zlib1g-dev \
> +			liblzma-dev \
> +			libcairo-dev \
> +			libpixman-1-dev \
> +			libudev-dev \
> +			libxrandr-dev \
> +			libxv-dev \
> +			x11proto-dri2-dev
> +
> +RUN apt-get install -t stretch-backports -y \
> +					meson \
> +					libdrm-dev
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2019-05-31 10:31 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-29 10:51 [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Petri Latvala
2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 2/3] meson: Mark pciaccess and kmod as required Petri Latvala
2019-05-29 10:54   ` Ser, Simon
2019-05-29 10:57     ` Petri Latvala
2019-05-29 10:51 ` [igt-dev] [PATCH i-g-t 3/3] gitlab-CI: Do a build-test without any optional dependencies installed Petri Latvala
2019-05-29 11:07   ` Petri Latvala
2019-05-31 10:31   ` Ser, Simon
2019-05-29 10:55 ` [igt-dev] [PATCH i-g-t 1/3] meson: Remove redundant libdw from lib_deps Ser, Simon
2019-05-29 12:34 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/3] " Patchwork
2019-05-29 16:37 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox