Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48
@ 2023-05-15 14:39 Mauro Carvalho Chehab
  2023-05-15 16:18 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2023-05-15 14:39 UTC (permalink / raw)
  To: igt-dev

From: Mauro Carvalho Chehab <mchehab@kernel.org>

changeset 8ac9440e7bec ("meson: add an option to control Xe test builds")
require updating a dict with values from another dict. This seems
to be supported only on version 0.48 or above.

So, update meson minimal version.

Fixes: 8ac9440e7bec ("meson: add an option to control Xe test builds")
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
---
 meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index 1c872cc9cb02..a1f646ee9239 100644
--- a/meson.build
+++ b/meson.build
@@ -7,7 +7,7 @@ project('igt-gpu-tools', 'c',
           'buildtype=debugoptimized',
         ],
 	license : 'MIT',
-	meson_version : '>=0.47.2')
+	meson_version : '>=0.48')
 
 if get_option('b_ndebug') != 'false'
 	error('Building without -Db_ndebug=false is not supported')
-- 
2.40.1

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

* [igt-dev] ✗ GitLab.Pipeline: warning for meson.build: update minimal version to 0.48
  2023-05-15 14:39 [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48 Mauro Carvalho Chehab
@ 2023-05-15 16:18 ` Patchwork
  2023-05-15 16:29 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
  2023-05-15 19:15 ` [igt-dev] [PATCH i-g-t] " Kamil Konieczny
  2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2023-05-15 16:18 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: igt-dev

== Series Details ==

Series: meson.build: update minimal version to 0.48
URL   : https://patchwork.freedesktop.org/series/117774/
State : warning

== Summary ==

Pipeline status: FAILED.

see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/881582 for the overview.

build:tests-fedora-oldest-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/41753426):
  Installing collected packages: meson
    Running setup.py install for meson: started
      Running setup.py install for meson: finished with status 'done'
  Successfully installed meson-0.47.2
  $ meson $MESON_OPTIONS build
  The Meson build system
  Version: 0.47.2
  Source dir: /builds/gfx-ci/igt-ci-tags
  Build dir: /builds/gfx-ci/igt-ci-tags/build
  Build type: native build
  
  meson.build:1:0: ERROR:  Meson version is 0.47.2 but project requires >=0.48.
  
  A full log can be found at /builds/gfx-ci/igt-ci-tags/build/meson-logs/meson-log.txt
  section_end:1684167162:step_script
  section_start:1684167162:cleanup_file_variables
  Cleaning up project directory and file based variables
  section_end:1684167165:cleanup_file_variables
  ERROR: Job failed: exit code 1

== Logs ==

For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/881582


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

* [igt-dev] ✗ Fi.CI.BAT: failure for meson.build: update minimal version to 0.48
  2023-05-15 14:39 [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48 Mauro Carvalho Chehab
  2023-05-15 16:18 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
@ 2023-05-15 16:29 ` Patchwork
  2023-05-15 19:15 ` [igt-dev] [PATCH i-g-t] " Kamil Konieczny
  2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2023-05-15 16:29 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 10415 bytes --]

== Series Details ==

Series: meson.build: update minimal version to 0.48
URL   : https://patchwork.freedesktop.org/series/117774/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_13148 -> IGTPW_8960
====================================================

Summary
-------

  **FAILURE**

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

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/index.html

Participating hosts (36 -> 36)
------------------------------

  Additional (1): fi-apl-guc 
  Missing    (1): fi-snb-2520m 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_suspend@basic-s2idle-without-i915:
    - bat-adls-5:         NOTRUN -> [ABORT][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@i915_suspend@basic-s2idle-without-i915.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@debugfs_test@basic-hwmon:
    - bat-adls-5:         NOTRUN -> [SKIP][2] ([i915#7456])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@debugfs_test@basic-hwmon.html

  * igt@fbdev@eof:
    - bat-adls-5:         NOTRUN -> [SKIP][3] ([i915#2582]) +4 similar issues
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@fbdev@eof.html

  * igt@gem_lmem_swapping@basic:
    - fi-apl-guc:         NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4613]) +3 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/fi-apl-guc/igt@gem_lmem_swapping@basic.html

  * igt@gem_lmem_swapping@random-engines:
    - bat-adls-5:         NOTRUN -> [SKIP][5] ([i915#4613]) +3 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@gem_lmem_swapping@random-engines.html

  * igt@gem_tiled_pread_basic:
    - bat-adls-5:         NOTRUN -> [SKIP][6] ([i915#3282])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@gem_tiled_pread_basic.html

  * igt@i915_pm_backlight@basic-brightness:
    - bat-adls-5:         NOTRUN -> [SKIP][7] ([i915#7561])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@i915_pm_backlight@basic-brightness.html

  * igt@i915_pm_backlight@basic-brightness@edp-1:
    - bat-rplp-1:         NOTRUN -> [ABORT][8] ([i915#7077])
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rplp-1/igt@i915_pm_backlight@basic-brightness@edp-1.html

  * igt@i915_selftest@live@hangcheck:
    - bat-adls-5:         NOTRUN -> [DMESG-WARN][9] ([i915#5591])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@i915_selftest@live@hangcheck.html

  * igt@i915_selftest@live@slpc:
    - bat-rpls-2:         NOTRUN -> [DMESG-WARN][10] ([i915#6367])
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rpls-2/igt@i915_selftest@live@slpc.html

  * igt@i915_suspend@basic-s2idle-without-i915:
    - bat-rpls-2:         NOTRUN -> [ABORT][11] ([i915#6687])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rpls-2/igt@i915_suspend@basic-s2idle-without-i915.html

  * igt@kms_chamelium_hpd@vga-hpd-fast:
    - bat-adls-5:         NOTRUN -> [SKIP][12] ([i915#7828]) +7 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_chamelium_hpd@vga-hpd-fast.html
    - fi-apl-guc:         NOTRUN -> [SKIP][13] ([fdo#109271]) +20 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/fi-apl-guc/igt@kms_chamelium_hpd@vga-hpd-fast.html

  * igt@kms_flip@basic-flip-vs-wf_vblank:
    - bat-adls-5:         NOTRUN -> [SKIP][14] ([i915#3637]) +3 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_flip@basic-flip-vs-wf_vblank.html

  * igt@kms_force_connector_basic@force-load-detect:
    - bat-adls-5:         NOTRUN -> [SKIP][15] ([fdo#109285])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_frontbuffer_tracking@basic:
    - bat-adls-5:         NOTRUN -> [SKIP][16] ([i915#1849])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_frontbuffer_tracking@basic.html

  * igt@kms_pipe_crc_basic@read-crc-frame-sequence:
    - bat-adls-5:         NOTRUN -> [SKIP][17] ([i915#1845]) +14 similar issues
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_pipe_crc_basic@read-crc-frame-sequence.html

  * igt@kms_psr@primary_mmap_gtt:
    - bat-rplp-1:         NOTRUN -> [SKIP][18] ([i915#1072])
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rplp-1/igt@kms_psr@primary_mmap_gtt.html

  * igt@kms_psr@sprite_plane_onoff:
    - bat-adls-5:         NOTRUN -> [SKIP][19] ([i915#1072]) +3 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_psr@sprite_plane_onoff.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - bat-rplp-1:         NOTRUN -> [SKIP][20] ([i915#3555] / [i915#4579])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rplp-1/igt@kms_setmode@basic-clone-single-crtc.html
    - bat-adls-5:         NOTRUN -> [SKIP][21] ([i915#3555] / [i915#4579])
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-apl-guc:         NOTRUN -> [SKIP][22] ([fdo#109271] / [i915#4579])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/fi-apl-guc/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-fence-flip:
    - bat-adls-5:         NOTRUN -> [SKIP][23] ([fdo#109295] / [i915#1845])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@prime_vgem@basic-fence-flip.html

  * igt@prime_vgem@basic-read:
    - bat-adls-5:         NOTRUN -> [SKIP][24] ([fdo#109295] / [i915#3291]) +2 similar issues
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@prime_vgem@basic-read.html

  
#### Possible fixes ####

  * igt@i915_module_load@load:
    - bat-adls-5:         [ABORT][25] -> [PASS][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13148/bat-adls-5/igt@i915_module_load@load.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-adls-5/igt@i915_module_load@load.html

  * igt@i915_selftest@live@reset:
    - bat-rpls-2:         [ABORT][27] ([i915#4983] / [i915#7461] / [i915#7913] / [i915#8347]) -> [PASS][28]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13148/bat-rpls-2/igt@i915_selftest@live@reset.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rpls-2/igt@i915_selftest@live@reset.html

  * igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-c-dp-1:
    - bat-dg2-8:          [FAIL][29] ([i915#7932]) -> [PASS][30] +1 similar issue
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13148/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-c-dp-1.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-dg2-8/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence@pipe-c-dp-1.html

  
#### Warnings ####

  * igt@kms_psr@sprite_plane_onoff:
    - bat-rplp-1:         [ABORT][31] ([i915#8442]) -> [SKIP][32] ([i915#1072])
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13148/bat-rplp-1/igt@kms_psr@sprite_plane_onoff.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/bat-rplp-1/igt@kms_psr@sprite_plane_onoff.html

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

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
  [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
  [i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591
  [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
  [i915#6687]: https://gitlab.freedesktop.org/drm/intel/issues/6687
  [i915#7077]: https://gitlab.freedesktop.org/drm/intel/issues/7077
  [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456
  [i915#7461]: https://gitlab.freedesktop.org/drm/intel/issues/7461
  [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561
  [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828
  [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913
  [i915#7920]: https://gitlab.freedesktop.org/drm/intel/issues/7920
  [i915#7932]: https://gitlab.freedesktop.org/drm/intel/issues/7932
  [i915#7953]: https://gitlab.freedesktop.org/drm/intel/issues/7953
  [i915#8347]: https://gitlab.freedesktop.org/drm/intel/issues/8347
  [i915#8442]: https://gitlab.freedesktop.org/drm/intel/issues/8442


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7290 -> IGTPW_8960

  CI-20190529: 20190529
  CI_DRM_13148: 57a535e042517014a85f33be6fa5ed22145c56e9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8960: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/index.html
  IGT_7290: 0261157319fe993ccefaf270b2fc7a8ebef418ae @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8960/index.html

[-- Attachment #2: Type: text/html, Size: 12238 bytes --]

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

* Re: [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48
  2023-05-15 14:39 [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48 Mauro Carvalho Chehab
  2023-05-15 16:18 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
  2023-05-15 16:29 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
@ 2023-05-15 19:15 ` Kamil Konieczny
  2023-05-16  7:01   ` Mauro Carvalho Chehab
  2 siblings, 1 reply; 5+ messages in thread
From: Kamil Konieczny @ 2023-05-15 19:15 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: igt-dev

Hi Mauro,

On 2023-05-15 at 16:39:23 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab@kernel.org>
> 
> changeset 8ac9440e7bec ("meson: add an option to control Xe test builds")
> require updating a dict with values from another dict. This seems
> to be supported only on version 0.48 or above.

I agree but this should wait until we update image fedora-oldest-meson
to have new meson 0.48 binary. Btw as you pointed out,
Debian buster LTS uses 0.49.2 so maybe we should use that ?

Btw maybe we should ask community if anyone is using this old
meson for building igt ? +cc Petri

> 
> So, update meson minimal version.
- ^^^
Please try to avoid starting sentence with "So" word, just:

Updated meson minimal version from 0.47.2 to 0.48

Regards,
Kamil

> 
> Fixes: 8ac9440e7bec ("meson: add an option to control Xe test builds")
> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
>  meson.build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meson.build b/meson.build
> index 1c872cc9cb02..a1f646ee9239 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -7,7 +7,7 @@ project('igt-gpu-tools', 'c',
>            'buildtype=debugoptimized',
>          ],
>  	license : 'MIT',
> -	meson_version : '>=0.47.2')
> +	meson_version : '>=0.48')
>  
>  if get_option('b_ndebug') != 'false'
>  	error('Building without -Db_ndebug=false is not supported')
> -- 
> 2.40.1
> 


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

* Re: [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48
  2023-05-15 19:15 ` [igt-dev] [PATCH i-g-t] " Kamil Konieczny
@ 2023-05-16  7:01   ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2023-05-16  7:01 UTC (permalink / raw)
  To: Kamil Konieczny; +Cc: igt-dev

On Mon, 15 May 2023 21:15:22 +0200
Kamil Konieczny <kamil.konieczny@linux.intel.com> wrote:

> Hi Mauro,
> 
> On 2023-05-15 at 16:39:23 +0200, Mauro Carvalho Chehab wrote:
> > From: Mauro Carvalho Chehab <mchehab@kernel.org>
> > 
> > changeset 8ac9440e7bec ("meson: add an option to control Xe test builds")
> > require updating a dict with values from another dict. This seems
> > to be supported only on version 0.48 or above.  
> 
> I agree but this should wait until we update image fedora-oldest-meson
> to have new meson 0.48 binary. 

I added such change at the pipeline script. Just submitted a v2:

	https://patchwork.freedesktop.org/patch/537229/?series=117774&rev=2

> Btw as you pointed out,
> Debian buster LTS uses 0.49.2 so maybe we should use that ?

I won't artificially increase the version. IMO, the best is to keep
the minimum version that it is technically possible there.

> Btw maybe we should ask community if anyone is using this old
> meson for building igt ? +cc Petri
> 
> > 
> > So, update meson minimal version.  
> - ^^^
> Please try to avoid starting sentence with "So" word, just:
> 
> Updated meson minimal version from 0.47.2 to 0.48

Ok.

Regards,
Mauro

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

end of thread, other threads:[~2023-05-16  7:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-15 14:39 [igt-dev] [PATCH i-g-t] meson.build: update minimal version to 0.48 Mauro Carvalho Chehab
2023-05-15 16:18 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2023-05-15 16:29 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2023-05-15 19:15 ` [igt-dev] [PATCH i-g-t] " Kamil Konieczny
2023-05-16  7:01   ` Mauro Carvalho Chehab

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