All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests
@ 2019-06-05 17:44 Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 2/4] lib/igt_core: 0 is a valid val for long options Lucas De Marchi
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-06-05 17:44 UTC (permalink / raw)
  To: igt-dev; +Cc: Lucas De Marchi, Petri Latvala

Start the core options from 500 so the individual tests can have their
own options starting from 0. This makes it easier to set the long
options without conflicting.

500 is just a magic number, higher than any ascii char that could be
used in the individual test.

While at it, fix the coding style to use tab rather than space.

v2: also fix the test for conflicting args

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
 lib/igt_core.c                   | 16 ++++++++++------
 lib/tests/igt_conflicting_args.c |  2 +-
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/lib/igt_core.c b/lib/igt_core.c
index 9c86d664..814f5c72 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -280,12 +280,16 @@ int test_children_sz;
 bool test_child;
 
 enum {
- OPT_LIST_SUBTESTS,
- OPT_RUN_SUBTEST,
- OPT_DESCRIPTION,
- OPT_DEBUG,
- OPT_INTERACTIVE_DEBUG,
- OPT_HELP = 'h'
+	/*
+	 * Let the first values be used by individual tests so options don't
+	 * conflict with core ones
+	 */
+	OPT_LIST_SUBTESTS = 500,
+	OPT_RUN_SUBTEST,
+	OPT_DESCRIPTION,
+	OPT_DEBUG,
+	OPT_INTERACTIVE_DEBUG,
+	OPT_HELP = 'h'
 };
 
 static int igt_exitcode = IGT_EXIT_SUCCESS;
diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c
index c357b6c5..f600abd4 100644
--- a/lib/tests/igt_conflicting_args.c
+++ b/lib/tests/igt_conflicting_args.c
@@ -91,7 +91,7 @@ int main(int argc, char **argv)
 	internal_assert_wsignaled(do_fork(), SIGABRT);
 
 	/* conflict on long option 'val' representations */
-	long_options[0] = (struct option) { "iterations", required_argument, NULL, 0};
+	long_options[0] = (struct option) { "iterations", required_argument, NULL, 500};
 	short_options = "";
 	internal_assert_wsignaled(do_fork(), SIGABRT);
 
-- 
2.21.0

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

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

* [igt-dev] [PATCH 2/4] lib/igt_core: 0 is a valid val for long options
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
@ 2019-06-05 17:44 ` Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 3/4] testdisplay: use first available option values Lucas De Marchi
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-06-05 17:44 UTC (permalink / raw)
  To: igt-dev; +Cc: Lucas De Marchi, Petri Latvala

This is usually used by long options when working with enum to set long
option values. So replace the strchr() with a memchr() to take that into
account.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
---
 lib/igt_core.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/igt_core.c b/lib/igt_core.c
index 814f5c72..a0b7e581 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -680,6 +680,7 @@ static int common_init(int *argc, char **argv,
 	};
 	char *short_opts;
 	const char *std_short_opts = "h";
+	size_t std_short_opts_len = strlen(std_short_opts);
 	struct option *combined_opts;
 	int extra_opt_count;
 	int all_opt_count;
@@ -713,7 +714,7 @@ static int common_init(int *argc, char **argv,
 
 		/* check for conflicts with standard short options */
 		if (extra_long_opts[extra_opt_count].val != ':'
-		    && (conflicting_char = strchr(std_short_opts, extra_long_opts[extra_opt_count].val))) {
+		    && (conflicting_char = memchr(std_short_opts, extra_long_opts[extra_opt_count].val, std_short_opts_len))) {
 			igt_critical("Conflicting long and short option 'val' representation between --%s and -%c\n",
 				     extra_long_opts[extra_opt_count].name,
 				     *conflicting_char);
@@ -727,7 +728,7 @@ static int common_init(int *argc, char **argv,
 			continue;
 
 		/* check for conflicts with standard short options */
-		if (strchr(std_short_opts, extra_short_opts[i])) {
+		if (memchr(std_short_opts, extra_short_opts[i], std_short_opts_len)) {
 			igt_critical("Conflicting short option: -%c\n", std_short_opts[i]);
 			assert(0);
 		}
-- 
2.21.0

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

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

* [igt-dev] [PATCH 3/4] testdisplay: use first available option values
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 2/4] lib/igt_core: 0 is a valid val for long options Lucas De Marchi
@ 2019-06-05 17:44 ` Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 4/4] lib/igt_core: add -h to usage Lucas De Marchi
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-06-05 17:44 UTC (permalink / raw)
  To: igt-dev; +Cc: Lucas De Marchi, Petri Latvala

Now that core options are set to 500 and above, start from the lowest
values without causing problems with conflicts. This also rename the
constants to follow the names from the core.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Acked-by: Antonio Argenziano <antonio.argenziano@intel.com>
---
 tests/testdisplay.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/tests/testdisplay.c b/tests/testdisplay.c
index b4f0d45f..32590547 100644
--- a/tests/testdisplay.c
+++ b/tests/testdisplay.c
@@ -69,8 +69,10 @@
 #include <stdlib.h>
 #include <signal.h>
 
-#define Yb_OPT		 5
-#define Yf_OPT		 6
+enum {
+	OPT_YB,
+	OPT_YF,
+};
 
 static int tio_fd;
 struct termios saved_tio;
@@ -573,8 +575,8 @@ static void set_termio_mode(void)
 
 static char optstr[] = "3iaf:s:d:p:mrto:j:y";
 static struct option long_opts[] = {
-	{"yb", 0, 0, Yb_OPT},
-	{"yf", 0, 0, Yf_OPT},
+	{"yb", 0, 0, OPT_YB},
+	{"yf", 0, 0, OPT_YF},
 	{ 0, 0, 0, 0 }
 };
 
@@ -648,10 +650,10 @@ static int opt_handler(int opt, int opt_index, void *data)
 		tiling = LOCAL_I915_FORMAT_MOD_X_TILED;
 		break;
 	case 'y':
-	case Yb_OPT:
+	case OPT_YB:
 		tiling = LOCAL_I915_FORMAT_MOD_Y_TILED;
 		break;
-	case Yf_OPT:
+	case OPT_YF:
 		tiling = LOCAL_I915_FORMAT_MOD_Yf_TILED;
 		break;
 	case 'r':
-- 
2.21.0

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

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

* [igt-dev] [PATCH 4/4] lib/igt_core: add -h to usage
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 2/4] lib/igt_core: 0 is a valid val for long options Lucas De Marchi
  2019-06-05 17:44 ` [igt-dev] [PATCH 3/4] testdisplay: use first available option values Lucas De Marchi
@ 2019-06-05 17:44 ` Lucas De Marchi
  2019-06-05 18:24 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [1/4] lib/igt_core: reserve long options for individual tests Patchwork
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-06-05 17:44 UTC (permalink / raw)
  To: igt-dev; +Cc: Lucas De Marchi, Petri Latvala

We also accept the short option -h.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
---
 lib/igt_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/igt_core.c b/lib/igt_core.c
index a0b7e581..6b9f0425 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -558,7 +558,7 @@ static void print_usage(const char *help_str, bool output_on_stderr)
 		   "  --debug[=log-domain]\n"
 		   "  --interactive-debug[=domain]\n"
 		   "  --help-description\n"
-		   "  --help\n");
+		   "  --help|-h\n");
 	if (help_str)
 		fprintf(f, "%s\n", help_str);
 }
-- 
2.21.0

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

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

* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [1/4] lib/igt_core: reserve long options for individual tests
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
                   ` (2 preceding siblings ...)
  2019-06-05 17:44 ` [igt-dev] [PATCH 4/4] lib/igt_core: add -h to usage Lucas De Marchi
@ 2019-06-05 18:24 ` Patchwork
  2019-06-06  9:20 ` [igt-dev] [PATCH 1/4] " Petri Latvala
  2019-06-07 11:53 ` [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [1/4] " Patchwork
  5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-06-05 18:24 UTC (permalink / raw)
  To: Lucas De Marchi; +Cc: igt-dev

== Series Details ==

Series: series starting with [1/4] lib/igt_core: reserve long options for individual tests
URL   : https://patchwork.freedesktop.org/series/61677/
State : success

== Summary ==

CI Bug Log - changes from IGT_5041 -> IGTPW_3123
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_mmap_gtt@basic-write-gtt-no-prefault:
    - fi-icl-dsi:         [PASS][1] -> [INCOMPLETE][2] ([fdo#107713])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/fi-icl-dsi/igt@gem_mmap_gtt@basic-write-gtt-no-prefault.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/fi-icl-dsi/igt@gem_mmap_gtt@basic-write-gtt-no-prefault.html

  
#### Possible fixes ####

  * igt@gem_ctx_create@basic-files:
    - {fi-icl-guc}:       [INCOMPLETE][3] ([fdo#107713] / [fdo#109100]) -> [PASS][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/fi-icl-guc/igt@gem_ctx_create@basic-files.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/fi-icl-guc/igt@gem_ctx_create@basic-files.html

  * igt@gem_ringfill@basic-default-forked:
    - fi-icl-u3:          [DMESG-WARN][5] ([fdo#107724]) -> [PASS][6] +1 similar issue
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/fi-icl-u3/igt@gem_ringfill@basic-default-forked.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/fi-icl-u3/igt@gem_ringfill@basic-default-forked.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-icl-u2:          [FAIL][7] ([fdo#103167]) -> [PASS][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html

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

  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100


Participating hosts (53 -> 46)
------------------------------

  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-kbl-7560u fi-byt-clapper fi-bdw-samus 


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

  * IGT: IGT_5041 -> IGTPW_3123

  CI_DRM_6196: 7a984cf09665b9ef4c63e82a8551bdde5da229ae @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3123: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/
  IGT_5041: 4f2b9f5930fa33d091cf89637dc6e7f76f632a88 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

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

* Re: [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
                   ` (3 preceding siblings ...)
  2019-06-05 18:24 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [1/4] lib/igt_core: reserve long options for individual tests Patchwork
@ 2019-06-06  9:20 ` Petri Latvala
  2019-06-06 16:50   ` Lucas De Marchi
  2019-06-07 11:53 ` [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [1/4] " Patchwork
  5 siblings, 1 reply; 8+ messages in thread
From: Petri Latvala @ 2019-06-06  9:20 UTC (permalink / raw)
  To: Lucas De Marchi; +Cc: igt-dev

On Wed, Jun 05, 2019 at 10:44:47AM -0700, Lucas De Marchi wrote:
> Start the core options from 500 so the individual tests can have their
> own options starting from 0. This makes it easier to set the long
> options without conflicting.
> 
> 500 is just a magic number, higher than any ascii char that could be
> used in the individual test.
> 
> While at it, fix the coding style to use tab rather than space.
> 
> v2: also fix the test for conflicting args
> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>

Reviewed-by: Petri Latvala <petri.latvala@intel.com>

> ---
>  lib/igt_core.c                   | 16 ++++++++++------
>  lib/tests/igt_conflicting_args.c |  2 +-
>  2 files changed, 11 insertions(+), 7 deletions(-)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c
> index 9c86d664..814f5c72 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -280,12 +280,16 @@ int test_children_sz;
>  bool test_child;
>  
>  enum {
> - OPT_LIST_SUBTESTS,
> - OPT_RUN_SUBTEST,
> - OPT_DESCRIPTION,
> - OPT_DEBUG,
> - OPT_INTERACTIVE_DEBUG,
> - OPT_HELP = 'h'
> +	/*
> +	 * Let the first values be used by individual tests so options don't
> +	 * conflict with core ones
> +	 */
> +	OPT_LIST_SUBTESTS = 500,
> +	OPT_RUN_SUBTEST,
> +	OPT_DESCRIPTION,
> +	OPT_DEBUG,
> +	OPT_INTERACTIVE_DEBUG,
> +	OPT_HELP = 'h'
>  };
>  
>  static int igt_exitcode = IGT_EXIT_SUCCESS;
> diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c
> index c357b6c5..f600abd4 100644
> --- a/lib/tests/igt_conflicting_args.c
> +++ b/lib/tests/igt_conflicting_args.c
> @@ -91,7 +91,7 @@ int main(int argc, char **argv)
>  	internal_assert_wsignaled(do_fork(), SIGABRT);
>  
>  	/* conflict on long option 'val' representations */
> -	long_options[0] = (struct option) { "iterations", required_argument, NULL, 0};
> +	long_options[0] = (struct option) { "iterations", required_argument, NULL, 500};
>  	short_options = "";
>  	internal_assert_wsignaled(do_fork(), SIGABRT);
>  
> -- 
> 2.21.0
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* Re: [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests
  2019-06-06  9:20 ` [igt-dev] [PATCH 1/4] " Petri Latvala
@ 2019-06-06 16:50   ` Lucas De Marchi
  0 siblings, 0 replies; 8+ messages in thread
From: Lucas De Marchi @ 2019-06-06 16:50 UTC (permalink / raw)
  To: igt-dev

On Thu, Jun 06, 2019 at 12:20:06PM +0300, Petri Latvala wrote:
>On Wed, Jun 05, 2019 at 10:44:47AM -0700, Lucas De Marchi wrote:
>> Start the core options from 500 so the individual tests can have their
>> own options starting from 0. This makes it easier to set the long
>> options without conflicting.
>>
>> 500 is just a magic number, higher than any ascii char that could be
>> used in the individual test.
>>
>> While at it, fix the coding style to use tab rather than space.
>>
>> v2: also fix the test for conflicting args
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
>
>Reviewed-by: Petri Latvala <petri.latvala@intel.com>

series applied. Thanks for the reviews.

Lucas De Marchi

>
>> ---
>>  lib/igt_core.c                   | 16 ++++++++++------
>>  lib/tests/igt_conflicting_args.c |  2 +-
>>  2 files changed, 11 insertions(+), 7 deletions(-)
>>
>> diff --git a/lib/igt_core.c b/lib/igt_core.c
>> index 9c86d664..814f5c72 100644
>> --- a/lib/igt_core.c
>> +++ b/lib/igt_core.c
>> @@ -280,12 +280,16 @@ int test_children_sz;
>>  bool test_child;
>>
>>  enum {
>> - OPT_LIST_SUBTESTS,
>> - OPT_RUN_SUBTEST,
>> - OPT_DESCRIPTION,
>> - OPT_DEBUG,
>> - OPT_INTERACTIVE_DEBUG,
>> - OPT_HELP = 'h'
>> +	/*
>> +	 * Let the first values be used by individual tests so options don't
>> +	 * conflict with core ones
>> +	 */
>> +	OPT_LIST_SUBTESTS = 500,
>> +	OPT_RUN_SUBTEST,
>> +	OPT_DESCRIPTION,
>> +	OPT_DEBUG,
>> +	OPT_INTERACTIVE_DEBUG,
>> +	OPT_HELP = 'h'
>>  };
>>
>>  static int igt_exitcode = IGT_EXIT_SUCCESS;
>> diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c
>> index c357b6c5..f600abd4 100644
>> --- a/lib/tests/igt_conflicting_args.c
>> +++ b/lib/tests/igt_conflicting_args.c
>> @@ -91,7 +91,7 @@ int main(int argc, char **argv)
>>  	internal_assert_wsignaled(do_fork(), SIGABRT);
>>
>>  	/* conflict on long option 'val' representations */
>> -	long_options[0] = (struct option) { "iterations", required_argument, NULL, 0};
>> +	long_options[0] = (struct option) { "iterations", required_argument, NULL, 500};
>>  	short_options = "";
>>  	internal_assert_wsignaled(do_fork(), SIGABRT);
>>
>> --
>> 2.21.0
>>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [1/4] lib/igt_core: reserve long options for individual tests
  2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
                   ` (4 preceding siblings ...)
  2019-06-06  9:20 ` [igt-dev] [PATCH 1/4] " Petri Latvala
@ 2019-06-07 11:53 ` Patchwork
  5 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-06-07 11:53 UTC (permalink / raw)
  To: Lucas De Marchi; +Cc: igt-dev

== Series Details ==

Series: series starting with [1/4] lib/igt_core: reserve long options for individual tests
URL   : https://patchwork.freedesktop.org/series/61677/
State : failure

== Summary ==

CI Bug Log - changes from IGT_5041_full -> IGTPW_3123_full
====================================================

Summary
-------

  **FAILURE**

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

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@kms_big_fb@yf-tiled-8bpp-rotate-90:
    - shard-iclb:         NOTRUN -> [SKIP][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb6/igt@kms_big_fb@yf-tiled-8bpp-rotate-90.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_mmap_gtt@forked-medium-copy-odd:
    - shard-iclb:         [PASS][2] -> [INCOMPLETE][3] ([fdo#107713])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb7/igt@gem_mmap_gtt@forked-medium-copy-odd.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb5/igt@gem_mmap_gtt@forked-medium-copy-odd.html

  * igt@gem_tiled_swapping@non-threaded:
    - shard-hsw:          [PASS][4] -> [INCOMPLETE][5] ([fdo#103540] / [fdo#108686])
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw1/igt@gem_tiled_swapping@non-threaded.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw7/igt@gem_tiled_swapping@non-threaded.html

  * igt@kms_flip@flip-vs-suspend:
    - shard-hsw:          [PASS][6] -> [INCOMPLETE][7] ([fdo#103540])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw1/igt@kms_flip@flip-vs-suspend.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw2/igt@kms_flip@flip-vs-suspend.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-msflip-blt:
    - shard-hsw:          [PASS][8] -> [SKIP][9] ([fdo#109271]) +11 similar issues
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw7/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-msflip-blt.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw1/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-indfb-msflip-blt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite:
    - shard-iclb:         [PASS][10] -> [FAIL][11] ([fdo#103167]) +2 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-pwrite.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
    - shard-apl:          [PASS][12] -> [DMESG-WARN][13] ([fdo#108566]) +4 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-apl6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-apl3/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html

  * igt@kms_psr@psr2_sprite_plane_onoff:
    - shard-iclb:         [PASS][14] -> [SKIP][15] ([fdo#109441]) +1 similar issue
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb2/igt@kms_psr@psr2_sprite_plane_onoff.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb1/igt@kms_psr@psr2_sprite_plane_onoff.html

  
#### Possible fixes ####

  * igt@gem_ctx_isolation@vecs0-s3:
    - shard-kbl:          [DMESG-WARN][16] ([fdo#103313]) -> [PASS][17]
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-kbl3/igt@gem_ctx_isolation@vecs0-s3.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-kbl3/igt@gem_ctx_isolation@vecs0-s3.html

  * {igt@gem_exec_balancer@bonded-imm}:
    - shard-iclb:         [FAIL][18] -> [PASS][19]
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb2/igt@gem_exec_balancer@bonded-imm.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb6/igt@gem_exec_balancer@bonded-imm.html

  * {igt@gem_exec_balancer@smoke}:
    - shard-iclb:         [SKIP][20] -> [PASS][21]
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb8/igt@gem_exec_balancer@smoke.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb1/igt@gem_exec_balancer@smoke.html

  * igt@gem_mmap_gtt@forked-medium-copy-xy:
    - shard-iclb:         [TIMEOUT][22] ([fdo#109673]) -> [PASS][23]
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb8/igt@gem_mmap_gtt@forked-medium-copy-xy.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb4/igt@gem_mmap_gtt@forked-medium-copy-xy.html

  * igt@i915_suspend@debugfs-reader:
    - shard-apl:          [DMESG-WARN][24] ([fdo#108566]) -> [PASS][25] +5 similar issues
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-apl7/igt@i915_suspend@debugfs-reader.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-apl1/igt@i915_suspend@debugfs-reader.html

  * igt@kms_big_fb@linear-8bpp-rotate-180:
    - shard-hsw:          [INCOMPLETE][26] ([fdo#103540]) -> [PASS][27]
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw1/igt@kms_big_fb@linear-8bpp-rotate-180.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw6/igt@kms_big_fb@linear-8bpp-rotate-180.html

  * igt@kms_cursor_crc@pipe-b-cursor-256x256-onscreen:
    - shard-snb:          [SKIP][28] ([fdo#109271]) -> [PASS][29]
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-snb6/igt@kms_cursor_crc@pipe-b-cursor-256x256-onscreen.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-snb1/igt@kms_cursor_crc@pipe-b-cursor-256x256-onscreen.html

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic:
    - shard-hsw:          [SKIP][30] ([fdo#109271]) -> [PASS][31] +67 similar issues
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw1/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw4/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-atomic.html

  * igt@kms_frontbuffer_tracking@fbc-stridechange:
    - shard-iclb:         [FAIL][32] ([fdo#103167]) -> [PASS][33] +2 similar issues
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-stridechange.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-stridechange.html

  * igt@kms_psr@psr2_sprite_mmap_cpu:
    - shard-iclb:         [SKIP][34] ([fdo#109441]) -> [PASS][35] +2 similar issues
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb3/igt@kms_psr@psr2_sprite_mmap_cpu.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_cpu.html

  * igt@kms_sysfs_edid_timing:
    - shard-hsw:          [FAIL][36] ([fdo#100047]) -> [PASS][37]
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-hsw1/igt@kms_sysfs_edid_timing.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-hsw5/igt@kms_sysfs_edid_timing.html

  
#### Warnings ####

  * igt@gem_mmap_gtt@forked-big-copy-xy:
    - shard-iclb:         [INCOMPLETE][38] ([fdo#107713] / [fdo#109100]) -> [TIMEOUT][39] ([fdo#109673])
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-iclb2/igt@gem_mmap_gtt@forked-big-copy-xy.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-iclb3/igt@gem_mmap_gtt@forked-big-copy-xy.html

  * igt@gem_tiled_swapping@non-threaded:
    - shard-glk:          [DMESG-WARN][40] ([fdo#108686]) -> [FAIL][41] ([fdo#108686])
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5041/shard-glk3/igt@gem_tiled_swapping@non-threaded.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/shard-glk8/igt@gem_tiled_swapping@non-threaded.html

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

  [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313
  [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
  [fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
  [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#109673]: https://bugs.freedesktop.org/show_bug.cgi?id=109673


Participating hosts (7 -> 6)
------------------------------

  Missing    (1): shard-skl 


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

  * IGT: IGT_5041 -> IGTPW_3123

  CI_DRM_6196: 7a984cf09665b9ef4c63e82a8551bdde5da229ae @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3123: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3123/
  IGT_5041: 4f2b9f5930fa33d091cf89637dc6e7f76f632a88 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

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

end of thread, other threads:[~2019-06-07 11:53 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-05 17:44 [igt-dev] [PATCH 1/4] lib/igt_core: reserve long options for individual tests Lucas De Marchi
2019-06-05 17:44 ` [igt-dev] [PATCH 2/4] lib/igt_core: 0 is a valid val for long options Lucas De Marchi
2019-06-05 17:44 ` [igt-dev] [PATCH 3/4] testdisplay: use first available option values Lucas De Marchi
2019-06-05 17:44 ` [igt-dev] [PATCH 4/4] lib/igt_core: add -h to usage Lucas De Marchi
2019-06-05 18:24 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [1/4] lib/igt_core: reserve long options for individual tests Patchwork
2019-06-06  9:20 ` [igt-dev] [PATCH 1/4] " Petri Latvala
2019-06-06 16:50   ` Lucas De Marchi
2019-06-07 11:53 ` [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [1/4] " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.