* [Intel-gfx] [v7 0/3] Set BPP in the kernel @ 2021-07-19 9:50 Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 1/3] drm/i915/display: Add write permissions for fec support Vandita Kulkarni ` (6 more replies) 0 siblings, 7 replies; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-19 9:50 UTC (permalink / raw) To: intel-gfx Patnana Venkata Sai (1): drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni (2): drm/i915/display: Add write permissions for fec support drm/i915/display/dsc: Force dsc BPP .../drm/i915/display/intel_display_debugfs.c | 78 ++++++++++++++++++- .../drm/i915/display/intel_display_types.h | 1 + drivers/gpu/drm/i915/display/intel_dp.c | 17 ++++ 3 files changed, 94 insertions(+), 2 deletions(-) -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 1/3] drm/i915/display: Add write permissions for fec support 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni @ 2021-07-19 9:50 ` Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 2/3] drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni ` (5 subsequent siblings) 6 siblings, 0 replies; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-19 9:50 UTC (permalink / raw) To: intel-gfx; +Cc: Jani Nikula Though there is a write option available on fec_suport debugfs file, so far it has been registering with read permissions only. Suggested-by: Jani Nikula <jani.nikula@intel.com> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/display/intel_display_debugfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c b/drivers/gpu/drm/i915/display/intel_display_debugfs.c index 65832c4d962f..717d5624839d 100644 --- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c +++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c @@ -2434,7 +2434,7 @@ int intel_connector_debugfs_add(struct drm_connector *connector) } if ((DISPLAY_VER(dev_priv) >= 11 || IS_CANNONLAKE(dev_priv)) && ((connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort && !to_intel_connector(connector)->mst_port) || connector->connector_type == DRM_MODE_CONNECTOR_eDP)) - debugfs_create_file("i915_dsc_fec_support", S_IRUGO, root, + debugfs_create_file("i915_dsc_fec_support", 0644, root, connector, &i915_dsc_fec_support_fops); /* Legacy panels doesn't lpsp on any platform */ -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 2/3] drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 1/3] drm/i915/display: Add write permissions for fec support Vandita Kulkarni @ 2021-07-19 9:50 ` Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni ` (4 subsequent siblings) 6 siblings, 0 replies; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-19 9:50 UTC (permalink / raw) To: intel-gfx; +Cc: Jani Nikula From: Patnana Venkata Sai <venkata.sai.patnana@intel.com> [What]: This patch creates a per connector debugfs node to expose the Input and Compressed BPP. The same node can be used from userspace to force DSC to a certain BPP(all accepted values). [Why]: Useful to verify all supported/requested compression bpp's through IGT v2: Remove unnecessary logic (Jani) v3: Drop pipe bpp in debugfs node (Vandita) v4: Minor cleanups (Vandita) v5: Fix NULL pointer dereference v6: Fix dim tool checkpatch errors Release the lock before return (Vandita) v7: Rename to file to dsc_bpp, remove unwanted dsc bpp range check from v6, permissions (Jani) Cc: Vandita Kulkarni <vandita.kulkarni@intel.com> Cc: Navare Manasi D <manasi.d.navare@intel.com> Cc: Jani Nikula <jani.nikula@linux.intel.com> Signed-off-by: Anusha Srivatsa <anusha.srivatsa@intel.com> Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@intel.com> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> Reviewed-by: Jani Nikula <jani.nikula@intel.com> --- .../drm/i915/display/intel_display_debugfs.c | 76 ++++++++++++++++++- .../drm/i915/display/intel_display_types.h | 1 + 2 files changed, 76 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c b/drivers/gpu/drm/i915/display/intel_display_debugfs.c index 717d5624839d..2cf742a0b957 100644 --- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c +++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c @@ -2395,6 +2395,73 @@ static const struct file_operations i915_dsc_fec_support_fops = { .write = i915_dsc_fec_support_write }; +static int i915_dsc_bpp_show(struct seq_file *m, void *data) +{ + struct drm_connector *connector = m->private; + struct drm_device *dev = connector->dev; + struct drm_crtc *crtc; + struct intel_crtc_state *crtc_state; + struct intel_encoder *encoder = intel_attached_encoder(to_intel_connector(connector)); + int ret; + + if (!encoder) + return -ENODEV; + + ret = drm_modeset_lock_single_interruptible(&dev->mode_config.connection_mutex); + if (ret) + return ret; + + crtc = connector->state->crtc; + if (connector->status != connector_status_connected || !crtc) { + ret = -ENODEV; + goto out; + } + + crtc_state = to_intel_crtc_state(crtc->state); + seq_printf(m, "Compressed_BPP: %d\n", crtc_state->dsc.compressed_bpp); + +out: drm_modeset_unlock(&dev->mode_config.connection_mutex); + + return ret; +} + +static ssize_t i915_dsc_bpp_write(struct file *file, + const char __user *ubuf, + size_t len, loff_t *offp) +{ + struct drm_connector *connector = + ((struct seq_file *)file->private_data)->private; + struct intel_encoder *encoder = intel_attached_encoder(to_intel_connector(connector)); + struct intel_dp *intel_dp = enc_to_intel_dp(encoder); + int dsc_bpp = 0; + int ret; + + ret = kstrtoint_from_user(ubuf, len, 0, &dsc_bpp); + if (ret < 0) + return ret; + + intel_dp->force_dsc_bpp = dsc_bpp; + *offp += len; + + return len; +} + +static int i915_dsc_bpp_open(struct inode *inode, + struct file *file) +{ + return single_open(file, i915_dsc_bpp_show, + inode->i_private); +} + +static const struct file_operations i915_dsc_bpp_fops = { + .owner = THIS_MODULE, + .open = i915_dsc_bpp_open, + .read = seq_read, + .llseek = seq_lseek, + .release = single_release, + .write = i915_dsc_bpp_write +}; + /** * intel_connector_debugfs_add - add i915 specific connector debugfs files * @connector: pointer to a registered drm_connector @@ -2433,10 +2500,17 @@ int intel_connector_debugfs_add(struct drm_connector *connector) connector, &i915_hdcp_sink_capability_fops); } - if ((DISPLAY_VER(dev_priv) >= 11 || IS_CANNONLAKE(dev_priv)) && ((connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort && !to_intel_connector(connector)->mst_port) || connector->connector_type == DRM_MODE_CONNECTOR_eDP)) + if ((DISPLAY_VER(dev_priv) >= 11 || IS_CANNONLAKE(dev_priv)) && + ((connector->connector_type == DRM_MODE_CONNECTOR_DisplayPort && + !to_intel_connector(connector)->mst_port) || + connector->connector_type == DRM_MODE_CONNECTOR_eDP)) { debugfs_create_file("i915_dsc_fec_support", 0644, root, connector, &i915_dsc_fec_support_fops); + debugfs_create_file("i915_dsc_bpp", 0644, root, + connector, &i915_dsc_bpp_fops); + } + /* Legacy panels doesn't lpsp on any platform */ if ((DISPLAY_VER(dev_priv) >= 9 || IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) && diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h index d94f361b548b..19d8d3eefbc2 100644 --- a/drivers/gpu/drm/i915/display/intel_display_types.h +++ b/drivers/gpu/drm/i915/display/intel_display_types.h @@ -1612,6 +1612,7 @@ struct intel_dp { /* Display stream compression testing */ bool force_dsc_en; + int force_dsc_bpp; bool hobl_failed; bool hobl_active; -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 1/3] drm/i915/display: Add write permissions for fec support Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 2/3] drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni @ 2021-07-19 9:50 ` Vandita Kulkarni 2021-07-19 10:40 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Set BPP in the kernel (rev5) Patchwork ` (3 subsequent siblings) 6 siblings, 0 replies; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-19 9:50 UTC (permalink / raw) To: intel-gfx Set DSC BPP to the value forced through debugfs. It can go from bpc to bpp-1. v2: Use default dsc bpp when we are just doing force_dsc_en, use default dsc bpp for invalid force_dsc_bpp values. (Jani) Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> Reviewed-by: Swati Sharma <swati2.sharma@intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 5b52beaddada..c386ef8eb200 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1274,6 +1274,23 @@ static int intel_dp_dsc_compute_config(struct intel_dp *intel_dp, pipe_config->pipe_bpp); pipe_config->dsc.slice_count = dsc_dp_slice_count; } + + /* As of today we support DSC for only RGB */ + if (intel_dp->force_dsc_bpp) { + if (intel_dp->force_dsc_bpp >= 8 && + intel_dp->force_dsc_bpp < pipe_bpp) { + drm_dbg_kms(&dev_priv->drm, + "DSC BPP forced to %d", + intel_dp->force_dsc_bpp); + pipe_config->dsc.compressed_bpp = + intel_dp->force_dsc_bpp; + } else { + drm_dbg_kms(&dev_priv->drm, + "Invalid DSC BPP %d", + intel_dp->force_dsc_bpp); + } + } + /* * VDSC engine operates at 1 Pixel per clock, so if peak pixel rate * is greater than the maximum Cdclock and if slice count is even -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Set BPP in the kernel (rev5) 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni ` (2 preceding siblings ...) 2021-07-19 9:50 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni @ 2021-07-19 10:40 ` Patchwork 2021-07-19 10:44 ` [Intel-gfx] ✗ Fi.CI.DOCS: " Patchwork ` (2 subsequent siblings) 6 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-19 10:40 UTC (permalink / raw) To: Sharma, Swati2; +Cc: intel-gfx == Series Details == Series: Set BPP in the kernel (rev5) URL : https://patchwork.freedesktop.org/series/92312/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately. - +drivers/gpu/drm/i915/display/intel_display.c:1896:21: expected struct i915_vma *[assigned] vma +drivers/gpu/drm/i915/display/intel_display.c:1896:21: got void [noderef] __iomem *[assigned] iomem +drivers/gpu/drm/i915/display/intel_display.c:1896:21: warning: incorrect type in assignment (different address spaces) +drivers/gpu/drm/i915/gem/i915_gem_context.c:1412:34: expected struct i915_address_space *vm +drivers/gpu/drm/i915/gem/i915_gem_context.c:1412:34: got struct i915_address_space [noderef] __rcu *vm +drivers/gpu/drm/i915/gem/i915_gem_context.c:1412:34: warning: incorrect type in argument 1 (different address spaces) +drivers/gpu/drm/i915/gem/selftests/mock_context.c:43:25: expected struct i915_address_space [noderef] __rcu *vm +drivers/gpu/drm/i915/gem/selftests/mock_context.c:43:25: got struct i915_address_space * +drivers/gpu/drm/i915/gem/selftests/mock_context.c:43:25: warning: incorrect type in assignment (different address spaces) +drivers/gpu/drm/i915/gem/selftests/mock_context.c:60:34: expected struct i915_address_space *vm +drivers/gpu/drm/i915/gem/selftests/mock_context.c:60:34: got struct i915_address_space [noderef] __rcu *vm +drivers/gpu/drm/i915/gem/selftests/mock_context.c:60:34: warning: incorrect type in argument 1 (different address spaces) +drivers/gpu/drm/i915/gt/intel_engine_stats.h:27:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:27:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:27:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:32:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:32:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:49:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:49:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:49:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:56:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_engine_stats.h:56:9: warning: trying to copy expression type 31 +drivers/gpu/drm/i915/gt/intel_reset.c:1396:5: warning: context imbalance in 'intel_gt_reset_trylock' - different lock contexts for basic block +./include/asm-generic/bitops/find.h:112:45: warning: shift count is negative (-262080) +./include/asm-generic/bitops/find.h:32:31: warning: shift count is negative (-262080) +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_read16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_read32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_read64' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_read8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_write16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_write32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'fwtable_write8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_read16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_read32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_read64' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_read8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_write16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_write32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen11_fwtable_write8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_read16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_read32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_read64' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_read8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_write16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_write32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen12_fwtable_write8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_read16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_read32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_read64' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_read8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_write16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_write32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen6_write8' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen8_write16' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen8_write32' - different lock contexts for basic block +./include/linux/spinlock.h:409:9: warning: context imbalance in 'gen8_write8' - different lock contexts for basic block _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.DOCS: warning for Set BPP in the kernel (rev5) 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni ` (3 preceding siblings ...) 2021-07-19 10:40 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Set BPP in the kernel (rev5) Patchwork @ 2021-07-19 10:44 ` Patchwork 2021-07-19 11:07 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2021-07-19 15:19 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 6 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-19 10:44 UTC (permalink / raw) To: Sharma, Swati2; +Cc: intel-gfx == Series Details == Series: Set BPP in the kernel (rev5) URL : https://patchwork.freedesktop.org/series/92312/ State : warning == Summary == $ make htmldocs 2>&1 > /dev/null | grep i915 ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'jump_whitelist' description in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'shadow_map' description in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'batch_map' description in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Function parameter or member 'trampoline' not described in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'jump_whitelist' description in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'shadow_map' description in 'intel_engine_cmd_parser' ./drivers/gpu/drm/i915/i915_cmd_parser.c:1436: warning: Excess function parameter 'batch_map' description in 'intel_engine_cmd_parser' _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Set BPP in the kernel (rev5) 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni ` (4 preceding siblings ...) 2021-07-19 10:44 ` [Intel-gfx] ✗ Fi.CI.DOCS: " Patchwork @ 2021-07-19 11:07 ` Patchwork 2021-07-19 15:19 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 6 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-19 11:07 UTC (permalink / raw) To: Sharma, Swati2; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 7451 bytes --] == Series Details == Series: Set BPP in the kernel (rev5) URL : https://patchwork.freedesktop.org/series/92312/ State : success == Summary == CI Bug Log - changes from CI_DRM_10350 -> Patchwork_20645 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/index.html Known issues ------------ Here are the changes found in Patchwork_20645 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@amdgpu/amd_basic@query-info: - fi-tgl-u2: NOTRUN -> [SKIP][1] ([fdo#109315]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@amdgpu/amd_basic@query-info.html - fi-tgl-y: NOTRUN -> [SKIP][2] ([fdo#109315]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@amdgpu/amd_basic@query-info.html * igt@amdgpu/amd_cs_nop@fork-compute0: - fi-tgl-u2: NOTRUN -> [SKIP][3] ([fdo#109315] / [i915#2575]) +16 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@amdgpu/amd_cs_nop@fork-compute0.html * igt@amdgpu/amd_cs_nop@fork-gfx0: - fi-tgl-y: NOTRUN -> [SKIP][4] ([fdo#109315] / [i915#2575]) +16 similar issues [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@amdgpu/amd_cs_nop@fork-gfx0.html * igt@amdgpu/amd_cs_nop@sync-fork-compute0: - fi-kbl-soraka: NOTRUN -> [SKIP][5] ([fdo#109271]) +13 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-kbl-soraka/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html * igt@gem_huc_copy@huc-copy: - fi-tgl-u2: NOTRUN -> [SKIP][6] ([i915#2190]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@gem_huc_copy@huc-copy.html - fi-tgl-y: NOTRUN -> [SKIP][7] ([i915#2190]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@gem_huc_copy@huc-copy.html * igt@i915_selftest@live@gt_lrc: - fi-tgl-y: NOTRUN -> [DMESG-FAIL][8] ([i915#2373]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@i915_selftest@live@gt_lrc.html * igt@i915_selftest@live@gt_pm: - fi-tgl-y: NOTRUN -> [DMESG-FAIL][9] ([i915#1759]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@i915_selftest@live@gt_pm.html * igt@kms_addfb_basic@addfb25-y-tiled-small-legacy: - fi-bdw-5557u: NOTRUN -> [SKIP][10] ([fdo#109271]) +3 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-bdw-5557u/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html * igt@kms_chamelium@dp-crc-fast: - fi-bdw-5557u: NOTRUN -> [SKIP][11] ([fdo#109271] / [fdo#111827]) +8 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-bdw-5557u/igt@kms_chamelium@dp-crc-fast.html * igt@kms_chamelium@dp-hpd-fast: - fi-tgl-u2: NOTRUN -> [SKIP][12] ([fdo#109284] / [fdo#111827]) +8 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@kms_chamelium@dp-hpd-fast.html * igt@kms_chamelium@vga-edid-read: - fi-tgl-y: NOTRUN -> [SKIP][13] ([fdo#111827]) +8 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@kms_chamelium@vga-edid-read.html * igt@kms_force_connector_basic@force-load-detect: - fi-tgl-u2: NOTRUN -> [SKIP][14] ([fdo#109285]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@kms_force_connector_basic@force-load-detect.html - fi-tgl-y: NOTRUN -> [SKIP][15] ([fdo#109285]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@kms_force_connector_basic@force-load-detect.html * igt@prime_vgem@basic-userptr: - fi-tgl-y: NOTRUN -> [SKIP][16] ([i915#3301]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-y/igt@prime_vgem@basic-userptr.html - fi-tgl-u2: NOTRUN -> [SKIP][17] ([i915#3301]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-u2/igt@prime_vgem@basic-userptr.html #### Possible fixes #### * igt@gem_exec_suspend@basic-s0: - {fi-tgl-1115g4}: [FAIL][18] ([i915#1888]) -> [PASS][19] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s0.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s0.html * igt@kms_chamelium@dp-crc-fast: - fi-kbl-7500u: [FAIL][20] ([i915#1372]) -> [PASS][21] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html #### Warnings #### * igt@runner@aborted: - fi-bdw-5557u: [FAIL][22] ([i915#2426]) -> [FAIL][23] ([i915#1602] / [i915#2029]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/fi-bdw-5557u/igt@runner@aborted.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/fi-bdw-5557u/igt@runner@aborted.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1372]: https://gitlab.freedesktop.org/drm/intel/issues/1372 [i915#1602]: https://gitlab.freedesktop.org/drm/intel/issues/1602 [i915#1759]: https://gitlab.freedesktop.org/drm/intel/issues/1759 [i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888 [i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2373]: https://gitlab.freedesktop.org/drm/intel/issues/2373 [i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426 [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575 [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301 [i915#3717]: https://gitlab.freedesktop.org/drm/intel/issues/3717 Participating hosts (40 -> 37) ------------------------------ Additional (2): fi-tgl-y fi-tgl-u2 Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan bat-jsl-1 fi-bdw-samus Build changes ------------- * Linux: CI_DRM_10350 -> Patchwork_20645 CI-20190529: 20190529 CI_DRM_10350: 0da7e60301374bc5d2d53573b061cad7f6e2959e @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6144: bc65ee9ee6593716306448c9fb82c77f284f2148 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_20645: 3826bab150798cbdaa6b8f9ee331dfcc04a31334 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 3826bab15079 drm/i915/display/dsc: Force dsc BPP 5ad527f4291c drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable 1ef1011c1398 drm/i915/display: Add write permissions for fec support == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/index.html [-- Attachment #1.2: Type: text/html, Size: 9214 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for Set BPP in the kernel (rev5) 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni ` (5 preceding siblings ...) 2021-07-19 11:07 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2021-07-19 15:19 ` Patchwork 6 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-19 15:19 UTC (permalink / raw) To: Sharma, Swati2; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 30251 bytes --] == Series Details == Series: Set BPP in the kernel (rev5) URL : https://patchwork.freedesktop.org/series/92312/ State : success == Summary == CI Bug Log - changes from CI_DRM_10350_full -> Patchwork_20645_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_20645_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_persistence@legacy-engines-hang@blt: - shard-skl: NOTRUN -> [SKIP][1] ([fdo#109271]) +102 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl9/igt@gem_ctx_persistence@legacy-engines-hang@blt.html * igt@gem_ctx_persistence@legacy-engines-mixed: - shard-snb: NOTRUN -> [SKIP][2] ([fdo#109271] / [i915#1099]) +3 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-snb7/igt@gem_ctx_persistence@legacy-engines-mixed.html * igt@gem_exec_fair@basic-deadline: - shard-kbl: [PASS][3] -> [FAIL][4] ([i915#2846]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl1/igt@gem_exec_fair@basic-deadline.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl6/igt@gem_exec_fair@basic-deadline.html - shard-glk: [PASS][5] -> [FAIL][6] ([i915#2846]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-glk9/igt@gem_exec_fair@basic-deadline.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-glk2/igt@gem_exec_fair@basic-deadline.html - shard-apl: NOTRUN -> [FAIL][7] ([i915#2846]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl1/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-flow@rcs0: - shard-kbl: [PASS][8] -> [SKIP][9] ([fdo#109271]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl7/igt@gem_exec_fair@basic-flow@rcs0.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl6/igt@gem_exec_fair@basic-flow@rcs0.html - shard-tglb: [PASS][10] -> [FAIL][11] ([i915#2842]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-tglb7/igt@gem_exec_fair@basic-flow@rcs0.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb3/igt@gem_exec_fair@basic-flow@rcs0.html * igt@gem_exec_fair@basic-pace@vcs1: - shard-kbl: [PASS][12] -> [FAIL][13] ([i915#2842]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl2/igt@gem_exec_fair@basic-pace@vcs1.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl1/igt@gem_exec_fair@basic-pace@vcs1.html * igt@gem_mmap_gtt@cpuset-big-copy-odd: - shard-glk: [PASS][14] -> [FAIL][15] ([i915#307]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-glk4/igt@gem_mmap_gtt@cpuset-big-copy-odd.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-glk3/igt@gem_mmap_gtt@cpuset-big-copy-odd.html * igt@gem_mmap_gtt@cpuset-big-copy-xy: - shard-iclb: [PASS][16] -> [FAIL][17] ([i915#307]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb3/igt@gem_mmap_gtt@cpuset-big-copy-xy.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb4/igt@gem_mmap_gtt@cpuset-big-copy-xy.html * igt@gem_mmap_gtt@cpuset-medium-copy-xy: - shard-iclb: [PASS][18] -> [FAIL][19] ([i915#2428]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb7/igt@gem_mmap_gtt@cpuset-medium-copy-xy.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb4/igt@gem_mmap_gtt@cpuset-medium-copy-xy.html * igt@gem_render_copy@y-tiled-to-vebox-linear: - shard-iclb: NOTRUN -> [SKIP][20] ([i915#768]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb1/igt@gem_render_copy@y-tiled-to-vebox-linear.html * igt@gem_userptr_blits@dmabuf-sync: - shard-apl: NOTRUN -> [SKIP][21] ([fdo#109271] / [i915#3323]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl3/igt@gem_userptr_blits@dmabuf-sync.html * igt@gem_userptr_blits@input-checking: - shard-skl: NOTRUN -> [DMESG-WARN][22] ([i915#3002]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl9/igt@gem_userptr_blits@input-checking.html * igt@gem_workarounds@suspend-resume: - shard-skl: [PASS][23] -> [INCOMPLETE][24] ([i915#198]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl4/igt@gem_workarounds@suspend-resume.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl4/igt@gem_workarounds@suspend-resume.html * igt@gem_workarounds@suspend-resume-context: - shard-apl: [PASS][25] -> [DMESG-WARN][26] ([i915#180]) +1 similar issue [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl1/igt@gem_workarounds@suspend-resume-context.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl3/igt@gem_workarounds@suspend-resume-context.html * igt@i915_pm_rpm@modeset-non-lpsp: - shard-iclb: NOTRUN -> [SKIP][27] ([fdo#110892]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@i915_pm_rpm@modeset-non-lpsp.html * igt@i915_pm_rpm@modeset-pc8-residency-stress: - shard-apl: NOTRUN -> [SKIP][28] ([fdo#109271]) +354 similar issues [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl1/igt@i915_pm_rpm@modeset-pc8-residency-stress.html * igt@i915_selftest@mock@scatterlist: - shard-apl: NOTRUN -> [DMESG-WARN][29] ([i915#3746]) +17 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl1/igt@i915_selftest@mock@scatterlist.html * igt@i915_selftest@mock@vma: - shard-snb: NOTRUN -> [DMESG-WARN][30] ([i915#3746]) +17 similar issues [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-snb5/igt@i915_selftest@mock@vma.html * igt@kms_atomic_transition@plane-all-modeset-transition-fencing: - shard-iclb: NOTRUN -> [SKIP][31] ([i915#1769]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html * igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-async-flip: - shard-skl: NOTRUN -> [FAIL][32] ([i915#3722]) [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl3/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-async-flip.html * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-async-flip: - shard-skl: NOTRUN -> [FAIL][33] ([i915#3763]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl9/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-async-flip.html * igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip: - shard-apl: NOTRUN -> [SKIP][34] ([fdo#109271] / [i915#3777]) +1 similar issue [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl6/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip.html * igt@kms_ccs@pipe-a-crc-primary-rotation-180-yf_tiled_ccs: - shard-tglb: NOTRUN -> [SKIP][35] ([i915#3689]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb6/igt@kms_ccs@pipe-a-crc-primary-rotation-180-yf_tiled_ccs.html * igt@kms_chamelium@hdmi-edid-change-during-suspend: - shard-apl: NOTRUN -> [SKIP][36] ([fdo#109271] / [fdo#111827]) +31 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl1/igt@kms_chamelium@hdmi-edid-change-during-suspend.html * igt@kms_chamelium@hdmi-hpd-fast: - shard-snb: NOTRUN -> [SKIP][37] ([fdo#109271] / [fdo#111827]) +14 similar issues [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-snb5/igt@kms_chamelium@hdmi-hpd-fast.html * igt@kms_chamelium@vga-hpd-for-each-pipe: - shard-skl: NOTRUN -> [SKIP][38] ([fdo#109271] / [fdo#111827]) +8 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl1/igt@kms_chamelium@vga-hpd-for-each-pipe.html * igt@kms_color@pipe-d-ctm-0-5: - shard-iclb: NOTRUN -> [SKIP][39] ([fdo#109278] / [i915#1149]) [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_color@pipe-d-ctm-0-5.html * igt@kms_content_protection@atomic-dpms: - shard-apl: NOTRUN -> [TIMEOUT][40] ([i915#1319]) [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl6/igt@kms_content_protection@atomic-dpms.html * igt@kms_content_protection@dp-mst-type-1: - shard-iclb: NOTRUN -> [SKIP][41] ([i915#3116]) [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_content_protection@dp-mst-type-1.html * igt@kms_cursor_crc@pipe-a-cursor-suspend: - shard-skl: [PASS][42] -> [INCOMPLETE][43] ([i915#2828] / [i915#300]) [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl9/igt@kms_cursor_crc@pipe-a-cursor-suspend.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl8/igt@kms_cursor_crc@pipe-a-cursor-suspend.html * igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement: - shard-glk: NOTRUN -> [SKIP][44] ([fdo#109271]) +1 similar issue [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-glk7/igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement.html - shard-tglb: NOTRUN -> [SKIP][45] ([i915#3319]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb7/igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement.html * igt@kms_cursor_crc@pipe-d-cursor-256x85-random: - shard-iclb: NOTRUN -> [SKIP][46] ([fdo#109278]) +7 similar issues [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb1/igt@kms_cursor_crc@pipe-d-cursor-256x85-random.html * igt@kms_cursor_crc@pipe-d-cursor-512x170-offscreen: - shard-tglb: NOTRUN -> [SKIP][47] ([fdo#109279] / [i915#3359]) +1 similar issue [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-512x170-offscreen.html * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size: - shard-iclb: NOTRUN -> [SKIP][48] ([fdo#109274] / [fdo#109278]) [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions: - shard-skl: NOTRUN -> [FAIL][49] ([i915#2346]) [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl5/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html * igt@kms_flip@2x-flip-vs-panning-vs-hang: - shard-iclb: NOTRUN -> [SKIP][50] ([fdo#109274]) [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_flip@2x-flip-vs-panning-vs-hang.html * igt@kms_flip@flip-vs-expired-vblank@c-edp1: - shard-skl: [PASS][51] -> [FAIL][52] ([i915#79]) +1 similar issue [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl5/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl9/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html * igt@kms_flip@flip-vs-suspend@c-dp1: - shard-kbl: [PASS][53] -> [DMESG-WARN][54] ([i915#180]) +8 similar issues [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl2/igt@kms_flip@flip-vs-suspend@c-dp1.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl7/igt@kms_flip@flip-vs-suspend@c-dp1.html * igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1: - shard-skl: [PASS][55] -> [FAIL][56] ([i915#2122]) [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl10/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl3/igt@kms_flip@plain-flip-ts-check-interruptible@b-edp1.html * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-gtt: - shard-iclb: NOTRUN -> [SKIP][57] ([fdo#109280]) +1 similar issue [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-gtt.html * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-pwrite: - shard-snb: NOTRUN -> [SKIP][58] ([fdo#109271]) +292 similar issues [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-snb5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-pwrite.html * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-fullscreen: - shard-tglb: NOTRUN -> [SKIP][59] ([fdo#111825]) [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-fullscreen.html * igt@kms_hdr@bpc-switch-dpms: - shard-skl: [PASS][60] -> [FAIL][61] ([i915#1188]) [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl1/igt@kms_hdr@bpc-switch-dpms.html [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl6/igt@kms_hdr@bpc-switch-dpms.html * igt@kms_multipipe_modeset@basic-max-pipe-crc-check: - shard-iclb: NOTRUN -> [SKIP][62] ([i915#1839]) [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb1/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html * igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d: - shard-apl: NOTRUN -> [SKIP][63] ([fdo#109271] / [i915#533]) +3 similar issues [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl6/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html * igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes: - shard-apl: NOTRUN -> [DMESG-WARN][64] ([i915#180]) [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes.html * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb: - shard-apl: NOTRUN -> [FAIL][65] ([fdo#108145] / [i915#265]) +5 similar issues [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl7/igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb.html * igt@kms_plane_alpha_blend@pipe-a-coverage-vs-premult-vs-constant: - shard-iclb: [PASS][66] -> [SKIP][67] ([fdo#109278]) [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb5/igt@kms_plane_alpha_blend@pipe-a-coverage-vs-premult-vs-constant.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb2/igt@kms_plane_alpha_blend@pipe-a-coverage-vs-premult-vs-constant.html * igt@kms_plane_alpha_blend@pipe-b-alpha-opaque-fb: - shard-skl: NOTRUN -> [FAIL][68] ([fdo#108145] / [i915#265]) +2 similar issues [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-alpha-opaque-fb.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-apl: NOTRUN -> [SKIP][69] ([fdo#109271] / [i915#658]) +7 similar issues [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl6/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-0: - shard-skl: NOTRUN -> [SKIP][70] ([fdo#109271] / [i915#658]) [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl3/igt@kms_psr2_sf@plane-move-sf-dmg-area-0.html * igt@kms_psr@psr2_primary_mmap_gtt: - shard-iclb: NOTRUN -> [SKIP][71] ([fdo#109441]) [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_psr@psr2_primary_mmap_gtt.html * igt@kms_psr@psr2_sprite_plane_move: - shard-iclb: [PASS][72] -> [SKIP][73] ([fdo#109441]) +2 similar issues [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb8/igt@kms_psr@psr2_sprite_plane_move.html * igt@kms_tv_load_detect@load-detect: - shard-iclb: NOTRUN -> [SKIP][74] ([fdo#109309]) [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@kms_tv_load_detect@load-detect.html * igt@kms_writeback@writeback-pixel-formats: - shard-apl: NOTRUN -> [SKIP][75] ([fdo#109271] / [i915#2437]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl7/igt@kms_writeback@writeback-pixel-formats.html * igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame: - shard-iclb: NOTRUN -> [SKIP][76] ([fdo#109278] / [i915#2530]) [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@nouveau_crc@pipe-d-ctx-flip-skip-current-frame.html * igt@prime_nv_test@i915_import_gtt_mmap: - shard-tglb: NOTRUN -> [SKIP][77] ([fdo#109291]) [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb6/igt@prime_nv_test@i915_import_gtt_mmap.html * igt@sysfs_clients@fair-7: - shard-apl: NOTRUN -> [SKIP][78] ([fdo#109271] / [i915#2994]) +6 similar issues [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl3/igt@sysfs_clients@fair-7.html - shard-skl: NOTRUN -> [SKIP][79] ([fdo#109271] / [i915#2994]) [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl8/igt@sysfs_clients@fair-7.html * igt@tools_test@sysfs_l3_parity: - shard-iclb: NOTRUN -> [SKIP][80] ([fdo#109307]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@tools_test@sysfs_l3_parity.html #### Possible fixes #### * igt@gem_eio@unwedge-stress: - shard-tglb: [TIMEOUT][81] ([i915#2369] / [i915#3063] / [i915#3648]) -> [PASS][82] [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-tglb2/igt@gem_eio@unwedge-stress.html [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb3/igt@gem_eio@unwedge-stress.html * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-tglb: [FAIL][83] ([i915#2842]) -> [PASS][84] +1 similar issue [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-tglb1/igt@gem_exec_fair@basic-pace-share@rcs0.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-tglb2/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-pace@vecs0: - shard-kbl: [FAIL][85] ([i915#2842]) -> [PASS][86] +1 similar issue [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl2/igt@gem_exec_fair@basic-pace@vecs0.html [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl1/igt@gem_exec_fair@basic-pace@vecs0.html * igt@gem_exec_fair@basic-throttle@rcs0: - shard-iclb: [FAIL][87] ([i915#2849]) -> [PASS][88] [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb6/igt@gem_exec_fair@basic-throttle@rcs0.html [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@gem_exec_fair@basic-throttle@rcs0.html * igt@gem_softpin@noreloc-s3: - shard-apl: [DMESG-WARN][89] ([i915#180]) -> [PASS][90] +2 similar issues [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl3/igt@gem_softpin@noreloc-s3.html [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@gem_softpin@noreloc-s3.html * igt@i915_pm_dc@dc5-psr: - shard-iclb: [DMESG-WARN][91] ([i915#3698]) -> [PASS][92] [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb2/igt@i915_pm_dc@dc5-psr.html [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb1/igt@i915_pm_dc@dc5-psr.html * igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled: - shard-skl: [FAIL][93] -> [PASS][94] +1 similar issue [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl8/igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled.html [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl7/igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-apl: [INCOMPLETE][95] ([i915#180]) -> [PASS][96] [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl8/igt@kms_fbcon_fbt@fbc-suspend.html [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl1/igt@kms_fbcon_fbt@fbc-suspend.html * igt@kms_flip@2x-plain-flip-ts-check@ab-hdmi-a1-hdmi-a2: - shard-glk: [FAIL][97] ([i915#2122]) -> [PASS][98] [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-glk9/igt@kms_flip@2x-plain-flip-ts-check@ab-hdmi-a1-hdmi-a2.html [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-glk6/igt@kms_flip@2x-plain-flip-ts-check@ab-hdmi-a1-hdmi-a2.html * igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1: - shard-skl: [FAIL][99] ([i915#79]) -> [PASS][100] [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl3/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl2/igt@kms_flip@flip-vs-expired-vblank-interruptible@c-edp1.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-skl: [INCOMPLETE][101] ([i915#198]) -> [PASS][102] +1 similar issue [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl10/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl5/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b: - shard-skl: [DMESG-WARN][103] ([i915#1982]) -> [PASS][104] [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc: - shard-skl: [FAIL][105] ([fdo#108145] / [i915#265]) -> [PASS][106] +3 similar issues [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl10/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl3/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html * igt@kms_psr@psr2_cursor_mmap_cpu: - shard-iclb: [SKIP][107] ([fdo#109441]) -> [PASS][108] [107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb5/igt@kms_psr@psr2_cursor_mmap_cpu.html [108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html * igt@sysfs_timeslice_duration@timeout@rcs0: - shard-skl: [FAIL][109] ([i915#3259]) -> [PASS][110] [109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-skl2/igt@sysfs_timeslice_duration@timeout@rcs0.html [110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-skl2/igt@sysfs_timeslice_duration@timeout@rcs0.html #### Warnings #### * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [FAIL][111] ([i915#2842]) -> [FAIL][112] ([i915#2851]) [111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-glk3/igt@gem_exec_fair@basic-pace-solo@rcs0.html [112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-glk4/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@i915_pm_dc@dc6-psr: - shard-iclb: [DMESG-WARN][113] ([i915#3698]) -> [FAIL][114] ([i915#454]) [113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb1/igt@i915_pm_dc@dc6-psr.html [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb6/igt@i915_pm_dc@dc6-psr.html * igt@i915_pm_rc6_residency@rc6-fence: - shard-iclb: [WARN][115] ([i915#2684]) -> [WARN][116] ([i915#1804] / [i915#2684]) [115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb8/igt@i915_pm_rc6_residency@rc6-fence.html [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb4/igt@i915_pm_rc6_residency@rc6-fence.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-0: - shard-iclb: [SKIP][117] ([i915#2920]) -> [SKIP][118] ([i915#658]) +1 similar issue [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb2/igt@kms_psr2_sf@plane-move-sf-dmg-area-0.html [118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb8/igt@kms_psr2_sf@plane-move-sf-dmg-area-0.html * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-4: - shard-iclb: [SKIP][119] ([i915#658]) -> [SKIP][120] ([i915#2920]) +1 similar issue [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb5/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-4.html [120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-4.html * igt@runner@aborted: - shard-kbl: ([FAIL][121], [FAIL][122]) ([i915#3002] / [i915#3363]) -> ([FAIL][123], [FAIL][124], [FAIL][125], [FAIL][126], [FAIL][127], [FAIL][128], [FAIL][129]) ([i915#1436] / [i915#180] / [i915#1814] / [i915#2505] / [i915#3002] / [i915#3363] / [i915#602]) [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl2/igt@runner@aborted.html [122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-kbl7/igt@runner@aborted.html [123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl6/igt@runner@aborted.html [124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl7/igt@runner@aborted.html [125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl4/igt@runner@aborted.html [126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl4/igt@runner@aborted.html [127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl7/igt@runner@aborted.html [128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl7/igt@runner@aborted.html [129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-kbl1/igt@runner@aborted.html - shard-iclb: ([FAIL][130], [FAIL][131], [FAIL][132], [FAIL][133], [FAIL][134]) ([i915#1814] / [i915#3002] / [i915#3702]) -> ([FAIL][135], [FAIL][136], [FAIL][137]) ([i915#1814] / [i915#3002]) [130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb3/igt@runner@aborted.html [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb6/igt@runner@aborted.html [132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb1/igt@runner@aborted.html [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb2/igt@runner@aborted.html [134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-iclb1/igt@runner@aborted.html [135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb5/igt@runner@aborted.html [136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb8/igt@runner@aborted.html [137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-iclb5/igt@runner@aborted.html - shard-apl: ([FAIL][138], [FAIL][139], [FAIL][140], [FAIL][141], [FAIL][142], [FAIL][143], [FAIL][144], [FAIL][145]) ([i915#180] / [i915#3002] / [i915#3363]) -> ([FAIL][146], [FAIL][147], [FAIL][148], [FAIL][149], [FAIL][150]) ([fdo#109271] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363]) [138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl2/igt@runner@aborted.html [139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl3/igt@runner@aborted.html [140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl3/igt@runner@aborted.html [141]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl7/igt@runner@aborted.html [142]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl8/igt@runner@aborted.html [143]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl6/igt@runner@aborted.html [144]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl3/igt@runner@aborted.html [145]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10350/shard-apl3/igt@runner@aborted.html [146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@runner@aborted.html [147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@runner@aborted.html [148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@runner@aborted.html [149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl8/igt@runner@aborted.html [150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/shard-apl3/igt@runner@aborted.html ### Piglit changes ### {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274 [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278 [fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279 [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280 [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291 [fdo#109307]: https://bugs.freedesktop.org/show_bug.cgi?id=109307 [fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309 [fdo#109441]: https://bugs == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20645/index.html [-- Attachment #1.2: Type: text/html, Size: 35192 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 0/3] Enable setting custom DSC BPP @ 2021-07-20 6:49 Vandita Kulkarni 2021-07-20 6:49 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni 0 siblings, 1 reply; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-20 6:49 UTC (permalink / raw) To: intel-gfx This series add debugfs entry to force dsc bpp to ceratin valid test value, for validation needs. This series has been tested locally. With the introduction of i915_dsc_bpp debugfs the expectation is that whenever there is force_dsc_en set, force_dsc_bpp should have a valid value for that format which is between bpp to bpp-1. This makes the older test kms_dp_dsc --basic fail as in that case force_dsc_bpp would be 0 and is not a valid value. Have tested with local changes on the same. The series https://patchwork.freedesktop.org/series/91772/ have the base patches and would need some work on the debugfs name change, giving default value for force_dsc_bpp in case of basic-dsc-enable test cases, clearing up of the force_dsc_bpp value while exiting the test. Which will be floated shortly. Have added minor fix on the feci debugfs interface. If further changes are needed on the same will float them in a different series. This series has been reviewed here https://patchwork.freedesktop.org/series/92312/#rev5 Resubmitting it here as the series submitter got overridden due to one of the review comment mishaps. Patnana Venkata Sai (1): drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni (2): drm/i915/display: Add write permissions for fec support drm/i915/display/dsc: Force dsc BPP .../drm/i915/display/intel_display_debugfs.c | 78 ++++++++++++++++++- .../drm/i915/display/intel_display_types.h | 1 + drivers/gpu/drm/i915/display/intel_dp.c | 17 ++++ 3 files changed, 94 insertions(+), 2 deletions(-) -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-20 6:49 [Intel-gfx] [v7 0/3] Enable setting custom DSC BPP Vandita Kulkarni @ 2021-07-20 6:49 ` Vandita Kulkarni 0 siblings, 0 replies; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-20 6:49 UTC (permalink / raw) To: intel-gfx Set DSC BPP to the value forced through debugfs. It can go from bpc to bpp-1. v2: Use default dsc bpp when we are just doing force_dsc_en, use default dsc bpp for invalid force_dsc_bpp values. (Jani) Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> Reviewed-by: Swati Sharma <swati2.sharma@intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 5b52beaddada..c386ef8eb200 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1274,6 +1274,23 @@ static int intel_dp_dsc_compute_config(struct intel_dp *intel_dp, pipe_config->pipe_bpp); pipe_config->dsc.slice_count = dsc_dp_slice_count; } + + /* As of today we support DSC for only RGB */ + if (intel_dp->force_dsc_bpp) { + if (intel_dp->force_dsc_bpp >= 8 && + intel_dp->force_dsc_bpp < pipe_bpp) { + drm_dbg_kms(&dev_priv->drm, + "DSC BPP forced to %d", + intel_dp->force_dsc_bpp); + pipe_config->dsc.compressed_bpp = + intel_dp->force_dsc_bpp; + } else { + drm_dbg_kms(&dev_priv->drm, + "Invalid DSC BPP %d", + intel_dp->force_dsc_bpp); + } + } + /* * VDSC engine operates at 1 Pixel per clock, so if peak pixel rate * is greater than the maximum Cdclock and if slice count is even -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 0/3] Set BPP in the kernel @ 2021-07-08 10:25 Vandita Kulkarni 2021-07-08 10:25 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni 0 siblings, 1 reply; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-08 10:25 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula This series add debugfs entry to force dsc bpp to ceratin valid test value, for validation needs. This series has been tested locally. With the introduction of i915_dsc_bpp debugfs the expectation is that whenever there is force_dsc_en set, force_dsc_bpp should have a valid value for that format which is between bpp to bpp-1. This makes the older test kms_dp_dsc --basic fail as in that case force_dsc_bpp would be 0 and is not a valid value. Have tested with local changes on the same. The series https://patchwork.freedesktop.org/series/91772/ have the base patches and would need some work on the debugfs name change, giving default value for force_dsc_bpp in case of basic-dsc-enable test cases, clearing up of the force_dsc_bpp value while exiting the test. Which will be floated shortly. Have added minor fix on the feci debugfs interface. If further changes are needed on the same will float them in a different series. Patnana Venkata Sai (1): drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni (2): drm/i915/display: Add write permissions for fec support drm/i915/display/dsc: Force dsc BPP .../drm/i915/display/intel_display_debugfs.c | 78 ++++++++++++++++++- .../drm/i915/display/intel_display_types.h | 1 + drivers/gpu/drm/i915/display/intel_dp.c | 17 ++++ 3 files changed, 94 insertions(+), 2 deletions(-) -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 10:25 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni @ 2021-07-08 10:25 ` Vandita Kulkarni 2021-07-08 13:09 ` Jani Nikula 0 siblings, 1 reply; 15+ messages in thread From: Vandita Kulkarni @ 2021-07-08 10:25 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Set DSC BPP to the value forced through debugfs. It can go from bpc to bpp-1. Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 5b52beaddada..3e50cdd7e448 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1240,6 +1240,23 @@ static int intel_dp_dsc_compute_config(struct intel_dp *intel_dp, pipe_config->port_clock = intel_dp->common_rates[limits->max_clock]; pipe_config->lane_count = limits->max_lane_count; + if (intel_dp->force_dsc_en) { + /* As of today we support DSC for only RGB */ + if (intel_dp->force_dsc_bpp >= 8 && + intel_dp->force_dsc_bpp < pipe_bpp) { + drm_dbg_kms(&dev_priv->drm, + "DSC BPP forced to %d", + intel_dp->force_dsc_bpp); + pipe_config->dsc.compressed_bpp = + intel_dp->force_dsc_bpp; + } else { + drm_dbg_kms(&dev_priv->drm, + "Invalid DSC BPP %d", + intel_dp->force_dsc_bpp); + return -EINVAL; + } + } + if (intel_dp_is_edp(intel_dp)) { pipe_config->dsc.compressed_bpp = min_t(u16, drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, -- 2.32.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 10:25 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni @ 2021-07-08 13:09 ` Jani Nikula 2021-07-08 13:13 ` Jani Nikula 0 siblings, 1 reply; 15+ messages in thread From: Jani Nikula @ 2021-07-08 13:09 UTC (permalink / raw) To: Vandita Kulkarni, intel-gfx On Thu, 08 Jul 2021, Vandita Kulkarni <vandita.kulkarni@intel.com> wrote: > Set DSC BPP to the value forced through > debugfs. It can go from bpc to bpp-1. > > Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 5b52beaddada..3e50cdd7e448 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -1240,6 +1240,23 @@ static int intel_dp_dsc_compute_config(struct intel_dp *intel_dp, > pipe_config->port_clock = intel_dp->common_rates[limits->max_clock]; > pipe_config->lane_count = limits->max_lane_count; > > + if (intel_dp->force_dsc_en) { > + /* As of today we support DSC for only RGB */ > + if (intel_dp->force_dsc_bpp >= 8 && > + intel_dp->force_dsc_bpp < pipe_bpp) { > + drm_dbg_kms(&dev_priv->drm, > + "DSC BPP forced to %d", > + intel_dp->force_dsc_bpp); > + pipe_config->dsc.compressed_bpp = > + intel_dp->force_dsc_bpp; > + } else { > + drm_dbg_kms(&dev_priv->drm, > + "Invalid DSC BPP %d", > + intel_dp->force_dsc_bpp); > + return -EINVAL; I'd just let it use the normal compressed_bpp, with the debug message, instead of returning -EINVAL. > + } > + } > + This should be *after* the below blocks, because otherwise compressed_bpp will be overridden by the normal case, not by the force case! BR, Jani. > if (intel_dp_is_edp(intel_dp)) { > pipe_config->dsc.compressed_bpp = > min_t(u16, drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 13:09 ` Jani Nikula @ 2021-07-08 13:13 ` Jani Nikula 2021-07-08 13:24 ` Kulkarni, Vandita 0 siblings, 1 reply; 15+ messages in thread From: Jani Nikula @ 2021-07-08 13:13 UTC (permalink / raw) To: Vandita Kulkarni, intel-gfx On Thu, 08 Jul 2021, Jani Nikula <jani.nikula@intel.com> wrote: > On Thu, 08 Jul 2021, Vandita Kulkarni <vandita.kulkarni@intel.com> wrote: >> Set DSC BPP to the value forced through >> debugfs. It can go from bpc to bpp-1. >> >> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> >> --- >> drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c >> index 5b52beaddada..3e50cdd7e448 100644 >> --- a/drivers/gpu/drm/i915/display/intel_dp.c >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c >> @@ -1240,6 +1240,23 @@ static int intel_dp_dsc_compute_config(struct intel_dp *intel_dp, >> pipe_config->port_clock = intel_dp->common_rates[limits->max_clock]; >> pipe_config->lane_count = limits->max_lane_count; >> >> + if (intel_dp->force_dsc_en) { Oh, this should check for intel_dp->force_dsc_bpp. We don't want to always force the bpp when we force dsc enable. >> + /* As of today we support DSC for only RGB */ >> + if (intel_dp->force_dsc_bpp >= 8 && >> + intel_dp->force_dsc_bpp < pipe_bpp) { >> + drm_dbg_kms(&dev_priv->drm, >> + "DSC BPP forced to %d", >> + intel_dp->force_dsc_bpp); >> + pipe_config->dsc.compressed_bpp = >> + intel_dp->force_dsc_bpp; >> + } else { >> + drm_dbg_kms(&dev_priv->drm, >> + "Invalid DSC BPP %d", >> + intel_dp->force_dsc_bpp); >> + return -EINVAL; > > I'd just let it use the normal compressed_bpp, with the debug message, > instead of returning -EINVAL. > >> + } >> + } >> + > > This should be *after* the below blocks, because otherwise > compressed_bpp will be overridden by the normal case, not by the force > case! > > BR, > Jani. > >> if (intel_dp_is_edp(intel_dp)) { >> pipe_config->dsc.compressed_bpp = >> min_t(u16, drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 13:13 ` Jani Nikula @ 2021-07-08 13:24 ` Kulkarni, Vandita 2021-07-08 16:23 ` Jani Nikula 0 siblings, 1 reply; 15+ messages in thread From: Kulkarni, Vandita @ 2021-07-08 13:24 UTC (permalink / raw) To: Nikula, Jani, intel-gfx > -----Original Message----- > From: Nikula, Jani <jani.nikula@intel.com> > Sent: Thursday, July 8, 2021 6:44 PM > To: Kulkarni, Vandita <vandita.kulkarni@intel.com>; intel- > gfx@lists.freedesktop.org > Cc: Kulkarni, Vandita <vandita.kulkarni@intel.com> > Subject: Re: [v7 3/3] drm/i915/display/dsc: Force dsc BPP > > On Thu, 08 Jul 2021, Jani Nikula <jani.nikula@intel.com> wrote: > > On Thu, 08 Jul 2021, Vandita Kulkarni <vandita.kulkarni@intel.com> wrote: > >> Set DSC BPP to the value forced through debugfs. It can go from bpc > >> to bpp-1. > >> > >> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> > >> --- > >> drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ > >> 1 file changed, 17 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > >> b/drivers/gpu/drm/i915/display/intel_dp.c > >> index 5b52beaddada..3e50cdd7e448 100644 > >> --- a/drivers/gpu/drm/i915/display/intel_dp.c > >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c > >> @@ -1240,6 +1240,23 @@ static int intel_dp_dsc_compute_config(struct > intel_dp *intel_dp, > >> pipe_config->port_clock = intel_dp->common_rates[limits- > >max_clock]; > >> pipe_config->lane_count = limits->max_lane_count; > >> > >> + if (intel_dp->force_dsc_en) { > > Oh, this should check for intel_dp->force_dsc_bpp. We don't want to always > force the bpp when we force dsc enable. Okay will fix this. And I was returning -EINVAL , to fail the test on setting invalid BPP. > > >> + /* As of today we support DSC for only RGB */ > >> + if (intel_dp->force_dsc_bpp >= 8 && > >> + intel_dp->force_dsc_bpp < pipe_bpp) { > >> + drm_dbg_kms(&dev_priv->drm, > >> + "DSC BPP forced to %d", > >> + intel_dp->force_dsc_bpp); > >> + pipe_config->dsc.compressed_bpp = > >> + intel_dp->force_dsc_bpp; > >> + } else { > >> + drm_dbg_kms(&dev_priv->drm, > >> + "Invalid DSC BPP %d", > >> + intel_dp->force_dsc_bpp); > >> + return -EINVAL; > > > > I'd just let it use the normal compressed_bpp, with the debug message, > > instead of returning -EINVAL. > > > >> + } > >> + } > >> + > > > > This should be *after* the below blocks, because otherwise > > compressed_bpp will be overridden by the normal case, not by the force > > case! > > > > BR, > > Jani. > > > >> if (intel_dp_is_edp(intel_dp)) { > >> pipe_config->dsc.compressed_bpp = > >> min_t(u16, > drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, > > -- > Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 13:24 ` Kulkarni, Vandita @ 2021-07-08 16:23 ` Jani Nikula 2021-07-08 16:40 ` Kulkarni, Vandita 0 siblings, 1 reply; 15+ messages in thread From: Jani Nikula @ 2021-07-08 16:23 UTC (permalink / raw) To: Kulkarni, Vandita, intel-gfx On Thu, 08 Jul 2021, "Kulkarni, Vandita" <vandita.kulkarni@intel.com> wrote: >> -----Original Message----- >> From: Nikula, Jani <jani.nikula@intel.com> >> Sent: Thursday, July 8, 2021 6:44 PM >> To: Kulkarni, Vandita <vandita.kulkarni@intel.com>; intel- >> gfx@lists.freedesktop.org >> Cc: Kulkarni, Vandita <vandita.kulkarni@intel.com> >> Subject: Re: [v7 3/3] drm/i915/display/dsc: Force dsc BPP >> >> On Thu, 08 Jul 2021, Jani Nikula <jani.nikula@intel.com> wrote: >> > On Thu, 08 Jul 2021, Vandita Kulkarni <vandita.kulkarni@intel.com> wrote: >> >> Set DSC BPP to the value forced through debugfs. It can go from bpc >> >> to bpp-1. >> >> >> >> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> >> >> --- >> >> drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ >> >> 1 file changed, 17 insertions(+) >> >> >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c >> >> b/drivers/gpu/drm/i915/display/intel_dp.c >> >> index 5b52beaddada..3e50cdd7e448 100644 >> >> --- a/drivers/gpu/drm/i915/display/intel_dp.c >> >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c >> >> @@ -1240,6 +1240,23 @@ static int intel_dp_dsc_compute_config(struct >> intel_dp *intel_dp, >> >> pipe_config->port_clock = intel_dp->common_rates[limits- >> >max_clock]; >> >> pipe_config->lane_count = limits->max_lane_count; >> >> >> >> + if (intel_dp->force_dsc_en) { >> >> Oh, this should check for intel_dp->force_dsc_bpp. We don't want to always >> force the bpp when we force dsc enable. > Okay will fix this. > And I was returning -EINVAL , to fail the test on setting invalid BPP. Okay, if it makes the test easier, I guess it's fine. Up to you. BR, Jani. > >> >> >> + /* As of today we support DSC for only RGB */ >> >> + if (intel_dp->force_dsc_bpp >= 8 && >> >> + intel_dp->force_dsc_bpp < pipe_bpp) { >> >> + drm_dbg_kms(&dev_priv->drm, >> >> + "DSC BPP forced to %d", >> >> + intel_dp->force_dsc_bpp); >> >> + pipe_config->dsc.compressed_bpp = >> >> + intel_dp->force_dsc_bpp; >> >> + } else { >> >> + drm_dbg_kms(&dev_priv->drm, >> >> + "Invalid DSC BPP %d", >> >> + intel_dp->force_dsc_bpp); >> >> + return -EINVAL; >> > >> > I'd just let it use the normal compressed_bpp, with the debug message, >> > instead of returning -EINVAL. >> > >> >> + } >> >> + } >> >> + >> > >> > This should be *after* the below blocks, because otherwise >> > compressed_bpp will be overridden by the normal case, not by the force >> > case! >> > >> > BR, >> > Jani. >> > >> >> if (intel_dp_is_edp(intel_dp)) { >> >> pipe_config->dsc.compressed_bpp = >> >> min_t(u16, >> drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, >> >> -- >> Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP 2021-07-08 16:23 ` Jani Nikula @ 2021-07-08 16:40 ` Kulkarni, Vandita 0 siblings, 0 replies; 15+ messages in thread From: Kulkarni, Vandita @ 2021-07-08 16:40 UTC (permalink / raw) To: Nikula, Jani, intel-gfx > -----Original Message----- > From: Nikula, Jani <jani.nikula@intel.com> > Sent: Thursday, July 8, 2021 9:53 PM > To: Kulkarni, Vandita <vandita.kulkarni@intel.com>; intel- > gfx@lists.freedesktop.org > Subject: RE: [v7 3/3] drm/i915/display/dsc: Force dsc BPP > > On Thu, 08 Jul 2021, "Kulkarni, Vandita" <vandita.kulkarni@intel.com> wrote: > >> -----Original Message----- > >> From: Nikula, Jani <jani.nikula@intel.com> > >> Sent: Thursday, July 8, 2021 6:44 PM > >> To: Kulkarni, Vandita <vandita.kulkarni@intel.com>; intel- > >> gfx@lists.freedesktop.org > >> Cc: Kulkarni, Vandita <vandita.kulkarni@intel.com> > >> Subject: Re: [v7 3/3] drm/i915/display/dsc: Force dsc BPP > >> > >> On Thu, 08 Jul 2021, Jani Nikula <jani.nikula@intel.com> wrote: > >> > On Thu, 08 Jul 2021, Vandita Kulkarni <vandita.kulkarni@intel.com> > wrote: > >> >> Set DSC BPP to the value forced through debugfs. It can go from > >> >> bpc to bpp-1. > >> >> > >> >> Signed-off-by: Vandita Kulkarni <vandita.kulkarni@intel.com> > >> >> --- > >> >> drivers/gpu/drm/i915/display/intel_dp.c | 17 +++++++++++++++++ > >> >> 1 file changed, 17 insertions(+) > >> >> > >> >> diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > >> >> b/drivers/gpu/drm/i915/display/intel_dp.c > >> >> index 5b52beaddada..3e50cdd7e448 100644 > >> >> --- a/drivers/gpu/drm/i915/display/intel_dp.c > >> >> +++ b/drivers/gpu/drm/i915/display/intel_dp.c > >> >> @@ -1240,6 +1240,23 @@ static int > >> >> intel_dp_dsc_compute_config(struct > >> intel_dp *intel_dp, > >> >> pipe_config->port_clock = intel_dp->common_rates[limits- > >> >max_clock]; > >> >> pipe_config->lane_count = limits->max_lane_count; > >> >> > >> >> + if (intel_dp->force_dsc_en) { > >> > >> Oh, this should check for intel_dp->force_dsc_bpp. We don't want to > >> always force the bpp when we force dsc enable. > > Okay will fix this. > > And I was returning -EINVAL , to fail the test on setting invalid BPP. > > Okay, if it makes the test easier, I guess it's fine. Up to you. Okay, for now I have sent a patch like you suggested, as I see that there are no negative test cases . Have sent the v2 of this patch. So, it wouldn't make much difference. Thanks, Vandita > > BR, > Jani. > > > > >> > >> >> + /* As of today we support DSC for only RGB */ > >> >> + if (intel_dp->force_dsc_bpp >= 8 && > >> >> + intel_dp->force_dsc_bpp < pipe_bpp) { > >> >> + drm_dbg_kms(&dev_priv->drm, > >> >> + "DSC BPP forced to %d", > >> >> + intel_dp->force_dsc_bpp); > >> >> + pipe_config->dsc.compressed_bpp = > >> >> + intel_dp->force_dsc_bpp; > >> >> + } else { > >> >> + drm_dbg_kms(&dev_priv->drm, > >> >> + "Invalid DSC BPP %d", > >> >> + intel_dp->force_dsc_bpp); > >> >> + return -EINVAL; > >> > > >> > I'd just let it use the normal compressed_bpp, with the debug > >> > message, instead of returning -EINVAL. > >> > > >> >> + } > >> >> + } > >> >> + > >> > > >> > This should be *after* the below blocks, because otherwise > >> > compressed_bpp will be overridden by the normal case, not by the > >> > force case! > >> > > >> > BR, > >> > Jani. > >> > > >> >> if (intel_dp_is_edp(intel_dp)) { > >> >> pipe_config->dsc.compressed_bpp = > >> >> min_t(u16, > >> drm_edp_dsc_sink_output_bpp(intel_dp->dsc_dpcd) >> 4, > >> > >> -- > >> Jani Nikula, Intel Open Source Graphics Center > > -- > Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2021-07-20 6:49 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-07-19 9:50 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 1/3] drm/i915/display: Add write permissions for fec support Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 2/3] drm/i915/display/dsc: Add Per connector debugfs node for DSC BPP enable Vandita Kulkarni 2021-07-19 9:50 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni 2021-07-19 10:40 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Set BPP in the kernel (rev5) Patchwork 2021-07-19 10:44 ` [Intel-gfx] ✗ Fi.CI.DOCS: " Patchwork 2021-07-19 11:07 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2021-07-19 15:19 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork -- strict thread matches above, loose matches on Subject: below -- 2021-07-20 6:49 [Intel-gfx] [v7 0/3] Enable setting custom DSC BPP Vandita Kulkarni 2021-07-20 6:49 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni 2021-07-08 10:25 [Intel-gfx] [v7 0/3] Set BPP in the kernel Vandita Kulkarni 2021-07-08 10:25 ` [Intel-gfx] [v7 3/3] drm/i915/display/dsc: Force dsc BPP Vandita Kulkarni 2021-07-08 13:09 ` Jani Nikula 2021-07-08 13:13 ` Jani Nikula 2021-07-08 13:24 ` Kulkarni, Vandita 2021-07-08 16:23 ` Jani Nikula 2021-07-08 16:40 ` Kulkarni, Vandita
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.