[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241008-bouncy-sawfish-of-temperance-2e9e5f@houat>
Date: Tue, 8 Oct 2024 10:18:49 +0200
From: Maxime Ripard <mripard@...nel.org>
To: Neil Armstrong <neil.armstrong@...aro.org>
Cc: Jessica Zhang <quic_jesszhan@...cinc.com>,
Rob Clark <robdclark@...il.com>, Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
quic_abhinavk@...cinc.com, Sean Paul <sean@...rly.run>,
Marijn Suijten <marijn.suijten@...ainline.org>, David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Thomas Zimmermann <tzimmermann@...e.de>, quic_ebharadw@...cinc.com, linux-arm-msm@...r.kernel.org,
dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
Rob Clark <robdclark@...omium.org>, Ville Syrjälä <ville.syrjala@...ux.intel.com>
Subject: Re: [PATCH v2 16/22] drm/msm/dpu: Configure CWB in writeback encoder
On Tue, Oct 08, 2024 at 10:00:57AM GMT, Neil Armstrong wrote:
> Hi,
>
> On 01/10/2024 09:37, neil.armstrong@...aro.org wrote:
> > Hi,
> >
> > On 30/09/2024 21:19, Jessica Zhang wrote:
> > >
> > >
> > > On 9/30/2024 7:17 AM, neil.armstrong@...aro.org wrote:
> > > > On 25/09/2024 00:59, Jessica Zhang wrote:
>
> <snip>
>
> > > >
> > > > 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 ]---
> > ==================><====================================================
>
> Anything I can try to get past the crash ?
The call to dpu_kms_get_existing_global_state in
dpu_encoder_helper_phys_setup_cwb looks suspicious to me, can you check
whether the returned global state is NULL?
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)
Powered by blists - more mailing lists