Skip to content

Remove all bad dp_mst_port uses and hide struct def

Lyude Paul dri-devel intel-gfx nouveau stable 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 of 6 patches 7 repliesIn review

4 of 6 patches are in this archive. The cover letter announces 6.

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6drm/dp_mst: Add drm_dp_get_payload_info()drivers/gpu/drm/drm_dp_mst_topology.c include/drm/drm_dp_mst_helper.h+60 -12 replies
3/6drm/nouveau: Stop reading port->mgr in nv50_mstc_get_modes()drivers/gpu/drm/nouveau/dispnv50/disp.c+2 -11 reply
4/6drm/nouveau: Stop reading port->mgr in nv50_mstc_detect()drivers/gpu/drm/nouveau/dispnv50/disp.c+1 -12 replies
6/6drm/i915: Start using struct drm_dp_mst_port againdrivers/gpu/drm/i915/intel_dp_mst.c drivers/gpu/drm/i915/intel_drv.h+2 -20 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchBen Skeggs
SeriesA-b
1/6
3/6
4/6
6/6

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/6] Remove all bad dp_mst_port uses and hide struct def

So we don't ever have to worry about drivers touching drm_dp_mst_port structs without verifying them and crashing again.

Lyude Paul (6):
drm/dp_mst: Add drm_dp_get_payload_info()
drm/nouveau: Use drm_dp_get_payload_info() for getting payload/vcpi drm/nouveau: Stop reading port->mgr in nv50_mstc_get_modes() drm/nouveau: Stop reading port->mgr in nv50_mstc_detect() drm/dp_mst: Hide drm_dp_mst_port contents from drivers drm/i915: Start using struct drm_dp_mst_port again

 drivers/gpu/drm/drm_dp_mst_topology.c   | 115 ++++++++++++++++++++++++
 drivers/gpu/drm/i915/intel_dp_mst.c     |   2 +-
 drivers/gpu/drm/i915/intel_drv.h        |   2 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c |  60 +++++--------
 include/drm/drm_dp_mst_helper.h         |  65 ++------------
 5 files changed, 146 insertions(+), 98 deletions(-)
Signature · 6 lines
2.19.1

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx