* [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
@ 2019-03-20 19:37 Dhinakaran Pandiyan
2019-03-20 19:37 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too Dhinakaran Pandiyan
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2019-03-20 19:37 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
Currently only Y tiled CCS modifier is tested, rewrite the functions so
that we can add support for other CCS modifiers.
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
---
tests/kms_ccs.c | 52 +++++++++++++++++++++++++++++++++----------------
1 file changed, 35 insertions(+), 17 deletions(-)
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index a74bd706..f780a01c 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -58,6 +58,7 @@ typedef struct {
enum test_flags flags;
igt_plane_t *plane;
igt_pipe_crc_t *pipe_crc;
+ uint64_t ccs_modifier;
} data_t;
static const struct {
@@ -69,6 +70,10 @@ static const struct {
{0.0, 1.0, 0.0}
};
+static const uint64_t ccs_modifiers[] = {
+ LOCAL_I915_FORMAT_MOD_Y_TILED_CCS
+};
+
/*
* Limit maximum used sprite plane width so this test will not mistakenly
* fail on hardware limitations which are not interesting to this test.
@@ -140,7 +145,8 @@ modifiers_ptr(struct local_drm_format_modifier_blob *blob)
return (struct local_drm_format_modifier *)(((char *)blob) + blob->modifiers_offset);
}
-static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane, uint32_t format)
+static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane,
+ uint32_t format)
{
drmModePropertyBlobPtr blob;
struct local_drm_format_modifier_blob *blob_data;
@@ -178,7 +184,7 @@ static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane, uint32_t
last_mod = &modifiers[blob_data->count_modifiers];
igt_assert_lte(((char *) last_mod - (char *) blob_data), blob->length);
for (int i = 0; i < blob_data->count_modifiers; i++) {
- if (modifiers[i].modifier != LOCAL_I915_FORMAT_MOD_Y_TILED_CCS)
+ if (modifiers[i].modifier != data->ccs_modifier)
continue;
if (modifiers[i].offset > fmt_idx ||
@@ -226,7 +232,7 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
* available FIFO configurations.
*/
if (fb_flags & FB_COMPRESSED)
- modifier = LOCAL_I915_FORMAT_MOD_Y_TILED_CCS;
+ modifier = data->ccs_modifier;
else if (!(fb_flags & FB_HAS_PLANE))
modifier = LOCAL_I915_FORMAT_MOD_Y_TILED;
else
@@ -313,6 +319,7 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
if (data->plane && fb_flags & FB_COMPRESSED) {
if (!plane_has_format_with_ccs(data, data->plane, DRM_FORMAT_XRGB8888))
return false;
+
generate_fb(data, &fb, min(MAX_SPRITE_PLANE_WIDTH, drm_mode->hdisplay),
drm_mode->vdisplay,
(fb_flags & ~FB_COMPRESSED) | FB_HAS_PLANE);
@@ -369,21 +376,10 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
return true;
}
-static int test_output(data_t *data)
-{
- igt_display_t *display = &data->display;
+static int test_ccs(data_t *data)
+{ int valid_tests = 0;
igt_crc_t crc, ref_crc;
enum test_fb_flags fb_flags = 0;
- int valid_tests = 0;
-
- igt_display_require_output_on_pipe(display, data->pipe);
-
- /* Sets data->output with a valid output. */
- for_each_valid_output_on_pipe(display, data->pipe, data->output) {
- break;
- }
-
- igt_output_set_pipe(data->output, data->pipe);
if (data->flags & TEST_CRC) {
data->pipe_crc = igt_pipe_crc_new(data->drm_fd, data->pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
@@ -411,7 +407,29 @@ static int test_output(data_t *data)
valid_tests += try_config(data, fb_flags | FB_COMPRESSED | FB_ZERO_AUX_STRIDE , NULL);
}
- igt_output_set_pipe(data->output, PIPE_ANY);
+ return valid_tests;
+}
+
+static int test_output(data_t *data)
+{
+ igt_display_t *display = &data->display;
+ int i, valid_tests = 0;
+
+ igt_display_require_output_on_pipe(display, data->pipe);
+
+ /* Sets data->output with a valid output. */
+ for_each_valid_output_on_pipe(display, data->pipe, data->output) {
+ break;
+ }
+
+ igt_output_set_pipe(data->output, data->pipe);
+
+ for (i = 0; i < ARRAY_SIZE(ccs_modifiers); i++) {
+ data->ccs_modifier = ccs_modifiers[i];
+ valid_tests += test_ccs(data);
+ }
+
+ igt_output_set_pipe(data->output, PIPE_NONE);
igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
return valid_tests;
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too.
2019-03-20 19:37 [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Dhinakaran Pandiyan
@ 2019-03-20 19:37 ` Dhinakaran Pandiyan
2019-03-20 19:51 ` Ville Syrjälä
2019-03-20 19:49 ` [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Ville Syrjälä
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2019-03-20 19:37 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
Extend the test to Yf tiled CCS, no reason not to.
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
tests/kms_ccs.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index f780a01c..cd42e747 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -71,7 +71,8 @@ static const struct {
};
static const uint64_t ccs_modifiers[] = {
- LOCAL_I915_FORMAT_MOD_Y_TILED_CCS
+ LOCAL_I915_FORMAT_MOD_Y_TILED_CCS,
+ LOCAL_I915_FORMAT_MOD_Yf_TILED_CCS
};
/*
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
2019-03-20 19:37 [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Dhinakaran Pandiyan
2019-03-20 19:37 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too Dhinakaran Pandiyan
@ 2019-03-20 19:49 ` Ville Syrjälä
2019-03-20 19:54 ` Dhinakaran Pandiyan
2019-03-20 20:15 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] " Patchwork
2019-03-21 3:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 1 reply; 7+ messages in thread
From: Ville Syrjälä @ 2019-03-20 19:49 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
On Wed, Mar 20, 2019 at 12:37:32PM -0700, Dhinakaran Pandiyan wrote:
> Currently only Y tiled CCS modifier is tested, rewrite the functions so
> that we can add support for other CCS modifiers.
>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> ---
> tests/kms_ccs.c | 52 +++++++++++++++++++++++++++++++++----------------
> 1 file changed, 35 insertions(+), 17 deletions(-)
>
> diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
> index a74bd706..f780a01c 100644
> --- a/tests/kms_ccs.c
> +++ b/tests/kms_ccs.c
> @@ -58,6 +58,7 @@ typedef struct {
> enum test_flags flags;
> igt_plane_t *plane;
> igt_pipe_crc_t *pipe_crc;
> + uint64_t ccs_modifier;
> } data_t;
>
> static const struct {
> @@ -69,6 +70,10 @@ static const struct {
> {0.0, 1.0, 0.0}
> };
>
> +static const uint64_t ccs_modifiers[] = {
> + LOCAL_I915_FORMAT_MOD_Y_TILED_CCS
> +};
> +
> /*
> * Limit maximum used sprite plane width so this test will not mistakenly
> * fail on hardware limitations which are not interesting to this test.
> @@ -140,7 +145,8 @@ modifiers_ptr(struct local_drm_format_modifier_blob *blob)
> return (struct local_drm_format_modifier *)(((char *)blob) + blob->modifiers_offset);
> }
>
> -static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane, uint32_t format)
> +static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane,
> + uint32_t format)
> {
> drmModePropertyBlobPtr blob;
> struct local_drm_format_modifier_blob *blob_data;
Hmm. I thought I changed this to use the igt_kms thing. I guess it never
got pushed.
> @@ -178,7 +184,7 @@ static bool plane_has_format_with_ccs(data_t *data, igt_plane_t *plane, uint32_t
> last_mod = &modifiers[blob_data->count_modifiers];
> igt_assert_lte(((char *) last_mod - (char *) blob_data), blob->length);
> for (int i = 0; i < blob_data->count_modifiers; i++) {
> - if (modifiers[i].modifier != LOCAL_I915_FORMAT_MOD_Y_TILED_CCS)
> + if (modifiers[i].modifier != data->ccs_modifier)
> continue;
>
> if (modifiers[i].offset > fmt_idx ||
> @@ -226,7 +232,7 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
> * available FIFO configurations.
> */
> if (fb_flags & FB_COMPRESSED)
> - modifier = LOCAL_I915_FORMAT_MOD_Y_TILED_CCS;
> + modifier = data->ccs_modifier;
> else if (!(fb_flags & FB_HAS_PLANE))
> modifier = LOCAL_I915_FORMAT_MOD_Y_TILED;
> else
> @@ -313,6 +319,7 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
> if (data->plane && fb_flags & FB_COMPRESSED) {
> if (!plane_has_format_with_ccs(data, data->plane, DRM_FORMAT_XRGB8888))
> return false;
> +
> generate_fb(data, &fb, min(MAX_SPRITE_PLANE_WIDTH, drm_mode->hdisplay),
> drm_mode->vdisplay,
> (fb_flags & ~FB_COMPRESSED) | FB_HAS_PLANE);
> @@ -369,21 +376,10 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
> return true;
> }
>
> -static int test_output(data_t *data)
> -{
> - igt_display_t *display = &data->display;
> +static int test_ccs(data_t *data)
> +{ int valid_tests = 0;
> igt_crc_t crc, ref_crc;
> enum test_fb_flags fb_flags = 0;
> - int valid_tests = 0;
> -
> - igt_display_require_output_on_pipe(display, data->pipe);
> -
> - /* Sets data->output with a valid output. */
> - for_each_valid_output_on_pipe(display, data->pipe, data->output) {
> - break;
> - }
> -
> - igt_output_set_pipe(data->output, data->pipe);
>
> if (data->flags & TEST_CRC) {
> data->pipe_crc = igt_pipe_crc_new(data->drm_fd, data->pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
> @@ -411,7 +407,29 @@ static int test_output(data_t *data)
> valid_tests += try_config(data, fb_flags | FB_COMPRESSED | FB_ZERO_AUX_STRIDE , NULL);
> }
>
> - igt_output_set_pipe(data->output, PIPE_ANY);
> + return valid_tests;
> +}
> +
> +static int test_output(data_t *data)
> +{
> + igt_display_t *display = &data->display;
> + int i, valid_tests = 0;
> +
> + igt_display_require_output_on_pipe(display, data->pipe);
> +
> + /* Sets data->output with a valid output. */
> + for_each_valid_output_on_pipe(display, data->pipe, data->output) {
> + break;
> + }
I think we have a nicer way to do this these days.
Patch is
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> +
> + igt_output_set_pipe(data->output, data->pipe);
> +
> + for (i = 0; i < ARRAY_SIZE(ccs_modifiers); i++) {
> + data->ccs_modifier = ccs_modifiers[i];
> + valid_tests += test_ccs(data);
> + }
> +
> + igt_output_set_pipe(data->output, PIPE_NONE);
> igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
>
> return valid_tests;
> --
> 2.17.1
--
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] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too.
2019-03-20 19:37 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too Dhinakaran Pandiyan
@ 2019-03-20 19:51 ` Ville Syrjälä
0 siblings, 0 replies; 7+ messages in thread
From: Ville Syrjälä @ 2019-03-20 19:51 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
On Wed, Mar 20, 2019 at 12:37:33PM -0700, Dhinakaran Pandiyan wrote:
> Extend the test to Yf tiled CCS, no reason not to.
>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
> tests/kms_ccs.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
> index f780a01c..cd42e747 100644
> --- a/tests/kms_ccs.c
> +++ b/tests/kms_ccs.c
> @@ -71,7 +71,8 @@ static const struct {
> };
>
> static const uint64_t ccs_modifiers[] = {
> - LOCAL_I915_FORMAT_MOD_Y_TILED_CCS
> + LOCAL_I915_FORMAT_MOD_Y_TILED_CCS,
> + LOCAL_I915_FORMAT_MOD_Yf_TILED_CCS
Missing comma.
Otherwise
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> };
>
> /*
> --
> 2.17.1
--
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] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
2019-03-20 19:49 ` [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Ville Syrjälä
@ 2019-03-20 19:54 ` Dhinakaran Pandiyan
0 siblings, 0 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2019-03-20 19:54 UTC (permalink / raw)
To: Ville Syrjälä; +Cc: igt-dev
On Wed, 2019-03-20 at 21:49 +0200, Ville Syrjälä wrote:
> On Wed, Mar 20, 2019 at 12:37:32PM -0700, Dhinakaran Pandiyan wrote:
> > Currently only Y tiled CCS modifier is tested, rewrite the
> > functions so
> > that we can add support for other CCS modifiers.
> >
> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
> > ---
> > tests/kms_ccs.c | 52 +++++++++++++++++++++++++++++++++----------
> > ------
> > 1 file changed, 35 insertions(+), 17 deletions(-)
> >
> > diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
> > index a74bd706..f780a01c 100644
> > --- a/tests/kms_ccs.c
> > +++ b/tests/kms_ccs.c
> > @@ -58,6 +58,7 @@ typedef struct {
> > enum test_flags flags;
> > igt_plane_t *plane;
> > igt_pipe_crc_t *pipe_crc;
> > + uint64_t ccs_modifier;
> > } data_t;
> >
> > static const struct {
> > @@ -69,6 +70,10 @@ static const struct {
> > {0.0, 1.0, 0.0}
> > };
> >
> > +static const uint64_t ccs_modifiers[] = {
> > + LOCAL_I915_FORMAT_MOD_Y_TILED_CCS
> > +};
> > +
> > /*
> > * Limit maximum used sprite plane width so this test will not
> > mistakenly
> > * fail on hardware limitations which are not interesting to this
> > test.
> > @@ -140,7 +145,8 @@ modifiers_ptr(struct
> > local_drm_format_modifier_blob *blob)
> > return (struct local_drm_format_modifier *)(((char *)blob) +
> > blob->modifiers_offset);
> > }
> >
> > -static bool plane_has_format_with_ccs(data_t *data, igt_plane_t
> > *plane, uint32_t format)
> > +static bool plane_has_format_with_ccs(data_t *data, igt_plane_t
> > *plane,
> > + uint32_t format)
> > {
> > drmModePropertyBlobPtr blob;
> > struct local_drm_format_modifier_blob *blob_data;
>
> Hmm. I thought I changed this to use the igt_kms thing. I guess it
> never
> got pushed.
I'll be happy to review if you can find and send it to the list. Thanks
for the reviews.
-DK
>
> > @@ -178,7 +184,7 @@ static bool plane_has_format_with_ccs(data_t
> > *data, igt_plane_t *plane, uint32_t
> > last_mod = &modifiers[blob_data->count_modifiers];
> > igt_assert_lte(((char *) last_mod - (char *) blob_data), blob-
> > >length);
> > for (int i = 0; i < blob_data->count_modifiers; i++) {
> > - if (modifiers[i].modifier !=
> > LOCAL_I915_FORMAT_MOD_Y_TILED_CCS)
> > + if (modifiers[i].modifier != data->ccs_modifier)
> > continue;
> >
> > if (modifiers[i].offset > fmt_idx ||
> > @@ -226,7 +232,7 @@ static void generate_fb(data_t *data, struct
> > igt_fb *fb,
> > * available FIFO configurations.
> > */
> > if (fb_flags & FB_COMPRESSED)
> > - modifier = LOCAL_I915_FORMAT_MOD_Y_TILED_CCS;
> > + modifier = data->ccs_modifier;
> > else if (!(fb_flags & FB_HAS_PLANE))
> > modifier = LOCAL_I915_FORMAT_MOD_Y_TILED;
> > else
> > @@ -313,6 +319,7 @@ static bool try_config(data_t *data, enum
> > test_fb_flags fb_flags,
> > if (data->plane && fb_flags & FB_COMPRESSED) {
> > if (!plane_has_format_with_ccs(data, data->plane,
> > DRM_FORMAT_XRGB8888))
> > return false;
> > +
> > generate_fb(data, &fb, min(MAX_SPRITE_PLANE_WIDTH,
> > drm_mode->hdisplay),
> > drm_mode->vdisplay,
> > (fb_flags & ~FB_COMPRESSED) |
> > FB_HAS_PLANE);
> > @@ -369,21 +376,10 @@ static bool try_config(data_t *data, enum
> > test_fb_flags fb_flags,
> > return true;
> > }
> >
> > -static int test_output(data_t *data)
> > -{
> > - igt_display_t *display = &data->display;
> > +static int test_ccs(data_t *data)
> > +{ int valid_tests = 0;
> > igt_crc_t crc, ref_crc;
> > enum test_fb_flags fb_flags = 0;
> > - int valid_tests = 0;
> > -
> > - igt_display_require_output_on_pipe(display, data->pipe);
> > -
> > - /* Sets data->output with a valid output. */
> > - for_each_valid_output_on_pipe(display, data->pipe, data-
> > >output) {
> > - break;
> > - }
> > -
> > - igt_output_set_pipe(data->output, data->pipe);
> >
> > if (data->flags & TEST_CRC) {
> > data->pipe_crc = igt_pipe_crc_new(data->drm_fd, data-
> > >pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
> > @@ -411,7 +407,29 @@ static int test_output(data_t *data)
> > valid_tests += try_config(data, fb_flags |
> > FB_COMPRESSED | FB_ZERO_AUX_STRIDE , NULL);
> > }
> >
> > - igt_output_set_pipe(data->output, PIPE_ANY);
> > + return valid_tests;
> > +}
> > +
> > +static int test_output(data_t *data)
> > +{
> > + igt_display_t *display = &data->display;
> > + int i, valid_tests = 0;
> > +
> > + igt_display_require_output_on_pipe(display, data->pipe);
> > +
> > + /* Sets data->output with a valid output. */
> > + for_each_valid_output_on_pipe(display, data->pipe, data-
> > >output) {
> > + break;
> > + }
>
> I think we have a nicer way to do this these days.
>
> Patch is
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> > +
> > + igt_output_set_pipe(data->output, data->pipe);
> > +
> > + for (i = 0; i < ARRAY_SIZE(ccs_modifiers); i++) {
> > + data->ccs_modifier = ccs_modifiers[i];
> > + valid_tests += test_ccs(data);
> > + }
> > +
> > + igt_output_set_pipe(data->output, PIPE_NONE);
> > igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC
> > : COMMIT_LEGACY);
> >
> > return valid_tests;
> > --
> > 2.17.1
>
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
2019-03-20 19:37 [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Dhinakaran Pandiyan
2019-03-20 19:37 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too Dhinakaran Pandiyan
2019-03-20 19:49 ` [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Ville Syrjälä
@ 2019-03-20 20:15 ` Patchwork
2019-03-21 3:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-03-20 20:15 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
URL : https://patchwork.freedesktop.org/series/58288/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5783 -> IGTPW_2670
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/58288/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2670 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live_execlists:
- fi-apl-guc: PASS -> INCOMPLETE [fdo#103927] / [fdo#109720]
* igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence:
- fi-byt-clapper: PASS -> FAIL [fdo#103191] / [fdo#107362]
* igt@kms_psr@primary_mmap_gtt:
- fi-blb-e6850: NOTRUN -> SKIP [fdo#109271] +27
* igt@runner@aborted:
- fi-apl-guc: NOTRUN -> FAIL [fdo#108622] / [fdo#109720]
#### Possible fixes ####
* igt@i915_pm_rpm@basic-pci-d3-state:
- fi-hsw-4770: SKIP [fdo#109271] -> PASS +3
* igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-byt-clapper: FAIL [fdo#103191] / [fdo#107362] -> PASS
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-blb-e6850: INCOMPLETE [fdo#107718] -> PASS
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#108622]: https://bugs.freedesktop.org/show_bug.cgi?id=108622
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109720]: https://bugs.freedesktop.org/show_bug.cgi?id=109720
Participating hosts (49 -> 39)
------------------------------
Missing (10): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-icl-u3 fi-pnv-d510 fi-bdw-samus
Build changes
-------------
* IGT: IGT_4893 -> IGTPW_2670
CI_DRM_5783: f7a4ef1924a1ba05633514f4845378f51a183681 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2670: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2670/
IGT_4893: 27939a179fcd143e3a179ffc7b0372718259587a @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2670/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [i-g-t,1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
2019-03-20 19:37 [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Dhinakaran Pandiyan
` (2 preceding siblings ...)
2019-03-20 20:15 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] " Patchwork
@ 2019-03-21 3:07 ` Patchwork
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-03-21 3:07 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t,1/2] tests/kms_ccs: Refactor code to add CCS modifiers.
URL : https://patchwork.freedesktop.org/series/58288/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5783_full -> IGTPW_2670_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/58288/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_2670_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_create@create-clear:
- shard-snb: PASS -> INCOMPLETE [fdo#105411]
* igt@gem_ctx_isolation@vcs1-nonpriv:
- shard-glk: NOTRUN -> SKIP [fdo#109271] +18
* igt@gem_exec_schedule@preempt-queue-render:
- shard-hsw: NOTRUN -> SKIP [fdo#109271] +19
* igt@kms_atomic_transition@5x-modeset-transitions-nonblocking:
- shard-hsw: NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +1
- shard-glk: NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +1
* igt@kms_atomic_transition@plane-all-modeset-transition:
- shard-apl: PASS -> INCOMPLETE [fdo#103927]
* igt@kms_color@pipe-b-ctm-max:
- shard-apl: PASS -> FAIL [fdo#108147]
- shard-kbl: PASS -> FAIL [fdo#108147]
* igt@kms_cursor_crc@cursor-64x21-random:
- shard-apl: PASS -> FAIL [fdo#103232]
- shard-kbl: PASS -> FAIL [fdo#103232]
* igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
- shard-glk: PASS -> FAIL [fdo#104873]
* igt@kms_flip@flip-vs-suspend:
- shard-hsw: PASS -> INCOMPLETE [fdo#103540]
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
- shard-apl: PASS -> DMESG-WARN [fdo#108566]
* igt@kms_rotation_crc@multiplane-rotation-cropping-bottom:
- shard-kbl: PASS -> DMESG-FAIL [fdo#105763]
* igt@kms_setmode@basic:
- shard-kbl: PASS -> FAIL [fdo#99912]
* igt@kms_vblank@pipe-a-ts-continuation-dpms-rpm:
- shard-snb: NOTRUN -> SKIP [fdo#109271] / [fdo#109278] +11
* igt@perf_pmu@busy-check-all-vecs0:
- shard-snb: NOTRUN -> SKIP [fdo#109271] +73
#### Possible fixes ####
* igt@gem_eio@unwedge-stress:
- shard-snb: FAIL [fdo#109661] -> PASS
* igt@gem_pwrite@big-cpu-random:
- shard-glk: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS
* igt@kms_atomic_interruptible@legacy-cursor:
- shard-hsw: INCOMPLETE [fdo#103540] -> PASS
* igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-snb: DMESG-WARN [fdo#107956] -> PASS
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
- shard-kbl: DMESG-WARN [fdo#107956] -> PASS +1
- shard-hsw: DMESG-WARN [fdo#107956] -> PASS
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-kbl: DMESG-WARN [fdo#108566] -> PASS
* igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
- shard-glk: FAIL [fdo#108145] -> PASS
- shard-kbl: FAIL [fdo#108145] -> PASS
* igt@kms_rotation_crc@multiplane-rotation-cropping-bottom:
- shard-glk: DMESG-FAIL [fdo#105763] / [fdo#106538] -> PASS
* igt@kms_vblank@pipe-b-ts-continuation-suspend:
- shard-kbl: FAIL [fdo#104894] -> PASS
- shard-apl: FAIL [fdo#104894] -> PASS +1
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#104873]: https://bugs.freedesktop.org/show_bug.cgi?id=104873
[fdo#104894]: https://bugs.freedesktop.org/show_bug.cgi?id=104894
[fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#106538]: https://bugs.freedesktop.org/show_bug.cgi?id=106538
[fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (10 -> 5)
------------------------------
Missing (5): shard-skl pig-hsw-4770r pig-glk-j5005 shard-iclb pig-skl-6260u
Build changes
-------------
* IGT: IGT_4893 -> IGTPW_2670
* Piglit: piglit_4509 -> None
CI_DRM_5783: f7a4ef1924a1ba05633514f4845378f51a183681 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2670: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2670/
IGT_4893: 27939a179fcd143e3a179ffc7b0372718259587a @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2670/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-03-21 3:07 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-20 19:37 [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Dhinakaran Pandiyan
2019-03-20 19:37 ` [igt-dev] [PATCH i-g-t 2/2] tests/kms_ccs: Test Yf CCS modifier too Dhinakaran Pandiyan
2019-03-20 19:51 ` Ville Syrjälä
2019-03-20 19:49 ` [igt-dev] [PATCH i-g-t 1/2] tests/kms_ccs: Refactor code to add CCS modifiers Ville Syrjälä
2019-03-20 19:54 ` Dhinakaran Pandiyan
2019-03-20 20:15 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] " Patchwork
2019-03-21 3:07 ` [igt-dev] ✓ Fi.CI.IGT: " 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.