Hi, On Wed, Dec 11, 2024 at 11:51 AM Abhinav Kumar <[email protected]> wrote: > > Similar to the r_pipe sspp protect, add a check to protect > the pipe state prints to avoid NULL ptr dereference for cases when > the state is dumped without a corresponding atomic_check() where the > pipe->sspp is assigned. > > Fixes: 31f7148fd370 ("drm/msm/dpu: move pstate->pipe initialization to > dpu_plane_atomic_check") > Reported-by: Stephen Boyd <[email protected]> > Closes: https://gitlab.freedesktop.org/drm/msm/-/issues/67 > Signed-off-by: Abhinav Kumar <[email protected]> > --- > To: Rob Clark <[email protected]> > To: Dmitry Baryshkov <[email protected]> > To: Sean Paul <[email protected]> > To: Marijn Suijten <[email protected]> > To: David Airlie <[email protected]> > To: Simona Vetter <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: Stephen Boyd <[email protected]> > --- > Changes in v2: > - move pstate->stage out of the pipe->sspp check > - add reported-by credits for Stephen > - Link to v1: > https://lore.kernel.org/r/[email protected] > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-)
Tested-by: Douglas Anderson <[email protected]>
