* [PATCH] drm/msm/dp: populate connector of struct dp_panel
@ 2021-12-28 17:51 Kuogee Hsieh
2021-12-28 18:50 ` Bjorn Andersson
0 siblings, 1 reply; 2+ messages in thread
From: Kuogee Hsieh @ 2021-12-28 17:51 UTC (permalink / raw)
To: robdclark, sean, swboyd, vkoul, daniel, airlied, agross,
dmitry.baryshkov, bjorn.andersson
Cc: quic_abhinavk, aravindh, quic_khsieh, quic_sbillaka, freedreno,
dri-devel, linux-arm-msm, linux-kernel
There is kernel crashed happen due to unable to handle kernel NULL
pointer dereference of dp_panel->connector while running DP link
layer compliance test case 4.2.2.6 (EDIDCorruption Detection).
This patch will fixed the problem by populating connector of dp_panel.
Fixes: 7948fe12d47 ("drm/msm/dp: return correct edid checksum after corrupted edid checksum read")
Signed-off-by: Kuogee Hsieh <quic_khsieh@quicinc.com>
---
drivers/gpu/drm/msm/dp/dp_panel.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/msm/dp/dp_panel.c b/drivers/gpu/drm/msm/dp/dp_panel.c
index 71db10c..6a938a2 100644
--- a/drivers/gpu/drm/msm/dp/dp_panel.c
+++ b/drivers/gpu/drm/msm/dp/dp_panel.c
@@ -197,6 +197,8 @@ int dp_panel_read_sink_caps(struct dp_panel *dp_panel,
kfree(dp_panel->edid);
dp_panel->edid = NULL;
+ dp_panel->connector = connector;
+
dp_panel->edid = drm_get_edid(connector,
&panel->aux->ddc);
if (!dp_panel->edid) {
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drm/msm/dp: populate connector of struct dp_panel
2021-12-28 17:51 [PATCH] drm/msm/dp: populate connector of struct dp_panel Kuogee Hsieh
@ 2021-12-28 18:50 ` Bjorn Andersson
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Andersson @ 2021-12-28 18:50 UTC (permalink / raw)
To: Kuogee Hsieh
Cc: robdclark, sean, swboyd, vkoul, daniel, airlied, agross,
dmitry.baryshkov, quic_abhinavk, aravindh, quic_sbillaka,
freedreno, dri-devel, linux-arm-msm, linux-kernel
On Tue 28 Dec 09:51 PST 2021, Kuogee Hsieh wrote:
> There is kernel crashed happen due to unable to handle kernel NULL
It would be wonderful, for my understanding today, as well as people in
the coming months to be able to search for the callstack etc on the
mailing list, if you could provide some details about the crash.
E.g. a callstack or description of when it happens.
> pointer dereference of dp_panel->connector while running DP link
> layer compliance test case 4.2.2.6 (EDIDCorruption Detection).
> This patch will fixed the problem by populating connector of dp_panel.
>
> Fixes: 7948fe12d47 ("drm/msm/dp: return correct edid checksum after corrupted edid checksum read")
> Signed-off-by: Kuogee Hsieh <quic_khsieh@quicinc.com>
> ---
> drivers/gpu/drm/msm/dp/dp_panel.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/msm/dp/dp_panel.c b/drivers/gpu/drm/msm/dp/dp_panel.c
> index 71db10c..6a938a2 100644
> --- a/drivers/gpu/drm/msm/dp/dp_panel.c
> +++ b/drivers/gpu/drm/msm/dp/dp_panel.c
> @@ -197,6 +197,8 @@ int dp_panel_read_sink_caps(struct dp_panel *dp_panel,
> kfree(dp_panel->edid);
> dp_panel->edid = NULL;
>
> + dp_panel->connector = connector;
So you have a dp_display_private object with a panel and a connector
allocated, but before you manage to associate the two you get a HPD
event and call this function, so you decide to stitch the two together
just here in some piece of code unrelated to the initialization of your
objects?
It sounds like we're lacking synchronization between the initialization
and the HPD interrupts and this would not be the correct solution.
Regards,
Bjorn
> +
> dp_panel->edid = drm_get_edid(connector,
> &panel->aux->ddc);
> if (!dp_panel->edid) {
> --
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-28 18:49 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-28 17:51 [PATCH] drm/msm/dp: populate connector of struct dp_panel Kuogee Hsieh
2021-12-28 18:50 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).