* [igt-dev] Add support for testing of 16 bpc fixed point framebuffers. @ 2021-05-03 18:25 Mario Kleiner 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Mario Kleiner ` (3 more replies) 0 siblings, 4 replies; 10+ messages in thread From: Mario Kleiner @ 2021-05-03 18:25 UTC (permalink / raw) To: igt-dev; +Cc: mario.kleiner.de This has been so far tested on AMD RaveRidge with DCN-1 display engine. It requires a Cairo 1.17.2 snapshot or later - tested with the current 1.17.4 snapshot. What was tested? kms_plane --run-subtest pixel-format-pipe-A-planes on current drm-next, with my 16 bpc framebuffer format patches for AMD DC applied. Thanks to Ville for pointers on what to change. Note that on my machine igt master needed additional hacks to make the kms_plane test sort of work at all on AMD, even without these new test case. Seems at least igt kms_plane is not quite up to date for current amdgpu-kms in recent kernels. E.g., it didn't get very far on Linux 5.8, but failed even faster on Linux 5.12 with the new DCC and DCC_RETILE modifiers exposed. Atm. only FMT_MOD_LINEAR seems to work on recent kernels. But this is a different topic... -mario _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. 2021-05-03 18:25 [igt-dev] Add support for testing of 16 bpc fixed point framebuffers Mario Kleiner @ 2021-05-03 18:25 ` Mario Kleiner 2021-05-03 18:59 ` Ville Syrjälä 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats Mario Kleiner ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: Mario Kleiner @ 2021-05-03 18:25 UTC (permalink / raw) To: igt-dev; +Cc: Alex Deucher, mario.kleiner.de These are 16 bits per color channel unsigned normalized formats. They are supported by at least AMD display hw, and suitable for direct scanout of Vulkan swapchain images in the format VK_FORMAT_R16G16B16A16_UNORM. Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Alex Deucher <alexander.deucher@amd.com> --- include/drm-uapi/drm_fourcc.h | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/include/drm-uapi/drm_fourcc.h b/include/drm-uapi/drm_fourcc.h index a7bc058c..5db4f55a 100644 --- a/include/drm-uapi/drm_fourcc.h +++ b/include/drm-uapi/drm_fourcc.h @@ -144,6 +144,13 @@ extern "C" { #define DRM_FORMAT_RGBA1010102 fourcc_code('R', 'A', '3', '0') /* [31:0] R:G:B:A 10:10:10:2 little endian */ #define DRM_FORMAT_BGRA1010102 fourcc_code('B', 'A', '3', '0') /* [31:0] B:G:R:A 10:10:10:2 little endian */ +/* 64 bpp RGB */ +#define DRM_FORMAT_XRGB16161616 fourcc_code('X', 'R', '4', '8') /* [63:0] x:R:G:B 16:16:16:16 little endian */ +#define DRM_FORMAT_XBGR16161616 fourcc_code('X', 'B', '4', '8') /* [63:0] x:B:G:R 16:16:16:16 little endian */ + +#define DRM_FORMAT_ARGB16161616 fourcc_code('A', 'R', '4', '8') /* [63:0] A:R:G:B 16:16:16:16 little endian */ +#define DRM_FORMAT_ABGR16161616 fourcc_code('A', 'B', '4', '8') /* [63:0] A:B:G:R 16:16:16:16 little endian */ + /* * Floating point 64bpp RGB * IEEE 754-2008 binary16 half-precision float -- 2.25.1 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Mario Kleiner @ 2021-05-03 18:59 ` Ville Syrjälä 0 siblings, 0 replies; 10+ messages in thread From: Ville Syrjälä @ 2021-05-03 18:59 UTC (permalink / raw) To: Mario Kleiner; +Cc: igt-dev, Alex Deucher On Mon, May 03, 2021 at 08:25:54PM +0200, Mario Kleiner wrote: > These are 16 bits per color channel unsigned normalized formats. > They are supported by at least AMD display hw, and suitable for > direct scanout of Vulkan swapchain images in the format > VK_FORMAT_R16G16B16A16_UNORM. I think we've been trying to just copy the whole thing from the kernel and include a reference to the relevant kernel commit. So need to land the kernel drm_fourcc.h bits first before we land these. > > Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Alex Deucher <alexander.deucher@amd.com> > --- > include/drm-uapi/drm_fourcc.h | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/include/drm-uapi/drm_fourcc.h b/include/drm-uapi/drm_fourcc.h > index a7bc058c..5db4f55a 100644 > --- a/include/drm-uapi/drm_fourcc.h > +++ b/include/drm-uapi/drm_fourcc.h > @@ -144,6 +144,13 @@ extern "C" { > #define DRM_FORMAT_RGBA1010102 fourcc_code('R', 'A', '3', '0') /* [31:0] R:G:B:A 10:10:10:2 little endian */ > #define DRM_FORMAT_BGRA1010102 fourcc_code('B', 'A', '3', '0') /* [31:0] B:G:R:A 10:10:10:2 little endian */ > > +/* 64 bpp RGB */ > +#define DRM_FORMAT_XRGB16161616 fourcc_code('X', 'R', '4', '8') /* [63:0] x:R:G:B 16:16:16:16 little endian */ > +#define DRM_FORMAT_XBGR16161616 fourcc_code('X', 'B', '4', '8') /* [63:0] x:B:G:R 16:16:16:16 little endian */ > + > +#define DRM_FORMAT_ARGB16161616 fourcc_code('A', 'R', '4', '8') /* [63:0] A:R:G:B 16:16:16:16 little endian */ > +#define DRM_FORMAT_ABGR16161616 fourcc_code('A', 'B', '4', '8') /* [63:0] A:B:G:R 16:16:16:16 little endian */ > + > /* > * Floating point 64bpp RGB > * IEEE 754-2008 binary16 half-precision float > -- > 2.25.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] 10+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats. 2021-05-03 18:25 [igt-dev] Add support for testing of 16 bpc fixed point framebuffers Mario Kleiner 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Mario Kleiner @ 2021-05-03 18:25 ` Mario Kleiner 2021-05-03 18:47 ` Ville Syrjälä 2021-05-03 19:19 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Patchwork 2021-05-04 0:07 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 3 siblings, 1 reply; 10+ messages in thread From: Mario Kleiner @ 2021-05-03 18:25 UTC (permalink / raw) To: igt-dev; +Cc: Alex Deucher, mario.kleiner.de This is used to support testing the 16 bpc formats, e.g., via: kms_plane --run-subtest pixel-format-pipe-A-planes So far this was successfully tested on AMD RavenRidge with DCN-1 display hw. The new conversion routines are slightly adapted copies of the convert_float_to_fp16() and convert_fp16_to_float() functions, with the conversion math modified for float <-> uint16 instead. Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Cc: Alex Deucher <alexander.deucher@amd.com> --- lib/igt_fb.c | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 123 insertions(+) diff --git a/lib/igt_fb.c b/lib/igt_fb.c index 954e1181..148e6c0f 100644 --- a/lib/igt_fb.c +++ b/lib/igt_fb.c @@ -220,6 +220,22 @@ static const struct format_desc_struct { .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, .num_planes = 1, .plane_bpp = { 64, }, }, + { .name = "XRGB16161616", .depth = -1, .drm_id = DRM_FORMAT_XRGB16161616, + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, + .num_planes = 1, .plane_bpp = { 64, }, + }, + { .name = "ARGB16161616", .depth = -1, .drm_id = DRM_FORMAT_ARGB16161616, + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, + .num_planes = 1, .plane_bpp = { 64, }, + }, + { .name = "XBGR16161616", .depth = -1, .drm_id = DRM_FORMAT_XBGR16161616, + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, + .num_planes = 1, .plane_bpp = { 64, }, + }, + { .name = "ABGR16161616", .depth = -1, .drm_id = DRM_FORMAT_ABGR16161616, + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, + .num_planes = 1, .plane_bpp = { 64, }, + }, { .name = "NV12", .depth = -1, .drm_id = DRM_FORMAT_NV12, .cairo_id = CAIRO_FORMAT_RGB24, .convert = true, .num_planes = 2, .plane_bpp = { 8, 16, }, @@ -3365,9 +3381,13 @@ static const unsigned char *rgbx_swizzle(uint32_t format) default: case DRM_FORMAT_XRGB16161616F: case DRM_FORMAT_ARGB16161616F: + case DRM_FORMAT_XRGB16161616: + case DRM_FORMAT_ARGB16161616: return swizzle_bgrx; case DRM_FORMAT_XBGR16161616F: case DRM_FORMAT_ABGR16161616F: + case DRM_FORMAT_XBGR16161616: + case DRM_FORMAT_ABGR16161616: return swizzle_rgbx; } } @@ -3451,6 +3471,97 @@ static void convert_float_to_fp16(struct fb_convert *cvt) } } +static void float_to_uint16(const float *f, uint16_t *h, unsigned int num) +{ + for (int i = 0; i < num; i++) + h[i] = f[i] * 65535.0f + 0.5f; +} + +static void uint16_to_float(const uint16_t *h, float *f, unsigned int num) +{ + for (int i = 0; i < num; i++) + f[i] = ((float) h[i]) / 65535.0f; +} + +static void convert_uint16_to_float(struct fb_convert *cvt) +{ + int i, j; + uint16_t *up16; + float *ptr = cvt->dst.ptr; + unsigned int float_stride = cvt->dst.fb->strides[0] / sizeof(*ptr); + unsigned int up16_stride = cvt->src.fb->strides[0] / sizeof(*up16); + const unsigned char *swz = rgbx_swizzle(cvt->src.fb->drm_format); + bool needs_reswizzle = swz != swizzle_rgbx; + + uint16_t *buf = convert_src_get(cvt); + up16 = buf + cvt->src.fb->offsets[0] / sizeof(*buf); + + for (i = 0; i < cvt->dst.fb->height; i++) { + if (needs_reswizzle) { + const uint16_t *u16_tmp = up16; + float *rgb_tmp = ptr; + + for (j = 0; j < cvt->dst.fb->width; j++) { + struct igt_vec4 rgb; + + uint16_to_float(u16_tmp, rgb.d, 4); + + rgb_tmp[0] = rgb.d[swz[0]]; + rgb_tmp[1] = rgb.d[swz[1]]; + rgb_tmp[2] = rgb.d[swz[2]]; + rgb_tmp[3] = rgb.d[swz[3]]; + + rgb_tmp += 4; + u16_tmp += 4; + } + } else { + uint16_to_float(up16, ptr, cvt->dst.fb->width * 4); + } + + ptr += float_stride; + up16 += up16_stride; + } + + convert_src_put(cvt, buf); +} + +static void convert_float_to_uint16(struct fb_convert *cvt) +{ + int i, j; + uint16_t *up16 = cvt->dst.ptr + cvt->dst.fb->offsets[0]; + const float *ptr = cvt->src.ptr; + unsigned float_stride = cvt->src.fb->strides[0] / sizeof(*ptr); + unsigned up16_stride = cvt->dst.fb->strides[0] / sizeof(*up16); + const unsigned char *swz = rgbx_swizzle(cvt->dst.fb->drm_format); + bool needs_reswizzle = swz != swizzle_rgbx; + + for (i = 0; i < cvt->dst.fb->height; i++) { + if (needs_reswizzle) { + const float *rgb_tmp = ptr; + uint16_t *u16_tmp = up16; + + for (j = 0; j < cvt->dst.fb->width; j++) { + struct igt_vec4 rgb; + + rgb.d[0] = rgb_tmp[swz[0]]; + rgb.d[1] = rgb_tmp[swz[1]]; + rgb.d[2] = rgb_tmp[swz[2]]; + rgb.d[3] = rgb_tmp[swz[3]]; + + float_to_uint16(rgb.d, u16_tmp, 4); + + rgb_tmp += 4; + u16_tmp += 4; + } + } else { + float_to_uint16(ptr, up16, cvt->dst.fb->width * 4); + } + + ptr += float_stride; + up16 += up16_stride; + } +} + static void convert_pixman(struct fb_convert *cvt) { pixman_format_code_t src_pixman = drm_format_to_pixman(cvt->src.fb->drm_format); @@ -3560,6 +3671,12 @@ static void fb_convert(struct fb_convert *cvt) case DRM_FORMAT_ABGR16161616F: convert_fp16_to_float(cvt); return; + case DRM_FORMAT_XRGB16161616: + case DRM_FORMAT_XBGR16161616: + case DRM_FORMAT_ARGB16161616: + case DRM_FORMAT_ABGR16161616: + convert_uint16_to_float(cvt); + return; } } else if (cvt->src.fb->drm_format == IGT_FORMAT_FLOAT) { switch (cvt->dst.fb->drm_format) { @@ -3589,6 +3706,12 @@ static void fb_convert(struct fb_convert *cvt) case DRM_FORMAT_ABGR16161616F: convert_float_to_fp16(cvt); return; + case DRM_FORMAT_XRGB16161616: + case DRM_FORMAT_XBGR16161616: + case DRM_FORMAT_ARGB16161616: + case DRM_FORMAT_ABGR16161616: + convert_float_to_uint16(cvt); + return; } } -- 2.25.1 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats. 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats Mario Kleiner @ 2021-05-03 18:47 ` Ville Syrjälä 2021-05-03 19:32 ` Mario Kleiner 0 siblings, 1 reply; 10+ messages in thread From: Ville Syrjälä @ 2021-05-03 18:47 UTC (permalink / raw) To: Mario Kleiner; +Cc: igt-dev, Alex Deucher On Mon, May 03, 2021 at 08:25:55PM +0200, Mario Kleiner wrote: > This is used to support testing the 16 bpc formats, e.g., via: > > kms_plane --run-subtest pixel-format-pipe-A-planes > > So far this was successfully tested on AMD RavenRidge with DCN-1 > display hw. > > The new conversion routines are slightly adapted copies of the > convert_float_to_fp16() and convert_fp16_to_float() functions, > with the conversion math modified for float <-> uint16 instead. > > Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Alex Deucher <alexander.deucher@amd.com> > --- > lib/igt_fb.c | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 123 insertions(+) > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > index 954e1181..148e6c0f 100644 > --- a/lib/igt_fb.c > +++ b/lib/igt_fb.c > @@ -220,6 +220,22 @@ static const struct format_desc_struct { > .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > .num_planes = 1, .plane_bpp = { 64, }, > }, > + { .name = "XRGB16161616", .depth = -1, .drm_id = DRM_FORMAT_XRGB16161616, > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > + .num_planes = 1, .plane_bpp = { 64, }, > + }, > + { .name = "ARGB16161616", .depth = -1, .drm_id = DRM_FORMAT_ARGB16161616, > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > + .num_planes = 1, .plane_bpp = { 64, }, > + }, > + { .name = "XBGR16161616", .depth = -1, .drm_id = DRM_FORMAT_XBGR16161616, > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > + .num_planes = 1, .plane_bpp = { 64, }, > + }, > + { .name = "ABGR16161616", .depth = -1, .drm_id = DRM_FORMAT_ABGR16161616, > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > + .num_planes = 1, .plane_bpp = { 64, }, > + }, > { .name = "NV12", .depth = -1, .drm_id = DRM_FORMAT_NV12, > .cairo_id = CAIRO_FORMAT_RGB24, .convert = true, > .num_planes = 2, .plane_bpp = { 8, 16, }, > @@ -3365,9 +3381,13 @@ static const unsigned char *rgbx_swizzle(uint32_t format) > default: > case DRM_FORMAT_XRGB16161616F: > case DRM_FORMAT_ARGB16161616F: > + case DRM_FORMAT_XRGB16161616: > + case DRM_FORMAT_ARGB16161616: > return swizzle_bgrx; > case DRM_FORMAT_XBGR16161616F: > case DRM_FORMAT_ABGR16161616F: > + case DRM_FORMAT_XBGR16161616: > + case DRM_FORMAT_ABGR16161616: > return swizzle_rgbx; > } > } > @@ -3451,6 +3471,97 @@ static void convert_float_to_fp16(struct fb_convert *cvt) > } > } > > +static void float_to_uint16(const float *f, uint16_t *h, unsigned int num) > +{ > + for (int i = 0; i < num; i++) > + h[i] = f[i] * 65535.0f + 0.5f; > +} > + > +static void uint16_to_float(const uint16_t *h, float *f, unsigned int num) > +{ > + for (int i = 0; i < num; i++) > + f[i] = ((float) h[i]) / 65535.0f; nit: the cast shouldn't be necessary. Looks good otherwise. Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > +} > + > +static void convert_uint16_to_float(struct fb_convert *cvt) > +{ > + int i, j; > + uint16_t *up16; > + float *ptr = cvt->dst.ptr; > + unsigned int float_stride = cvt->dst.fb->strides[0] / sizeof(*ptr); > + unsigned int up16_stride = cvt->src.fb->strides[0] / sizeof(*up16); > + const unsigned char *swz = rgbx_swizzle(cvt->src.fb->drm_format); > + bool needs_reswizzle = swz != swizzle_rgbx; > + > + uint16_t *buf = convert_src_get(cvt); > + up16 = buf + cvt->src.fb->offsets[0] / sizeof(*buf); > + > + for (i = 0; i < cvt->dst.fb->height; i++) { > + if (needs_reswizzle) { > + const uint16_t *u16_tmp = up16; > + float *rgb_tmp = ptr; > + > + for (j = 0; j < cvt->dst.fb->width; j++) { > + struct igt_vec4 rgb; > + > + uint16_to_float(u16_tmp, rgb.d, 4); > + > + rgb_tmp[0] = rgb.d[swz[0]]; > + rgb_tmp[1] = rgb.d[swz[1]]; > + rgb_tmp[2] = rgb.d[swz[2]]; > + rgb_tmp[3] = rgb.d[swz[3]]; > + > + rgb_tmp += 4; > + u16_tmp += 4; > + } > + } else { > + uint16_to_float(up16, ptr, cvt->dst.fb->width * 4); > + } > + > + ptr += float_stride; > + up16 += up16_stride; > + } > + > + convert_src_put(cvt, buf); > +} > + > +static void convert_float_to_uint16(struct fb_convert *cvt) > +{ > + int i, j; > + uint16_t *up16 = cvt->dst.ptr + cvt->dst.fb->offsets[0]; > + const float *ptr = cvt->src.ptr; > + unsigned float_stride = cvt->src.fb->strides[0] / sizeof(*ptr); > + unsigned up16_stride = cvt->dst.fb->strides[0] / sizeof(*up16); > + const unsigned char *swz = rgbx_swizzle(cvt->dst.fb->drm_format); > + bool needs_reswizzle = swz != swizzle_rgbx; > + > + for (i = 0; i < cvt->dst.fb->height; i++) { > + if (needs_reswizzle) { > + const float *rgb_tmp = ptr; > + uint16_t *u16_tmp = up16; > + > + for (j = 0; j < cvt->dst.fb->width; j++) { > + struct igt_vec4 rgb; > + > + rgb.d[0] = rgb_tmp[swz[0]]; > + rgb.d[1] = rgb_tmp[swz[1]]; > + rgb.d[2] = rgb_tmp[swz[2]]; > + rgb.d[3] = rgb_tmp[swz[3]]; > + > + float_to_uint16(rgb.d, u16_tmp, 4); > + > + rgb_tmp += 4; > + u16_tmp += 4; > + } > + } else { > + float_to_uint16(ptr, up16, cvt->dst.fb->width * 4); > + } > + > + ptr += float_stride; > + up16 += up16_stride; > + } > +} > + > static void convert_pixman(struct fb_convert *cvt) > { > pixman_format_code_t src_pixman = drm_format_to_pixman(cvt->src.fb->drm_format); > @@ -3560,6 +3671,12 @@ static void fb_convert(struct fb_convert *cvt) > case DRM_FORMAT_ABGR16161616F: > convert_fp16_to_float(cvt); > return; > + case DRM_FORMAT_XRGB16161616: > + case DRM_FORMAT_XBGR16161616: > + case DRM_FORMAT_ARGB16161616: > + case DRM_FORMAT_ABGR16161616: > + convert_uint16_to_float(cvt); > + return; > } > } else if (cvt->src.fb->drm_format == IGT_FORMAT_FLOAT) { > switch (cvt->dst.fb->drm_format) { > @@ -3589,6 +3706,12 @@ static void fb_convert(struct fb_convert *cvt) > case DRM_FORMAT_ABGR16161616F: > convert_float_to_fp16(cvt); > return; > + case DRM_FORMAT_XRGB16161616: > + case DRM_FORMAT_XBGR16161616: > + case DRM_FORMAT_ARGB16161616: > + case DRM_FORMAT_ABGR16161616: > + convert_float_to_uint16(cvt); > + return; > } > } > > -- > 2.25.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] 10+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats. 2021-05-03 18:47 ` Ville Syrjälä @ 2021-05-03 19:32 ` Mario Kleiner 2021-05-03 19:43 ` Ville Syrjälä 0 siblings, 1 reply; 10+ messages in thread From: Mario Kleiner @ 2021-05-03 19:32 UTC (permalink / raw) To: Ville Syrjälä; +Cc: igt-dev, Alex Deucher On Mon, May 3, 2021 at 8:48 PM Ville Syrjälä <ville.syrjala@linux.intel.com> wrote: > > On Mon, May 03, 2021 at 08:25:55PM +0200, Mario Kleiner wrote: > > This is used to support testing the 16 bpc formats, e.g., via: > > > > kms_plane --run-subtest pixel-format-pipe-A-planes > > > > So far this was successfully tested on AMD RavenRidge with DCN-1 > > display hw. > > > > The new conversion routines are slightly adapted copies of the > > convert_float_to_fp16() and convert_fp16_to_float() functions, > > with the conversion math modified for float <-> uint16 instead. > > > > Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> > > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Cc: Alex Deucher <alexander.deucher@amd.com> > > --- > > lib/igt_fb.c | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 123 insertions(+) > > > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > > index 954e1181..148e6c0f 100644 > > --- a/lib/igt_fb.c > > +++ b/lib/igt_fb.c > > @@ -220,6 +220,22 @@ static const struct format_desc_struct { > > .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > .num_planes = 1, .plane_bpp = { 64, }, > > }, > > + { .name = "XRGB16161616", .depth = -1, .drm_id = DRM_FORMAT_XRGB16161616, > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > + .num_planes = 1, .plane_bpp = { 64, }, > > + }, > > + { .name = "ARGB16161616", .depth = -1, .drm_id = DRM_FORMAT_ARGB16161616, > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > + .num_planes = 1, .plane_bpp = { 64, }, > > + }, > > + { .name = "XBGR16161616", .depth = -1, .drm_id = DRM_FORMAT_XBGR16161616, > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > + .num_planes = 1, .plane_bpp = { 64, }, > > + }, > > + { .name = "ABGR16161616", .depth = -1, .drm_id = DRM_FORMAT_ABGR16161616, > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > + .num_planes = 1, .plane_bpp = { 64, }, > > + }, > > { .name = "NV12", .depth = -1, .drm_id = DRM_FORMAT_NV12, > > .cairo_id = CAIRO_FORMAT_RGB24, .convert = true, > > .num_planes = 2, .plane_bpp = { 8, 16, }, > > @@ -3365,9 +3381,13 @@ static const unsigned char *rgbx_swizzle(uint32_t format) > > default: > > case DRM_FORMAT_XRGB16161616F: > > case DRM_FORMAT_ARGB16161616F: > > + case DRM_FORMAT_XRGB16161616: > > + case DRM_FORMAT_ARGB16161616: > > return swizzle_bgrx; > > case DRM_FORMAT_XBGR16161616F: > > case DRM_FORMAT_ABGR16161616F: > > + case DRM_FORMAT_XBGR16161616: > > + case DRM_FORMAT_ABGR16161616: > > return swizzle_rgbx; > > } > > } > > @@ -3451,6 +3471,97 @@ static void convert_float_to_fp16(struct fb_convert *cvt) > > } > > } > > > > +static void float_to_uint16(const float *f, uint16_t *h, unsigned int num) > > +{ > > + for (int i = 0; i < num; i++) > > + h[i] = f[i] * 65535.0f + 0.5f; > > +} > > + > > +static void uint16_to_float(const uint16_t *h, float *f, unsigned int num) > > +{ > > + for (int i = 0; i < num; i++) > > + f[i] = ((float) h[i]) / 65535.0f; > > nit: the cast shouldn't be necessary. > > Looks good otherwise. > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > Thanks for the quick review. Compiler and test results agree with you :). However, all other uintsomething_to_float() conversion routines, e.g., for the yuv formats and for fp16_to_float() all use that apparently redundant cast, which is why I left it for consistency with the rest of the code. And in case those wiser people who wrote those bits may know something that i don't see? Should I change the patch to drop the cast, and resend? -mario > > +} > > + > > +static void convert_uint16_to_float(struct fb_convert *cvt) > > +{ > > + int i, j; > > + uint16_t *up16; > > + float *ptr = cvt->dst.ptr; > > + unsigned int float_stride = cvt->dst.fb->strides[0] / sizeof(*ptr); > > + unsigned int up16_stride = cvt->src.fb->strides[0] / sizeof(*up16); > > + const unsigned char *swz = rgbx_swizzle(cvt->src.fb->drm_format); > > + bool needs_reswizzle = swz != swizzle_rgbx; > > + > > + uint16_t *buf = convert_src_get(cvt); > > + up16 = buf + cvt->src.fb->offsets[0] / sizeof(*buf); > > + > > + for (i = 0; i < cvt->dst.fb->height; i++) { > > + if (needs_reswizzle) { > > + const uint16_t *u16_tmp = up16; > > + float *rgb_tmp = ptr; > > + > > + for (j = 0; j < cvt->dst.fb->width; j++) { > > + struct igt_vec4 rgb; > > + > > + uint16_to_float(u16_tmp, rgb.d, 4); > > + > > + rgb_tmp[0] = rgb.d[swz[0]]; > > + rgb_tmp[1] = rgb.d[swz[1]]; > > + rgb_tmp[2] = rgb.d[swz[2]]; > > + rgb_tmp[3] = rgb.d[swz[3]]; > > + > > + rgb_tmp += 4; > > + u16_tmp += 4; > > + } > > + } else { > > + uint16_to_float(up16, ptr, cvt->dst.fb->width * 4); > > + } > > + > > + ptr += float_stride; > > + up16 += up16_stride; > > + } > > + > > + convert_src_put(cvt, buf); > > +} > > + > > +static void convert_float_to_uint16(struct fb_convert *cvt) > > +{ > > + int i, j; > > + uint16_t *up16 = cvt->dst.ptr + cvt->dst.fb->offsets[0]; > > + const float *ptr = cvt->src.ptr; > > + unsigned float_stride = cvt->src.fb->strides[0] / sizeof(*ptr); > > + unsigned up16_stride = cvt->dst.fb->strides[0] / sizeof(*up16); > > + const unsigned char *swz = rgbx_swizzle(cvt->dst.fb->drm_format); > > + bool needs_reswizzle = swz != swizzle_rgbx; > > + > > + for (i = 0; i < cvt->dst.fb->height; i++) { > > + if (needs_reswizzle) { > > + const float *rgb_tmp = ptr; > > + uint16_t *u16_tmp = up16; > > + > > + for (j = 0; j < cvt->dst.fb->width; j++) { > > + struct igt_vec4 rgb; > > + > > + rgb.d[0] = rgb_tmp[swz[0]]; > > + rgb.d[1] = rgb_tmp[swz[1]]; > > + rgb.d[2] = rgb_tmp[swz[2]]; > > + rgb.d[3] = rgb_tmp[swz[3]]; > > + > > + float_to_uint16(rgb.d, u16_tmp, 4); > > + > > + rgb_tmp += 4; > > + u16_tmp += 4; > > + } > > + } else { > > + float_to_uint16(ptr, up16, cvt->dst.fb->width * 4); > > + } > > + > > + ptr += float_stride; > > + up16 += up16_stride; > > + } > > +} > > + > > static void convert_pixman(struct fb_convert *cvt) > > { > > pixman_format_code_t src_pixman = drm_format_to_pixman(cvt->src.fb->drm_format); > > @@ -3560,6 +3671,12 @@ static void fb_convert(struct fb_convert *cvt) > > case DRM_FORMAT_ABGR16161616F: > > convert_fp16_to_float(cvt); > > return; > > + case DRM_FORMAT_XRGB16161616: > > + case DRM_FORMAT_XBGR16161616: > > + case DRM_FORMAT_ARGB16161616: > > + case DRM_FORMAT_ABGR16161616: > > + convert_uint16_to_float(cvt); > > + return; > > } > > } else if (cvt->src.fb->drm_format == IGT_FORMAT_FLOAT) { > > switch (cvt->dst.fb->drm_format) { > > @@ -3589,6 +3706,12 @@ static void fb_convert(struct fb_convert *cvt) > > case DRM_FORMAT_ABGR16161616F: > > convert_float_to_fp16(cvt); > > return; > > + case DRM_FORMAT_XRGB16161616: > > + case DRM_FORMAT_XBGR16161616: > > + case DRM_FORMAT_ARGB16161616: > > + case DRM_FORMAT_ABGR16161616: > > + convert_float_to_uint16(cvt); > > + return; > > } > > } > > > > -- > > 2.25.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] 10+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats. 2021-05-03 19:32 ` Mario Kleiner @ 2021-05-03 19:43 ` Ville Syrjälä 2021-05-03 19:47 ` Mario Kleiner 0 siblings, 1 reply; 10+ messages in thread From: Ville Syrjälä @ 2021-05-03 19:43 UTC (permalink / raw) To: Mario Kleiner; +Cc: igt-dev, Alex Deucher On Mon, May 03, 2021 at 09:32:19PM +0200, Mario Kleiner wrote: > On Mon, May 3, 2021 at 8:48 PM Ville Syrjälä > <ville.syrjala@linux.intel.com> wrote: > > > > On Mon, May 03, 2021 at 08:25:55PM +0200, Mario Kleiner wrote: > > > This is used to support testing the 16 bpc formats, e.g., via: > > > > > > kms_plane --run-subtest pixel-format-pipe-A-planes > > > > > > So far this was successfully tested on AMD RavenRidge with DCN-1 > > > display hw. > > > > > > The new conversion routines are slightly adapted copies of the > > > convert_float_to_fp16() and convert_fp16_to_float() functions, > > > with the conversion math modified for float <-> uint16 instead. > > > > > > Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> > > > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Cc: Alex Deucher <alexander.deucher@amd.com> > > > --- > > > lib/igt_fb.c | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++ > > > 1 file changed, 123 insertions(+) > > > > > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > > > index 954e1181..148e6c0f 100644 > > > --- a/lib/igt_fb.c > > > +++ b/lib/igt_fb.c > > > @@ -220,6 +220,22 @@ static const struct format_desc_struct { > > > .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > .num_planes = 1, .plane_bpp = { 64, }, > > > }, > > > + { .name = "XRGB16161616", .depth = -1, .drm_id = DRM_FORMAT_XRGB16161616, > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > + }, > > > + { .name = "ARGB16161616", .depth = -1, .drm_id = DRM_FORMAT_ARGB16161616, > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > + }, > > > + { .name = "XBGR16161616", .depth = -1, .drm_id = DRM_FORMAT_XBGR16161616, > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > + }, > > > + { .name = "ABGR16161616", .depth = -1, .drm_id = DRM_FORMAT_ABGR16161616, > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > + }, > > > { .name = "NV12", .depth = -1, .drm_id = DRM_FORMAT_NV12, > > > .cairo_id = CAIRO_FORMAT_RGB24, .convert = true, > > > .num_planes = 2, .plane_bpp = { 8, 16, }, > > > @@ -3365,9 +3381,13 @@ static const unsigned char *rgbx_swizzle(uint32_t format) > > > default: > > > case DRM_FORMAT_XRGB16161616F: > > > case DRM_FORMAT_ARGB16161616F: > > > + case DRM_FORMAT_XRGB16161616: > > > + case DRM_FORMAT_ARGB16161616: > > > return swizzle_bgrx; > > > case DRM_FORMAT_XBGR16161616F: > > > case DRM_FORMAT_ABGR16161616F: > > > + case DRM_FORMAT_XBGR16161616: > > > + case DRM_FORMAT_ABGR16161616: > > > return swizzle_rgbx; > > > } > > > } > > > @@ -3451,6 +3471,97 @@ static void convert_float_to_fp16(struct fb_convert *cvt) > > > } > > > } > > > > > > +static void float_to_uint16(const float *f, uint16_t *h, unsigned int num) > > > +{ > > > + for (int i = 0; i < num; i++) > > > + h[i] = f[i] * 65535.0f + 0.5f; > > > +} > > > + > > > +static void uint16_to_float(const uint16_t *h, float *f, unsigned int num) > > > +{ > > > + for (int i = 0; i < num; i++) > > > + f[i] = ((float) h[i]) / 65535.0f; > > > > nit: the cast shouldn't be necessary. > > > > Looks good otherwise. > > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > Thanks for the quick review. Compiler and test results agree with you :). > > However, all other uintsomething_to_float() conversion routines, e.g., > for the yuv formats and for fp16_to_float() all use that apparently > redundant cast, which is why I left it for consistency with the rest > of the code. And in case those wiser people who wrote those bits may > know something that i don't see? > > Should I change the patch to drop the cast, and resend? Maybe not be worth the hassl esp. if there are redundant casts all over anyway. We could do a global pass later to clear them all up, or just squint and pretend they're not there :P -- 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] 10+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats. 2021-05-03 19:43 ` Ville Syrjälä @ 2021-05-03 19:47 ` Mario Kleiner 0 siblings, 0 replies; 10+ messages in thread From: Mario Kleiner @ 2021-05-03 19:47 UTC (permalink / raw) To: Ville Syrjälä; +Cc: igt-dev, Alex Deucher On Mon, May 3, 2021 at 9:43 PM Ville Syrjälä <ville.syrjala@linux.intel.com> wrote: > > On Mon, May 03, 2021 at 09:32:19PM +0200, Mario Kleiner wrote: > > On Mon, May 3, 2021 at 8:48 PM Ville Syrjälä > > <ville.syrjala@linux.intel.com> wrote: > > > > > > On Mon, May 03, 2021 at 08:25:55PM +0200, Mario Kleiner wrote: > > > > This is used to support testing the 16 bpc formats, e.g., via: > > > > > > > > kms_plane --run-subtest pixel-format-pipe-A-planes > > > > > > > > So far this was successfully tested on AMD RavenRidge with DCN-1 > > > > display hw. > > > > > > > > The new conversion routines are slightly adapted copies of the > > > > convert_float_to_fp16() and convert_fp16_to_float() functions, > > > > with the conversion math modified for float <-> uint16 instead. > > > > > > > > Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com> > > > > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > Cc: Alex Deucher <alexander.deucher@amd.com> > > > > --- > > > > lib/igt_fb.c | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++ > > > > 1 file changed, 123 insertions(+) > > > > > > > > diff --git a/lib/igt_fb.c b/lib/igt_fb.c > > > > index 954e1181..148e6c0f 100644 > > > > --- a/lib/igt_fb.c > > > > +++ b/lib/igt_fb.c > > > > @@ -220,6 +220,22 @@ static const struct format_desc_struct { > > > > .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > > .num_planes = 1, .plane_bpp = { 64, }, > > > > }, > > > > + { .name = "XRGB16161616", .depth = -1, .drm_id = DRM_FORMAT_XRGB16161616, > > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > > + }, > > > > + { .name = "ARGB16161616", .depth = -1, .drm_id = DRM_FORMAT_ARGB16161616, > > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > > + }, > > > > + { .name = "XBGR16161616", .depth = -1, .drm_id = DRM_FORMAT_XBGR16161616, > > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > > + }, > > > > + { .name = "ABGR16161616", .depth = -1, .drm_id = DRM_FORMAT_ABGR16161616, > > > > + .cairo_id = CAIRO_FORMAT_RGBA128F, .convert = true, > > > > + .num_planes = 1, .plane_bpp = { 64, }, > > > > + }, > > > > { .name = "NV12", .depth = -1, .drm_id = DRM_FORMAT_NV12, > > > > .cairo_id = CAIRO_FORMAT_RGB24, .convert = true, > > > > .num_planes = 2, .plane_bpp = { 8, 16, }, > > > > @@ -3365,9 +3381,13 @@ static const unsigned char *rgbx_swizzle(uint32_t format) > > > > default: > > > > case DRM_FORMAT_XRGB16161616F: > > > > case DRM_FORMAT_ARGB16161616F: > > > > + case DRM_FORMAT_XRGB16161616: > > > > + case DRM_FORMAT_ARGB16161616: > > > > return swizzle_bgrx; > > > > case DRM_FORMAT_XBGR16161616F: > > > > case DRM_FORMAT_ABGR16161616F: > > > > + case DRM_FORMAT_XBGR16161616: > > > > + case DRM_FORMAT_ABGR16161616: > > > > return swizzle_rgbx; > > > > } > > > > } > > > > @@ -3451,6 +3471,97 @@ static void convert_float_to_fp16(struct fb_convert *cvt) > > > > } > > > > } > > > > > > > > +static void float_to_uint16(const float *f, uint16_t *h, unsigned int num) > > > > +{ > > > > + for (int i = 0; i < num; i++) > > > > + h[i] = f[i] * 65535.0f + 0.5f; > > > > +} > > > > + > > > > +static void uint16_to_float(const uint16_t *h, float *f, unsigned int num) > > > > +{ > > > > + for (int i = 0; i < num; i++) > > > > + f[i] = ((float) h[i]) / 65535.0f; > > > > > > nit: the cast shouldn't be necessary. > > > > > > Looks good otherwise. > > > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > > Thanks for the quick review. Compiler and test results agree with you :). > > > > However, all other uintsomething_to_float() conversion routines, e.g., > > for the yuv formats and for fp16_to_float() all use that apparently > > redundant cast, which is why I left it for consistency with the rest > > of the code. And in case those wiser people who wrote those bits may > > know something that i don't see? > > > > Should I change the patch to drop the cast, and resend? > > Maybe not be worth the hassl esp. if there are redundant casts all over > anyway. We could do a global pass later to clear them all up, or just > squint and pretend they're not there :P > > -- > Ville Syrjälä > Intel Ok, then i'll leave it as is. -mario _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. 2021-05-03 18:25 [igt-dev] Add support for testing of 16 bpc fixed point framebuffers Mario Kleiner 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Mario Kleiner 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats Mario Kleiner @ 2021-05-03 19:19 ` Patchwork 2021-05-04 0:07 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 3 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2021-05-03 19:19 UTC (permalink / raw) To: Mario Kleiner; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 4865 bytes --] == Series Details == Series: series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. URL : https://patchwork.freedesktop.org/series/89761/ State : success == Summary == CI Bug Log - changes from CI_DRM_10039 -> IGTPW_5776 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/index.html Known issues ------------ Here are the changes found in IGTPW_5776 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@amdgpu/amd_cs_nop@sync-fork-compute0: - fi-snb-2600: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html * igt@gem_exec_fence@basic-busy@bcs0: - fi-kbl-soraka: NOTRUN -> [SKIP][2] ([fdo#109271]) +24 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-kbl-soraka/igt@gem_exec_fence@basic-busy@bcs0.html * igt@gem_huc_copy@huc-copy: - fi-kbl-soraka: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#2190]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html * igt@i915_selftest@live@gt_pm: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][4] ([i915#1886] / [i915#2291]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-kbl-soraka: NOTRUN -> [SKIP][5] ([fdo#109271] / [fdo#111827]) +8 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-kbl-soraka/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d: - fi-kbl-soraka: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#533]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-kbl-soraka/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - fi-skl-6600u: [PASS][7] -> [INCOMPLETE][8] ([i915#198]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/fi-skl-6600u/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-skl-6600u/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s3: - {fi-tgl-1115g4}: [FAIL][9] ([i915#1888]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html * igt@i915_pm_rpm@basic-rte: - {fi-tgl-1115g4}: [DMESG-WARN][11] -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/fi-tgl-1115g4/igt@i915_pm_rpm@basic-rte.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-tgl-1115g4/igt@i915_pm_rpm@basic-rte.html * igt@i915_selftest@live@hangcheck: - fi-snb-2600: [INCOMPLETE][13] ([i915#2782]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/fi-snb-2600/igt@i915_selftest@live@hangcheck.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/fi-snb-2600/igt@i915_selftest@live@hangcheck.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886 [i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888 [i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291 [i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782 [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533 Participating hosts (45 -> 40) ------------------------------ Additional (1): fi-kbl-soraka Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-dg1-1 fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_6077 -> IGTPW_5776 CI-20190529: 20190529 CI_DRM_10039: a707d569366fe3e474683e56e8acf0f6d5cb2cab @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_5776: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/index.html IGT_6077: 126a3f6fc0e97786e2819085efc84e741093aed5 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/index.html [-- Attachment #1.2: Type: text/html, Size: 5988 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 10+ messages in thread
* [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. 2021-05-03 18:25 [igt-dev] Add support for testing of 16 bpc fixed point framebuffers Mario Kleiner ` (2 preceding siblings ...) 2021-05-03 19:19 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Patchwork @ 2021-05-04 0:07 ` Patchwork 3 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2021-05-04 0:07 UTC (permalink / raw) To: Mario Kleiner; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 30317 bytes --] == Series Details == Series: series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats. URL : https://patchwork.freedesktop.org/series/89761/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10039_full -> IGTPW_5776_full ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_5776_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_5776_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_5776/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_5776_full: ### IGT changes ### #### Possible regressions #### * igt@kms_atomic@plane-immutable-zpos: - shard-tglb: [PASS][1] -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb1/igt@kms_atomic@plane-immutable-zpos.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb3/igt@kms_atomic@plane-immutable-zpos.html * igt@kms_cursor_crc@pipe-a-cursor-dpms: - shard-tglb: [PASS][3] -> [INCOMPLETE][4] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb6/igt@kms_cursor_crc@pipe-a-cursor-dpms.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb8/igt@kms_cursor_crc@pipe-a-cursor-dpms.html * igt@kms_cursor_crc@pipe-c-cursor-128x42-random: - shard-apl: NOTRUN -> [FAIL][5] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl8/igt@kms_cursor_crc@pipe-c-cursor-128x42-random.html - shard-kbl: NOTRUN -> [FAIL][6] [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_cursor_crc@pipe-c-cursor-128x42-random.html * igt@kms_cursor_crc@pipe-c-cursor-256x85-random: - shard-kbl: [PASS][7] -> [FAIL][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl2/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html - shard-apl: [PASS][9] -> [FAIL][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-apl7/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl8/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html Known issues ------------ Here are the changes found in IGTPW_5776_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_param@set-priority-not-supported: - shard-tglb: NOTRUN -> [SKIP][11] ([fdo#109314]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@gem_ctx_param@set-priority-not-supported.html - shard-iclb: NOTRUN -> [SKIP][12] ([fdo#109314]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@gem_ctx_param@set-priority-not-supported.html * igt@gem_ctx_persistence@idempotent: - shard-snb: NOTRUN -> [SKIP][13] ([fdo#109271] / [i915#1099]) +5 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-snb6/igt@gem_ctx_persistence@idempotent.html * igt@gem_eio@unwedge-stress: - shard-snb: NOTRUN -> [FAIL][14] ([i915#3354]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-snb6/igt@gem_eio@unwedge-stress.html * igt@gem_exec_fair@basic-none-share@rcs0: - shard-tglb: [PASS][15] -> [FAIL][16] ([i915#2842]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb7/igt@gem_exec_fair@basic-none-share@rcs0.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@gem_exec_fair@basic-none-share@rcs0.html * igt@gem_exec_fair@basic-none@rcs0: - shard-glk: NOTRUN -> [FAIL][17] ([i915#2842]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk1/igt@gem_exec_fair@basic-none@rcs0.html * igt@gem_exec_fair@basic-none@vcs0: - shard-tglb: NOTRUN -> [FAIL][18] ([i915#2842]) +4 similar issues [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb6/igt@gem_exec_fair@basic-none@vcs0.html * igt@gem_exec_fair@basic-none@vecs0: - shard-iclb: NOTRUN -> [FAIL][19] ([i915#2842]) +3 similar issues [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@gem_exec_fair@basic-none@vecs0.html * igt@gem_exec_reloc@basic-wide-active@bcs0: - shard-apl: NOTRUN -> [FAIL][20] ([i915#2389]) +3 similar issues [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl2/igt@gem_exec_reloc@basic-wide-active@bcs0.html * igt@gem_exec_reloc@basic-wide-active@vcs1: - shard-iclb: NOTRUN -> [FAIL][21] ([i915#2389]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb1/igt@gem_exec_reloc@basic-wide-active@vcs1.html * igt@gem_mmap_gtt@cpuset-big-copy: - shard-iclb: [PASS][22] -> [FAIL][23] ([i915#307]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb6/igt@gem_mmap_gtt@cpuset-big-copy.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@gem_mmap_gtt@cpuset-big-copy.html * igt@gem_mmap_gtt@cpuset-big-copy-odd: - shard-iclb: [PASS][24] -> [FAIL][25] ([i915#2428]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb4/igt@gem_mmap_gtt@cpuset-big-copy-odd.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@gem_mmap_gtt@cpuset-big-copy-odd.html * igt@gem_pread@exhaustion: - shard-tglb: NOTRUN -> [WARN][26] ([i915#2658]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb6/igt@gem_pread@exhaustion.html - shard-glk: NOTRUN -> [WARN][27] ([i915#2658]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk4/igt@gem_pread@exhaustion.html - shard-apl: NOTRUN -> [WARN][28] ([i915#2658]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl7/igt@gem_pread@exhaustion.html - shard-iclb: NOTRUN -> [WARN][29] ([i915#2658]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@gem_pread@exhaustion.html - shard-snb: NOTRUN -> [WARN][30] ([i915#2658]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-snb7/igt@gem_pread@exhaustion.html * igt@gem_render_copy@y-tiled-mc-ccs-to-vebox-y-tiled: - shard-iclb: NOTRUN -> [SKIP][31] ([i915#768]) +1 similar issue [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@gem_render_copy@y-tiled-mc-ccs-to-vebox-y-tiled.html * igt@gem_userptr_blits@dmabuf-sync: - shard-apl: NOTRUN -> [SKIP][32] ([fdo#109271] / [i915#3323]) [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl2/igt@gem_userptr_blits@dmabuf-sync.html * igt@gem_userptr_blits@dmabuf-unsync: - shard-tglb: NOTRUN -> [SKIP][33] ([i915#3297]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb3/igt@gem_userptr_blits@dmabuf-unsync.html - shard-iclb: NOTRUN -> [SKIP][34] ([i915#3297]) [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb4/igt@gem_userptr_blits@dmabuf-unsync.html * igt@gen3_render_tiledx_blits: - shard-iclb: NOTRUN -> [SKIP][35] ([fdo#109289]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb4/igt@gen3_render_tiledx_blits.html - shard-tglb: NOTRUN -> [SKIP][36] ([fdo#109289]) [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@gen3_render_tiledx_blits.html * igt@gen9_exec_parse@allowed-all: - shard-iclb: NOTRUN -> [SKIP][37] ([fdo#112306]) +1 similar issue [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@gen9_exec_parse@allowed-all.html - shard-tglb: NOTRUN -> [SKIP][38] ([fdo#112306]) +1 similar issue [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb8/igt@gen9_exec_parse@allowed-all.html - shard-glk: NOTRUN -> [DMESG-WARN][39] ([i915#1436] / [i915#716]) [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk5/igt@gen9_exec_parse@allowed-all.html * igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp: - shard-kbl: NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#1937]) [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl1/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html - shard-apl: NOTRUN -> [SKIP][41] ([fdo#109271] / [i915#1937]) [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl8/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html * igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a: - shard-glk: NOTRUN -> [SKIP][42] ([fdo#109271] / [i915#1937]) [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk8/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html * igt@kms_big_fb@yf-tiled-addfb-size-overflow: - shard-tglb: NOTRUN -> [SKIP][43] ([fdo#111615]) +4 similar issues [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb5/igt@kms_big_fb@yf-tiled-addfb-size-overflow.html * igt@kms_chamelium@hdmi-hpd-fast: - shard-snb: NOTRUN -> [SKIP][44] ([fdo#109271] / [fdo#111827]) +26 similar issues [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-snb5/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_color@pipe-a-degamma: - shard-tglb: NOTRUN -> [FAIL][45] ([i915#1149]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@kms_color@pipe-a-degamma.html - shard-iclb: NOTRUN -> [FAIL][46] ([i915#1149]) [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb4/igt@kms_color@pipe-a-degamma.html * igt@kms_color_chamelium@pipe-a-ctm-limited-range: - shard-apl: NOTRUN -> [SKIP][47] ([fdo#109271] / [fdo#111827]) +24 similar issues [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl2/igt@kms_color_chamelium@pipe-a-ctm-limited-range.html * igt@kms_color_chamelium@pipe-b-ctm-red-to-blue: - shard-iclb: NOTRUN -> [SKIP][48] ([fdo#109284] / [fdo#111827]) +2 similar issues [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@kms_color_chamelium@pipe-b-ctm-red-to-blue.html - shard-glk: NOTRUN -> [SKIP][49] ([fdo#109271] / [fdo#111827]) +3 similar issues [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk5/igt@kms_color_chamelium@pipe-b-ctm-red-to-blue.html - shard-kbl: NOTRUN -> [SKIP][50] ([fdo#109271] / [fdo#111827]) +2 similar issues [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_color_chamelium@pipe-b-ctm-red-to-blue.html * igt@kms_color_chamelium@pipe-d-ctm-red-to-blue: - shard-tglb: NOTRUN -> [SKIP][51] ([fdo#109284] / [fdo#111827]) +4 similar issues [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html - shard-iclb: NOTRUN -> [SKIP][52] ([fdo#109278] / [fdo#109284] / [fdo#111827]) [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html * igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen: - shard-iclb: NOTRUN -> [SKIP][53] ([fdo#109278] / [fdo#109279]) [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb1/igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen.html - shard-tglb: NOTRUN -> [SKIP][54] ([fdo#109279] / [i915#3359]) [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen.html * igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen: - shard-kbl: NOTRUN -> [SKIP][55] ([fdo#109271]) +45 similar issues [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl6/igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen.html - shard-tglb: NOTRUN -> [SKIP][56] ([i915#3319]) +1 similar issue [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb2/igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen.html * igt@kms_cursor_crc@pipe-d-cursor-dpms: - shard-iclb: NOTRUN -> [SKIP][57] ([fdo#109278]) +10 similar issues [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@kms_cursor_crc@pipe-d-cursor-dpms.html * igt@kms_cursor_crc@pipe-d-cursor-max-size-rapid-movement: - shard-tglb: NOTRUN -> [SKIP][58] ([i915#3359]) [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@kms_cursor_crc@pipe-d-cursor-max-size-rapid-movement.html * igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge: - shard-snb: NOTRUN -> [SKIP][59] ([fdo#109271]) +500 similar issues [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-snb2/igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge.html * igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size: - shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109274] / [fdo#109278]) [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb3/igt@kms_cursor_legacy@cursorb-vs-flipa-varying-size.html * igt@kms_dp_tiled_display@basic-test-pattern-with-chamelium: - shard-tglb: NOTRUN -> [SKIP][61] ([i915#2065]) [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb3/igt@kms_dp_tiled_display@basic-test-pattern-with-chamelium.html - shard-iclb: NOTRUN -> [SKIP][62] ([i915#2065]) [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@kms_dp_tiled_display@basic-test-pattern-with-chamelium.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-kbl: [PASS][63] -> [INCOMPLETE][64] ([i915#155] / [i915#180] / [i915#636]) [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl6/igt@kms_fbcon_fbt@fbc-suspend.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@kms_fbcon_fbt@fbc-suspend.html * igt@kms_flip@2x-flip-vs-wf_vblank-interruptible: - shard-iclb: NOTRUN -> [SKIP][65] ([fdo#109274]) +1 similar issue [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@kms_flip@2x-flip-vs-wf_vblank-interruptible.html * igt@kms_flip@2x-plain-flip-ts-check-interruptible@ac-hdmi-a1-hdmi-a2: - shard-glk: [PASS][66] -> [FAIL][67] ([i915#2122]) [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-glk7/igt@kms_flip@2x-plain-flip-ts-check-interruptible@ac-hdmi-a1-hdmi-a2.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk1/igt@kms_flip@2x-plain-flip-ts-check-interruptible@ac-hdmi-a1-hdmi-a2.html * igt@kms_flip@flip-vs-suspend@c-dp1: - shard-kbl: [PASS][68] -> [DMESG-WARN][69] ([i915#180]) +3 similar issues [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl3/igt@kms_flip@flip-vs-suspend@c-dp1.html [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_flip@flip-vs-suspend@c-dp1.html - shard-apl: [PASS][70] -> [DMESG-WARN][71] ([i915#180]) +1 similar issue [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-apl6/igt@kms_flip@flip-vs-suspend@c-dp1.html [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl8/igt@kms_flip@flip-vs-suspend@c-dp1.html * igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs: - shard-tglb: NOTRUN -> [SKIP][72] ([i915#2587]) [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb6/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs.html * igt@kms_frontbuffer_tracking@fbc-suspend: - shard-iclb: [PASS][73] -> [INCOMPLETE][74] ([i915#1185]) [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-suspend.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb3/igt@kms_frontbuffer_tracking@fbc-suspend.html * igt@kms_frontbuffer_tracking@fbcpsr-suspend: - shard-glk: NOTRUN -> [SKIP][75] ([fdo#109271]) +46 similar issues [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk7/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html * igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-plflip-blt: - shard-iclb: NOTRUN -> [SKIP][76] ([fdo#109280]) +12 similar issues [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-plflip-blt.html * igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite: - shard-tglb: NOTRUN -> [SKIP][77] ([fdo#111825]) +15 similar issues [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb8/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite.html * igt@kms_hdmi_inject@inject-audio: - shard-tglb: [PASS][78] -> [SKIP][79] ([i915#433]) [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb2/igt@kms_hdmi_inject@inject-audio.html [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@kms_hdmi_inject@inject-audio.html * igt@kms_hdr@static-toggle: - shard-iclb: NOTRUN -> [SKIP][80] ([i915#1187]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb4/igt@kms_hdr@static-toggle.html - shard-tglb: NOTRUN -> [SKIP][81] ([i915#1187]) [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb1/igt@kms_hdr@static-toggle.html * igt@kms_plane_alpha_blend@pipe-b-alpha-transparent-fb: - shard-apl: NOTRUN -> [FAIL][82] ([i915#265]) [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl3/igt@kms_plane_alpha_blend@pipe-b-alpha-transparent-fb.html * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max: - shard-apl: NOTRUN -> [FAIL][83] ([fdo#108145] / [i915#265]) +4 similar issues [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl3/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max.html * igt@kms_plane_alpha_blend@pipe-c-alpha-basic: - shard-kbl: NOTRUN -> [FAIL][84] ([fdo#108145] / [i915#265]) [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_plane_alpha_blend@pipe-c-alpha-basic.html - shard-glk: NOTRUN -> [FAIL][85] ([fdo#108145] / [i915#265]) [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk6/igt@kms_plane_alpha_blend@pipe-c-alpha-basic.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-4: - shard-apl: NOTRUN -> [SKIP][86] ([fdo#109271] / [i915#658]) +7 similar issues [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl3/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-4.html * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1: - shard-iclb: NOTRUN -> [SKIP][87] ([i915#658]) [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html - shard-glk: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#658]) +1 similar issue [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk5/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html - shard-kbl: NOTRUN -> [SKIP][89] ([fdo#109271] / [i915#658]) +1 similar issue [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html * igt@kms_psr2_su@page_flip: - shard-tglb: NOTRUN -> [SKIP][90] ([i915#1911]) [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@kms_psr2_su@page_flip.html - shard-iclb: NOTRUN -> [SKIP][91] ([fdo#109642] / [fdo#111068] / [i915#658]) [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@kms_psr2_su@page_flip.html * igt@kms_psr@psr2_primary_page_flip: - shard-iclb: NOTRUN -> [SKIP][92] ([fdo#109441]) +1 similar issue [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@kms_psr@psr2_primary_page_flip.html * igt@kms_psr@psr2_sprite_blt: - shard-iclb: [PASS][93] -> [SKIP][94] ([fdo#109441]) [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb5/igt@kms_psr@psr2_sprite_blt.html * igt@kms_psr@psr2_sprite_render: - shard-tglb: NOTRUN -> [FAIL][95] ([i915#132]) +1 similar issue [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb2/igt@kms_psr@psr2_sprite_render.html * igt@kms_sysfs_edid_timing: - shard-apl: NOTRUN -> [FAIL][96] ([IGT#2]) [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl2/igt@kms_sysfs_edid_timing.html * igt@kms_tv_load_detect@load-detect: - shard-tglb: NOTRUN -> [SKIP][97] ([fdo#109309]) [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb6/igt@kms_tv_load_detect@load-detect.html - shard-iclb: NOTRUN -> [SKIP][98] ([fdo#109309]) [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@kms_tv_load_detect@load-detect.html * igt@kms_vrr@flip-suspend: - shard-iclb: NOTRUN -> [SKIP][99] ([fdo#109502]) [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb7/igt@kms_vrr@flip-suspend.html - shard-tglb: NOTRUN -> [SKIP][100] ([fdo#109502]) [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb6/igt@kms_vrr@flip-suspend.html * igt@kms_writeback@writeback-check-output: - shard-apl: NOTRUN -> [SKIP][101] ([fdo#109271] / [i915#2437]) [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl6/igt@kms_writeback@writeback-check-output.html * igt@nouveau_crc@pipe-a-source-rg: - shard-iclb: NOTRUN -> [SKIP][102] ([i915#2530]) [102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb2/igt@nouveau_crc@pipe-a-source-rg.html - shard-tglb: NOTRUN -> [SKIP][103] ([i915#2530]) +1 similar issue [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@nouveau_crc@pipe-a-source-rg.html * igt@nouveau_crc@pipe-d-source-outp-complete: - shard-iclb: NOTRUN -> [SKIP][104] ([fdo#109278] / [i915#2530]) [104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb6/igt@nouveau_crc@pipe-d-source-outp-complete.html * igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name: - shard-apl: NOTRUN -> [SKIP][105] ([fdo#109271]) +239 similar issues [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl1/igt@prime_nv_api@i915_nv_reimport_twice_check_flink_name.html * igt@prime_nv_api@nv_self_import: - shard-iclb: NOTRUN -> [SKIP][106] ([fdo#109291]) +1 similar issue [106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@prime_nv_api@nv_self_import.html * igt@prime_nv_pcopy@test1_macro: - shard-tglb: NOTRUN -> [SKIP][107] ([fdo#109291]) +1 similar issue [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@prime_nv_pcopy@test1_macro.html * igt@sysfs_clients@create: - shard-glk: NOTRUN -> [SKIP][108] ([fdo#109271] / [i915#2994]) [108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk2/igt@sysfs_clients@create.html - shard-tglb: NOTRUN -> [SKIP][109] ([i915#2994]) [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb2/igt@sysfs_clients@create.html - shard-apl: NOTRUN -> [SKIP][110] ([fdo#109271] / [i915#2994]) +3 similar issues [110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl6/igt@sysfs_clients@create.html - shard-iclb: NOTRUN -> [SKIP][111] ([i915#2994]) [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb1/igt@sysfs_clients@create.html - shard-kbl: NOTRUN -> [SKIP][112] ([fdo#109271] / [i915#2994]) [112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@sysfs_clients@create.html #### Possible fixes #### * igt@gem_eio@unwedge-stress: - shard-tglb: [TIMEOUT][113] ([i915#2369] / [i915#3063]) -> [PASS][114] [113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb5/igt@gem_eio@unwedge-stress.html [114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb8/igt@gem_eio@unwedge-stress.html * igt@gem_exec_fair@basic-pace@bcs0: - shard-iclb: [FAIL][115] ([i915#2842]) -> [PASS][116] [115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb1/igt@gem_exec_fair@basic-pace@bcs0.html [116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb7/igt@gem_exec_fair@basic-pace@bcs0.html * igt@gem_exec_fair@basic-pace@rcs0: - shard-kbl: [FAIL][117] ([i915#2842]) -> [PASS][118] [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl3/igt@gem_exec_fair@basic-pace@rcs0.html [118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl3/igt@gem_exec_fair@basic-pace@rcs0.html * igt@gem_exec_fair@basic-pace@vcs1: - shard-kbl: [SKIP][119] ([fdo#109271]) -> [PASS][120] [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl3/igt@gem_exec_fair@basic-pace@vcs1.html [120]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl3/igt@gem_exec_fair@basic-pace@vcs1.html * igt@gem_exec_whisper@basic-fds-forked-all: - shard-glk: [DMESG-WARN][121] ([i915#118] / [i915#95]) -> [PASS][122] [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-glk6/igt@gem_exec_whisper@basic-fds-forked-all.html [122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk2/igt@gem_exec_whisper@basic-fds-forked-all.html * igt@gem_mmap_gtt@big-copy-xy: - shard-glk: [FAIL][123] ([i915#307]) -> [PASS][124] [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-glk6/igt@gem_mmap_gtt@big-copy-xy.html [124]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-glk9/igt@gem_mmap_gtt@big-copy-xy.html * igt@kms_big_fb@y-tiled-64bpp-rotate-180: - shard-tglb: [FAIL][125] ([i915#3434]) -> [PASS][126] [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-tglb3/igt@kms_big_fb@y-tiled-64bpp-rotate-180.html [126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-tglb7/igt@kms_big_fb@y-tiled-64bpp-rotate-180.html * igt@kms_cursor_crc@pipe-b-cursor-suspend: - shard-kbl: [DMESG-WARN][127] ([i915#180]) -> [PASS][128] [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl7/igt@kms_cursor_crc@pipe-b-cursor-suspend.html [128]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl6/igt@kms_cursor_crc@pipe-b-cursor-suspend.html * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc: - shard-apl: [FAIL][129] ([fdo#108145] / [i915#265]) -> [PASS][130] [129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-apl8/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html [130]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-apl6/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html * igt@kms_psr2_su@frontbuffer: - shard-iclb: [SKIP][131] ([fdo#109642] / [fdo#111068] / [i915#658]) -> [PASS][132] [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb5/igt@kms_psr2_su@frontbuffer.html [132]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb2/igt@kms_psr2_su@frontbuffer.html * igt@kms_psr@psr2_sprite_plane_move: - shard-iclb: [SKIP][133] ([fdo#109441]) -> [PASS][134] [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb3/igt@kms_psr@psr2_sprite_plane_move.html [134]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html #### Warnings #### * igt@i915_pm_dc@dc3co-vpb-simulation: - shard-iclb: [SKIP][135] ([i915#658]) -> [SKIP][136] ([i915#588]) [135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb6/igt@i915_pm_dc@dc3co-vpb-simulation.html [136]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-2: - shard-iclb: [SKIP][137] ([i915#2920]) -> [SKIP][138] ([i915#658]) +1 similar issue [137]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-iclb2/igt@kms_psr2_sf@plane-move-sf-dmg-area-2.html [138]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-iclb8/igt@kms_psr2_sf@plane-move-sf-dmg-area-2.html * igt@runner@aborted: - shard-kbl: ([FAIL][139], [FAIL][140], [FAIL][141]) ([i915#1814] / [i915#3002] / [i915#3363]) -> ([FAIL][142], [FAIL][143], [FAIL][144], [FAIL][145], [FAIL][146]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#92]) [139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl7/igt@runner@aborted.html [140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl7/igt@runner@aborted.html [141]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10039/shard-kbl7/igt@runner@aborted.html [142]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl4/igt@runner@aborted.html [143]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@runner@aborted.html [144]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@runner@aborted.html [145]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@runner@aborted.html [146]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/shard-kbl7/igt@runner@aborted.html - shard-apl: ([FAIL][147], [FAIL][148]) ([i915#3002] / [i915#3363]) -> ([FAIL][149], [FAIL][150]) ([i915#18 == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5776/index.html [-- Attachment #1.2: Type: text/html, Size: 34465 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-05-04 0:07 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-05-03 18:25 [igt-dev] Add support for testing of 16 bpc fixed point framebuffers Mario Kleiner 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Mario Kleiner 2021-05-03 18:59 ` Ville Syrjälä 2021-05-03 18:25 ` [igt-dev] [PATCH i-g-t 2/2] lib/igt_fb: Add support for testing of 16 bpc fixed point formats Mario Kleiner 2021-05-03 18:47 ` Ville Syrjälä 2021-05-03 19:32 ` Mario Kleiner 2021-05-03 19:43 ` Ville Syrjälä 2021-05-03 19:47 ` Mario Kleiner 2021-05-03 19:19 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/2] drm-uapi: Add fourcc's for 16 bpc fixed point framebuffer formats Patchwork 2021-05-04 0:07 ` [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.