Re: [PATCH v2 16/22] drm/msm/dpu: Configure CWB in writeback encoder

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 30/09/2024 21:19, Jessica Zhang wrote:


On 9/30/2024 7:17 AM, neil.armstrong@xxxxxxxxxx wrote:
On 25/09/2024 00:59, Jessica Zhang wrote:
Cache the CWB block mask in the DPU virtual encoder and configure CWB
according to the CWB block mask within the writeback phys encoder

Signed-off-by: Jessica Zhang <quic_jesszhan@xxxxxxxxxxx>
---
  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        | 83 ++++++++++++ +++++++++-
  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h   | 16 ++++-
  .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c    |  4 +-
  3 files changed, 100 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c b/drivers/ gpu/drm/msm/disp/dpu1/dpu_encoder.c
index b2f0bf412451..2628f2d55cb3 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
@@ -24,6 +24,7 @@
  #include "dpu_hw_catalog.h"
  #include "dpu_hw_intf.h"
  #include "dpu_hw_ctl.h"
+#include "dpu_hw_cwb.h"
  #include "dpu_hw_dspp.h"
  #include "dpu_hw_dsc.h"
  #include "dpu_hw_merge3d.h"
@@ -139,6 +140,7 @@ enum dpu_enc_rc_states {
   *            num_phys_encs.
   * @hw_dsc:        Handle to the DSC blocks used for the display.
   * @dsc_mask:        Bitmask of used DSC blocks.
+ * @cwb_mask        Bitmask of used CWB muxes
   * @intfs_swapped:    Whether or not the phys_enc interfaces have been swapped
   *            for partial update right-only cases, such as pingpong
   *            split where virtual pingpong does not generate IRQs
@@ -185,6 +187,7 @@ struct dpu_encoder_virt {
      struct dpu_hw_dsc *hw_dsc[MAX_CHANNELS_PER_ENC];
      unsigned int dsc_mask;
+    unsigned int cwb_mask;
      bool intfs_swapped;
@@ -1063,6 +1066,7 @@ static void dpu_encoder_virt_atomic_mode_set(struct drm_encoder *drm_enc,
      int num_cwb = 0;
      bool is_cwb_encoder;
      unsigned int dsc_mask = 0;
+    unsigned int cwb_mask = 0;
      int i;
      if (!drm_enc) {
@@ -1103,8 +1107,12 @@ static void dpu_encoder_virt_atomic_mode_set(struct drm_encoder *drm_enc,
                                 ARRAY_SIZE(hw_pp));
      }
-    for (i = 0; i < num_cwb; i++)
+    for (i = 0; i < num_cwb; i++) {
          dpu_enc->hw_cwb[i] = to_dpu_hw_cwb(hw_cwb[i]);
+        cwb_mask |= BIT(dpu_enc->hw_cwb[i]->idx - CWB_0);
+    }
+
+    dpu_enc->cwb_mask = cwb_mask;
      dpu_rm_get_assigned_resources(&dpu_kms->rm, global_state,
              drm_enc->crtc, DPU_HW_BLK_CTL, hw_ctl, ARRAY_SIZE(hw_ctl));
@@ -2071,6 +2079,9 @@ void dpu_encoder_helper_phys_cleanup(struct dpu_encoder_phys *phys_enc)
          }
      }
+    if (dpu_enc->cwb_mask)
+        dpu_encoder_helper_phys_setup_cwb(phys_enc, false);
+
      /* reset the merge 3D HW block */
      if (phys_enc->hw_pp && phys_enc->hw_pp->merge_3d) {
          phys_enc->hw_pp->merge_3d->ops.setup_3d_mode(phys_enc- >hw_pp->merge_3d,
@@ -2114,6 +2125,68 @@ void dpu_encoder_helper_phys_cleanup(struct dpu_encoder_phys *phys_enc)
      ctl->ops.clear_pending_flush(ctl);
  }
+void dpu_encoder_helper_phys_setup_cwb(struct dpu_encoder_phys *phys_enc,
+                       bool enable)
+{
+    struct dpu_encoder_virt *dpu_enc = to_dpu_encoder_virt(phys_enc- >parent);
+    struct dpu_hw_cwb *hw_cwb;
+    struct dpu_hw_cwb_setup_cfg cwb_cfg;
+
+    struct dpu_kms *dpu_kms;
+    struct dpu_global_state *global_state;
+    struct dpu_hw_blk *rt_pp_list[MAX_CHANNELS_PER_ENC];
+    int num_pp, rt_pp_idx[MAX_CHANNELS_PER_ENC];
+
+    if (!phys_enc || !phys_enc->hw_wb || !dpu_enc->cwb_mask)
+        return;
+
+    dpu_kms = phys_enc->dpu_kms;
+    global_state = dpu_kms_get_existing_global_state(dpu_kms);
+    num_pp = dpu_rm_get_assigned_resources(&dpu_kms->rm, global_state,
+                           phys_enc->parent->crtc,
+                           DPU_HW_BLK_PINGPONG, rt_pp_list,
+                           ARRAY_SIZE(rt_pp_list));
+
+    if (num_pp == 0 || num_pp > MAX_CHANNELS_PER_ENC) {
+        DPU_DEBUG_ENC(dpu_enc, "invalid num_pp %d\n", num_pp);
+        return;
+    }
+
+    for (int i = 0; i < num_pp; i++) {
+        struct dpu_hw_pingpong *hw_pp = to_dpu_hw_pingpong(rt_pp_list[i]);
+
+        for (int j = 0; j < ARRAY_SIZE(dpu_enc->hw_cwb); j++) {
+            hw_cwb = dpu_enc->hw_cwb[i];
+
+            /*
+             * Even CWB muxes must take input from even real-time
+             * pingpongs and odd CWB muxes must take input from odd
+             * pingpongs
+             */
+            if (hw_pp->idx % 2 == hw_cwb->idx % 2) {

When running igt-test on QRD8650, I get:
# IGT_FRAME_DUMP_PATH=$PWD FRAME_PNG_FILE_NAME=pwet /usr/libexec/igt- gpu-tools/kms_writeback -d

Hi Neil,

Thanks for reporting this. Unfortunately, I'm not able to recreate this on the MTP8650.

How many/which non-WB outputs are you testing with?

Here's the modetest output:
==================><====================================================
Encoders:
id	crtc	type	possible crtcs	possible clones	
32	103	DSI	0x00000007	0x00000005
34	0	TMDS	0x00000007	0x00000006
37	0	Virtual	0x00000007	0x00000007

Connectors:
id	encoder	status		name		size (mm)	modes	encoders
33	32	connected	DSI-1          	71x157		1	32
  modes:
	index name refresh (Hz) hdisp hss hse htot vdisp vss vse vtot
  #0 1080x2400 144.00 1080 1100 1102 1122 2400 2420 2422 2440 394225 flags: ; type: preferred, driver
  props:
	1 EDID:
		flags: immutable blob
		blobs:

		value:
	2 DPMS:
		flags: enum
		enums: On=0 Standby=1 Suspend=2 Off=3
		value: 0
	5 link-status:
		flags: enum
		enums: Good=0 Bad=1
		value: 0
	6 non-desktop:
		flags: immutable range
		values: 0 1
		value: 0
	4 TILE:
		flags: immutable blob
		blobs:

		value:
35	0	disconnected	DP-1           	0x0		0	34
  props:
	1 EDID:
		flags: immutable blob
		blobs:

		value:
	2 DPMS:
		flags: enum
		enums: On=0 Standby=1 Suspend=2 Off=3
		value: 0
	5 link-status:
		flags: enum
		enums: Good=0 Bad=1
		value: 0
	6 non-desktop:
		flags: immutable range
		values: 0 1
		value: 0
	4 TILE:
		flags: immutable blob
		blobs:

		value:
	36 subconnector:
		flags: immutable enum
		enums: Unknown=0 VGA=1 DVI-D=3 HDMI=11 DP=10 Wireless=18 Native=15
		value: 0
==================><====================================================

and dri state:
==================><====================================================
# cat /sys/kernel/debug/dri/0/state
plane[43]: plane-0
	crtc=crtc-0
	fb=106
		allocated by = [fbcon]
		refcount=2
		format=XR24 little-endian (0x34325258)
		modifier=0x0
		size=1080x2400
		layers:
			size[0]=1080x2400
			pitch[0]=4352
			offset[0]=0
			obj[0]:
				name=0
				refcount=1
				start=0010102d
				size=10444800
				imported=no
	crtc-pos=1080x2400+0+0
	src-pos=1080.000000x2400.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=1
	sspp[0]=sspp_0
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=1080x2400+0+0
	dst[0]=1080x2400+0+0
plane[49]: plane-1
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_1
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[55]: plane-2
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_2
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[61]: plane-3
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_3
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[67]: plane-4
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_8
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[73]: plane-5
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_9
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[79]: plane-6
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_10
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[85]: plane-7
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_11
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[91]: plane-8
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_12
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
plane[97]: plane-9
	crtc=(null)
	fb=0
	crtc-pos=0x0+0+0
	src-pos=0.000000x0.000000+0.000000+0.000000
	rotation=1
	normalized-zpos=0
	color-encoding=ITU-R BT.601 YCbCr
	color-range=YCbCr limited range
	color_mgmt_changed=0
	stage=0
	sspp[0]=sspp_13
	multirect_mode[0]=none
	multirect_index[0]=solo
	src[0]=0x0+0+0
	dst[0]=0x0+0+0
crtc[103]: crtc-0
	enable=1
	active=1
	self_refresh_active=0
	planes_changed=1
	mode_changed=0
	active_changed=0
	connectors_changed=0
	color_mgmt_changed=0
	plane_mask=1
	connector_mask=1
	encoder_mask=1
	mode: "1080x2400": 144 394225 1080 1100 1102 1122 2400 2420 2422 2440 0x48 0x0
	lm[0]=0
	ctl[0]=2
crtc[104]: crtc-1
	enable=0
	active=0
	self_refresh_active=0
	planes_changed=0
	mode_changed=0
	active_changed=0
	connectors_changed=0
	color_mgmt_changed=0
	plane_mask=0
	connector_mask=0
	encoder_mask=0
	mode: "": 0 0 0 0 0 0 0 0 0 0 0x0 0x0
crtc[105]: crtc-2
	enable=0
	active=0
	self_refresh_active=0
	planes_changed=0
	mode_changed=0
	active_changed=0
	connectors_changed=0
	color_mgmt_changed=0
	plane_mask=0
	connector_mask=0
	encoder_mask=0
	mode: "": 0 0 0 0 0 0 0 0 0 0 0x0 0x0
connector[33]: DSI-1
	crtc=crtc-0
	self_refresh_aware=0
	max_requested_bpc=0
	colorspace=Default
connector[35]: DP-1
	crtc=(null)
	self_refresh_aware=0
	max_requested_bpc=0
	colorspace=Default
connector[42]: Writeback-1
	crtc=(null)
	self_refresh_aware=0
	max_requested_bpc=0
	colorspace=Default
resource mapping:
	pingpong=103 # # # # # # # # # -
	mixer=103 # # # # # -
	ctl=# # 103 # # #
	dspp=# # # #
	dsc=# # # # # #
	cdm=-
	cwb=# # # #
==================><====================================================

I pasted all the kms_writeback log, I have nothing more.

If I specify `--run-subtest dump-valid-clones` I get:
==================><====================================================
IGT_FRAME_DUMP_PATH=$PWD FRAME_PNG_FILE_NAME=out.png /usr/libexec/igt-gpu-tools/kms_writeback -d --run-subtest dump-valid-clones
[   33.250236] Console: switching to colour dummy device 80x25
IGT-Version: 1.29-1.28 (aarch64) (Linux: 6.12.0-rc1-00022-ge581f752bf79 aarch64)
Using IGT_SRANDOM=1709054789 for randomisation[   33.256171] [IGT] kms_writeback: executing

Opened device: /dev/dri/card0
[   33.360023] [IGT] kms_writeback: starting subtest dump-valid-clones
Starting subtest: dump-valid-clones
[   34.063316] [drm:dpu_encoder_virt_atomic_disable:1314] [dpu error]enc32 timeout pending
[   34.244272] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000010
[   34.253385] Mem abort info:
[   34.256328]   ESR = 0x0000000096000006
[   34.260272]   EC = 0x25: DABT (current EL), IL = 32 bits
[   34.265816]   SET = 0, FnV = 0
[   34.269043]   EA = 0, S1PTW = 0
[   34.272332]   FSC = 0x06: level 2 translation fault
[   34.277430] Data abort info:
[   34.280460]   ISV = 0, ISS = 0x00000006, ISS2 = 0x00000000
[   34.286170]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
[   34.291438]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
[   34.296975] user pgtable: 4k pages, 48-bit VAs, pgdp=00000008824fc000
[   34.303673] [0000000000000010] pgd=08000008dc4e0003, p4d=08000008dc4e0003, pud=08000008dd4af003, pmd=0000000000000000
[   34.314647] Internal error: Oops: 0000000096000006 [#1] PREEMPT SMP
[   34.321144] Modules linked in: snd_soc_wsa884x q6prm_clocks q6apm_dai q6apm_lpass_dais snd_q6dsp_common q6prm 8021q garp mrp stp llc usb_f_fs libcomposite qrtr_mhi snd_q6apm rpmsg_ctrl fastrpc apr qrtr_smd rpmsg_char snd_soc_hdmi_codec ath12k mac80211 libarc4 mhi panel_visionox_vtdr6130 qcom_pd_mapper goodix_berlin_spi ucsi_glink pmic_glink_altmode pci_pwrctl_pwrseq pci_pwrctl_core typec_ucsi aux_hpd_bridge qcom_battmgr nb7vpq904m wcd939x_usbss goodix_berlin_core crct10dif_ce phy_qcom_eusb2_repeater msm sm3_ce sm3 qcom_q6v5_pas sha3_ce hci_uart sha512_ce sha512_arm64 leds_qcom_lpg ocmem qcom_pil_info qcom_q6v5 qcom_pbs btqca ipa btbcm drm_exec qcom_sysmon pwrseq_qcom_wcn snd_soc_sc8280xp led_class_multicolor snd_soc_qcom_sdw qrtr qcom_common gpu_sched snd_soc_wcd939x drm_dp_aux_bus qcom_spmi_temp_alarm snd_soc_qcom_common qcom_glink_smem snd_soc_wcd939x_sdw rtc_pm8xxx drm_display_helper pinctrl_sm8650_lpass_lpi regmap_sdw cfg80211 bluetooth qcom_pon pmic_glink ecdh_generic pdr_interface phy_qcom_qmp_combo ecc rfkill
[   34.321268]  nvmem_qcom_spmi_sdam qcom_stats spi_geni_qcom pwrseq_core i2c_qcom_geni aux_bridge phy_qcom_snps_eusb2 dispcc_sm8550 drm_kms_helper gpi soundwire_qcom snd_soc_lpass_va_macro pinctrl_lpass_lpi snd_soc_wcd_mbhc snd_soc_lpass_tx_macro snd_soc_lpass_rx_macro snd_soc_lpass_wsa_macro llcc_qcom snd_soc_lpass_macro_common slimbus snd_soc_wcd_classh mdt_loader qcom_pdr_msg qcrypto gpucc_sm8650 icc_bwmon qmi_helpers authenc phy_qcom_qmp_ufs libdes soundwire_bus ufs_qcom nvmem_reboot_mode phy_qcom_qmp_pcie typec qcom_rng rmtfs_mem socinfo fuse drm backlight ipv6
[   34.464862] CPU: 5 UID: 0 PID: 513 Comm: kms_writeback Tainted: G S                 6.12.0-rc1-00022-ge581f752bf79 #2
[   34.475812] Tainted: [S]=CPU_OUT_OF_SPEC
[   34.479905] Hardware name: Qualcomm Technologies, Inc. SM8650 QRD (DT)
[   34.486667] pstate: 81400005 (Nzcv daif +PAN -UAO -TCO +DIT -SSBS BTYPE=--)
[   34.493880] pc : dpu_encoder_helper_phys_setup_cwb+0xb8/0x1ec [msm]
[   34.500441] lr : dpu_encoder_helper_phys_setup_cwb+0x88/0x1ec [msm]
[   34.506969] sp : ffff800085fc37e0
[   34.510437] x29: ffff800085fc3810 x28: ffffb8c93c953068 x27: ffff5af315c90880
[   34.517826] x26: ffff5af359c55780 x25: ffff800085fc3878 x24: ffff5af35a956e80
[   34.525217] x23: 0000000000000000 x22: ffff5af355dc2080 x21: ffff5af35a956e80
[   34.532607] x20: ffff5af315c90880 x19: ffff5af315c90c80 x18: 0000000000000001
[   34.539997] x17: 0000000000000018 x16: ffffb8c95c9c8c64 x15: 0000000000000038
[   34.547385] x14: 0000001971602a24 x13: 00000000000000e1 x12: 000000000000000b
[   34.554774] x11: 0000000000000000 x10: e7125de8a27ae014 x9 : 5aef79bd13b1e2a7
[   34.562162] x8 : ffff5af355dc2718 x7 : 0000000000000004 x6 : ffff5af356374d98
[   34.569550] x5 : 0000000000000002 x4 : ffff800085fc37f8 x3 : ffff5af315c90950
[   34.576938] x2 : 0000000000000002 x1 : 0000000000000000 x0 : 0000000000000001
[   34.584328] Call trace:
[   34.586905]  dpu_encoder_helper_phys_setup_cwb+0xb8/0x1ec [msm]
[   34.593075]  dpu_encoder_helper_phys_cleanup+0x328/0x3c4 [msm]
[   34.599165]  dpu_encoder_phys_wb_disable+0x80/0xac [msm]
[   34.604713]  dpu_encoder_virt_atomic_disable+0xb4/0x160 [msm]
[   34.610711]  disable_outputs+0x108/0x32c [drm_kms_helper]
[   34.616351]  drm_atomic_helper_commit_modeset_disables+0x1c/0x4c [drm_kms_helper]
[   34.624110]  msm_atomic_commit_tail+0x188/0x514 [msm]
[   34.629396]  commit_tail+0xa4/0x18c [drm_kms_helper]
[   34.634570]  drm_atomic_helper_commit+0x17c/0x194 [drm_kms_helper]
[   34.640990]  drm_atomic_commit+0xb8/0xf4 [drm]
[   34.645690]  drm_mode_atomic_ioctl+0xad4/0xd88 [drm]
[   34.650889]  drm_ioctl_kernel+0xc0/0x128 [drm]
[   34.655564]  drm_ioctl+0x218/0x49c [drm]
[   34.659697]  __arm64_sys_ioctl+0xac/0xf0
[   34.663804]  invoke_syscall+0x48/0x10c
[   34.667755]  el0_svc_common.constprop.0+0xc0/0xe0
[   34.672648]  do_el0_svc+0x1c/0x28
[   34.676117]  el0_svc+0x34/0xd8
[   34.679330]  el0t_64_sync_handler+0x120/0x12c
[   34.683864]  el0t_64_sync+0x190/0x194
[   34.687699] Code: 910063e1 f8607822 f8607861 b9401042 (b9401021)
[   34.694014] ---[ end trace 0000000000000000 ]---
==================><====================================================

Neil


Also, can you share the IGT debug logs?

FWIW, I haven't had the chance to test with DP yet so that might be why you're hitting this issue and I'm not.

Thanks,

Jessica Zhang

[ 2566.668998] Console: switching to colour dummy device 80x25
IGT-Version: 1.29-1.28 (aarch64) (Linux: 6.12.0-rc1-00022-ge581f752bf79 aarch64)
[ 2566.674859] [IGT] kms_writeback: executing
Using IGT_SRANDOM=1709057323 for randomisation
Opened device: /dev/dri/card0
[ 2566.741375] [IGT] kms_writeback: starting subtest dump-writeback
Starting subtest: dump-writeback
Subtest dump-writeback: SUCCESS (0.305s)[ 2567.053189] [IGT] kms_writeback: finished subtest dump-writeback, SUCCESS

[ 2567.064505] [IGT] kms_writeback: starting subtest dump-valid-clones
Starting subtest: dump-valid-clones
[ 2567.762793] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000010
[ 2567.771919] Mem abort info:
[ 2567.774888]   ESR = 0x0000000096000006
[ 2567.778831]   EC = 0x25: DABT (current EL), IL = 32 bits
[ 2567.784371]   SET = 0, FnV = 0
[ 2567.787601]   EA = 0, S1PTW = 0
[ 2567.790942]   FSC = 0x06: level 2 translation fault
[ 2567.796044] Data abort info:
[ 2567.799083]   ISV = 0, ISS = 0x00000006, ISS2 = 0x00000000
[ 2567.804793]   CM = 0, WnR = 0, TnD = 0, TagAccess = 0
[ 2567.810057]   GCS = 0, Overlay = 0, DirtyBit = 0, Xs = 0
[ 2567.815600] user pgtable: 4k pages, 48-bit VAs, pgdp=00000008d60cf000
[ 2567.822290] [0000000000000010] pgd=08000008d6049003, p4d=08000008d6049003, pud=080000089397e003, pmd=0000000000000000
[ 2567.833254] Internal error: Oops: 0000000096000006 [#1] PREEMPT SMP
[ 2567.839747] Modules linked in: snd_soc_wsa884x q6prm_clocks q6apm_lpass_dais snd_q6dsp_common q6apm_dai q6prm 8021q garp mrp stp llc usb_f_fs libcomposite qrtr_mhi snd_soc_hdmi_codec ath12k mac80211 libarc4 mhi panel_visionox_vtdr6130 snd_q6apm pci_pwrctl_pwrseq pci_pwrctl_core rpmsg_ctrl apr fastrpc qrtr_smd rpmsg_char wcd939x_usbss nb7vpq904m qcom_pd_mapper goodix_berlin_spi goodix_berlin_core ucsi_glink typec_ucsi pmic_glink_altmode aux_hpd_bridge qcom_battmgr leds_qcom_lpg msm ocmem drm_exec hci_uart qcom_pbs gpu_sched led_class_multicolor btqca phy_qcom_eusb2_repeater btbcm qcom_spmi_temp_alarm drm_dp_aux_bus phy_qcom_qmp_combo crct10dif_ce bluetooth drm_display_helper sm3_ce ecdh_generic aux_bridge sm3 snd_soc_sc8280xp pwrseq_qcom_wcn sha3_ce snd_soc_qcom_sdw rtc_pm8xxx qcom_pon ecc nvmem_qcom_spmi_sdam sha512_ce qcom_stats spi_geni_qcom snd_soc_qcom_common sha512_arm64 pwrseq_core i2c_qcom_geni cfg80211 drm_kms_helper dispcc_sm8550 gpi ipa snd_soc_lpass_va_macro snd_soc_lpass_tx_macro soundwire_qcom
[ 2567.839860]  pinctrl_sm8650_lpass_lpi snd_soc_lpass_wsa_macro snd_soc_lpass_rx_macro rfkill slimbus phy_qcom_snps_eusb2 pinctrl_lpass_lpi gpucc_sm8650 snd_soc_lpass_macro_common qcom_q6v5_pas qcom_pil_info qcom_q6v5 qcrypto authenc icc_bwmon qcom_sysmon qcom_common qrtr qcom_glink_smem phy_qcom_qmp_pcie mdt_loader libdes llcc_qcom ufs_qcom phy_qcom_qmp_ufs pmic_glink snd_soc_wcd939x rmtfs_mem pdr_interface snd_soc_wcd939x_sdw regmap_sdw qcom_pdr_msg snd_soc_wcd_mbhc qmi_helpers snd_soc_wcd_classh soundwire_bus typec nvmem_reboot_mode qcom_rng socinfo fuse drm backlight ipv6
[ 2567.983445] CPU: 5 UID: 0 PID: 554 Comm: kms_writeback Tainted: G S                 6.12.0-rc1-00022-ge581f752bf79 #2
[ 2567.994390] Tainted: [S]=CPU_OUT_OF_SPEC
[ 2567.998483] Hardware name: Qualcomm Technologies, Inc. SM8650 QRD (DT)
[ 2568.005244] pstate: 81400005 (Nzcv daif +PAN -UAO -TCO +DIT -SSBS BTYPE=--)
[ 2568.012455] pc : dpu_encoder_helper_phys_setup_cwb+0xb8/0x1ec [msm]
[ 2568.019009] lr : dpu_encoder_helper_phys_setup_cwb+0x88/0x1ec [msm]
[ 2568.025532] sp : ffff80008939b7e0
[ 2568.028999] x29: ffff80008939b810 x28: ffffcbcb66f26068 x27: ffff37ad962cb080
[ 2568.036388] x26: ffff37ad9887ed80 x25: ffff80008939b878 x24: ffff37ad43642a80
[ 2568.043775] x23: 0000000000000000 x22: ffff37ad42812080 x21: ffff37ad43642a80
[ 2568.051163] x20: ffff37ad962cb080 x19: ffff37ad962c8080 x18: 0000000000000001
[ 2568.058552] x17: 000000040044ffff x16: ffffcbcbb0fc8c64 x15: 00003d08ffff9c00
[ 2568.065939] x14: 00000013519b2832 x13: ffff37ad9d392200 x12: 000000000000000b
[ 2568.073325] x11: ffff37ad40dc56c0 x10: ffff37ad9d392200 x9 : ffff37afbe7bba80
[ 2568.080712] x8 : ffff37ad42812718 x7 : 0000000000000004 x6 : ffff37ad989ac798
[ 2568.088098] x5 : 0000000000000002 x4 : ffff80008939b7f8 x3 : ffff37ad962cb150
[ 2568.095480] x2 : 0000000000000002 x1 : 0000000000000000 x0 : 0000000000000001
[ 2568.102868] Call trace:
[ 2568.105446]  dpu_encoder_helper_phys_setup_cwb+0xb8/0x1ec [msm]
[ 2568.111608]  dpu_encoder_helper_phys_cleanup+0x328/0x3c4 [msm]
[ 2568.117692]  dpu_encoder_phys_wb_disable+0x80/0xac [msm]
[ 2568.123233]  dpu_encoder_virt_atomic_disable+0xb4/0x160 [msm]
[ 2568.129224]  disable_outputs+0x108/0x32c [drm_kms_helper]
[ 2568.134858]  drm_atomic_helper_commit_modeset_disables+0x1c/0x4c [drm_kms_helper]
[ 2568.142614]  msm_atomic_commit_tail+0x188/0x514 [msm]
[ 2568.147894]  commit_tail+0xa4/0x18c [drm_kms_helper]
[ 2568.153065]  drm_atomic_helper_commit+0x17c/0x194 [drm_kms_helper]
[ 2568.159482]  drm_atomic_commit+0xb8/0xf4 [drm]
[ 2568.164176]  drm_mode_atomic_ioctl+0xad4/0xd88 [drm]
[ 2568.169369]  drm_ioctl_kernel+0xc0/0x128 [drm]
[ 2568.174039]  drm_ioctl+0x218/0x49c [drm]
[ 2568.178165]  __arm64_sys_ioctl+0xac/0xf0
[ 2568.182271]  invoke_syscall+0x48/0x10c
[ 2568.186217]  el0_svc_common.constprop.0+0xc0/0xe0
[ 2568.191109]  do_el0_svc+0x1c/0x28
[ 2568.194576]  el0_svc+0x34/0xd8
[ 2568.197788]  el0t_64_sync_handler+0x120/0x12c
[ 2568.202321]  el0t_64_sync+0x190/0x194
[ 2568.206157] Code: 910063e1 f8607822 f8607861 b9401042 (b9401021)
[ 2568.212484] ---[ end trace 0000000000000000 ]---

Neil

+                rt_pp_idx[i] = enable ? hw_pp->idx : PINGPONG_NONE;
+                break;
+            }
+        }
+    }
+
+    /*
+     * The CWB mux supports using LM or DSPP as tap points. For now,
+     * always use LM tap point
+     */
+    cwb_cfg.input = INPUT_MODE_LM_OUT;
+
+    for (int i = 0; i < MAX_CHANNELS_PER_ENC; i++) {
+        hw_cwb = dpu_enc->hw_cwb[i];
+        if (!hw_cwb)
+            continue;
+
+        cwb_cfg.pp_idx = rt_pp_idx[i];
+
+        hw_cwb->ops.config_cwb(hw_cwb, &cwb_cfg);
+    }
+}
+
  void dpu_encoder_helper_phys_setup_cdm(struct dpu_encoder_phys *phys_enc,
                         const struct msm_format *dpu_fmt,
                         u32 output_type)
@@ -2557,6 +2630,14 @@ enum dpu_intf_mode dpu_encoder_get_intf_mode(struct drm_encoder *encoder)
      return INTF_MODE_NONE;
  }
+unsigned int dpu_encoder_helper_get_cwb(struct dpu_encoder_phys *phys_enc)
+{
+    struct drm_encoder *encoder = phys_enc->parent;
+    struct dpu_encoder_virt *dpu_enc = to_dpu_encoder_virt(encoder);
+
+    return dpu_enc->cwb_mask;
+}
+
  unsigned int dpu_encoder_helper_get_dsc(struct dpu_encoder_phys *phys_enc)
  {
      struct drm_encoder *encoder = phys_enc->parent;
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h b/ drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
index e77ebe3a68da..d7a02d1f8053 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
@@ -1,6 +1,6 @@
  /* SPDX-License-Identifier: GPL-2.0-only */
  /*
- * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2022-2024 Qualcomm Innovation Center, Inc. All rights reserved.
   * Copyright (c) 2015-2018 The Linux Foundation. All rights reserved.
   */
@@ -331,6 +331,12 @@ static inline enum dpu_3d_blend_mode dpu_encoder_helper_get_3d_blend_mode(
      return BLEND_3D_NONE;
  }
+/**
+ * dpu_encoder_helper_get_cwb - get CWB blocks mask for the DPU encoder
+ * @phys_enc: Pointer to physical encoder structure
+ */
+unsigned int dpu_encoder_helper_get_cwb(struct dpu_encoder_phys *phys_enc);
+
  /**
   * dpu_encoder_helper_get_dsc - get DSC blocks mask for the DPU encoder
   *   This helper function is used by physical encoder to get DSC blocks mask
@@ -400,6 +406,14 @@ int dpu_encoder_helper_wait_for_irq(struct dpu_encoder_phys *phys_enc,
   */
  void dpu_encoder_helper_phys_cleanup(struct dpu_encoder_phys *phys_enc);
+/**
+ * dpu_encoder_helper_phys_setup_cwb - helper to configure CWB muxes
+ * @phys_enc: Pointer to physical encoder structure
+ * @enable: Enable CWB mux
+ */
+void dpu_encoder_helper_phys_setup_cwb(struct dpu_encoder_phys *phys_enc,
+                       bool enable);
+
  /**
   * dpu_encoder_helper_phys_setup_cdm - setup chroma down sampling block
   * @phys_enc: Pointer to physical encoder
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c b/ drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
index 882c717859ce..e88c4d91041f 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_wb.c
@@ -1,6 +1,6 @@
  // SPDX-License-Identifier: GPL-2.0-only
  /*
- * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2022-2024 Qualcomm Innovation Center, Inc. All rights reserved.
   */
  #define pr_fmt(fmt)    "[drm:%s:%d] " fmt, __func__, __LINE__
@@ -342,6 +342,8 @@ static void dpu_encoder_phys_wb_setup(
      dpu_encoder_helper_phys_setup_cdm(phys_enc, dpu_fmt, CDM_CDWN_OUTPUT_WB);
+    dpu_encoder_helper_phys_setup_cwb(phys_enc, true);
+
      dpu_encoder_phys_wb_setup_ctl(phys_enc);
  }








[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux