From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
Abhinav Kumar <abhinavk@codeaurora.org>,
Stephen Boyd <sboyd@kernel.org>, David Airlie <airlied@linux.ie>,
Daniel Vetter <daniel@ffwll.ch>,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
freedreno@lists.freedesktop.org,
Kuogee Hsieh <khsieh@codeaurora.org>
Subject: Re: [PATCH v2 2/2] drm/msm/dp: rewrite dss_module_power to use bulk clock functions
Date: Tue, 28 Dec 2021 22:12:21 -0600 [thread overview]
Message-ID: <YcvgJcxWDfaSMVVg@builder.lan> (raw)
In-Reply-To: <20211126023516.1108411-3-dmitry.baryshkov@linaro.org>
On Thu 25 Nov 20:35 CST 2021, Dmitry Baryshkov wrote:
> In order to simplify DP code, drop hand-coded loops over clock arrays,
> replacing them with clk_bulk_* functions.
>
I've yet to debug this, but applying the two patches and attaching an
HDMI cable to my USB dongle results in the follwing splat on the 8350
HDK.
[ 39.658840] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000000
[ 39.667883] Mem abort info:
[ 39.670774] ESR = 0x96000006
[ 39.673940] EC = 0x25: DABT (current EL), IL = 32 bits
[ 39.679417] SET = 0, FnV = 0
[ 39.682582] EA = 0, S1PTW = 0
[ 39.685825] FSC = 0x06: level 2 translation fault
[ 39.690851] Data abort info:
[ 39.693838] ISV = 0, ISS = 0x00000006
[ 39.697797] CM = 0, WnR = 0
[ 39.700864] user pgtable: 4k pages, 48-bit VAs, pgdp=000000010eb8d000
[ 39.707501] [0000000000000000] pgd=080000010f097003, p4d=080000010f097003, pud=080000010ba58003, pmd=0000000000000000
[ 39.718425] Internal error: Oops: 96000006 [#1] PREEMPT SMP
[ 39.724169] Modules linked in: pmic_glink_altmode qcom_pmic_glink_power cfg80211 rfkill 8021q garp mrp stp llc microchip lan78xx snd_soc_hdmi_codec pmic_glink pdr_interface rpmsg_char qrtr_smd qrtr fsa4480 qcom_q6v5_pas qcom_pil_info i2c_qcom_geni qcom_q6v5 msm qcom_sysmon qcom_stats gpu_sched crct10dif_ce drm_kms_helper qcom_common qcom_glink_smem gpucc_sm8350 phy_qcom_qmp mdt_loader typec ufs_qcom qmi_helpers qcom_rng socinfo qnoc_sm8350 rmtfs_mem fuse drm ipv6
[ 39.766330] CPU: 0 PID: 85 Comm: kworker/0:3 Not tainted 5.16.0-rc5-next-20211215-00046-g2f90133452d9 #280
[ 39.776256] Hardware name: Qualcomm Technologies, Inc. SM8350 HDK (DT)
[ 39.782969] Workqueue: events pmic_glink_altmode_worker [pmic_glink_altmode]
[ 39.790235] pstate: 40400005 (nZcv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--)
[ 39.797393] pc : __pi_strcmp+0x1c/0xf0
[ 39.801271] lr : dp_ctrl_set_clock_rate+0x8c/0xf0 [msm]
[ 39.806737] sp : ffff800008adbbc0
[ 39.810153] x29: ffff800008adbbc0 x28: 0000000000000000 x27: 0000000000000000
[ 39.817501] x26: ffffb60107f74270 x25: ffff297a49b33c80 x24: 00000000202fbf00
[ 39.824846] x23: 0000000000000001 x22: ffff297a4b400580 x21: 0000000000000020
[ 39.832188] x20: ffffb600e3d78068 x19: 0000000000000000 x18: ffffffffffffffff
[ 39.839541] x17: 6b6c63206e6f2030 x16: ffffb601063476c0 x15: 0720072007200720
[ 39.846893] x14: 0720072007200720 x13: ffffb60107c622c8 x12: 0000000000000765
[ 39.854229] x11: 0000000000000277 x10: 0101010101010101 x9 : ffffb60107c622c8
[ 39.861565] x8 : 00000000ffffefff x7 : 0000000000000000 x6 : ffffb60107cba2c8
[ 39.868902] x5 : 0000000000000000 x4 : ffff297cbe8619d8 x3 : 0000000000000000
[ 39.876240] x2 : 0000000000000000 x1 : ffffb600e3d78068 x0 : 0000000000000000
[ 39.883594] Call trace:
[ 39.886124] __pi_strcmp+0x1c/0xf0
[ 39.889638] dp_ctrl_enable_mainlink_clocks+0x98/0x110 [msm]
[ 39.895537] dp_ctrl_on_link+0x98/0x3f0 [msm]
[ 39.900096] dp_display_process_hpd_high+0xa8/0x100 [msm]
[ 39.905731] dp_display_usbpd_attention_cb+0x164/0x1a4 [msm]
[ 39.911629] dp_hpd_oob_event+0x74/0xa4 [msm]
[ 39.916195] dp_display_oob_hotplug_event+0x1c/0x2c [msm]
[ 39.921831] dp_oob_hotplug_event+0x18/0x24 [msm]
[ 39.926756] drm_connector_oob_hotplug_event+0x40/0x60 [drm]
[ 39.932686] pmic_glink_altmode_worker+0x7c/0x194 [pmic_glink_altmode]
[ 39.939398] process_one_work+0x1d0/0x350
[ 39.943526] worker_thread+0x13c/0x460
[ 39.947390] kthread+0x17c/0x190
[ 39.950722] ret_from_fork+0x10/0x20
[ 39.954416] Code: f24008ff 540003e1 f2400807 54000241 (f8408402)
[ 39.960684] ---[ end trace 0000000000000000 ]---
Regards,
Bjorn
WARNING: multiple messages have this Message-ID (diff)
From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: freedreno@lists.freedesktop.org, Stephen Boyd <sboyd@kernel.org>,
linux-arm-msm@vger.kernel.org,
Abhinav Kumar <abhinavk@codeaurora.org>,
Kuogee Hsieh <khsieh@codeaurora.org>,
David Airlie <airlied@linux.ie>,
dri-devel@lists.freedesktop.org, Sean Paul <sean@poorly.run>
Subject: Re: [PATCH v2 2/2] drm/msm/dp: rewrite dss_module_power to use bulk clock functions
Date: Tue, 28 Dec 2021 22:12:21 -0600 [thread overview]
Message-ID: <YcvgJcxWDfaSMVVg@builder.lan> (raw)
In-Reply-To: <20211126023516.1108411-3-dmitry.baryshkov@linaro.org>
On Thu 25 Nov 20:35 CST 2021, Dmitry Baryshkov wrote:
> In order to simplify DP code, drop hand-coded loops over clock arrays,
> replacing them with clk_bulk_* functions.
>
I've yet to debug this, but applying the two patches and attaching an
HDMI cable to my USB dongle results in the follwing splat on the 8350
HDK.
[ 39.658840] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000000
[ 39.667883] Mem abort info:
[ 39.670774] ESR = 0x96000006
[ 39.673940] EC = 0x25: DABT (current EL), IL = 32 bits
[ 39.679417] SET = 0, FnV = 0
[ 39.682582] EA = 0, S1PTW = 0
[ 39.685825] FSC = 0x06: level 2 translation fault
[ 39.690851] Data abort info:
[ 39.693838] ISV = 0, ISS = 0x00000006
[ 39.697797] CM = 0, WnR = 0
[ 39.700864] user pgtable: 4k pages, 48-bit VAs, pgdp=000000010eb8d000
[ 39.707501] [0000000000000000] pgd=080000010f097003, p4d=080000010f097003, pud=080000010ba58003, pmd=0000000000000000
[ 39.718425] Internal error: Oops: 96000006 [#1] PREEMPT SMP
[ 39.724169] Modules linked in: pmic_glink_altmode qcom_pmic_glink_power cfg80211 rfkill 8021q garp mrp stp llc microchip lan78xx snd_soc_hdmi_codec pmic_glink pdr_interface rpmsg_char qrtr_smd qrtr fsa4480 qcom_q6v5_pas qcom_pil_info i2c_qcom_geni qcom_q6v5 msm qcom_sysmon qcom_stats gpu_sched crct10dif_ce drm_kms_helper qcom_common qcom_glink_smem gpucc_sm8350 phy_qcom_qmp mdt_loader typec ufs_qcom qmi_helpers qcom_rng socinfo qnoc_sm8350 rmtfs_mem fuse drm ipv6
[ 39.766330] CPU: 0 PID: 85 Comm: kworker/0:3 Not tainted 5.16.0-rc5-next-20211215-00046-g2f90133452d9 #280
[ 39.776256] Hardware name: Qualcomm Technologies, Inc. SM8350 HDK (DT)
[ 39.782969] Workqueue: events pmic_glink_altmode_worker [pmic_glink_altmode]
[ 39.790235] pstate: 40400005 (nZcv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--)
[ 39.797393] pc : __pi_strcmp+0x1c/0xf0
[ 39.801271] lr : dp_ctrl_set_clock_rate+0x8c/0xf0 [msm]
[ 39.806737] sp : ffff800008adbbc0
[ 39.810153] x29: ffff800008adbbc0 x28: 0000000000000000 x27: 0000000000000000
[ 39.817501] x26: ffffb60107f74270 x25: ffff297a49b33c80 x24: 00000000202fbf00
[ 39.824846] x23: 0000000000000001 x22: ffff297a4b400580 x21: 0000000000000020
[ 39.832188] x20: ffffb600e3d78068 x19: 0000000000000000 x18: ffffffffffffffff
[ 39.839541] x17: 6b6c63206e6f2030 x16: ffffb601063476c0 x15: 0720072007200720
[ 39.846893] x14: 0720072007200720 x13: ffffb60107c622c8 x12: 0000000000000765
[ 39.854229] x11: 0000000000000277 x10: 0101010101010101 x9 : ffffb60107c622c8
[ 39.861565] x8 : 00000000ffffefff x7 : 0000000000000000 x6 : ffffb60107cba2c8
[ 39.868902] x5 : 0000000000000000 x4 : ffff297cbe8619d8 x3 : 0000000000000000
[ 39.876240] x2 : 0000000000000000 x1 : ffffb600e3d78068 x0 : 0000000000000000
[ 39.883594] Call trace:
[ 39.886124] __pi_strcmp+0x1c/0xf0
[ 39.889638] dp_ctrl_enable_mainlink_clocks+0x98/0x110 [msm]
[ 39.895537] dp_ctrl_on_link+0x98/0x3f0 [msm]
[ 39.900096] dp_display_process_hpd_high+0xa8/0x100 [msm]
[ 39.905731] dp_display_usbpd_attention_cb+0x164/0x1a4 [msm]
[ 39.911629] dp_hpd_oob_event+0x74/0xa4 [msm]
[ 39.916195] dp_display_oob_hotplug_event+0x1c/0x2c [msm]
[ 39.921831] dp_oob_hotplug_event+0x18/0x24 [msm]
[ 39.926756] drm_connector_oob_hotplug_event+0x40/0x60 [drm]
[ 39.932686] pmic_glink_altmode_worker+0x7c/0x194 [pmic_glink_altmode]
[ 39.939398] process_one_work+0x1d0/0x350
[ 39.943526] worker_thread+0x13c/0x460
[ 39.947390] kthread+0x17c/0x190
[ 39.950722] ret_from_fork+0x10/0x20
[ 39.954416] Code: f24008ff 540003e1 f2400807 54000241 (f8408402)
[ 39.960684] ---[ end trace 0000000000000000 ]---
Regards,
Bjorn
next prev parent reply other threads:[~2021-12-29 4:12 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-26 2:35 [PATCH v2 0/2] drm/msm: rework clock handling Dmitry Baryshkov
2021-11-26 2:35 ` Dmitry Baryshkov
2021-11-26 2:35 ` [PATCH v2 1/2] drm/msm/dpu: simplify clocks handling Dmitry Baryshkov
2021-11-26 2:35 ` Dmitry Baryshkov
2022-01-19 2:32 ` [Freedreno] " Jessica Zhang
2022-01-19 2:32 ` Jessica Zhang
2022-01-19 20:31 ` Dmitry Baryshkov
2022-01-19 20:31 ` Dmitry Baryshkov
2021-11-26 2:35 ` [PATCH v2 2/2] drm/msm/dp: rewrite dss_module_power to use bulk clock functions Dmitry Baryshkov
2021-11-26 2:35 ` Dmitry Baryshkov
2021-12-29 0:29 ` Dmitry Baryshkov
2021-12-29 0:29 ` Dmitry Baryshkov
2021-12-29 22:03 ` Kuogee Hsieh
2021-12-29 22:03 ` Kuogee Hsieh
2021-12-29 4:12 ` Bjorn Andersson [this message]
2021-12-29 4:12 ` Bjorn Andersson
2021-12-31 4:48 ` Dmitry Baryshkov
2021-12-31 4:48 ` Dmitry Baryshkov
2021-12-31 4:49 ` Dmitry Baryshkov
2021-12-31 4:49 ` Dmitry Baryshkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YcvgJcxWDfaSMVVg@builder.lan \
--to=bjorn.andersson@linaro.org \
--cc=abhinavk@codeaurora.org \
--cc=airlied@linux.ie \
--cc=daniel@ffwll.ch \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=khsieh@codeaurora.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=robdclark@gmail.com \
--cc=sboyd@kernel.org \
--cc=sean@poorly.run \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.