All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium.
@ 2020-03-05  3:33 Kunal Joshi
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 1/2] lib/igt_fb: Added macro to retrieve char code of format Kunal Joshi
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Kunal Joshi @ 2020-03-05  3:33 UTC (permalink / raw)
  To: igt-dev; +Cc: Kunal Joshi

Added capability in kms_chamelium to test both rgb and yuv
pixel formats supported per platform through chamelium.
Earlier, only hard coded pixel formats used to be tested which were
not generic across platforms resulting is SKIPS; to overcome these SKIPS
now only platform supported pixel formats will be tested.

Kunal Joshi (2):
  lib/igt_fb: Added macro to retrieve char code of format
  tests/kms_chamelium: Add support to validate RGB/YUV pixel formats

 lib/igt_fb.h          |   2 +
 tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
 2 files changed, 56 insertions(+), 86 deletions(-)

-- 
2.7.4

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

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

* [igt-dev] [PATCH i-g-t v2 1/2] lib/igt_fb: Added macro to retrieve char code of format
  2020-03-05  3:33 [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium Kunal Joshi
@ 2020-03-05  3:33 ` Kunal Joshi
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats Kunal Joshi
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 9+ messages in thread
From: Kunal Joshi @ 2020-03-05  3:33 UTC (permalink / raw)
  To: igt-dev; +Cc: Kunal Joshi

Added a macro to retrieve the fourcc of the pixel format.

Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
---
 lib/igt_fb.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/igt_fb.h b/lib/igt_fb.h
index 587f7a4..0383d83 100644
--- a/lib/igt_fb.h
+++ b/lib/igt_fb.h
@@ -48,6 +48,8 @@
 #define IGT_FORMAT_FMT "%c%c%c%c(0x%08x)"
 #define IGT_FORMAT_ARGS(f) ((f) >> 0) & 0xff, ((f) >> 8) & 0xff, \
 		((f) >> 16) & 0xff, ((f) >> 24) & 0xff, (f)
+#define IGT_FORMAT_ARGS_CODE(f) ((f) >> 0) & 0xff, ((f) >> 8) & 0xff, \
+		((f) >> 16) & 0xff, ((f) >> 24) & 0xff
 
 /**
  * igt_fb_t:
-- 
2.7.4

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

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

* [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats
  2020-03-05  3:33 [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium Kunal Joshi
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 1/2] lib/igt_fb: Added macro to retrieve char code of format Kunal Joshi
@ 2020-03-05  3:33 ` Kunal Joshi
  2020-03-05 13:01   ` Ville Syrjälä
  2020-03-05 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success for Test platform supported pixel formats through chamelium. (rev2) Patchwork
  2020-03-06  0:01 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  3 siblings, 1 reply; 9+ messages in thread
From: Kunal Joshi @ 2020-03-05  3:33 UTC (permalink / raw)
  To: igt-dev; +Cc: Kunal Joshi

Currently, only hard coded pixel formats are validated which are not
generic for all the platforms which leads to SKIPS.
Added support to validate only those pixel formats which are supported by
platform. Both RGB and YUV pixel formats are covered.

Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
---
 tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
 1 file changed, 54 insertions(+), 86 deletions(-)

diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
index 5c4a189..5c96017 100644
--- a/tests/kms_chamelium.c
+++ b/tests/kms_chamelium.c
@@ -2612,6 +2612,14 @@ static const struct edid *get_edid(enum test_edid edid)
 			if (chamelium_port_get_type(port) == \
 			    DRM_MODE_CONNECTOR_ ## type__)
 
+#define connector_subtest_start(name__, type__)              \
+        igt_subtest_with_dynamic(name__)                     \
+                for_each_port(p, port)                       \
+                        if (chamelium_port_get_type(port) == \
+                            DRM_MODE_CONNECTOR_ ## type__) {
+
+#define connector_subtest_end }
+
 static data_t data;
 
 IGT_TEST_DESCRIPTION("Tests requiring a Chamelium board");
@@ -2817,54 +2825,29 @@ igt_main
 					       CHAMELIUM_CHECK_CRC, 3);
 
 		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-argb8888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_ARGB8888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-abgr8888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_ABGR8888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-xrgb8888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_XRGB8888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-xbgr8888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_XBGR8888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-rgb888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_RGB888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-bgr888", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_BGR888,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-rgb565", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_RGB565,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-bgr565", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_BGR565,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-argb1555", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_ARGB1555,
-					      CHAMELIUM_CHECK_CRC, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-crc-xrgb1555", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_XRGB1555,
-					      CHAMELIUM_CHECK_CRC, 1);
+		connector_subtest_start("hdmi-rgb-pixel-format", HDMIA)
+			int k;
+			igt_output_t *output;
+			igt_plane_t *primary;
+
+			output = prepare_output(&data, port, TEST_EDID_BASE);
+			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
+			igt_assert(primary);
+
+			for (k = 0; k < primary->format_mod_count; k++) {
+				if (!igt_fb_supported_format(primary->formats[k]))
+					continue;
+
+				if ((igt_format_is_yuv(primary->formats[k]) ||
+				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
+					continue;
+
+				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
+					test_display_one_mode(&data, port, primary->formats[k],
+							      CHAMELIUM_CHECK_CRC,
+							      1);
+			}
+		connector_subtest_end
 
 		igt_describe(test_display_planes_random_desc);
 		connector_subtest("hdmi-crc-planes-random", HDMIA)
@@ -2872,44 +2855,29 @@ igt_main
 						   CHAMELIUM_CHECK_CRC);
 
 		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-nv12", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_NV12,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-nv16", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_NV16,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-nv21", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_NV21,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-nv61", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_NV61,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-yu12", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_YUV420,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-yu16", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_YUV422,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-yv12", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_YVU420,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
-
-		igt_describe(test_display_one_mode_desc);
-		connector_subtest("hdmi-cmp-yv16", HDMIA)
-			test_display_one_mode(&data, port, DRM_FORMAT_YVU422,
-					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
+		connector_subtest_start("hdmi-yuv-pixel-format", HDMIA)
+			int k;
+			igt_output_t *output;
+			igt_plane_t *primary;
+
+			output = prepare_output(&data, port, TEST_EDID_BASE);
+			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
+			igt_assert(primary);
+
+			for (k = 0; k < primary->format_mod_count; k++) {
+				if (!igt_fb_supported_format(primary->formats[k]))
+					continue;
+
+				if (!(igt_format_is_yuv(primary->formats[k]) &&
+				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
+					continue;
+
+				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
+					test_display_one_mode(&data, port, primary->formats[k],
+							      CHAMELIUM_CHECK_CHECKERBOARD,
+							      1);
+			}
+		connector_subtest_end
 
 		igt_describe(test_display_planes_random_desc);
 		connector_subtest("hdmi-cmp-planes-random", HDMIA)
-- 
2.7.4

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

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

* Re: [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats
  2020-03-05 13:01   ` Ville Syrjälä
@ 2020-03-05  8:42     ` Kunal Joshi
  2020-03-05 16:50       ` Ville Syrjälä
  0 siblings, 1 reply; 9+ messages in thread
From: Kunal Joshi @ 2020-03-05  8:42 UTC (permalink / raw)
  To: Ville Syrjälä, igt-dev

On 2020-03-05 at 15:01:15 +0200, Ville Syrjälä wrote:
> On Thu, Mar 05, 2020 at 09:03:38AM +0530, Kunal Joshi wrote:
> > Currently, only hard coded pixel formats are validated which are not
> > generic for all the platforms which leads to SKIPS.
> > Added support to validate only those pixel formats which are supported by
> > platform. Both RGB and YUV pixel formats are covered.
> > 
> > Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> > ---
> >  tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
> >  1 file changed, 54 insertions(+), 86 deletions(-)
> > 
> > diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> > index 5c4a189..5c96017 100644
> > --- a/tests/kms_chamelium.c
> > +++ b/tests/kms_chamelium.c
> > @@ -2612,6 +2612,14 @@ static const struct edid *get_edid(enum test_edid edid)
> >  			if (chamelium_port_get_type(port) == \
> >  			    DRM_MODE_CONNECTOR_ ## type__)
> >  
> > +#define connector_subtest_start(name__, type__)              \
> > +        igt_subtest_with_dynamic(name__)                     \
> > +                for_each_port(p, port)                       \
> > +                        if (chamelium_port_get_type(port) == \
> > +                            DRM_MODE_CONNECTOR_ ## type__) {
> > +
> > +#define connector_subtest_end }
> > +
> >  static data_t data;
> >  
> >  IGT_TEST_DESCRIPTION("Tests requiring a Chamelium board");
> > @@ -2817,54 +2825,29 @@ igt_main
> >  					       CHAMELIUM_CHECK_CRC, 3);
> >  
> >  		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-argb8888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB8888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-abgr8888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_ABGR8888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-xrgb8888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB8888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-xbgr8888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_XBGR8888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-rgb888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-bgr888", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR888,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-rgb565", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB565,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-bgr565", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR565,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-argb1555", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB1555,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-crc-xrgb1555", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB1555,
> > -					      CHAMELIUM_CHECK_CRC, 1);
> > +		connector_subtest_start("hdmi-rgb-pixel-format", HDMIA)
> > +			int k;
> > +			igt_output_t *output;
> > +			igt_plane_t *primary;
> > +
> > +			output = prepare_output(&data, port, TEST_EDID_BASE);
> > +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> > +			igt_assert(primary);
> > +
> > +			for (k = 0; k < primary->format_mod_count; k++) {
> > +				if (!igt_fb_supported_format(primary->formats[k]))
> > +					continue;
> > +
> > +				if ((igt_format_is_yuv(primary->formats[k]) ||
> > +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> > +					continue;
> > +
> > +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> 
> We have a igt_format_str() which is probably better for stuff like this.
> 

Thanks ville for pointing that out
I tried using igt_format_str()
but it doesn't seem to support that i tried it like below

igt_dynamic_f("%s",igt_format_str("testing-format-%c%c%c%c",
				  IGT_FORMAT_ARGS_CODE(primary->formats[k]))

Can i use igt_dynamic_f?
because it also accepts a printf format string
https://drm.pages.freedesktop.org/igt-gpu-tools/igt-gpu-tools-Core.html#igt-subtest-with-dynamic
> > +					test_display_one_mode(&data, port, primary->formats[k],
> > +							      CHAMELIUM_CHECK_CRC,
> > +							      1);
> > +			}
> > +		connector_subtest_end
> >  
> >  		igt_describe(test_display_planes_random_desc);
> >  		connector_subtest("hdmi-crc-planes-random", HDMIA)
> > @@ -2872,44 +2855,29 @@ igt_main
> >  						   CHAMELIUM_CHECK_CRC);
> >  
> >  		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-nv12", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_NV12,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-nv16", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_NV16,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-nv21", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_NV21,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-nv61", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_NV61,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-yu12", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_YUV420,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-yu16", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_YUV422,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-yv12", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_YVU420,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > -
> > -		igt_describe(test_display_one_mode_desc);
> > -		connector_subtest("hdmi-cmp-yv16", HDMIA)
> > -			test_display_one_mode(&data, port, DRM_FORMAT_YVU422,
> > -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> > +		connector_subtest_start("hdmi-yuv-pixel-format", HDMIA)
> > +			int k;
> > +			igt_output_t *output;
> > +			igt_plane_t *primary;
> > +
> > +			output = prepare_output(&data, port, TEST_EDID_BASE);
> > +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> > +			igt_assert(primary);
> > +
> > +			for (k = 0; k < primary->format_mod_count; k++) {
> > +				if (!igt_fb_supported_format(primary->formats[k]))
> > +					continue;
> > +
> > +				if (!(igt_format_is_yuv(primary->formats[k]) &&
> > +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> > +					continue;
> > +
> > +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> > +					test_display_one_mode(&data, port, primary->formats[k],
> > +							      CHAMELIUM_CHECK_CHECKERBOARD,
> > +							      1);
> > +			}
> > +		connector_subtest_end
> >  
> >  		igt_describe(test_display_planes_random_desc);
> >  		connector_subtest("hdmi-cmp-planes-random", HDMIA)
> > -- 
> > 2.7.4
> > 
> > _______________________________________________
> > igt-dev mailing list
> > igt-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> 
> -- 
> Ville Syrjälä
> Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* [igt-dev] ✓ Fi.CI.BAT: success for Test platform supported pixel formats through chamelium. (rev2)
  2020-03-05  3:33 [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium Kunal Joshi
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 1/2] lib/igt_fb: Added macro to retrieve char code of format Kunal Joshi
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats Kunal Joshi
@ 2020-03-05 11:44 ` Patchwork
  2020-03-06  0:01 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  3 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2020-03-05 11:44 UTC (permalink / raw)
  To: Kunal Joshi; +Cc: igt-dev

== Series Details ==

Series: Test platform supported pixel formats through chamelium. (rev2)
URL   : https://patchwork.freedesktop.org/series/74317/
State : success

== Summary ==

CI Bug Log - changes from IGT_5493 -> IGTPW_4262
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_suspend@basic-s4-devices:
    - fi-tgl-y:           [PASS][1] -> [FAIL][2] ([CI#94])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-tgl-y/igt@gem_exec_suspend@basic-s4-devices.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-tgl-y/igt@gem_exec_suspend@basic-s4-devices.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       [PASS][3] -> [FAIL][4] ([fdo#111407])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@vgem_basic@dmabuf-fence:
    - fi-tgl-y:           [PASS][5] -> [DMESG-WARN][6] ([CI#94] / [i915#402]) +1 similar issue
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-tgl-y/igt@vgem_basic@dmabuf-fence.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-tgl-y/igt@vgem_basic@dmabuf-fence.html

  
#### Possible fixes ####

  * igt@i915_selftest@live@active:
    - fi-icl-y:           [DMESG-FAIL][7] ([i915#765]) -> [PASS][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-icl-y/igt@i915_selftest@live@active.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-icl-y/igt@i915_selftest@live@active.html

  * igt@i915_selftest@live@gem_contexts:
    - fi-cfl-8700k:       [INCOMPLETE][9] ([i915#424]) -> [PASS][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-cfl-8700k/igt@i915_selftest@live@gem_contexts.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-cfl-8700k/igt@i915_selftest@live@gem_contexts.html

  * igt@prime_vgem@basic-gtt:
    - fi-tgl-y:           [DMESG-WARN][11] ([CI#94] / [i915#402]) -> [PASS][12] +2 similar issues
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/fi-tgl-y/igt@prime_vgem@basic-gtt.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/fi-tgl-y/igt@prime_vgem@basic-gtt.html

  
  [CI#94]: https://gitlab.freedesktop.org/gfx-ci/i915-infra/issues/94
  [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407
  [i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
  [i915#424]: https://gitlab.freedesktop.org/drm/intel/issues/424
  [i915#765]: https://gitlab.freedesktop.org/drm/intel/issues/765


Participating hosts (44 -> 43)
------------------------------

  Additional (7): fi-kbl-soraka fi-bdw-5557u fi-cfl-8109u fi-bsw-kefka fi-skl-lmem fi-bsw-nick fi-skl-6600u 
  Missing    (8): fi-ilk-m540 fi-hsw-4200u fi-byt-j1900 fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-byt-clapper fi-bdw-samus 


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

  * CI: CI-20190529 -> None
  * IGT: IGT_5493 -> IGTPW_4262

  CI-20190529: 20190529
  CI_DRM_8068: f8e69af5cca45947ebce78f677b75b0ecc4ba744 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_4262: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/index.html
  IGT_5493: 41422f8384047e2ec4c4cb896f6955bf51490f53 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools



== Testlist changes ==

+igt@kms_chamelium@hdmi-rgb-pixel-format
+igt@kms_chamelium@hdmi-yuv-pixel-format
-igt@kms_chamelium@hdmi-cmp-nv12
-igt@kms_chamelium@hdmi-cmp-nv16
-igt@kms_chamelium@hdmi-cmp-nv21
-igt@kms_chamelium@hdmi-cmp-nv61
-igt@kms_chamelium@hdmi-cmp-yu12
-igt@kms_chamelium@hdmi-cmp-yu16
-igt@kms_chamelium@hdmi-cmp-yv12
-igt@kms_chamelium@hdmi-cmp-yv16
-igt@kms_chamelium@hdmi-crc-abgr8888
-igt@kms_chamelium@hdmi-crc-argb1555
-igt@kms_chamelium@hdmi-crc-argb8888
-igt@kms_chamelium@hdmi-crc-bgr565
-igt@kms_chamelium@hdmi-crc-bgr888
-igt@kms_chamelium@hdmi-crc-rgb565
-igt@kms_chamelium@hdmi-crc-rgb888
-igt@kms_chamelium@hdmi-crc-xbgr8888
-igt@kms_chamelium@hdmi-crc-xrgb1555
-igt@kms_chamelium@hdmi-crc-xrgb8888

== Logs ==

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

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

* Re: [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats
  2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats Kunal Joshi
@ 2020-03-05 13:01   ` Ville Syrjälä
  2020-03-05  8:42     ` Kunal Joshi
  0 siblings, 1 reply; 9+ messages in thread
From: Ville Syrjälä @ 2020-03-05 13:01 UTC (permalink / raw)
  To: Kunal Joshi; +Cc: igt-dev

On Thu, Mar 05, 2020 at 09:03:38AM +0530, Kunal Joshi wrote:
> Currently, only hard coded pixel formats are validated which are not
> generic for all the platforms which leads to SKIPS.
> Added support to validate only those pixel formats which are supported by
> platform. Both RGB and YUV pixel formats are covered.
> 
> Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> ---
>  tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
>  1 file changed, 54 insertions(+), 86 deletions(-)
> 
> diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> index 5c4a189..5c96017 100644
> --- a/tests/kms_chamelium.c
> +++ b/tests/kms_chamelium.c
> @@ -2612,6 +2612,14 @@ static const struct edid *get_edid(enum test_edid edid)
>  			if (chamelium_port_get_type(port) == \
>  			    DRM_MODE_CONNECTOR_ ## type__)
>  
> +#define connector_subtest_start(name__, type__)              \
> +        igt_subtest_with_dynamic(name__)                     \
> +                for_each_port(p, port)                       \
> +                        if (chamelium_port_get_type(port) == \
> +                            DRM_MODE_CONNECTOR_ ## type__) {
> +
> +#define connector_subtest_end }
> +
>  static data_t data;
>  
>  IGT_TEST_DESCRIPTION("Tests requiring a Chamelium board");
> @@ -2817,54 +2825,29 @@ igt_main
>  					       CHAMELIUM_CHECK_CRC, 3);
>  
>  		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-argb8888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB8888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-abgr8888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_ABGR8888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-xrgb8888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB8888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-xbgr8888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_XBGR8888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-rgb888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_RGB888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-bgr888", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_BGR888,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-rgb565", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_RGB565,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-bgr565", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_BGR565,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-argb1555", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB1555,
> -					      CHAMELIUM_CHECK_CRC, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-crc-xrgb1555", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB1555,
> -					      CHAMELIUM_CHECK_CRC, 1);
> +		connector_subtest_start("hdmi-rgb-pixel-format", HDMIA)
> +			int k;
> +			igt_output_t *output;
> +			igt_plane_t *primary;
> +
> +			output = prepare_output(&data, port, TEST_EDID_BASE);
> +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> +			igt_assert(primary);
> +
> +			for (k = 0; k < primary->format_mod_count; k++) {
> +				if (!igt_fb_supported_format(primary->formats[k]))
> +					continue;
> +
> +				if ((igt_format_is_yuv(primary->formats[k]) ||
> +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> +					continue;
> +
> +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))

We have a igt_format_str() which is probably better for stuff like this.

> +					test_display_one_mode(&data, port, primary->formats[k],
> +							      CHAMELIUM_CHECK_CRC,
> +							      1);
> +			}
> +		connector_subtest_end
>  
>  		igt_describe(test_display_planes_random_desc);
>  		connector_subtest("hdmi-crc-planes-random", HDMIA)
> @@ -2872,44 +2855,29 @@ igt_main
>  						   CHAMELIUM_CHECK_CRC);
>  
>  		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-nv12", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_NV12,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-nv16", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_NV16,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-nv21", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_NV21,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-nv61", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_NV61,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-yu12", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_YUV420,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-yu16", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_YUV422,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-yv12", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_YVU420,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> -
> -		igt_describe(test_display_one_mode_desc);
> -		connector_subtest("hdmi-cmp-yv16", HDMIA)
> -			test_display_one_mode(&data, port, DRM_FORMAT_YVU422,
> -					      CHAMELIUM_CHECK_CHECKERBOARD, 1);
> +		connector_subtest_start("hdmi-yuv-pixel-format", HDMIA)
> +			int k;
> +			igt_output_t *output;
> +			igt_plane_t *primary;
> +
> +			output = prepare_output(&data, port, TEST_EDID_BASE);
> +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> +			igt_assert(primary);
> +
> +			for (k = 0; k < primary->format_mod_count; k++) {
> +				if (!igt_fb_supported_format(primary->formats[k]))
> +					continue;
> +
> +				if (!(igt_format_is_yuv(primary->formats[k]) &&
> +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> +					continue;
> +
> +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> +					test_display_one_mode(&data, port, primary->formats[k],
> +							      CHAMELIUM_CHECK_CHECKERBOARD,
> +							      1);
> +			}
> +		connector_subtest_end
>  
>  		igt_describe(test_display_planes_random_desc);
>  		connector_subtest("hdmi-cmp-planes-random", HDMIA)
> -- 
> 2.7.4
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* Re: [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats
  2020-03-05  8:42     ` Kunal Joshi
@ 2020-03-05 16:50       ` Ville Syrjälä
  2020-03-06  0:09         ` Kunal Joshi
  0 siblings, 1 reply; 9+ messages in thread
From: Ville Syrjälä @ 2020-03-05 16:50 UTC (permalink / raw)
  To: Kunal Joshi; +Cc: igt-dev

On Thu, Mar 05, 2020 at 02:12:38PM +0530, Kunal Joshi wrote:
> On 2020-03-05 at 15:01:15 +0200, Ville Syrjälä wrote:
> > On Thu, Mar 05, 2020 at 09:03:38AM +0530, Kunal Joshi wrote:
> > > Currently, only hard coded pixel formats are validated which are not
> > > generic for all the platforms which leads to SKIPS.
> > > Added support to validate only those pixel formats which are supported by
> > > platform. Both RGB and YUV pixel formats are covered.
> > > 
> > > Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> > > ---
> > >  tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
> > >  1 file changed, 54 insertions(+), 86 deletions(-)
> > > 
> > > diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> > > index 5c4a189..5c96017 100644
> > > --- a/tests/kms_chamelium.c
> > > +++ b/tests/kms_chamelium.c
> > > @@ -2612,6 +2612,14 @@ static const struct edid *get_edid(enum test_edid edid)
> > >  			if (chamelium_port_get_type(port) == \
> > >  			    DRM_MODE_CONNECTOR_ ## type__)
> > >  
> > > +#define connector_subtest_start(name__, type__)              \
> > > +        igt_subtest_with_dynamic(name__)                     \
> > > +                for_each_port(p, port)                       \
> > > +                        if (chamelium_port_get_type(port) == \
> > > +                            DRM_MODE_CONNECTOR_ ## type__) {
> > > +
> > > +#define connector_subtest_end }
> > > +
> > >  static data_t data;
> > >  
> > >  IGT_TEST_DESCRIPTION("Tests requiring a Chamelium board");
> > > @@ -2817,54 +2825,29 @@ igt_main
> > >  					       CHAMELIUM_CHECK_CRC, 3);
> > >  
> > >  		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-argb8888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB8888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-abgr8888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_ABGR8888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-xrgb8888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB8888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-xbgr8888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_XBGR8888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-rgb888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-bgr888", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR888,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-rgb565", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB565,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-bgr565", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR565,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-argb1555", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB1555,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > -
> > > -		igt_describe(test_display_one_mode_desc);
> > > -		connector_subtest("hdmi-crc-xrgb1555", HDMIA)
> > > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB1555,
> > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > +		connector_subtest_start("hdmi-rgb-pixel-format", HDMIA)
> > > +			int k;
> > > +			igt_output_t *output;
> > > +			igt_plane_t *primary;
> > > +
> > > +			output = prepare_output(&data, port, TEST_EDID_BASE);
> > > +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> > > +			igt_assert(primary);
> > > +
> > > +			for (k = 0; k < primary->format_mod_count; k++) {
> > > +				if (!igt_fb_supported_format(primary->formats[k]))
> > > +					continue;
> > > +
> > > +				if ((igt_format_is_yuv(primary->formats[k]) ||
> > > +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> > > +					continue;
> > > +
> > > +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> > 
> > We have a igt_format_str() which is probably better for stuff like this.
> > 
> 
> Thanks ville for pointing that out
> I tried using igt_format_str()
> but it doesn't seem to support that i tried it like below
> 
> igt_dynamic_f("%s",igt_format_str("testing-format-%c%c%c%c",
> 				  IGT_FORMAT_ARGS_CODE(primary->formats[k]))

I don't know what you're trying to do there.

igt_dynamic_f("testing-format-%s", igt_format_str(format))
or whatever.

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

* [igt-dev] ✗ Fi.CI.IGT: failure for Test platform supported pixel formats through chamelium. (rev2)
  2020-03-05  3:33 [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium Kunal Joshi
                   ` (2 preceding siblings ...)
  2020-03-05 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success for Test platform supported pixel formats through chamelium. (rev2) Patchwork
@ 2020-03-06  0:01 ` Patchwork
  3 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2020-03-06  0:01 UTC (permalink / raw)
  To: Kunal Joshi; +Cc: igt-dev

== Series Details ==

Series: Test platform supported pixel formats through chamelium. (rev2)
URL   : https://patchwork.freedesktop.org/series/74317/
State : failure

== Summary ==

CI Bug Log - changes from IGT_5493_full -> IGTPW_4262_full
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_4262_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_4262_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://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/index.html

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_pm_dc@dc5-dpms:
    - shard-iclb:         [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb7/igt@i915_pm_dc@dc5-dpms.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb3/igt@i915_pm_dc@dc5-dpms.html

  * {igt@kms_chamelium@hdmi-yuv-pixel-format} (NEW):
    - shard-tglb:         NOTRUN -> [SKIP][3] +1 similar issue
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb6/igt@kms_chamelium@hdmi-yuv-pixel-format.html

  
#### Warnings ####

  * igt@runner@aborted:
    - shard-hsw:          ([FAIL][4], [FAIL][5], [FAIL][6], [FAIL][7], [FAIL][8], [FAIL][9], [FAIL][10], [FAIL][11]) ([fdo#111870] / [i915#478]) -> ([FAIL][12], [FAIL][13], [FAIL][14], [FAIL][15], [FAIL][16], [FAIL][17]) ([fdo#111870])
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw1/igt@runner@aborted.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw2/igt@runner@aborted.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw2/igt@runner@aborted.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw8/igt@runner@aborted.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw8/igt@runner@aborted.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw6/igt@runner@aborted.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw2/igt@runner@aborted.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw6/igt@runner@aborted.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw1/igt@runner@aborted.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw4/igt@runner@aborted.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw5/igt@runner@aborted.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw8/igt@runner@aborted.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw5/igt@runner@aborted.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw6/igt@runner@aborted.html

  
New tests
---------

  New tests have been introduced between IGT_5493_full and IGTPW_4262_full:

### New IGT tests (2) ###

  * igt@kms_chamelium@hdmi-rgb-pixel-format:
    - Statuses : 7 skip(s)
    - Exec time: [0.0] s

  * igt@kms_chamelium@hdmi-yuv-pixel-format:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_shared@exec-shared-gtt-vebox:
    - shard-kbl:          [PASS][18] -> [FAIL][19] ([i915#616])
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-kbl4/igt@gem_ctx_shared@exec-shared-gtt-vebox.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-kbl4/igt@gem_ctx_shared@exec-shared-gtt-vebox.html

  * igt@gem_ctx_shared@exec-single-timeline-bsd:
    - shard-iclb:         [PASS][20] -> [SKIP][21] ([fdo#110841])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb8/igt@gem_ctx_shared@exec-single-timeline-bsd.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb4/igt@gem_ctx_shared@exec-single-timeline-bsd.html

  * igt@gem_exec_balancer@smoke:
    - shard-iclb:         [PASS][22] -> [SKIP][23] ([fdo#110854])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb4/igt@gem_exec_balancer@smoke.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb7/igt@gem_exec_balancer@smoke.html

  * igt@gem_exec_schedule@implicit-write-read-bsd2:
    - shard-iclb:         [PASS][24] -> [SKIP][25] ([fdo#109276] / [i915#677])
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb4/igt@gem_exec_schedule@implicit-write-read-bsd2.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb5/igt@gem_exec_schedule@implicit-write-read-bsd2.html

  * igt@gem_exec_schedule@promotion-bsd1:
    - shard-iclb:         [PASS][26] -> [SKIP][27] ([fdo#109276]) +15 similar issues
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb1/igt@gem_exec_schedule@promotion-bsd1.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb3/igt@gem_exec_schedule@promotion-bsd1.html

  * igt@gem_exec_schedule@wide-bsd:
    - shard-iclb:         [PASS][28] -> [SKIP][29] ([fdo#112146]) +1 similar issue
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb3/igt@gem_exec_schedule@wide-bsd.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb2/igt@gem_exec_schedule@wide-bsd.html

  * igt@i915_pm_rpm@basic-rte:
    - shard-glk:          [PASS][30] -> [FAIL][31] ([i915#579])
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-glk1/igt@i915_pm_rpm@basic-rte.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-glk2/igt@i915_pm_rpm@basic-rte.html
    - shard-iclb:         [PASS][32] -> [FAIL][33] ([i915#579])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb8/igt@i915_pm_rpm@basic-rte.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb4/igt@i915_pm_rpm@basic-rte.html
    - shard-tglb:         [PASS][34] -> [FAIL][35] ([i915#579])
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb2/igt@i915_pm_rpm@basic-rte.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb6/igt@i915_pm_rpm@basic-rte.html

  * igt@i915_pm_rpm@fences:
    - shard-tglb:         [PASS][36] -> [SKIP][37] ([i915#1316])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb7/igt@i915_pm_rpm@fences.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb6/igt@i915_pm_rpm@fences.html
    - shard-iclb:         [PASS][38] -> [SKIP][39] ([i915#1316])
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb6/igt@i915_pm_rpm@fences.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb4/igt@i915_pm_rpm@fences.html
    - shard-glk:          [PASS][40] -> [SKIP][41] ([fdo#109271])
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-glk7/igt@i915_pm_rpm@fences.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-glk2/igt@i915_pm_rpm@fences.html

  * igt@kms_cursor_crc@pipe-a-cursor-suspend:
    - shard-kbl:          [PASS][42] -> [DMESG-WARN][43] ([i915#180]) +3 similar issues
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-kbl4/igt@kms_cursor_crc@pipe-a-cursor-suspend.html

  * igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen:
    - shard-kbl:          [PASS][44] -> [FAIL][45] ([i915#54])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-kbl3/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-kbl3/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html
    - shard-apl:          [PASS][46] -> [FAIL][47] ([i915#54])
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-apl4/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-apl3/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html
    - shard-glk:          [PASS][48] -> [FAIL][49] ([i915#54])
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-glk9/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-glk7/igt@kms_cursor_crc@pipe-c-cursor-128x128-onscreen.html

  * igt@kms_flip@flip-vs-suspend-interruptible:
    - shard-tglb:         [PASS][50] -> [INCOMPLETE][51] ([i915#456] / [i915#460] / [i915#516])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb5/igt@kms_flip@flip-vs-suspend-interruptible.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb7/igt@kms_flip@flip-vs-suspend-interruptible.html

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
    - shard-apl:          [PASS][52] -> [DMESG-WARN][53] ([i915#180])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-apl1/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-apl4/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html

  * igt@kms_psr@psr2_primary_mmap_cpu:
    - shard-iclb:         [PASS][54] -> [SKIP][55] ([fdo#109441]) +2 similar issues
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb2/igt@kms_psr@psr2_primary_mmap_cpu.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb4/igt@kms_psr@psr2_primary_mmap_cpu.html

  * igt@kms_setmode@basic:
    - shard-hsw:          [PASS][56] -> [FAIL][57] ([i915#31])
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw6/igt@kms_setmode@basic.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw4/igt@kms_setmode@basic.html

  * igt@perf_pmu@busy-check-all-vcs1:
    - shard-iclb:         [PASS][58] -> [SKIP][59] ([fdo#112080]) +11 similar issues
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb2/igt@perf_pmu@busy-check-all-vcs1.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb6/igt@perf_pmu@busy-check-all-vcs1.html

  
#### Possible fixes ####

  * igt@gem_busy@close-race:
    - shard-tglb:         [INCOMPLETE][60] ([i915#977]) -> [PASS][61]
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb5/igt@gem_busy@close-race.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb6/igt@gem_busy@close-race.html

  * igt@gem_ctx_persistence@legacy-engines-mixed-process@vebox:
    - shard-kbl:          [FAIL][62] ([i915#679]) -> [PASS][63]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-kbl6/igt@gem_ctx_persistence@legacy-engines-mixed-process@vebox.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-kbl4/igt@gem_ctx_persistence@legacy-engines-mixed-process@vebox.html

  * igt@gem_ctx_shared@exec-shared-gtt-blt:
    - shard-tglb:         [FAIL][64] ([i915#616]) -> [PASS][65]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb7/igt@gem_ctx_shared@exec-shared-gtt-blt.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb5/igt@gem_ctx_shared@exec-shared-gtt-blt.html

  * igt@gem_exec_balancer@hang:
    - shard-tglb:         [FAIL][66] ([i915#1277]) -> [PASS][67]
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb7/igt@gem_exec_balancer@hang.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb1/igt@gem_exec_balancer@hang.html

  * igt@gem_exec_schedule@fifo-bsd1:
    - shard-iclb:         [SKIP][68] ([fdo#109276]) -> [PASS][69] +15 similar issues
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb7/igt@gem_exec_schedule@fifo-bsd1.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb1/igt@gem_exec_schedule@fifo-bsd1.html

  * igt@gem_exec_schedule@implicit-read-write-bsd1:
    - shard-iclb:         [SKIP][70] ([fdo#109276] / [i915#677]) -> [PASS][71] +1 similar issue
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb6/igt@gem_exec_schedule@implicit-read-write-bsd1.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb2/igt@gem_exec_schedule@implicit-read-write-bsd1.html

  * igt@gem_exec_schedule@pi-common-bsd:
    - shard-iclb:         [SKIP][72] ([i915#677]) -> [PASS][73]
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb1/igt@gem_exec_schedule@pi-common-bsd.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb7/igt@gem_exec_schedule@pi-common-bsd.html

  * igt@gem_exec_schedule@pi-ringfull-bsd:
    - shard-iclb:         [SKIP][74] ([fdo#112146]) -> [PASS][75] +2 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb2/igt@gem_exec_schedule@pi-ringfull-bsd.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb5/igt@gem_exec_schedule@pi-ringfull-bsd.html

  * igt@gem_mmap_offset@pf-nonblock:
    - shard-hsw:          [DMESG-WARN][76] ([i915#478]) -> [PASS][77]
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw6/igt@gem_mmap_offset@pf-nonblock.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw2/igt@gem_mmap_offset@pf-nonblock.html

  * igt@gem_userptr_blits@sync-unmap-cycles:
    - shard-snb:          [DMESG-WARN][78] ([fdo#111870] / [i915#478]) -> [PASS][79]
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb5/igt@gem_userptr_blits@sync-unmap-cycles.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb6/igt@gem_userptr_blits@sync-unmap-cycles.html
    - shard-hsw:          [DMESG-WARN][80] ([fdo#111870]) -> [PASS][81]
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw2/igt@gem_userptr_blits@sync-unmap-cycles.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw5/igt@gem_userptr_blits@sync-unmap-cycles.html

  * igt@i915_pm_rpm@gem-pread:
    - shard-tglb:         [SKIP][82] ([i915#1316]) -> [PASS][83]
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-tglb1/igt@i915_pm_rpm@gem-pread.html
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-tglb8/igt@i915_pm_rpm@gem-pread.html
    - shard-iclb:         [SKIP][84] ([i915#1316]) -> [PASS][85]
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb3/igt@i915_pm_rpm@gem-pread.html
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb2/igt@i915_pm_rpm@gem-pread.html
    - shard-glk:          [SKIP][86] ([fdo#109271]) -> [PASS][87]
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-glk2/igt@i915_pm_rpm@gem-pread.html
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-glk9/igt@i915_pm_rpm@gem-pread.html

  * igt@kms_cursor_crc@pipe-a-cursor-size-change:
    - shard-snb:          [SKIP][88] ([fdo#109271]) -> [PASS][89] +1 similar issue
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb2/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb2/igt@kms_cursor_crc@pipe-a-cursor-size-change.html

  * igt@kms_flip@flip-vs-suspend-interruptible:
    - shard-apl:          [DMESG-WARN][90] ([i915#180]) -> [PASS][91] +6 similar issues
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-apl6/igt@kms_flip@flip-vs-suspend-interruptible.html
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-apl3/igt@kms_flip@flip-vs-suspend-interruptible.html
    - shard-kbl:          [DMESG-WARN][92] ([i915#180]) -> [PASS][93] +2 similar issues
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-kbl4/igt@kms_flip@flip-vs-suspend-interruptible.html
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-kbl7/igt@kms_flip@flip-vs-suspend-interruptible.html

  * igt@kms_frontbuffer_tracking@fbc-suspend:
    - shard-snb:          [DMESG-WARN][94] ([i915#478]) -> [PASS][95]
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb6/igt@kms_frontbuffer_tracking@fbc-suspend.html
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb6/igt@kms_frontbuffer_tracking@fbc-suspend.html

  * igt@kms_plane_lowres@pipe-a-tiling-y:
    - shard-glk:          [FAIL][96] ([i915#899]) -> [PASS][97]
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-glk9/igt@kms_plane_lowres@pipe-a-tiling-y.html
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-glk8/igt@kms_plane_lowres@pipe-a-tiling-y.html

  * igt@kms_psr@psr2_dpms:
    - shard-iclb:         [SKIP][98] ([fdo#109441]) -> [PASS][99] +1 similar issue
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb8/igt@kms_psr@psr2_dpms.html
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb2/igt@kms_psr@psr2_dpms.html

  * igt@perf_pmu@busy-no-semaphores-vcs1:
    - shard-iclb:         [SKIP][100] ([fdo#112080]) -> [PASS][101] +9 similar issues
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb6/igt@perf_pmu@busy-no-semaphores-vcs1.html
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb1/igt@perf_pmu@busy-no-semaphores-vcs1.html

  
#### Warnings ####

  * igt@gem_exec_schedule@pi-userfault-bsd:
    - shard-iclb:         [INCOMPLETE][102] ([i915#1381]) -> [SKIP][103] ([i915#677])
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb5/igt@gem_exec_schedule@pi-userfault-bsd.html
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb2/igt@gem_exec_schedule@pi-userfault-bsd.html

  * igt@gem_exec_schedule@pi-userfault-bsd2:
    - shard-iclb:         [SKIP][104] ([fdo#109276]) -> [INCOMPLETE][105] ([i915#1381])
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-iclb7/igt@gem_exec_schedule@pi-userfault-bsd2.html
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-iclb1/igt@gem_exec_schedule@pi-userfault-bsd2.html

  * igt@gem_userptr_blits@dmabuf-sync:
    - shard-snb:          [DMESG-WARN][106] ([fdo#110789] / [fdo#111870] / [i915#478]) -> [DMESG-WARN][107] ([fdo#111870] / [i915#478])
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb4/igt@gem_userptr_blits@dmabuf-sync.html
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb5/igt@gem_userptr_blits@dmabuf-sync.html
    - shard-hsw:          [DMESG-WARN][108] ([fdo#110789] / [fdo#111870]) -> [DMESG-WARN][109] ([fdo#111870])
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-hsw2/igt@gem_userptr_blits@dmabuf-sync.html
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-hsw6/igt@gem_userptr_blits@dmabuf-sync.html

  * igt@runner@aborted:
    - shard-snb:          ([FAIL][110], [FAIL][111], [FAIL][112], [FAIL][113], [FAIL][114], [FAIL][115], [FAIL][116], [FAIL][117]) ([fdo#111870] / [i915#1077] / [i915#698]) -> ([FAIL][118], [FAIL][119], [FAIL][120], [FAIL][121], [FAIL][122], [FAIL][123]) ([fdo#111870] / [i915#1077])
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb2/igt@runner@aborted.html
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb5/igt@runner@aborted.html
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb4/igt@runner@aborted.html
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb2/igt@runner@aborted.html
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb6/igt@runner@aborted.html
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb5/igt@runner@aborted.html
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb2/igt@runner@aborted.html
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5493/shard-snb4/igt@runner@aborted.html
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb6/igt@runner@aborted.html
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb5/igt@runner@aborted.html
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb2/igt@runner@aborted.html
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb5/igt@runner@aborted.html
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb5/igt@runner@aborted.html
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/shard-snb4/igt@runner@aborted.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#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#110789]: https://bugs.freedesktop.org/show_bug.cgi?id=110789
  [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841
  [fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
  [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870
  [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
  [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
  [i915#1077]: https://gitlab.freedesktop.org/drm/intel/issues/1077
  [i915#1277]: https://gitlab.freedesktop.org/drm/intel/issues/1277
  [i915#1316]: https://gitlab.freedesktop.org/drm/intel/issues/1316
  [i915#1381]: https://gitlab.freedesktop.org/drm/intel/issues/1381
  [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
  [i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31
  [i915#456]: https://gitlab.freedesktop.org/drm/intel/issues/456
  [i915#460]: https://gitlab.freedesktop.org/drm/intel/issues/460
  [i915#478]: https://gitlab.freedesktop.org/drm/intel/issues/478
  [i915#516]: https://gitlab.freedesktop.org/drm/intel/issues/516
  [i915#54]: https://gitlab.freedesktop.org/drm/intel/issues/54
  [i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579
  [i915#616]: https://gitlab.freedesktop.org/drm/intel/issues/616
  [i915#677]: https://gitlab.freedesktop.org/drm/intel/issues/677
  [i915#679]: https://gitlab.freedesktop.org/drm/intel/issues/679
  [i915#698]: https://gitlab.freedesktop.org/drm/intel/issues/698
  [i915#899]: https://gitlab.freedesktop.org/drm/intel/issues/899
  [i915#977]: https://gitlab.freedesktop.org/drm/intel/issues/977


Participating hosts (8 -> 8)
------------------------------

  No changes in participating hosts


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

  * CI: CI-20190529 -> None
  * IGT: IGT_5493 -> IGTPW_4262

  CI-20190529: 20190529
  CI_DRM_8068: f8e69af5cca45947ebce78f677b75b0ecc4ba744 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_4262: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_4262/index.html
  IGT_5493: 41422f8384047e2ec4c4cb896f6955bf51490f53 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

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

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

* Re: [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats
  2020-03-05 16:50       ` Ville Syrjälä
@ 2020-03-06  0:09         ` Kunal Joshi
  0 siblings, 0 replies; 9+ messages in thread
From: Kunal Joshi @ 2020-03-06  0:09 UTC (permalink / raw)
  To: Ville Syrjälä, igt-dev

On 2020-03-05 at 18:50:10 +0200, Ville Syrjälä wrote:
> On Thu, Mar 05, 2020 at 02:12:38PM +0530, Kunal Joshi wrote:
> > On 2020-03-05 at 15:01:15 +0200, Ville Syrjälä wrote:
> > > On Thu, Mar 05, 2020 at 09:03:38AM +0530, Kunal Joshi wrote:
> > > > Currently, only hard coded pixel formats are validated which are not
> > > > generic for all the platforms which leads to SKIPS.
> > > > Added support to validate only those pixel formats which are supported by
> > > > platform. Both RGB and YUV pixel formats are covered.
> > > > 
> > > > Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> > > > ---
> > > >  tests/kms_chamelium.c | 140 +++++++++++++++++++-------------------------------
> > > >  1 file changed, 54 insertions(+), 86 deletions(-)
> > > > 
> > > > diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
> > > > index 5c4a189..5c96017 100644
> > > > --- a/tests/kms_chamelium.c
> > > > +++ b/tests/kms_chamelium.c
> > > > @@ -2612,6 +2612,14 @@ static const struct edid *get_edid(enum test_edid edid)
> > > >  			if (chamelium_port_get_type(port) == \
> > > >  			    DRM_MODE_CONNECTOR_ ## type__)
> > > >  
> > > > +#define connector_subtest_start(name__, type__)              \
> > > > +        igt_subtest_with_dynamic(name__)                     \
> > > > +                for_each_port(p, port)                       \
> > > > +                        if (chamelium_port_get_type(port) == \
> > > > +                            DRM_MODE_CONNECTOR_ ## type__) {
> > > > +
> > > > +#define connector_subtest_end }
> > > > +
> > > >  static data_t data;
> > > >  
> > > >  IGT_TEST_DESCRIPTION("Tests requiring a Chamelium board");
> > > > @@ -2817,54 +2825,29 @@ igt_main
> > > >  					       CHAMELIUM_CHECK_CRC, 3);
> > > >  
> > > >  		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-argb8888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB8888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-abgr8888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_ABGR8888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-xrgb8888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB8888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-xbgr8888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_XBGR8888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-rgb888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-bgr888", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR888,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-rgb565", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_RGB565,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-bgr565", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_BGR565,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-argb1555", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_ARGB1555,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > -
> > > > -		igt_describe(test_display_one_mode_desc);
> > > > -		connector_subtest("hdmi-crc-xrgb1555", HDMIA)
> > > > -			test_display_one_mode(&data, port, DRM_FORMAT_XRGB1555,
> > > > -					      CHAMELIUM_CHECK_CRC, 1);
> > > > +		connector_subtest_start("hdmi-rgb-pixel-format", HDMIA)
> > > > +			int k;
> > > > +			igt_output_t *output;
> > > > +			igt_plane_t *primary;
> > > > +
> > > > +			output = prepare_output(&data, port, TEST_EDID_BASE);
> > > > +			primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> > > > +			igt_assert(primary);
> > > > +
> > > > +			for (k = 0; k < primary->format_mod_count; k++) {
> > > > +				if (!igt_fb_supported_format(primary->formats[k]))
> > > > +					continue;
> > > > +
> > > > +				if ((igt_format_is_yuv(primary->formats[k]) ||
> > > > +				     primary->modifiers[k] != LOCAL_DRM_FORMAT_MOD_NONE))
> > > > +					continue;
> > > > +
> > > > +				igt_dynamic_f("testing-format-%c%c%c%c", IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> > > 
> > > We have a igt_format_str() which is probably better for stuff like this.
> > > 
> > 
> > Thanks ville for pointing that out
> > I tried using igt_format_str()
> > but it doesn't seem to support that i tried it like below
> > 
> > igt_dynamic_f("%s",igt_format_str("testing-format-%c%c%c%c",
> > 				  IGT_FORMAT_ARGS_CODE(primary->formats[k]))
> 
> I don't know what you're trying to do there.
> 
> igt_dynamic_f("testing-format-%s", igt_format_str(format))
> or whatever.
> 
Ville got your point i have addressed your comment in the next version

> -- 
> Ville Syrjälä
> Intel

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

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

end of thread, other threads:[~2020-03-06  7:07 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-05  3:33 [igt-dev] [PATCH i-g-t v2 0/2] Test platform supported pixel formats through chamelium Kunal Joshi
2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 1/2] lib/igt_fb: Added macro to retrieve char code of format Kunal Joshi
2020-03-05  3:33 ` [igt-dev] [PATCH i-g-t v2 2/2] tests/kms_chamelium: Add support to validate RGB/YUV pixel formats Kunal Joshi
2020-03-05 13:01   ` Ville Syrjälä
2020-03-05  8:42     ` Kunal Joshi
2020-03-05 16:50       ` Ville Syrjälä
2020-03-06  0:09         ` Kunal Joshi
2020-03-05 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success for Test platform supported pixel formats through chamelium. (rev2) Patchwork
2020-03-06  0:01 ` [igt-dev] ✗ Fi.CI.IGT: failure " 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.