All of lore.kernel.org
 help / color / mirror / Atom feed
* [Nouveau] [RESEND RFC 00/18] drm/display/dp_mst: Drop Radeon MST support, make MST atomic-only
@ 2022-06-07 19:29 ` Lyude Paul
  0 siblings, 0 replies; 155+ messages in thread
From: Lyude Paul @ 2022-06-07 19:29 UTC (permalink / raw)
  To: dri-devel, nouveau, amd-gfx
  Cc: Jani Nikula, Imre Deak, Fangzhi Zuo, Wayne Lin, Sean Paul,
	Ville Syrjälä

Ugh, thanks ./scripts/get_maintainers.pl for confusing and breaking
git-send email <<. Sorry for the resend everyone.

For quite a while we've been carrying around a lot of legacy modesetting
code in the MST helpers that has been rather annoying to keep around,
and very often gets in the way of trying to implement additional
functionality in MST such as fallback link rate retraining, dynamic BPC
management and DSC support, etc. because of the fact that we can't rely
on atomic for everything.

Luckily, we only actually have one user of the legacy MST code in the
kernel - radeon. Originally I was thinking of trying to maintain this
code and keep it around in some form, but I'm pretty unconvinced anyone
is actually using this. My reasoning for that is because I've seen
nearly no issues regarding MST on radeon for quite a while now - despite
the fact my local testing seems to indicate it's quite broken. This
isn't too surprising either, as MST support in radeon.ko is gated behind
a module parameter that isn't enabled by default. This isn't to say I
wouldn't be open to alternative suggestions, but I'd rather not be the
one to have to spend time on that if at all possible! Plus, I already
floated the idea of dropping this code by AMD folks a few times and
didn't get much resistance.

As well, this series has some basic refactoring that I did along the way
and some bugs I had to fix in order to get my atomic-only MST code
working. Most of this is pretty straight forward and simply renaming
things to more closely match the DisplayPort specification, as I think
this will also make maintaining this code a lot easier in the long run
(I've gotten myself confused way too many times because of this).

So far I've tested this on all three MST drivers: amdgpu, i915 and
nouveau, along with making sure that removing the radeon MST code
doesn't break anything else. The one thing I very much could use help
with regarding testing though is making sure that this works with
amdgpu's DSC support on MST.

So, with this we should be using the atomic state as much as possible
with MST modesetting, hooray!

Cc: Wayne Lin <Wayne.Lin@amd.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Fangzhi Zuo <Jerry.Zuo@amd.com>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Imre Deak <imre.deak@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Sean Paul <sean@poorly.run>

Lyude Paul (18):
  drm/amdgpu/dc/mst: Rename dp_mst_stream_allocation(_table)
  drm/amdgpu/dm/mst: Rename get_payload_table()
  drm/display/dp_mst: Rename drm_dp_mst_vcpi_allocation
  drm/display/dp_mst: Call them time slots, not VCPI slots
  drm/display/dp_mst: Fix confusing docs for
    drm_dp_atomic_release_time_slots()
  drm/display/dp_mst: Add some missing kdocs for atomic MST structs
  drm/display/dp_mst: Add helper for finding payloads in atomic MST
    state
  drm/display/dp_mst: Add nonblocking helpers for DP MST
  drm/display/dp_mst: Don't open code modeset checks for releasing time
    slots
  drm/display/dp_mst: Fix modeset tracking in
    drm_dp_atomic_release_vcpi_slots()
  drm/nouveau/kms: Cache DP encoders in nouveau_connector
  drm/nouveau/kms: Pull mst state in for all modesets
  drm/display/dp_mst: Add helpers for serializing SST <-> MST
    transitions
  drm/display/dp_mst: Drop all ports from topology on CSNs before
    queueing link address work
  drm/display/dp_mst: Skip releasing payloads if last connected port
    isn't connected
  drm/display/dp_mst: Maintain time slot allocations when deleting
    payloads
  drm/radeon: Drop legacy MST support
  drm/display/dp_mst: Move all payload info into the atomic state

 .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |   72 +-
 .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c |  111 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  126 +-
 drivers/gpu/drm/amd/display/dc/core/dc_link.c |   10 +-
 drivers/gpu/drm/amd/display/dc/dm_helpers.h   |    4 +-
 .../amd/display/include/link_service_types.h  |   18 +-
 drivers/gpu/drm/display/drm_dp_mst_topology.c | 1160 ++++++++---------
 drivers/gpu/drm/i915/display/intel_display.c  |   11 +
 drivers/gpu/drm/i915/display/intel_dp.c       |    9 +
 drivers/gpu/drm/i915/display/intel_dp_mst.c   |   91 +-
 drivers/gpu/drm/i915/display/intel_hdcp.c     |   24 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  202 ++-
 drivers/gpu/drm/nouveau/dispnv50/disp.h       |    2 +
 drivers/gpu/drm/nouveau/nouveau_connector.c   |   18 +-
 drivers/gpu/drm/nouveau/nouveau_connector.h   |    3 +
 drivers/gpu/drm/radeon/Makefile               |    2 +-
 drivers/gpu/drm/radeon/atombios_crtc.c        |   11 +-
 drivers/gpu/drm/radeon/atombios_encoders.c    |   59 -
 drivers/gpu/drm/radeon/radeon_atombios.c      |    2 -
 drivers/gpu/drm/radeon/radeon_connectors.c    |   61 +-
 drivers/gpu/drm/radeon/radeon_device.c        |    1 -
 drivers/gpu/drm/radeon/radeon_dp_mst.c        |  778 -----------
 drivers/gpu/drm/radeon/radeon_drv.c           |    4 -
 drivers/gpu/drm/radeon/radeon_encoders.c      |   14 +-
 drivers/gpu/drm/radeon/radeon_irq_kms.c       |   10 +-
 drivers/gpu/drm/radeon/radeon_mode.h          |   40 -
 include/drm/display/drm_dp_mst_helper.h       |  230 ++--
 27 files changed, 991 insertions(+), 2082 deletions(-)
 delete mode 100644 drivers/gpu/drm/radeon/radeon_dp_mst.c

-- 
2.35.3


^ permalink raw reply	[flat|nested] 155+ messages in thread

end of thread, other threads:[~2022-08-10 22:14 UTC | newest]

Thread overview: 155+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-07 19:29 [Nouveau] [RESEND RFC 00/18] drm/display/dp_mst: Drop Radeon MST support, make MST atomic-only Lyude Paul
2022-06-07 19:29 ` Lyude Paul
2022-06-07 19:29 ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 01/18] drm/amdgpu/dc/mst: Rename dp_mst_stream_allocation(_table) Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 02/18] drm/amdgpu/dm/mst: Rename get_payload_table() Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 03/18] drm/display/dp_mst: Rename drm_dp_mst_vcpi_allocation Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [RESEND RFC 04/18] drm/display/dp_mst: Call them time slots, not VCPI slots Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Nouveau] " Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Intel-gfx] " Lyude Paul
2022-06-15  4:28   ` Lin, Wayne
2022-06-15  4:28     ` [Nouveau] " Lin, Wayne
2022-06-15  4:28     ` Lin, Wayne
2022-06-15  4:28     ` Lin, Wayne
2022-08-02 21:29     ` Lyude Paul
2022-08-02 21:29       ` [Nouveau] " Lyude Paul
2022-08-02 21:29       ` [Intel-gfx] " Lyude Paul
2022-08-02 21:29       ` Lyude Paul
2022-08-02 21:29       ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 05/18] drm/display/dp_mst: Fix confusing docs for drm_dp_atomic_release_time_slots() Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 06/18] drm/display/dp_mst: Add some missing kdocs for atomic MST structs Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-15  4:43   ` Lin, Wayne
2022-06-15  4:43     ` [Nouveau] " Lin, Wayne
2022-06-15  4:43     ` Lin, Wayne
2022-06-15  4:43     ` Lin, Wayne
2022-08-08 23:07     ` Lyude Paul
2022-08-08 23:07       ` Lyude Paul
2022-08-08 23:07       ` Lyude Paul
2022-08-08 23:07       ` [Nouveau] " Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 07/18] drm/display/dp_mst: Add helper for finding payloads in atomic MST state Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-29 13:22   ` Jani Nikula
2022-06-29 13:22     ` Jani Nikula
2022-06-29 13:22     ` Jani Nikula
2022-06-29 13:22     ` [Nouveau] " Jani Nikula
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 08/18] drm/display/dp_mst: Add nonblocking helpers for DP MST Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Intel-gfx] " Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-29 13:30   ` Jani Nikula
2022-06-29 13:30     ` Jani Nikula
2022-06-29 13:30     ` [Intel-gfx] " Jani Nikula
2022-06-29 13:30     ` Jani Nikula
2022-06-29 13:30     ` [Nouveau] " Jani Nikula
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 09/18] drm/display/dp_mst: Don't open code modeset checks for releasing time slots Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Intel-gfx] " Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 10/18] drm/display/dp_mst: Fix modeset tracking in drm_dp_atomic_release_vcpi_slots() Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 11/18] drm/nouveau/kms: Cache DP encoders in nouveau_connector Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 12/18] drm/nouveau/kms: Pull mst state in for all modesets Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [RESEND RFC 13/18] drm/display/dp_mst: Add helpers for serializing SST <-> MST transitions Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Nouveau] " Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Intel-gfx] " Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 14/18] drm/display/dp_mst: Drop all ports from topology on CSNs before queueing link address work Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 15/18] drm/display/dp_mst: Skip releasing payloads if last connected port isn't connected Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-07-05  8:45   ` Lin, Wayne
2022-07-05  8:45     ` [Nouveau] " Lin, Wayne
2022-07-05  8:45     ` Lin, Wayne
2022-07-05  8:45     ` Lin, Wayne
2022-08-02 22:12     ` [Nouveau] " Lyude Paul
2022-08-02 22:12       ` Lyude Paul
2022-08-02 22:12       ` Lyude Paul
2022-08-02 22:12       ` Lyude Paul
2022-08-10  3:28       ` Lin, Wayne
2022-08-10  3:28         ` Lin, Wayne
2022-08-10  3:28         ` Lin, Wayne
2022-08-10  3:28         ` [Nouveau] " Lin, Wayne
2022-08-10 22:13         ` Lyude Paul
2022-08-10 22:13           ` Lyude Paul
2022-08-10 22:13           ` Lyude Paul
2022-08-10 22:13           ` [Nouveau] " Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 16/18] drm/display/dp_mst: Maintain time slot allocations when deleting payloads Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29 ` [Nouveau] [RESEND RFC 17/18] drm/radeon: Drop legacy MST support Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 20:48   ` [Nouveau] " Alex Deucher
2022-06-07 20:48     ` Alex Deucher
2022-06-07 20:48     ` Alex Deucher
2022-06-07 19:29 ` [RESEND RFC 18/18] drm/display/dp_mst: Move all payload info into the atomic state Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Nouveau] " Lyude Paul
2022-06-07 19:29   ` Lyude Paul
2022-06-07 19:29   ` [Intel-gfx] " Lyude Paul
2022-07-05  9:10   ` Lin, Wayne
2022-07-05  9:10     ` [Nouveau] " Lin, Wayne
2022-07-05  9:10     ` Lin, Wayne
2022-07-05  9:10     ` Lin, Wayne
2022-07-06 21:57     ` Lyude Paul
2022-07-06 21:57       ` [Nouveau] " Lyude Paul
2022-07-06 21:57       ` Lyude Paul
2022-07-06 21:57       ` [Intel-gfx] " Lyude Paul
2022-07-06 21:57       ` Lyude Paul
2022-08-03 20:27     ` Lyude Paul
2022-08-03 20:27       ` [Nouveau] " Lyude Paul
2022-08-03 20:27       ` Lyude Paul
2022-08-03 20:27       ` [Intel-gfx] " Lyude Paul
2022-08-03 20:27       ` Lyude Paul
2022-08-08 10:02       ` Lin, Wayne
2022-08-08 10:02         ` [Nouveau] " Lin, Wayne
2022-08-08 10:02         ` [Intel-gfx] " Lin, Wayne
2022-08-08 10:02         ` Lin, Wayne
2022-08-08 10:02         ` Lin, Wayne
2022-08-08 22:53         ` Lyude Paul
2022-08-08 22:53           ` [Nouveau] " Lyude Paul
2022-08-08 22:53           ` Lyude Paul
2022-08-08 22:53           ` [Intel-gfx] " Lyude Paul
2022-08-08 22:53           ` Lyude Paul
2022-06-29 13:33 ` [Nouveau] [RESEND RFC 00/18] drm/display/dp_mst: Drop Radeon MST support, make MST atomic-only Jani Nikula
2022-06-29 13:33   ` Jani Nikula
2022-06-29 13:33   ` Jani Nikula
2022-07-28 22:21 ` [Nouveau] " Lyude Paul
2022-07-28 22:21   ` Lyude Paul
2022-07-28 22:21   ` Lyude Paul

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.