mbox series

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

Message ID 20181117002120.28703-1-lyude@redhat.com (mailing list archive)
Headers show
Series Remove all bad dp_mst_port uses and hide struct def | expand

Message

Lyude Paul Nov. 17, 2018, 12:21 a.m. UTC 
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(-)

Comments

Karol Herbst Nov. 24, 2018, 3:55 p.m. UTC | #1 
Patches 1 and 5 are Acked-by: Karol Herbst <kherbst@redhat.com>
Patches 2-4 are Reviewed-by: Karol Herbst <kherbst@redhat.com>
On Sat, Nov 17, 2018 at 1:21 AM Lyude Paul <lyude@redhat.com> wrote:
>
> 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(-)
>
> --
> 2.19.1
>
> _______________________________________________
> Nouveau mailing list
> Nouveau@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/nouveau
Ben Skeggs Nov. 27, 2018, 10:17 p.m. UTC | #2 
For the series:

Acked-by: Ben Skeggs <bskeggs@redhat.com>
On Sat, 17 Nov 2018 at 10:21, Lyude Paul <lyude@redhat.com> wrote:
>
> 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(-)
>
> --
> 2.19.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel