MST refcounting/atomic helpers cleanup

Submitted by Lyude Paul on Dec. 14, 2018, 1:25 a.m.

Details

Reviewer None
Submitted Dec. 14, 2018, 1:25 a.m.
Last Updated Jan. 11, 2019, 11:41 a.m.
Revision 7

Cover Letter(s)

Revision 1
      This is a WIP version of the series I've been working on for a while now
to get all of the atomic DRM drivers in the tree to use the atomic MST
helpers, and to make the atomic MST helpers actually idempotent. Turns
out it's a lot more difficult to do that without also fixing how port
and branch device refcounting works so that it actually makes sense,
since the current upstream implementation requires a ton of magic in the
atomic helpers to work around properly and in many situations just plain
doesn't work as intended.

This patch series is starting to get bigger, and since there's still a
few bits here and there regarding the new refcount implementation that I
haven't quite decided on yet I figured I should get an opinion from
everyone else.

Currently I've got a couple of thoughts on how I could improve this
further:

* Get rid of drm_dp_mst_get_*_validated() entirely - I'm 90% sure that
  with the new refcounting scheme we might not actually need port
  validation at all anymore, assuming we make the use of malloc references
  in all of the DRM drivers. Either way, I don't think validation was ever
  actually a concept that worked: without malloc references, the port or
  branch device that's being passed to drm_dp_mst_get_*_validated()
  could be freed which also in turn means that that the stale pointer
  could in theory have gotten reused for a new port and thus-cause us to
  consider a freed port validated.
* Get rid of drm_dp_mst_get_vcpi_slots() - with malloc references, I
  don't think there's any use for this either
* Get rid of drm_dp_mst_reset_vcpi_slots() - I think the only time this
  function ever made sense was with port validation? Honestly, I wonder
  if we ever needed this at all...

Note: I haven't applied some of the comments from the reviews for the
series this is based off of:

drm/dp_mst: Improve VCPI helpers, use in nouveau
https://patchwork.freedesktop.org/series/51414/

This is just getting put on the ML so I can get some feedback on this.

Lyude Paul (15):
  drm/dp_mst: Remove bogus conditional in drm_dp_update_payload_part1()
  drm/dp_mst: Refactor drm_dp_update_payload_part1()
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Fix potential use-after-frees for MSTCs
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  31 +
 .../gpu/dp-mst/topology-figure-2.dot          |  37 +
 .../gpu/dp-mst/topology-figure-3.dot          |  40 +
 Documentation/gpu/drm-kms-helpers.rst         | 125 ++-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 910 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  66 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 139 ++-
 10 files changed, 1178 insertions(+), 272 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 2
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 937 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1207 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 3
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 938 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1208 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 4
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 938 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1208 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 5
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 938 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1208 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 6
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 938 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1208 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    
Revision 7
      This is the series I've been working on for a while now to get all of
the atomic DRM drivers in the tree to use the atomic MST helpers, and to
make the atomic MST helpers actually idempotent. Turns out it's a lot
more difficult to do that without also fixing how port and branch device
refcounting works so that it actually makes sense, since the current
upstream implementation requires a ton of magic in the atomic helpers to
work around properly and in many situations just plain doesn't work as
intended.

There's still more cleanup that can be done, but I think this is a good
place to start off for now :).

Lyude Paul (16):
  drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and
    friends
  drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
  drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref
    fails
  drm/dp_mst: Stop releasing VCPI when removing ports from topology
  drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
  drm/i915: Keep malloc references to MST ports
  drm/amdgpu/display: Keep malloc ref to MST port
  drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
  drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
  drm/nouveau: Keep malloc references to MST ports
  drm/nouveau: Stop unsetting mstc->port, use malloc refs
  drm/nouveau: Grab payload lock in nv50_msto_payload()
  drm/dp_mst: Add some atomic state iterator macros
  drm/dp_mst: Start tracking per-port VCPI allocations
  drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
  drm/nouveau: Use atomic VCPI helpers for MST

 .../gpu/dp-mst/topology-figure-1.dot          |  52 +
 .../gpu/dp-mst/topology-figure-2.dot          |  56 ++
 .../gpu/dp-mst/topology-figure-3.dot          |  59 ++
 Documentation/gpu/drm-kms-helpers.rst         |  26 +-
 .../display/amdgpu_dm/amdgpu_dm_mst_types.c   |  11 +-
 drivers/gpu/drm/drm_dp_mst_topology.c         | 938 ++++++++++++++----
 drivers/gpu/drm/i915/intel_connector.c        |   4 +
 drivers/gpu/drm/i915/intel_display.c          |   4 +
 drivers/gpu/drm/i915/intel_dp_mst.c           |  65 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c       |  94 +-
 include/drm/drm_dp_mst_helper.h               | 151 ++-
 11 files changed, 1208 insertions(+), 252 deletions(-)
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-1.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-2.dot
 create mode 100644 Documentation/gpu/dp-mst/topology-figure-3.dot
    

Revisions

Patches download mbox

Tests

CI Bug Log - changes from CI_DRM_5316 -> Patchwork_11093
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_11093 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11093, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/1/mbox/

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11093:

### IGT changes ###

#### Possible regressions ####

  * igt@pm_rpm@basic-rte:
    - fi-byt-j1900:       PASS -> FAIL

  
#### Warnings ####

  * igt@pm_rpm@basic-pci-d3-state:
    - fi-byt-j1900:       PASS -> SKIP
    - fi-bsw-kefka:       SKIP -> PASS

  
Known issues
------------

  Here are the changes found in Patchwork_11093 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362]

  * {igt@runner@aborted}:
    - fi-icl-y:           NOTRUN -> FAIL [fdo#108070]

  
#### Possible fixes ####

  * igt@i915_selftest@live_hangcheck:
    - fi-kbl-7560u:       INCOMPLETE [fdo#108044] -> PASS

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       FAIL [fdo#108767] -> PASS

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS +1

  * igt@pm_rpm@basic-rte:
    - fi-bsw-kefka:       FAIL -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108044]: https://bugs.freedesktop.org/show_bug.cgi?id=108044
  [fdo#108070]: https://bugs.freedesktop.org/show_bug.cgi?id=108070
  [fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767


Participating hosts (52 -> 47)
------------------------------

  Additional (1): fi-icl-y 
  Missing    (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 


Build changes
-------------

    * Linux: CI_DRM_5316 -> Patchwork_11093

  CI_DRM_5316: caa765e4d3924701c07164819f47e9c7ee6565e5 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4747: ad821d1dc5d0eea4ac3a0e8e29c56c7f66191108 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11093: 36339baac207cf4f6aae4c6c99f6d7aa4d3f8943 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

36339baac207 drm/nouveau: Use atomic VCPI helpers for MST
6f6ba4dab707 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
ccded9c0a4ba drm/dp_mst: Start tracking per-port VCPI allocations
38ced18302b4 drm/dp_mst: Add some atomic state iterator macros
da2150a45314 drm/nouveau: Grab payload lock in nv50_msto_payload()
47a8052bcd5e drm/nouveau: Stop unsetting mstc->port, use malloc refs
106eea116ce9 drm/nouveau: Fix potential use-after-frees for MSTCs
13984c5ea301 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
cf8a68c1efd5 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
f7fb04a02f7c drm/i915: Keep malloc references to MST ports
bb69acc1a685 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
45508c149b9e drm/dp_mst: Stop releasing VCPI when removing ports from topology
e2f94fce005b drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
6b4005731a4b drm/dp_mst: Refactor drm_dp_update_payload_part1()
69cf74524440 drm/dp_mst: Remove bogus conditional in drm_dp_update_payload_part1()
$ dim checkpatch origin/drm-tip
69cf74524440 drm/dp_mst: Remove bogus conditional in drm_dp_update_payload_part1()
6b4005731a4b drm/dp_mst: Refactor drm_dp_update_payload_part1()
e2f94fce005b drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:22: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 91a25e463130 ("drm/dp/mst: deallocate payload on port destruction")'
#22: 
91a25e463130 ("drm/dp/mst: deallocate payload on port destruction")

-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:27: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")'
#27: 
263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:46: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit c54c7374ff44 ("drm/dp_mst: Skip validating ports during destruction, just ref")'
#46: 
c54c7374ff44 ("drm/dp_mst: Skip validating ports during destruction, just ref")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:101: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#101: 
new file mode 100644

-:428: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#428: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:894:
+	DRM_DEBUG("mstb %p (%d)\n", mstb, kref_read(&mstb->malloc_kref)-1);
 	                                                               ^

-:471: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#471: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:937:
+	DRM_DEBUG("port %p (%d)\n", port, kref_read(&port->malloc_kref)-1);
 	                                                               ^

-:605: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#605: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1058:
+	DRM_DEBUG("mstb %p (%d)\n", mstb, kref_read(&mstb->topology_kref)-1);
 	                                                                 ^

-:714: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#714: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1181:
+	DRM_DEBUG("port %p (%d)\n", port, kref_read(&port->topology_kref)-1);
 	                                                                 ^

-:734: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#734: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1197:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:754: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#754: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1214:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:780: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#780: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1235:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:800: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#800: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1252:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 4 errors, 2 warnings, 8 checks, 1258 lines checked
45508c149b9e drm/dp_mst: Stop releasing VCPI when removing ports from topology
bb69acc1a685 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:92: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#92: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2155:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
f7fb04a02f7c drm/i915: Keep malloc references to MST ports
cf8a68c1efd5 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
13984c5ea301 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
106eea116ce9 drm/nouveau: Fix potential use-after-frees for MSTCs
47a8052bcd5e drm/nouveau: Stop unsetting mstc->port, use malloc refs
da2150a45314 drm/nouveau: Grab payload lock in nv50_msto_payload()
38ced18302b4 drm/dp_mst: Add some atomic state iterator macros
-:106: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#106: FILE: include/drm/drm_dp_mst_helper.h:697:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:106: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#106: FILE: include/drm/drm_dp_mst_helper.h:697:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:108: WARNING:LONG_LINE: line over 100 characters
#108: FILE: include/drm/drm_dp_mst_helper.h:699:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:123: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#123: FILE: include/drm/drm_dp_mst_helper.h:714:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:123: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#123: FILE: include/drm/drm_dp_mst_helper.h:714:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:125: WARNING:LONG_LINE: line over 100 characters
#125: FILE: include/drm/drm_dp_mst_helper.h:716:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:141: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#141: FILE: include/drm/drm_dp_mst_helper.h:732:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:141: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#141: FILE: include/drm/drm_dp_mst_helper.h:732:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:143: WARNING:LONG_LINE: line over 100 characters
#143: FILE: include/drm/drm_dp_mst_helper.h:734:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
ccded9c0a4ba drm/dp_mst: Start tracking per-port VCPI allocations
6f6ba4dab707 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
36339baac207 drm/nouveau: Use atomic VCPI helpers for MST
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Remove bogus conditional in drm_dp_update_payload_part1()
Okay!

Commit: drm/dp_mst: Refactor drm_dp_update_payload_part1()
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Fix potential use-after-frees for MSTCs
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
+./include/linux/slab.h:332:43: warning: dubious: x & !y

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

Tests

CI Bug Log - changes from CI_DRM_5333 -> Patchwork_11132
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_11132 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11132, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/2/mbox/

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11132:

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live_contexts:
    - fi-bsw-kefka:       PASS -> DMESG-FAIL

  
#### Warnings ####

  * igt@kms_busy@basic-flip-c:
    - fi-kbl-7500u:       PASS -> SKIP +2

  
Known issues
------------

  Here are the changes found in Patchwork_11132 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_suspend@basic-s3:
    - fi-blb-e6850:       PASS -> INCOMPLETE [fdo#107718]

  * igt@gem_mmap_gtt@basic:
    - fi-glk-dsi:         PASS -> INCOMPLETE [fdo#103359] / [k.org#198133]

  * igt@kms_chamelium@dp-hpd-fast:
    - fi-kbl-7500u:       PASS -> DMESG-WARN [fdo#102505] / [fdo#103558] / [fdo#105602]

  * igt@kms_frontbuffer_tracking@basic:
    - fi-icl-u3:          PASS -> FAIL [fdo#103167]

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a:
    - fi-byt-clapper:     PASS -> FAIL [fdo#107362]

  
#### Possible fixes ####

  * igt@i915_selftest@live_hangcheck:
    - fi-bwr-2160:        DMESG-FAIL [fdo#108735] -> PASS
    - fi-apl-guc:         DMESG-FAIL -> PASS
    - fi-kbl-7560u:       INCOMPLETE [fdo#108044] -> PASS

  * igt@kms_flip@basic-flip-vs-dpms:
    - fi-skl-6700hq:      DMESG-WARN [fdo#105998] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
    - fi-byt-clapper:     FAIL [fdo#103167] -> PASS

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  
#### Warnings ####

  * igt@i915_selftest@live_contexts:
    - fi-icl-u2:          INCOMPLETE [fdo#108315] -> DMESG-FAIL [fdo#108569]

  
  [fdo#102505]: https://bugs.freedesktop.org/show_bug.cgi?id=102505
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#105998]: https://bugs.freedesktop.org/show_bug.cgi?id=105998
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
  [fdo#108044]: https://bugs.freedesktop.org/show_bug.cgi?id=108044
  [fdo#108315]: https://bugs.freedesktop.org/show_bug.cgi?id=108315
  [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
  [fdo#108735]: https://bugs.freedesktop.org/show_bug.cgi?id=108735
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (49 -> 44)
------------------------------

  Additional (1): fi-pnv-d510 
  Missing    (6): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus 


Build changes
-------------

    * Linux: CI_DRM_5333 -> Patchwork_11132

  CI_DRM_5333: c758693b615deff56e5e2098379b587486cfff8a @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4750: f05c8c2739dce89185349703062784a7745cab14 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11132: b6b2a3fc307688511351dc937250867c56fd8f00 @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

b6b2a3fc3076 drm/nouveau: Use atomic VCPI helpers for MST
4d20dde05c36 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
945779405902 drm/dp_mst: Start tracking per-port VCPI allocations
260fc11b1f9a drm/dp_mst: Add some atomic state iterator macros
d242f9257a4b drm/nouveau: Grab payload lock in nv50_msto_payload()
e4743c105b87 drm/nouveau: Stop unsetting mstc->port, use malloc refs
f0d136a0f492 drm/nouveau: Keep malloc references to MST ports
371889f7b60e drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
6d9550106577 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
3c24606c381a drm/amdgpu/display: Keep malloc ref to MST port
514f014c71f3 drm/i915: Keep malloc references to MST ports
859904901106 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
3b35c50d8406 drm/dp_mst: Stop releasing VCPI when removing ports from topology
a4378c0bb6f7 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
5e53f025958a drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
f950c3fa3447 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
$ dim checkpatch origin/drm-tip
f950c3fa3447 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1373:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 399 lines checked
5e53f025958a drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:22: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 91a25e463130 ("drm/dp/mst: deallocate payload on port destruction")'
#22: 
91a25e463130 ("drm/dp/mst: deallocate payload on port destruction")

-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:27: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")'
#27: 
263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:46: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit c54c7374ff44 ("drm/dp_mst: Skip validating ports during destruction, just ref")'
#46: 
c54c7374ff44 ("drm/dp_mst: Skip validating ports during destruction, just ref")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:129: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#129: 
new file mode 100644

-:511: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#511: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:988:
+	DRM_DEBUG("mstb %p (%d)\n", mstb, kref_read(&mstb->malloc_kref)-1);
 	                                                               ^

-:562: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#562: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1039:
+	DRM_DEBUG("port %p (%d)\n", port, kref_read(&port->malloc_kref)-1);
 	                                                               ^

-:679: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#679: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1149:
+	DRM_DEBUG("mstb %p (%d)\n", mstb, kref_read(&mstb->topology_kref)-1);
 	                                                                 ^

-:779: CHECK:SPACING: spaces preferred around that '-' (ctx:VxV)
#779: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1273:
+	DRM_DEBUG("port %p (%d)\n", port, kref_read(&port->topology_kref)-1);
 	                                                                 ^

-:832: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#832: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1327:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:846: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#846: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1344:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

-:1012: WARNING:BRACES: braces {} are not necessary for any arm of this statement
#1012: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2058:
+			if (drm_dp_mst_topology_try_get_mstb(rmstb)) {
[...]
+			} else {
[...]

total: 4 errors, 3 warnings, 6 checks, 974 lines checked
a4378c0bb6f7 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
3b35c50d8406 drm/dp_mst: Stop releasing VCPI when removing ports from topology
859904901106 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2274:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
514f014c71f3 drm/i915: Keep malloc references to MST ports
3c24606c381a drm/amdgpu/display: Keep malloc ref to MST port
6d9550106577 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
371889f7b60e drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
f0d136a0f492 drm/nouveau: Keep malloc references to MST ports
e4743c105b87 drm/nouveau: Stop unsetting mstc->port, use malloc refs
d242f9257a4b drm/nouveau: Grab payload lock in nv50_msto_payload()
260fc11b1f9a drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:712:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:729:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:747:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
945779405902 drm/dp_mst: Start tracking per-port VCPI allocations
-:388: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#388: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:3672:
+
+	}

total: 0 errors, 0 warnings, 1 checks, 464 lines checked
4d20dde05c36 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
b6b2a3fc3076 drm/nouveau: Use atomic VCPI helpers for MST
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
+       ^~
+                ^~
+drivers/gpu/drm/drm_dp_mst_topology.c:3064:16: error: ‘struct drm_dp_mst_topology_state’ has no member named ‘vcpi_allocated’
+drivers/gpu/drm/drm_dp_mst_topology.c:3064:23: error: no member 'vcpi_allocated' in struct drm_dp_mst_topology_state
+drivers/gpu/drm/drm_dp_mst_topology.c:3064:23: warning: generating address of non-lvalue (8)
+drivers/gpu/drm/drm_dp_mst_topology.c:3602:14: error: no member 'vcpi_allocated' in struct drm_dp_mst_topology_state
+drivers/gpu/drm/drm_dp_mst_topology.c:3602:14: warning: generating address of non-lvalue (8)
+drivers/gpu/drm/drm_dp_mst_topology.c:3602:7: error: ‘struct drm_dp_mst_topology_state’ has no member named ‘vcpi_allocated’
+drivers/gpu/drm/drm_dp_mst_topology.c: In function ‘drm_dp_atomic_find_vcpi_slots’:
+drivers/gpu/drm/drm_dp_mst_topology.c: In function ‘drm_dp_mst_duplicate_state’:
+make[1]: *** [drivers/gpu/drm/drm_dp_mst_topology.o] Error 1
+make[1]: *** Waiting for unfinished jobs....
+make: *** [drivers/gpu/drm/] Error 2
+  state->vcpi_allocated = false;
+  topology_state->vcpi_allocated = true;

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
-       ^~
-                ^~
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3064:16: error: ‘struct drm_dp_mst_topology_state’ has no member named ‘vcpi_allocated’
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3064:23: error: no member 'vcpi_allocated' in struct drm_dp_mst_topology_state
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3064:23: warning: generating address of non-lvalue (8)
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3602:14: error: no member 'vcpi_allocated' in struct drm_dp_mst_topology_state
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3602:14: warning: generating address of non-lvalue (8)
-O:drivers/gpu/drm/drm_dp_mst_topology.c:3602:7: error: ‘struct drm_dp_mst_topology_state’ has no member named ‘vcpi_allocated’
-drivers/gpu/drm/drm_dp_mst_topology.c: In function ‘drm_dp_atomic_find_vcpi_slots’:
-drivers/gpu/drm/drm_dp_mst_topology.c: In function ‘drm_dp_mst_duplicate_state’:
+./include/linux/slab.h:332:43: warning: dubious: x & !y
-make[1]: *** [drivers/gpu/drm/drm_dp_mst_topology.o] Error 1
-make[1]: *** Waiting for unfinished jobs....
-make: *** [drivers/gpu/drm/] Error 2
-  state->vcpi_allocated = false;
-  topology_state->vcpi_allocated = true;

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

Tests

CI Bug Log - changes from CI_DRM_5358 -> Patchwork_11177
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11177 need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11177, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/3/mbox/

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11177:

### IGT changes ###

#### Warnings ####

  * igt@kms_flip@basic-flip-vs-dpms:
    - fi-skl-6770hq:      SKIP -> PASS +36

  
Known issues
------------

  Here are the changes found in Patchwork_11177 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_basic@userptr:
    - fi-kbl-8809g:       PASS -> DMESG-WARN [fdo#108965]

  * igt@kms_frontbuffer_tracking@basic:
    - fi-icl-u3:          PASS -> FAIL [fdo#103167]
    - fi-byt-clapper:     PASS -> FAIL [fdo#103167]

  * igt@kms_pipe_crc_basic@read-crc-pipe-a:
    - fi-byt-clapper:     PASS -> FAIL [fdo#107362]

  
#### Possible fixes ####

  * igt@i915_selftest@live_evict:
    - fi-bsw-kefka:       DMESG-WARN [fdo#107709] -> PASS

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       FAIL [fdo#108767] -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
    - fi-blb-e6850:       INCOMPLETE [fdo#107718] -> PASS

  
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107709]: https://bugs.freedesktop.org/show_bug.cgi?id=107709
  [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
  [fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767
  [fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965


Participating hosts (50 -> 47)
------------------------------

  Additional (3): fi-snb-2600 fi-gdg-551 fi-pnv-d510 
  Missing    (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 


Build changes
-------------

    * Linux: CI_DRM_5358 -> Patchwork_11177

  CI_DRM_5358: c6a0276a5007c01c64a8a80552b78c115e8a0dae @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4755: 0ba18cf75cafb51d1e72557528de4a1be640a85c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11177: c6d9ff62c32637d2b4cee64901ccb0bad6ab752f @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

c6d9ff62c326 drm/nouveau: Use atomic VCPI helpers for MST
7dd0558cdf87 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
16130bfa38c6 drm/dp_mst: Start tracking per-port VCPI allocations
302222f0d27b drm/dp_mst: Add some atomic state iterator macros
0aa7825762c5 drm/nouveau: Grab payload lock in nv50_msto_payload()
39d62a14d5d2 drm/nouveau: Stop unsetting mstc->port, use malloc refs
309d529712a6 drm/nouveau: Keep malloc references to MST ports
7c0f2ae7cab3 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
9d1d60cf283c drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
3768f7774d8b drm/amdgpu/display: Keep malloc ref to MST port
0dd0be44242b drm/i915: Keep malloc references to MST ports
e0da7592aee7 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
0ff3af9e3edf drm/dp_mst: Stop releasing VCPI when removing ports from topology
5641c25f8c24 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
cca38b72e03b drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
220fc69cfc1d drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
$ dim checkpatch origin/drm-tip
220fc69cfc1d drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1373:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 399 lines checked
cca38b72e03b drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:136: WARNING:BAD_SIGN_OFF: Duplicate signature
#136: 
Signed-off-by: Lyude Paul <lyude@redhat.com>

-:139: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#139: 
new file mode 100644

-:844: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#844: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1329:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:858: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#858: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1346:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 1 errors, 3 warnings, 2 checks, 975 lines checked
5641c25f8c24 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
0ff3af9e3edf drm/dp_mst: Stop releasing VCPI when removing ports from topology
e0da7592aee7 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2276:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
0dd0be44242b drm/i915: Keep malloc references to MST ports
3768f7774d8b drm/amdgpu/display: Keep malloc ref to MST port
9d1d60cf283c drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
7c0f2ae7cab3 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
309d529712a6 drm/nouveau: Keep malloc references to MST ports
39d62a14d5d2 drm/nouveau: Stop unsetting mstc->port, use malloc refs
0aa7825762c5 drm/nouveau: Grab payload lock in nv50_msto_payload()
302222f0d27b drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:712:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:729:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:747:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
16130bfa38c6 drm/dp_mst: Start tracking per-port VCPI allocations
7dd0558cdf87 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
c6d9ff62c326 drm/nouveau: Use atomic VCPI helpers for MST
CI Bug Log - changes from CI_DRM_5358_full -> Patchwork_11177_full
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11177_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11177_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11177_full:

### IGT changes ###

#### Warnings ####

  * igt@pm_rc6_residency@rc6-accuracy:
    - shard-kbl:          SKIP -> PASS

  * igt@tools_test@tools_test:
    - shard-glk:          SKIP -> PASS

  
Known issues
------------

  Here are the changes found in Patchwork_11177_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_workarounds@suspend-resume-context:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@kms_available_modes_crc@available_mode_test_crc:
    - shard-apl:          PASS -> FAIL [fdo#106641]

  * igt@kms_busy@extended-modeset-hang-newfb-render-c:
    - shard-hsw:          PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_ccs@pipe-b-crc-sprite-planes-basic:
    - shard-apl:          PASS -> FAIL [fdo#106510] / [fdo#108145]

  * igt@kms_cursor_crc@cursor-64x21-sliding:
    - shard-apl:          PASS -> FAIL [fdo#103232] +5

  * igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
    - shard-glk:          PASS -> FAIL [fdo#105363]

  * igt@kms_flip@modeset-vs-vblank-race-interruptible:
    - shard-apl:          PASS -> DMESG-WARN [fdo#103558] / [fdo#105602] +5

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
    - shard-glk:          PASS -> FAIL [fdo#103167] +1

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move:
    - shard-iclb:         PASS -> FAIL [fdo#103167] +4

  * igt@kms_frontbuffer_tracking@fbc-1p-rte:
    - shard-apl:          PASS -> DMESG-WARN [fdo#103558] / [fdo#108131]

  * igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite:
    - shard-iclb:         PASS -> DMESG-FAIL [fdo#107724]

  * igt@kms_frontbuffer_tracking@psr-slowdraw:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +2

  * igt@kms_plane@pixel-format-pipe-a-planes:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#106885]

  * igt@kms_plane@plane-position-covered-pipe-c-planes:
    - shard-apl:          PASS -> FAIL [fdo#103166]
    - shard-glk:          PASS -> FAIL [fdo#103166]

  * igt@kms_plane_alpha_blend@pipe-a-alpha-transparant-fb:
    - shard-skl:          NOTRUN -> FAIL [fdo#108145] +2

  * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
    - shard-glk:          PASS -> FAIL [fdo#108145] +1

  * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
    - shard-skl:          PASS -> FAIL [fdo#107815]

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-x:
    - shard-iclb:         PASS -> FAIL [fdo#103166] +1

  * igt@kms_sysfs_edid_timing:
    - shard-iclb:         PASS -> FAIL [fdo#100047]

  * igt@kms_vblank@pipe-a-ts-continuation-idle-hang:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] +6

  * igt@pm_rpm@dpms-mode-unset-lpsp:
    - shard-skl:          NOTRUN -> INCOMPLETE [fdo#107807]

  * igt@pm_rpm@gem-execbuf-stress-extra-wait:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#108840]

  * igt@pm_rpm@i2c:
    - shard-skl:          PASS -> INCOMPLETE [fdo#107807]

  
#### Possible fixes ####

  * igt@gem_ctx_isolation@rcs0-s3:
    - shard-skl:          INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS

  * igt@kms_color@pipe-a-legacy-gamma:
    - shard-apl:          FAIL [fdo#104782] / [fdo#108145] -> PASS

  * igt@kms_cursor_legacy@flip-vs-cursor-atomic:
    - shard-skl:          FAIL [fdo#102670] -> PASS

  * igt@kms_draw_crc@draw-method-xrgb8888-render-xtiled:
    - shard-skl:          FAIL [fdo#107791] -> PASS

  * igt@kms_draw_crc@draw-method-xrgb8888-render-ytiled:
    - shard-skl:          FAIL [fdo#103232] -> PASS

  * igt@kms_flip@wf_vblank-ts-check:
    - shard-apl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +14

  * igt@kms_flip_tiling@flip-x-tiled:
    - shard-skl:          FAIL [fdo#108145] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
    - shard-apl:          FAIL [fdo#103167] -> PASS +2

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-iclb:         FAIL [fdo#103167] -> PASS +5

  * igt@kms_frontbuffer_tracking@fbcpsr-indfb-scaledprimary:
    - shard-skl:          FAIL [fdo#105682] -> PASS

  * igt@kms_frontbuffer_tracking@fbcpsr-stridechange:
    - shard-iclb:         FAIL [fdo#105683] -> PASS

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-mmap-cpu:
    - shard-skl:          FAIL [fdo#103167] -> PASS

  * igt@kms_frontbuffer_tracking@psr-suspend:
    - shard-iclb:         INCOMPLETE [fdo#106978] / [fdo#107713] -> PASS

  * igt@kms_plane@plane-position-covered-pipe-b-planes:
    - shard-glk:          FAIL [fdo#103166] -> PASS

  * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max:
    - shard-glk:          FAIL [fdo#108145] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
    - shard-iclb:         FAIL [fdo#103166] -> PASS

  * igt@kms_plane_scaling@pipe-b-scaler-with-rotation:
    - shard-iclb:         DMESG-WARN [fdo#107724] -> PASS

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-glk:          DMESG-FAIL [fdo#105763] / [fdo#106538] -> PASS

  * igt@kms_rotation_crc@primary-rotation-90:
    - shard-skl:          FAIL [fdo#103925] / [fdo#107815] -> PASS

  * igt@kms_setmode@basic:
    - shard-apl:          FAIL [fdo#99912] -> PASS
    - shard-iclb:         FAIL [fdo#99912] -> PASS

  * igt@pm_rpm@modeset-non-lpsp-stress-no-wait:
    - shard-skl:          INCOMPLETE [fdo#107807] -> SKIP

  
#### Warnings ####

  * igt@i915_suspend@shrink:
    - shard-skl:          INCOMPLETE [fdo#106886] -> DMESG-WARN [fdo#107886] / [fdo#108784]

  * igt@kms_color@pipe-b-legacy-gamma:
    - shard-apl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x128-sliding:
    - shard-iclb:         FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] / [fdo#108336]

  * igt@kms_cursor_crc@cursor-64x64-random:
    - shard-apl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-64x64-suspend:
    - shard-apl:          FAIL [fdo#103191] / [fdo#103232] -> DMESG-FAIL [fdo#103232] / [fdo#103558] / [fdo#105602]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-apl:          FAIL [fdo#103167] -> DMESG-FAIL [fdo#103167] / [fdo#103558] / [fdo#105602]

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
    - shard-skl:          FAIL [fdo#103166] -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@kms_plane@plane-position-covered-pipe-a-planes:
    - shard-apl:          FAIL [fdo#103166] -> DMESG-FAIL [fdo#103166] / [fdo#103558] / [fdo#105602]

  * igt@kms_plane_alpha_blend@pipe-a-alpha-basic:
    - shard-apl:          FAIL [fdo#108145] -> DMESG-FAIL [fdo#103558] / [fdo#105602] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb:
    - shard-apl:          DMESG-FAIL [fdo#103558] / [fdo#105602] / [fdo#108145] -> FAIL [fdo#108145]

  
  [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
  [fdo#102670]: https://bugs.freedesktop.org/show_bug.cgi?id=102670
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#103925]: https://bugs.freedesktop.org/show_bug.cgi?id=103925
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
  [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
  [fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
  [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
  [fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
  [fdo#106538]: https://bugs.freedesktop.org/show_bug.cgi?id=106538
  [fdo#106641]: https://bugs.freedesktop.org/show_bug.cgi?id=106641
  [fdo#106885]: https://bugs.freedesktop.org/show_bug.cgi?id=106885
  [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
  [fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
  [fdo#107791]: https://bugs.freedesktop.org/show_bug.cgi?id=107791
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
  [fdo#107886]: https://bugs.freedesktop.org/show_bug.cgi?id=107886
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108131]: https://bugs.freedesktop.org/show_bug.cgi?id=108131
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
  [fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
  [fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
  [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912


Participating hosts (7 -> 7)
------------------------------

  No changes in participating hosts


Build changes
-------------

    * Linux: CI_DRM_5358 -> Patchwork_11177

  CI_DRM_5358: c6a0276a5007c01c64a8a80552b78c115e8a0dae @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4755: 0ba18cf75cafb51d1e72557528de4a1be640a85c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11177: c6d9ff62c32637d2b4cee64901ccb0bad6ab752f @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
+./include/linux/slab.h:332:43: warning: dubious: x & !y

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

Tests

CI Bug Log - changes from CI_DRM_5363 -> Patchwork_11191
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/4/mbox/

Known issues
------------

  Here are the changes found in Patchwork_11191 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@i915_module_load@reload-no-display:
    - fi-byt-clapper:     PASS -> WARN [fdo#108688]

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362]

  
#### Possible fixes ####

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       FAIL [fdo#108767] -> PASS

  * igt@kms_frontbuffer_tracking@basic:
    - fi-hsw-peppy:       DMESG-WARN [fdo#102614] -> PASS

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-a:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  [fdo#102614]: https://bugs.freedesktop.org/show_bug.cgi?id=102614
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108688]: https://bugs.freedesktop.org/show_bug.cgi?id=108688
  [fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767


Participating hosts (47 -> 38)
------------------------------

  Additional (3): fi-byt-n2820 fi-apl-guc fi-pnv-d510 
  Missing    (12): fi-kbl-soraka fi-kbl-7567u fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-kbl-7560u fi-skl-6700hq fi-skl-6600u fi-snb-2600 


Build changes
-------------

    * Linux: CI_DRM_5363 -> Patchwork_11191

  CI_DRM_5363: f141806c68e4cbd56e3a5a582eb1a6f5b7edfc84 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4756: 75081c6bfb9998bd7cbf35a7ac0578c683fe55a8 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11191: b2223e97312f7dc451725c1cefbea5785e09078a @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

b2223e97312f drm/nouveau: Use atomic VCPI helpers for MST
7e8e5b7e31ad drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
31f003c7f24b drm/dp_mst: Start tracking per-port VCPI allocations
42197be73f27 drm/dp_mst: Add some atomic state iterator macros
b6f6bbe3d1bc drm/nouveau: Grab payload lock in nv50_msto_payload()
81af58e1850c drm/nouveau: Stop unsetting mstc->port, use malloc refs
0d5dc5d62fd6 drm/nouveau: Keep malloc references to MST ports
03cee9c12c05 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
90a9631dfbe1 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
5dbb33b4a196 drm/amdgpu/display: Keep malloc ref to MST port
c9b9ef293e41 drm/i915: Keep malloc references to MST ports
922a91cd84d2 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
42483b36cce0 drm/dp_mst: Stop releasing VCPI when removing ports from topology
a20c087d93b3 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
395ddef17891 drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
2a2093f380b0 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
$ dim checkpatch origin/drm-tip
2a2093f380b0 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1373:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 399 lines checked
395ddef17891 drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:136: WARNING:BAD_SIGN_OFF: Duplicate signature
#136: 
Signed-off-by: Lyude Paul <lyude@redhat.com>

-:139: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#139: 
new file mode 100644

-:844: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#844: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1329:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:858: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#858: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1346:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 1 errors, 3 warnings, 2 checks, 975 lines checked
a20c087d93b3 drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
42483b36cce0 drm/dp_mst: Stop releasing VCPI when removing ports from topology
922a91cd84d2 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2276:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
c9b9ef293e41 drm/i915: Keep malloc references to MST ports
5dbb33b4a196 drm/amdgpu/display: Keep malloc ref to MST port
90a9631dfbe1 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
03cee9c12c05 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
0d5dc5d62fd6 drm/nouveau: Keep malloc references to MST ports
81af58e1850c drm/nouveau: Stop unsetting mstc->port, use malloc refs
b6f6bbe3d1bc drm/nouveau: Grab payload lock in nv50_msto_payload()
42197be73f27 drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:710:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:712:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:727:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:729:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:745:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:747:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
31f003c7f24b drm/dp_mst: Start tracking per-port VCPI allocations
-:42: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#42: 
* Add some missing changes that were requested by danvet that I forgot about after

total: 0 errors, 1 warnings, 0 checks, 444 lines checked
7e8e5b7e31ad drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
b2223e97312f drm/nouveau: Use atomic VCPI helpers for MST
CI Bug Log - changes from CI_DRM_5363_full -> Patchwork_11191_full
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11191_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11191_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11191_full:

### IGT changes ###

#### Warnings ####

  * igt@kms_cursor_legacy@cursora-vs-flipa-atomic:
    - shard-apl:          PASS -> SKIP +13

  
Known issues
------------

  Here are the changes found in Patchwork_11191_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ppgtt@blt-vs-render-ctx0:
    - shard-skl:          NOTRUN -> TIMEOUT [fdo#108039]

  * igt@kms_atomic_transition@plane-all-transition-nonblocking-fencing:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] / [fdo#109225]

  * igt@kms_busy@extended-modeset-hang-newfb-render-c:
    - shard-kbl:          PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#107956] +1

  * igt@kms_busy@extended-pageflip-hang-newfb-render-b:
    - shard-snb:          PASS -> INCOMPLETE [fdo#105411]

  * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-b:
    - shard-iclb:         NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_ccs@pipe-b-crc-sprite-planes-basic:
    - shard-iclb:         NOTRUN -> FAIL [fdo#107725]

  * igt@kms_chv_cursor_fail@pipe-a-64x64-top-edge:
    - shard-apl:          PASS -> DMESG-WARN [fdo#103558] / [fdo#105602] +62

  * igt@kms_color@pipe-b-ctm-0-25:
    - shard-skl:          PASS -> FAIL [fdo#108682]

  * igt@kms_color@pipe-b-legacy-gamma:
    - shard-apl:          PASS -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x128-dpms:
    - shard-apl:          PASS -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-128x128-suspend:
    - shard-glk:          PASS -> FAIL [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-256x256-sliding:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103232] +3

  * igt@kms_draw_crc@draw-method-xrgb2101010-mmap-cpu-untiled:
    - shard-iclb:         PASS -> WARN [fdo#108336] +3

  * igt@kms_flip@2x-flip-vs-expired-vblank:
    - shard-glk:          PASS -> FAIL [fdo#105363]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-onoff:
    - shard-apl:          PASS -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-pwrite:
    - shard-glk:          PASS -> FAIL [fdo#103167] +2

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-shrfb-msflip-blt:
    - shard-iclb:         PASS -> DMESG-FAIL [fdo#107724] +5

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-blt:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-pwrite:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +10

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-fullscreen:
    - shard-iclb:         PASS -> FAIL [fdo#103167] +2

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@kms_plane@pixel-format-pipe-a-planes-source-clamping:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#106885] +1

  * igt@kms_plane@plane-position-covered-pipe-b-planes:
    - shard-glk:          PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
    - shard-apl:          PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-x:
    - shard-iclb:         PASS -> FAIL [fdo#103166] +1

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-yf:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103166]

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-glk:          PASS -> DMESG-FAIL [fdo#105763] / [fdo#106538]

  * igt@kms_sysfs_edid_timing:
    - shard-iclb:         PASS -> FAIL [fdo#100047]
    - shard-skl:          NOTRUN -> FAIL [fdo#100047]

  * igt@perf_pmu@event-wait-rcs0:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] +21

  * igt@perf_pmu@rc6-runtime-pm-long:
    - shard-iclb:         PASS -> FAIL [fdo#105010]

  * igt@pm_rpm@gem-evict-pwrite:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#108840]

  * igt@pm_rpm@modeset-non-lpsp-stress:
    - shard-skl:          SKIP -> INCOMPLETE [fdo#107807]

  * igt@pm_rpm@system-suspend-execbuf:
    - shard-iclb:         PASS -> INCOMPLETE [fdo#107713] +1

  * igt@pm_rps@waitboost:
    - shard-iclb:         NOTRUN -> FAIL [fdo#102250] / [fdo#108059]

  
#### Possible fixes ####

  * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
    - shard-glk:          FAIL [fdo#108145] -> PASS

  * igt@kms_color@pipe-c-ctm-max:
    - shard-apl:          FAIL [fdo#108147] -> SKIP

  * igt@kms_cursor_crc@cursor-256x256-suspend:
    - shard-skl:          INCOMPLETE [fdo#104108] -> PASS
    - shard-apl:          FAIL [fdo#103191] / [fdo#103232] -> PASS

  * igt@kms_cursor_crc@cursor-256x85-random:
    - shard-apl:          FAIL [fdo#103232] -> PASS +3

  * igt@kms_cursor_legacy@flip-vs-cursor-legacy:
    - shard-apl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +16

  * igt@kms_draw_crc@draw-method-xrgb2101010-mmap-wc-untiled:
    - shard-iclb:         WARN [fdo#108336] -> PASS +1

  * igt@kms_flip@flip-vs-expired-vblank:
    - shard-skl:          FAIL [fdo#105363] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw:
    - shard-iclb:         DMESG-FAIL [fdo#107720] / [fdo#107724] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-glk:          FAIL [fdo#103167] -> PASS +2

  * igt@kms_frontbuffer_tracking@fbc-rgb565-draw-mmap-gtt:
    - shard-kbl:          DMESG-WARN [fdo#103313] / [fdo#103558] / [fdo#105602] -> PASS +1

  * igt@kms_frontbuffer_tracking@fbc-stridechange:
    - shard-iclb:         DMESG-FAIL [fdo#107724] -> PASS +7

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-indfb-draw-mmap-cpu:
    - shard-iclb:         DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +13

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-iclb:         FAIL [fdo#103167] -> PASS

  * igt@kms_plane@pixel-format-pipe-a-planes-source-clamping:
    - shard-apl:          FAIL [fdo#108948] -> PASS

  * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-b-planes:
    - shard-iclb:         INCOMPLETE [fdo#107713] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
    - shard-glk:          FAIL [fdo#103166] -> PASS
    - shard-apl:          FAIL [fdo#103166] -> PASS

  * igt@kms_sequence@get-busy:
    - shard-iclb:         DMESG-WARN [fdo#107724] -> PASS +25

  * igt@kms_setmode@basic:
    - shard-apl:          FAIL [fdo#99912] -> PASS

  * igt@kms_vblank@pipe-a-ts-continuation-dpms-suspend:
    - shard-kbl:          INCOMPLETE [fdo#103665] -> PASS

  * igt@kms_vblank@pipe-a-ts-continuation-modeset-hang:
    - shard-kbl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +14

  * igt@perf_pmu@rc6-runtime-pm-long:
    - shard-skl:          FAIL [fdo#105010] -> PASS

  * igt@pm_rpm@cursor-dpms:
    - shard-iclb:         DMESG-WARN [fdo#108654] -> PASS

  * igt@pm_rpm@system-suspend-modeset:
    - shard-skl:          INCOMPLETE [fdo#104108] / [fdo#107807] -> PASS

  * igt@pm_rps@min-max-config-loaded:
    - shard-glk:          FAIL [fdo#102250] -> PASS

  
#### Warnings ####

  * igt@i915_suspend@shrink:
    - shard-snb:          DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#105411] / [fdo#106886]

  * igt@kms_cursor_crc@cursor-256x256-random:
    - shard-iclb:         FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] / [fdo#108336] +1

  * igt@kms_cursor_crc@cursor-64x64-onscreen:
    - shard-iclb:         DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103232]

  * igt@kms_plane@pixel-format-pipe-a-planes-source-clamping:
    - shard-iclb:         FAIL [fdo#108948] -> DMESG-WARN [fdo#106885] / [fdo#107724] / [fdo#108336]

  * igt@kms_plane@pixel-format-pipe-c-planes-source-clamping:
    - shard-apl:          FAIL [fdo#108948] -> DMESG-WARN [fdo#103558] / [fdo#105602]

  * igt@kms_plane@plane-position-covered-pipe-b-planes:
    - shard-iclb:         FAIL [fdo#103166] -> DMESG-WARN [fdo#107724] / [fdo#108336]

  * igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
    - shard-apl:          FAIL [fdo#108145] -> DMESG-FAIL [fdo#103558] / [fdo#105602] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
    - shard-apl:          DMESG-FAIL [fdo#103558] / [fdo#105602] / [fdo#108145] -> FAIL [fdo#108145]
    - shard-kbl:          DMESG-FAIL [fdo#103558] / [fdo#105602] / [fdo#108145] -> FAIL [fdo#108145]

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-x:
    - shard-apl:          FAIL [fdo#103166] -> DMESG-WARN [fdo#103558] / [fdo#105602]

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-yf:
    - shard-apl:          DMESG-FAIL [fdo#103166] / [fdo#103558] / [fdo#105602] -> FAIL [fdo#103166]
    - shard-kbl:          DMESG-FAIL [fdo#103166] / [fdo#103558] / [fdo#105602] -> FAIL [fdo#103166]

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-kbl:          DMESG-FAIL [fdo#108950] -> DMESG-WARN [fdo#105604]

  
  [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
  [fdo#102250]: https://bugs.freedesktop.org/show_bug.cgi?id=102250
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
  [fdo#105010]: https://bugs.freedesktop.org/show_bug.cgi?id=105010
  [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
  [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#105604]: https://bugs.freedesktop.org/show_bug.cgi?id=105604
  [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
  [fdo#106538]: https://bugs.freedesktop.org/show_bug.cgi?id=106538
  [fdo#106885]: https://bugs.freedesktop.org/show_bug.cgi?id=106885
  [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107720]: https://bugs.freedesktop.org/show_bug.cgi?id=107720
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
  [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108039]: https://bugs.freedesktop.org/show_bug.cgi?id=108039
  [fdo#108059]: https://bugs.freedesktop.org/show_bug.cgi?id=108059
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
  [fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
  [fdo#108654]: https://bugs.freedesktop.org/show_bug.cgi?id=108654
  [fdo#108682]: https://bugs.freedesktop.org/show_bug.cgi?id=108682
  [fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
  [fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
  [fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948
  [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
  [fdo#109225]: https://bugs.freedesktop.org/show_bug.cgi?id=109225
  [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912


Participating hosts (7 -> 7)
------------------------------

  No changes in participating hosts


Build changes
-------------

    * Linux: CI_DRM_5363 -> Patchwork_11191

  CI_DRM_5363: f141806c68e4cbd56e3a5a582eb1a6f5b7edfc84 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4756: 75081c6bfb9998bd7cbf35a7ac0578c683fe55a8 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11191: b2223e97312f7dc451725c1cefbea5785e09078a @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
+./include/linux/slab.h:332:43: warning: dubious: x & !y

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

# Name Submitter State A F R T
[v5,01/20] drm/dp_mst: Fix some formatting in drm_dp_add_port() Lyude Paul Accepted 1
[v5,02/20] drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg() Lyude Paul Accepted 1
[v5,03/20] drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi() Lyude Paul Accepted 1
[v5,04/20] drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi() Lyude Paul Accepted 1
[v5,05/20] drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends Lyude Paul Accepted 2
[v5,06/20] drm/dp_mst: Introduce new refcounting scheme for mstbs and ports Lyude Paul New 2
[v5,07/20] drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails Lyude Paul New 2
[v5,08/20] drm/dp_mst: Stop releasing VCPI when removing ports from topology Lyude Paul New 2
[v5,09/20] drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs Lyude Paul New 2
[v5,10/20] drm/i915: Keep malloc references to MST ports Lyude Paul New 1
[v5,11/20] drm/amdgpu/display: Keep malloc ref to MST port Lyude Paul New 1
[v5,12/20] drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector() Lyude Paul New
[v5,13/20] drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup() Lyude Paul New
[v5,14/20] drm/nouveau: Keep malloc references to MST ports Lyude Paul New
[v5,15/20] drm/nouveau: Stop unsetting mstc->port, use malloc refs Lyude Paul New
[v5,16/20] drm/nouveau: Grab payload lock in nv50_msto_payload() Lyude Paul New
[v5,17/20] drm/dp_mst: Add some atomic state iterator macros Lyude Paul New 2
[v5,18/20] drm/dp_mst: Start tracking per-port VCPI allocations Lyude Paul New 1
[v5,19/20] drm/dp_mst: Check payload count in drm_dp_mst_atomic_check() Lyude Paul New 2
[v5,20/20] drm/nouveau: Use atomic VCPI helpers for MST Lyude Paul New

Tests

CI Bug Log - changes from CI_DRM_5380 -> Patchwork_11258
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11258 need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11258, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/5/

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11258:

### IGT changes ###

#### Warnings ####

  * igt@kms_flip@basic-flip-vs-dpms:
    - fi-skl-6770hq:      SKIP -> PASS +36

  
Known issues
------------

  Here are the changes found in Patchwork_11258 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_basic@userptr:
    - fi-kbl-8809g:       PASS -> DMESG-WARN [fdo#108965]

  * igt@i915_selftest@live_hangcheck:
    - fi-skl-guc:         PASS -> DMESG-FAIL [fdo#108593]

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       PASS -> FAIL [fdo#108767]

  * igt@kms_pipe_crc_basic@read-crc-pipe-b:
    - fi-byt-clapper:     PASS -> FAIL [fdo#107362]

  * igt@pm_rpm@module-reload:
    - fi-icl-u2:          PASS -> DMESG-WARN [fdo#108654]

  
#### Possible fixes ####

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#108593]: https://bugs.freedesktop.org/show_bug.cgi?id=108593
  [fdo#108654]: https://bugs.freedesktop.org/show_bug.cgi?id=108654
  [fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767
  [fdo#108965]: https://bugs.freedesktop.org/show_bug.cgi?id=108965
  [fdo#109241]: https://bugs.freedesktop.org/show_bug.cgi?id=109241


Participating hosts (49 -> 44)
------------------------------

  Missing    (5): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-icl-y fi-bdw-samus 


Build changes
-------------

    * Linux: CI_DRM_5380 -> Patchwork_11258

  CI_DRM_5380: f7836843c875734c42d61d755d822635390ab097 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4757: 738f43a54d626f08e250c926a5aeec53458fbd3c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11258: d6b0067e30e569204fbf1adab5df2e552713fefc @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

d6b0067e30e5 drm/nouveau: Use atomic VCPI helpers for MST
1acb8253f5b7 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
9077cacf44e8 drm/dp_mst: Start tracking per-port VCPI allocations
d1f83b76bee9 drm/dp_mst: Add some atomic state iterator macros
01a9632ebf65 drm/nouveau: Grab payload lock in nv50_msto_payload()
916a684cd19b drm/nouveau: Stop unsetting mstc->port, use malloc refs
a7f4c8e68b5d drm/nouveau: Keep malloc references to MST ports
171456c757e5 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
523545b36c93 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
f0300e01a8cf drm/amdgpu/display: Keep malloc ref to MST port
89d20c7c627b drm/i915: Keep malloc references to MST ports
33e82d971527 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
11a9df239381 drm/dp_mst: Stop releasing VCPI when removing ports from topology
a1f9438797bb drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
f212460470bf drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
bc368b90cb12 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
0b7daea7eab8 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
f7a476bd0e0f drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
2fd355b3a9aa drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
5778d100cac7 drm/dp_mst: Fix some formatting in drm_dp_add_port()
$ dim checkpatch origin/drm-tip
5778d100cac7 drm/dp_mst: Fix some formatting in drm_dp_add_port()
2fd355b3a9aa drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
f7a476bd0e0f drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
0b7daea7eab8 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
bc368b90cb12 drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1379:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 401 lines checked
f212460470bf drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:137: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#137: 
new file mode 100644

-:842: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#842: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1329:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:856: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#856: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1346:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 1 errors, 2 warnings, 2 checks, 915 lines checked
a1f9438797bb drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
11a9df239381 drm/dp_mst: Stop releasing VCPI when removing ports from topology
33e82d971527 drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2278:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
89d20c7c627b drm/i915: Keep malloc references to MST ports
f0300e01a8cf drm/amdgpu/display: Keep malloc ref to MST port
523545b36c93 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
171456c757e5 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
a7f4c8e68b5d drm/nouveau: Keep malloc references to MST ports
916a684cd19b drm/nouveau: Stop unsetting mstc->port, use malloc refs
01a9632ebf65 drm/nouveau: Grab payload lock in nv50_msto_payload()
d1f83b76bee9 drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:713:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:730:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:748:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
9077cacf44e8 drm/dp_mst: Start tracking per-port VCPI allocations
-:45: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#45: 
  * Cleanup atomic check logic for VCPI allocations - all we need to check in

total: 0 errors, 1 warnings, 0 checks, 444 lines checked
1acb8253f5b7 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
d6b0067e30e5 drm/nouveau: Use atomic VCPI helpers for MST
CI Bug Log - changes from CI_DRM_5380_full -> Patchwork_11258_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Known issues
------------

  Here are the changes found in Patchwork_11258_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_schedule@pi-ringfull-blt:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103158]

  * igt@gem_exec_schedule@pi-ringfull-bsd:
    - shard-skl:          NOTRUN -> FAIL [fdo#103158] +1

  * igt@gem_ppgtt@blt-vs-render-ctx0:
    - shard-skl:          PASS -> TIMEOUT [fdo#108039]

  * igt@kms_busy@extended-modeset-hang-newfb-render-a:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#107956] +2

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-a:
    - shard-iclb:         NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_ccs@pipe-a-crc-primary-basic:
    - shard-iclb:         NOTRUN -> FAIL [fdo#107725]

  * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
    - shard-apl:          PASS -> FAIL [fdo#106510] / [fdo#108145]

  * igt@kms_color@pipe-a-ctm-max:
    - shard-skl:          NOTRUN -> FAIL [fdo#108147]

  * igt@kms_color@pipe-a-gamma:
    - shard-skl:          NOTRUN -> FAIL [fdo#104782]

  * igt@kms_color@pipe-a-legacy-gamma:
    - shard-apl:          PASS -> FAIL [fdo#104782] / [fdo#108145]

  * igt@kms_color@pipe-c-legacy-gamma:
    - shard-apl:          PASS -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x42-random:
    - shard-skl:          NOTRUN -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-64x21-random:
    - shard-apl:          PASS -> FAIL [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-64x64-suspend:
    - shard-apl:          PASS -> FAIL [fdo#103191] / [fdo#103232]

  * igt@kms_draw_crc@draw-method-rgb565-blt-xtiled:
    - shard-skl:          PASS -> FAIL [fdo#103184]

  * igt@kms_draw_crc@draw-method-xrgb8888-pwrite-untiled:
    - shard-skl:          PASS -> FAIL [fdo#108472]

  * igt@kms_fbcon_fbt@psr-suspend:
    - shard-skl:          NOTRUN -> FAIL [fdo#107882]

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
    - shard-skl:          PASS -> FAIL [fdo#105363]

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-gtt:
    - shard-skl:          NOTRUN -> FAIL [fdo#105682]

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-onoff:
    - shard-iclb:         PASS -> FAIL [fdo#103167] +1

  * igt@kms_frontbuffer_tracking@fbcpsr-farfromfence:
    - shard-skl:          NOTRUN -> FAIL [fdo#103167] +6

  * igt@kms_frontbuffer_tracking@fbcpsr-suspend:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108] / [fdo#106978] / [fdo#107773]

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-shrfb-pgflip-blt:
    - shard-skl:          PASS -> FAIL [fdo#103167] +1

  * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@kms_plane@pixel-format-pipe-b-planes-source-clamping:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#106885] +1

  * igt@kms_plane_alpha_blend@pipe-a-alpha-7efc:
    - shard-skl:          NOTRUN -> FAIL [fdo#107815] / [fdo#108145] +2

  * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
    - shard-skl:          PASS -> FAIL [fdo#107815] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-b-alpha-7efc:
    - shard-kbl:          NOTRUN -> FAIL [fdo#108145] / [fdo#108590]

  * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
    - shard-apl:          PASS -> FAIL [fdo#108145]

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-yf:
    - shard-iclb:         PASS -> FAIL [fdo#103166] +3

  * igt@kms_psr@no_drrs:
    - shard-iclb:         PASS -> FAIL [fdo#108341]

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-apl:          PASS -> DMESG-FAIL [fdo#108950]

  * igt@pm_rpm@gem-mmap-gtt:
    - shard-kbl:          PASS -> DMESG-WARN [fdo#103558] / [fdo#105602] +8

  * igt@syncobj_basic@bad-pad-fd-to-handle:
    - shard-apl:          PASS -> INCOMPLETE [fdo#103927]

  
#### Possible fixes ####

  * igt@gem_exec_whisper@normal:
    - shard-skl:          TIMEOUT [fdo#108592] -> PASS

  * igt@gem_workarounds@suspend-resume-fd:
    - shard-skl:          INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS +1

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
    - shard-iclb:         DMESG-WARN [fdo#107956] -> PASS

  * igt@kms_cursor_crc@cursor-128x128-random:
    - shard-kbl:          DMESG-WARN [fdo#103313] -> PASS

  * igt@kms_cursor_crc@cursor-64x64-random:
    - shard-apl:          FAIL [fdo#103232] -> PASS

  * igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions-varying-size:
    - shard-iclb:         FAIL [fdo#103355] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-iclb:         FAIL [fdo#103167] -> PASS +2

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
    - shard-iclb:         FAIL [fdo#103166] -> PASS +2

  * igt@kms_setmode@basic:
    - shard-apl:          FAIL [fdo#99912] -> PASS
    - shard-kbl:          FAIL [fdo#99912] -> PASS

  * igt@kms_vblank@pipe-c-wait-forked:
    - shard-kbl:          DMESG-WARN [fdo#103313] / [fdo#105345] -> PASS

  * igt@pm_rpm@fences-dpms:
    - shard-iclb:         DMESG-WARN [fdo#108654] -> PASS +1

  * igt@pm_rpm@modeset-stress-extra-wait:
    - shard-skl:          INCOMPLETE [fdo#107807] -> PASS

  
#### Warnings ####

  * igt@kms_cursor_crc@cursor-128x128-suspend:
    - shard-iclb:         INCOMPLETE [fdo#107713] -> FAIL [fdo#103232]

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-kbl:          DMESG-FAIL [fdo#108950] -> DMESG-WARN [fdo#105604]

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#103158]: https://bugs.freedesktop.org/show_bug.cgi?id=103158
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313
  [fdo#103355]: https://bugs.freedesktop.org/show_bug.cgi?id=103355
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
  [fdo#105345]: https://bugs.freedesktop.org/show_bug.cgi?id=105345
  [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#105604]: https://bugs.freedesktop.org/show_bug.cgi?id=105604
  [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
  [fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
  [fdo#106885]: https://bugs.freedesktop.org/show_bug.cgi?id=106885
  [fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978
  [fdo#107469]: https://bugs.freedesktop.org/show_bug.cgi?id=107469
  [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
  [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
  [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
  [fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108039]: https://bugs.freedesktop.org/show_bug.cgi?id=108039
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
  [fdo#108341]: https://bugs.freedesktop.org/show_bug.cgi?id=108341
  [fdo#108472]: https://bugs.freedesktop.org/show_bug.cgi?id=108472
  [fdo#108590]: https://bugs.freedesktop.org/show_bug.cgi?id=108590
  [fdo#108592]: https://bugs.freedesktop.org/show_bug.cgi?id=108592
  [fdo#108654]: https://bugs.freedesktop.org/show_bug.cgi?id=108654
  [fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
  [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
  [fdo#109241]: https://bugs.freedesktop.org/show_bug.cgi?id=109241
  [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912


Participating hosts (6 -> 6)
------------------------------

  No changes in participating hosts


Build changes
-------------

    * Linux: CI_DRM_5380 -> Patchwork_11258

  CI_DRM_5380: f7836843c875734c42d61d755d822635390ab097 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4757: 738f43a54d626f08e250c926a5aeec53458fbd3c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11258: d6b0067e30e569204fbf1adab5df2e552713fefc @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Fix some formatting in drm_dp_add_port()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
Okay!

Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
Okay!

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

# Name Submitter State A F R T
[v6,01/20] drm/dp_mst: Fix some formatting in drm_dp_add_port() Lyude Paul New 2
[v6,02/20] drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg() Lyude Paul New 2
[v6,03/20] drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi() Lyude Paul New 2
[v6,04/20] drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi() Lyude Paul New 2
[v6,05/20] drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends Lyude Paul New 2
[v6,06/20] drm/dp_mst: Introduce new refcounting scheme for mstbs and ports Lyude Paul Accepted 2
[v6,07/20] drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails Lyude Paul Accepted 2
[v6,08/20] drm/dp_mst: Stop releasing VCPI when removing ports from topology Lyude Paul New 2
[v6,09/20] drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs Lyude Paul New 2
[v6,10/20] drm/i915: Keep malloc references to MST ports Lyude Paul New 1
[v6,11/20] drm/amdgpu/display: Keep malloc ref to MST port Lyude Paul New 1
[v6,12/20] drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector() Lyude Paul New
[v6,13/20] drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup() Lyude Paul New
[v6,14/20] drm/nouveau: Keep malloc references to MST ports Lyude Paul New
[v6,15/20] drm/nouveau: Stop unsetting mstc->port, use malloc refs Lyude Paul New
[v6,16/20] drm/nouveau: Grab payload lock in nv50_msto_payload() Lyude Paul New
[v6,17/20] drm/dp_mst: Add some atomic state iterator macros Lyude Paul New 1
[v6,18/20] drm/dp_mst: Start tracking per-port VCPI allocations Lyude Paul New 1
[v6,19/20] drm/dp_mst: Check payload count in drm_dp_mst_atomic_check() Lyude Paul New 1
[v6,20/20] drm/nouveau: Use atomic VCPI helpers for MST Lyude Paul New

Tests

CI Bug Log - changes from CI_DRM_5396 -> Patchwork_11275
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/6/mbox/

Known issues
------------

  Here are the changes found in Patchwork_11275 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_suspend@basic-s3:
    - fi-blb-e6850:       PASS -> INCOMPLETE [fdo#107718]

  * igt@i915_selftest@live_execlists:
    - fi-apl-guc:         PASS -> INCOMPLETE [fdo#103927]

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-7500u:       PASS -> FAIL [fdo#108767]

  * igt@kms_pipe_crc_basic@hang-read-crc-pipe-b:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362]

  
#### Possible fixes ####

  * igt@i915_selftest@live_hangcheck:
    - fi-bwr-2160:        DMESG-FAIL [fdo#108735] -> PASS

  * igt@kms_flip@basic-flip-vs-dpms:
    - fi-apl-guc:         DMESG-WARN -> PASS

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
    - fi-byt-clapper:     FAIL [fdo#103191] / [fdo#107362] -> PASS

  
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
  [fdo#108735]: https://bugs.freedesktop.org/show_bug.cgi?id=108735
  [fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767


Participating hosts (45 -> 42)
------------------------------

  Additional (2): fi-whl-u fi-skl-iommu 
  Missing    (5): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-pnv-d510 


Build changes
-------------

    * Linux: CI_DRM_5396 -> Patchwork_11275

  CI_DRM_5396: 8aa43e01ae8e72537445ce6c00c625ad144eb153 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4758: f01796214bbde31e37b0593e547ad9436fdd02ba @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11275: eda808298487174cc9eac53857058ad855754aad @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

eda808298487 drm/nouveau: Use atomic VCPI helpers for MST
c56bb94c3792 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
f9d81992c9e5 drm/dp_mst: Start tracking per-port VCPI allocations
b82d3c430cb1 drm/dp_mst: Add some atomic state iterator macros
afbb35c6e94b drm/nouveau: Grab payload lock in nv50_msto_payload()
64aa969674e9 drm/nouveau: Stop unsetting mstc->port, use malloc refs
141e44f950c7 drm/nouveau: Keep malloc references to MST ports
4f5695fc9072 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
defda10847fc drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
1ef9dfa3cc33 drm/amdgpu/display: Keep malloc ref to MST port
1171364b70bf drm/i915: Keep malloc references to MST ports
144f62fdfcaa drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
c47ff6b09ccd drm/dp_mst: Stop releasing VCPI when removing ports from topology
5d7e538f768f drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
1f70bc12c76d drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
0e76331779ed drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
ed2e6efe3038 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
a01f387a014f drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
2816687b107d drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
bc4462c0913d drm/dp_mst: Fix some formatting in drm_dp_add_port()
$ dim checkpatch origin/drm-tip
bc4462c0913d drm/dp_mst: Fix some formatting in drm_dp_add_port()
2816687b107d drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
a01f387a014f drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
ed2e6efe3038 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
0e76331779ed drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1379:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 401 lines checked
1f70bc12c76d drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:142: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#142: 
new file mode 100644

-:848: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#848: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1330:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:862: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#862: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1347:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 1 errors, 2 warnings, 2 checks, 916 lines checked
5d7e538f768f drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
c47ff6b09ccd drm/dp_mst: Stop releasing VCPI when removing ports from topology
144f62fdfcaa drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2269:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
1171364b70bf drm/i915: Keep malloc references to MST ports
1ef9dfa3cc33 drm/amdgpu/display: Keep malloc ref to MST port
defda10847fc drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
4f5695fc9072 drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
141e44f950c7 drm/nouveau: Keep malloc references to MST ports
64aa969674e9 drm/nouveau: Stop unsetting mstc->port, use malloc refs
afbb35c6e94b drm/nouveau: Grab payload lock in nv50_msto_payload()
b82d3c430cb1 drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:713:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:730:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:748:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
f9d81992c9e5 drm/dp_mst: Start tracking per-port VCPI allocations
-:45: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#45: 
  * Cleanup atomic check logic for VCPI allocations - all we need to check in

total: 0 errors, 1 warnings, 0 checks, 444 lines checked
c56bb94c3792 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
eda808298487 drm/nouveau: Use atomic VCPI helpers for MST
CI Bug Log - changes from CI_DRM_5396_full -> Patchwork_11275_full
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11275_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11275_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11275_full:

### IGT changes ###

#### Warnings ####

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-cur-indfb-draw-pwrite:
    - shard-hsw:          PASS -> {SKIP}

  
Known issues
------------

  Here are the changes found in Patchwork_11275_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_cpu_reloc@full:
    - shard-skl:          PASS -> TIMEOUT [fdo#108248]

  * igt@gem_exec_schedule@pi-ringfull-bsd:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103158]
    - shard-skl:          NOTRUN -> FAIL [fdo#103158] +1

  * igt@gem_exec_schedule@pi-ringfull-vebox:
    - shard-kbl:          NOTRUN -> FAIL [fdo#103158]

  * igt@gem_exec_whisper@normal:
    - shard-skl:          NOTRUN -> TIMEOUT [fdo#108592]

  * igt@gem_ppgtt@blt-vs-render-ctxn:
    - shard-skl:          PASS -> TIMEOUT [fdo#108039]

  * igt@gem_userptr_blits@readonly-unsync:
    - shard-skl:          PASS -> TIMEOUT [fdo#108887]

  * igt@gem_workarounds@suspend-resume:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108] / [fdo#107773]

  * igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#107724] +3

  * igt@kms_busy@extended-modeset-hang-newfb-render-a:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#107956] +1

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-a:
    - shard-iclb:         NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-pageflip-hang-newfb-render-b:
    - shard-glk:          PASS -> DMESG-WARN [fdo#107956]

  * igt@kms_ccs@pipe-c-crc-sprite-planes-basic:
    - shard-iclb:         NOTRUN -> FAIL [fdo#107725]

  * igt@kms_cursor_crc@cursor-128x128-random:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-64x64-dpms:
    - shard-skl:          PASS -> FAIL [fdo#103232] +1

  * igt@kms_draw_crc@fill-fb:
    - shard-skl:          PASS -> FAIL [fdo#103184]

  * igt@kms_fbcon_fbt@psr:
    - shard-skl:          NOTRUN -> FAIL [fdo#107882]

  * igt@kms_flip@flip-vs-expired-vblank:
    - shard-glk:          PASS -> FAIL [fdo#105363]

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
    - shard-skl:          PASS -> FAIL [fdo#105363]

  * igt@kms_flip_tiling@flip-changes-tiling-yf:
    - shard-skl:          NOTRUN -> FAIL [fdo#108228] / [fdo#108303]

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-cpu:
    - shard-skl:          NOTRUN -> FAIL [fdo#103167]

  * igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-pwrite:
    - shard-skl:          PASS -> FAIL [fdo#103167] / [fdo#105682]

  * igt@kms_frontbuffer_tracking@fbc-rgb565-draw-mmap-wc:
    - shard-skl:          PASS -> FAIL [fdo#105682]

  * igt@kms_frontbuffer_tracking@fbc-stridechange:
    - shard-skl:          NOTRUN -> FAIL [fdo#105683]

  * igt@kms_frontbuffer_tracking@fbcpsr-stridechange:
    - shard-iclb:         PASS -> FAIL [fdo#105683] / [fdo#108040]

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-pwrite:
    - shard-skl:          PASS -> FAIL [fdo#103167] +1

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt:
    - shard-iclb:         PASS -> FAIL [fdo#103167] +2

  * igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence:
    - shard-skl:          PASS -> FAIL [fdo#103191] / [fdo#107362]

  * igt@kms_plane@plane-position-covered-pipe-a-planes:
    - shard-iclb:         NOTRUN -> FAIL [fdo#103166] +1

  * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb:
    - shard-skl:          NOTRUN -> FAIL [fdo#108145] +4

  * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
    - shard-skl:          PASS -> FAIL [fdo#107815] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-c-alpha-basic:
    - shard-skl:          NOTRUN -> FAIL [fdo#107815] / [fdo#108145]

  * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc:
    - shard-skl:          PASS -> FAIL [fdo#107815]

  * igt@kms_plane_multiple@atomic-pipe-b-tiling-yf:
    - shard-iclb:         PASS -> FAIL [fdo#103166] +2

  * igt@kms_plane_multiple@atomic-pipe-c-tiling-y:
    - shard-apl:          PASS -> FAIL [fdo#103166]

  * igt@pm_rpm@cursor-dpms:
    - shard-iclb:         NOTRUN -> DMESG-WARN [fdo#107724] +1

  * igt@pm_rpm@universal-planes:
    - shard-iclb:         PASS -> DMESG-WARN [fdo#108654] / [fdo#108756]

  * igt@pm_rps@reset:
    - shard-iclb:         NOTRUN -> FAIL [fdo#102250] / [fdo#108059]

  
#### Possible fixes ####

  * igt@kms_atomic_transition@plane-all-transition-nonblocking-fencing:
    - shard-iclb:         DMESG-WARN [fdo#107724] / [fdo#109225] -> PASS

  * igt@kms_available_modes_crc@available_mode_test_crc:
    - shard-apl:          FAIL [fdo#106641] -> PASS

  * igt@kms_draw_crc@draw-method-xrgb8888-mmap-wc-xtiled:
    - shard-iclb:         WARN [fdo#108336] -> PASS

  * igt@kms_flip@flip-vs-wf_vblank-interruptible:
    - shard-iclb:         INCOMPLETE -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-onoff:
    - shard-iclb:         FAIL [fdo#103167] -> PASS

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
    - shard-iclb:         DMESG-FAIL [fdo#107724] -> PASS +3

  * igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-mmap-wc:
    - shard-iclb:         DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +6

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-c:
    - shard-iclb:         DMESG-WARN [fdo#107724] -> PASS +4

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
    - shard-apl:          FAIL [fdo#103166] -> PASS +1

  * igt@kms_psr@no_drrs:
    - shard-iclb:         FAIL [fdo#108341] -> PASS

  * igt@kms_rotation_crc@multiplane-rotation:
    - shard-apl:          DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +1

  * igt@kms_setmode@basic:
    - shard-kbl:          FAIL [fdo#99912] -> PASS

  * igt@pm_rpm@basic-rte:
    - shard-iclb:         INCOMPLETE [fdo#108990] -> PASS

  * igt@pm_rpm@legacy-planes:
    - shard-skl:          INCOMPLETE [fdo#107807] -> PASS +1

  * igt@pm_rpm@system-suspend-execbuf:
    - shard-skl:          INCOMPLETE [fdo#104108] / [fdo#107807] -> PASS

  * igt@prime_vgem@basic-fence-flip:
    - shard-apl:          FAIL [fdo#104008] -> PASS

  
#### Warnings ####

  * igt@i915_suspend@shrink:
    - shard-skl:          INCOMPLETE [fdo#106886] -> DMESG-WARN [fdo#107886] / [fdo#109244]

  * igt@kms_cursor_crc@cursor-128x42-sliding:
    - shard-iclb:         DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103232]

  * igt@kms_cursor_crc@cursor-64x21-random:
    - shard-iclb:         FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] / [fdo#108336]

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#102250]: https://bugs.freedesktop.org/show_bug.cgi?id=102250
  [fdo#103158]: https://bugs.freedesktop.org/show_bug.cgi?id=103158
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#104008]: https://bugs.freedesktop.org/show_bug.cgi?id=104008
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
  [fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
  [fdo#106641]: https://bugs.freedesktop.org/show_bug.cgi?id=106641
  [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
  [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
  [fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
  [fdo#107886]: https://bugs.freedesktop.org/show_bug.cgi?id=107886
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108039]: https://bugs.freedesktop.org/show_bug.cgi?id=108039
  [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
  [fdo#108059]: https://bugs.freedesktop.org/show_bug.cgi?id=108059
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108228]: https://bugs.freedesktop.org/show_bug.cgi?id=108228
  [fdo#108248]: https://bugs.freedesktop.org/show_bug.cgi?id=108248
  [fdo#108303]: https://bugs.freedesktop.org/show_bug.cgi?id=108303
  [fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
  [fdo#108341]: https://bugs.freedesktop.org/show_bug.cgi?id=108341
  [fdo#108592]: https://bugs.freedesktop.org/show_bug.cgi?id=108592
  [fdo#108654]: https://bugs.freedesktop.org/show_bug.cgi?id=108654
  [fdo#108756]: https://bugs.freedesktop.org/show_bug.cgi?id=108756
  [fdo#108887]: https://bugs.freedesktop.org/show_bug.cgi?id=108887
  [fdo#108990]: https://bugs.freedesktop.org/show_bug.cgi?id=108990
  [fdo#109225]: https://bugs.freedesktop.org/show_bug.cgi?id=109225
  [fdo#109241]: https://bugs.freedesktop.org/show_bug.cgi?id=109241
  [fdo#109244]: https://bugs.freedesktop.org/show_bug.cgi?id=109244
  [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912


Participating hosts (7 -> 7)
------------------------------

  No changes in participating hosts


Build changes
-------------

    * Linux: CI_DRM_5396 -> Patchwork_11275

  CI_DRM_5396: 8aa43e01ae8e72537445ce6c00c625ad144eb153 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4758: f01796214bbde31e37b0593e547ad9436fdd02ba @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11275: eda808298487174cc9eac53857058ad855754aad @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Fix some formatting in drm_dp_add_port()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
Okay!

Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
Okay!

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!

Patches download mbox

# Name Submitter State A F R T
[v7,01/20] drm/dp_mst: Fix some formatting in drm_dp_add_port() Lyude Paul New 2
[v7,02/20] drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg() Lyude Paul New 2
[v7,03/20] drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi() Lyude Paul New 2
[v7,04/20] drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi() Lyude Paul New 2
[v7,05/20] drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends Lyude Paul New 2
[v7,06/20] drm/dp_mst: Introduce new refcounting scheme for mstbs and ports Lyude Paul New 2
[v7,07/20] drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails Lyude Paul New 2
[v7,08/20] drm/dp_mst: Stop releasing VCPI when removing ports from topology Lyude Paul New 2
[v7,09/20] drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs Lyude Paul New 2
[v7,10/20] drm/i915: Keep malloc references to MST ports Lyude Paul New 1
[v7,11/20] drm/amdgpu/display: Keep malloc ref to MST port Lyude Paul New 1
[v7,12/20] drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector() Lyude Paul New 1
[v7,13/20] drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup() Lyude Paul New 1
[v7,14/20] drm/nouveau: Keep malloc references to MST ports Lyude Paul New 1
[v7,15/20] drm/nouveau: Stop unsetting mstc->port, use malloc refs Lyude Paul New 1
[v7,16/20] drm/nouveau: Grab payload lock in nv50_msto_payload() Lyude Paul New 1
[v7,17/20] drm/dp_mst: Add some atomic state iterator macros Lyude Paul New 1
[v7,18/20] drm/dp_mst: Start tracking per-port VCPI allocations Lyude Paul New 1
[v7,19/20] drm/dp_mst: Check payload count in drm_dp_mst_atomic_check() Lyude Paul New 1
[v7,20/20] drm/nouveau: Use atomic VCPI helpers for MST Lyude Paul New 1

Tests

CI Bug Log - changes from CI_DRM_5400 -> Patchwork_11278
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://patchwork.freedesktop.org/api/1.0/series/54030/revisions/7/mbox/

Known issues
------------

  Here are the changes found in Patchwork_11278 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
    - fi-byt-clapper:     PASS -> FAIL [fdo#103191] / [fdo#107362]

  
#### Possible fixes ####

  * igt@kms_frontbuffer_tracking@basic:
    - fi-byt-clapper:     FAIL [fdo#103167] -> PASS

  
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362


Participating hosts (47 -> 43)
------------------------------

  Missing    (4): fi-kbl-soraka fi-ilk-m540 fi-byt-squawks fi-bsw-cyan 


Build changes
-------------

    * Linux: CI_DRM_5400 -> Patchwork_11278

  CI_DRM_5400: 4982584021f0b046841f196efc25735bd71ebdcf @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4759: 478452fece3997dfacaa4d6babe6b8bf6fef784f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11278: 7058ad33b999acee846993440df70c8759da6c5e @ git://anongit.freedesktop.org/gfx-ci/linux


== Linux commits ==

7058ad33b999 drm/nouveau: Use atomic VCPI helpers for MST
96f08bd75cd3 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
6dc4e60a7f77 drm/dp_mst: Start tracking per-port VCPI allocations
9f0550204390 drm/dp_mst: Add some atomic state iterator macros
9d1a374f3b0a drm/nouveau: Grab payload lock in nv50_msto_payload()
fb2916eaf801 drm/nouveau: Stop unsetting mstc->port, use malloc refs
f6bc6e58cb36 drm/nouveau: Keep malloc references to MST ports
fa65fc6573ef drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
e22b33d521e3 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
1659ca4c5976 drm/amdgpu/display: Keep malloc ref to MST port
1215e951d338 drm/i915: Keep malloc references to MST ports
f44216dab40f drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
5432def72627 drm/dp_mst: Stop releasing VCPI when removing ports from topology
33ed2b8bf69c drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
f3167088bed1 drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
48650b1b91ef drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
3e64f7a83561 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
bab922f7f6c4 drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
5613e665e2e0 drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
4dc81bd3d040 drm/dp_mst: Fix some formatting in drm_dp_add_port()
$ dim checkpatch origin/drm-tip
4dc81bd3d040 drm/dp_mst: Fix some formatting in drm_dp_add_port()
5613e665e2e0 drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
bab922f7f6c4 drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
3e64f7a83561 drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
48650b1b91ef drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
-:84: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#84: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:990:
+			rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:102: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#102: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1006:
+		rmstb = drm_dp_mst_topology_get_mstb_validated_locked(

-:150: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#150: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1379:
+			mstb_child = drm_dp_mst_topology_get_mstb_validated(

total: 0 errors, 0 warnings, 3 checks, 401 lines checked
f3167088bed1 drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
-:27: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#27: 
commit 263efde31f97 ("drm/dp/mst: Get validated port ref in drm_dp_update_payload_part1()")

-:51: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")'
#51: 
commit 9765635b3075 ("Revert "drm/dp_mst: Skip validating ports during destruction, just ref"")

-:142: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#142: 
new file mode 100644

-:848: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#848: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1330:
+			mport = drm_dp_mst_topology_get_port_validated_locked(

-:862: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#862: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:1347:
+		rport = drm_dp_mst_topology_get_port_validated_locked(

total: 1 errors, 2 warnings, 2 checks, 916 lines checked
33ed2b8bf69c drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
5432def72627 drm/dp_mst: Stop releasing VCPI when removing ports from topology
f44216dab40f drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
-:97: CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#97: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:2269:
+				port = drm_dp_mst_topology_get_port_validated(

total: 0 errors, 0 warnings, 1 checks, 124 lines checked
1215e951d338 drm/i915: Keep malloc references to MST ports
1659ca4c5976 drm/amdgpu/display: Keep malloc ref to MST port
e22b33d521e3 drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
-:13: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#13: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 12 lines checked
fa65fc6573ef drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
-:20: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#20: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 23 lines checked
f6bc6e58cb36 drm/nouveau: Keep malloc references to MST ports
-:19: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#19: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 25 lines checked
fb2916eaf801 drm/nouveau: Stop unsetting mstc->port, use malloc refs
-:12: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#12: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 54 lines checked
9d1a374f3b0a drm/nouveau: Grab payload lock in nv50_msto_payload()
-:12: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#12: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 25 lines checked
9f0550204390 drm/dp_mst: Add some atomic state iterator macros
-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:110: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#110: FILE: include/drm/drm_dp_mst_helper.h:711:
+#define for_each_oldnew_mst_mgr_in_state(__state, mgr, old_state, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:112: WARNING:LONG_LINE: line over 100 characters
#112: FILE: include/drm/drm_dp_mst_helper.h:713:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), &(new_state), (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:127: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#127: FILE: include/drm/drm_dp_mst_helper.h:728:
+#define for_each_old_mst_mgr_in_state(__state, mgr, old_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:129: WARNING:LONG_LINE: line over 100 characters
#129: FILE: include/drm/drm_dp_mst_helper.h:730:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), &(old_state), NULL, (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__state' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:145: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__i' - possible side-effects?
#145: FILE: include/drm/drm_dp_mst_helper.h:746:
+#define for_each_new_mst_mgr_in_state(__state, mgr, new_state, __i) \
+	for ((__i) = 0; (__i) < (__state)->num_private_objs; (__i)++) \
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

-:147: WARNING:LONG_LINE: line over 100 characters
#147: FILE: include/drm/drm_dp_mst_helper.h:748:
+		for_each_if(__drm_dp_mst_state_iter_get((__state), &(mgr), NULL, &(new_state), (__i)))

total: 0 errors, 3 warnings, 6 checks, 120 lines checked
6dc4e60a7f77 drm/dp_mst: Start tracking per-port VCPI allocations
-:45: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#45: 
  * Cleanup atomic check logic for VCPI allocations - all we need to check in

total: 0 errors, 1 warnings, 0 checks, 444 lines checked
96f08bd75cd3 drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
7058ad33b999 drm/nouveau: Use atomic VCPI helpers for MST
-:25: WARNING:BAD_SIGN_OFF: 'Reviewed-by:' is the preferred signature form
#25: 
Reviewed-By: Ben Skeggs <bskeggs@redhat.com>

total: 0 errors, 1 warnings, 0 checks, 82 lines checked
CI Bug Log - changes from CI_DRM_5400_full -> Patchwork_11278_full
====================================================

Summary
-------

  **WARNING**

  Minor unknown changes coming with Patchwork_11278_full need to be verified
  manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_11278_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in Patchwork_11278_full:

### IGT changes ###

#### Warnings ####

  * igt@tools_test@tools_test:
    - shard-glk:          {SKIP} -> PASS

  
Known issues
------------

  Here are the changes found in Patchwork_11278_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ppgtt@blt-vs-render-ctxn:
    - shard-skl:          NOTRUN -> TIMEOUT [fdo#108039]

  * igt@kms_atomic@plane_primary_legacy:
    - shard-apl:          PASS -> DMESG-WARN [fdo#103558] / [fdo#105602]

  * igt@kms_busy@extended-modeset-hang-newfb-render-b:
    - shard-glk:          NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
    - shard-snb:          NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
    - shard-kbl:          NOTRUN -> DMESG-WARN [fdo#107956]

  * igt@kms_color@pipe-c-gamma:
    - shard-skl:          PASS -> FAIL [fdo#104782] / [fdo#108228]

  * igt@kms_color@pipe-c-legacy-gamma:
    - shard-apl:          PASS -> FAIL [fdo#104782]

  * igt@kms_cursor_crc@cursor-128x128-sliding:
    - shard-apl:          PASS -> FAIL [fdo#103232] +1

  * igt@kms_cursor_crc@cursor-256x256-suspend:
    - shard-skl:          PASS -> INCOMPLETE [fdo#104108]

  * igt@kms_cursor_crc@cursor-64x64-suspend:
    - shard-apl:          PASS -> FAIL [fdo#103191] / [fdo#103232]

  * igt@kms_draw_crc@draw-method-xrgb2101010-blt-ytiled:
    - shard-skl:          PASS -> FAIL [fdo#103184]

  * igt@kms_flip_tiling@flip-changes-tiling-yf:
    - shard-skl:          PASS -> FAIL [fdo#108228] / [fdo#108303]

  * igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-render:
    - shard-skl:          PASS -> FAIL [fdo#103167] +2

  * igt@kms_plane@pixel-format-pipe-c-planes-source-clamping:
    - shard-skl:          NOTRUN -> DMESG-WARN [fdo#106885] +1

  * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max:
    - shard-glk:          NOTRUN -> FAIL [fdo#108145]

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
    - shard-apl:          PASS -> FAIL [fdo#103166] +1

  * igt@kms_rotation_crc@multiplane-rotation-cropping-top:
    - shard-kbl:          PASS -> DMESG-FAIL [fdo#108950]

  * igt@kms_rotation_crc@primary-rotation-90:
    - shard-kbl:          PASS -> DMESG-WARN [fdo#103313] / [fdo#105345]

  * igt@kms_setmode@basic:
    - shard-kbl:          PASS -> FAIL [fdo#99912]

  * igt@pm_rpm@drm-resources-equal:
    - shard-skl:          PASS -> INCOMPLETE [fdo#107807]

  
#### Possible fixes ####

  * igt@gem_ctx_isolation@vecs0-s3:
    - shard-skl:          INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS

  * igt@gem_ppgtt@blt-vs-render-ctx0:
    - shard-skl:          TIMEOUT [fdo#108039] -> PASS

  * igt@gem_workarounds@suspend-resume-fd:
    - shard-glk:          INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS

  * igt@kms_busy@extended-pageflip-modeset-hang-oldfb-render-a:
    - shard-kbl:          DMESG-WARN [fdo#107956] -> PASS

  * igt@kms_chv_cursor_fail@pipe-b-64x64-right-edge:
    - shard-skl:          FAIL [fdo#104671] -> PASS

  * igt@kms_color@pipe-c-ctm-max:
    - shard-apl:          FAIL [fdo#108147] -> PASS

  * igt@kms_color@pipe-c-degamma:
    - shard-apl:          FAIL [fdo#104782] -> PASS

  * igt@kms_cursor_crc@cursor-256x85-random:
    - shard-apl:          FAIL [fdo#103232] -> PASS +1

  * igt@kms_draw_crc@draw-method-xrgb8888-mmap-cpu-ytiled:
    - shard-skl:          FAIL [fdo#107791] -> PASS

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-blt:
    - shard-skl:          FAIL [fdo#103167] -> PASS +2

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
    - shard-apl:          FAIL [fdo#103167] -> PASS +2

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-render:
    - shard-glk:          FAIL [fdo#103167] -> PASS +1

  * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
    - shard-skl:          FAIL [fdo#107815] / [fdo#108145] -> PASS

  * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
    - shard-skl:          FAIL [fdo#107815] -> PASS

  * igt@kms_plane_multiple@atomic-pipe-a-tiling-y:
    - shard-apl:          FAIL [fdo#103166] -> PASS +1

  
#### Warnings ####

  * igt@kms_cursor_crc@cursor-128x128-suspend:
    - shard-skl:          FAIL [fdo#103191] / [fdo#103232] -> INCOMPLETE [fdo#104108]

  
  [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
  [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
  [fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
  [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
  [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
  [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313
  [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
  [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558
  [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
  [fdo#104671]: https://bugs.freedesktop.org/show_bug.cgi?id=104671
  [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
  [fdo#105345]: https://bugs.freedesktop.org/show_bug.cgi?id=105345
  [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602
  [fdo#106885]: https://bugs.freedesktop.org/show_bug.cgi?id=106885
  [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
  [fdo#107791]: https://bugs.freedesktop.org/show_bug.cgi?id=107791
  [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
  [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
  [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
  [fdo#108039]: https://bugs.freedesktop.org/show_bug.cgi?id=108039
  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#108147]: https://bugs.freedesktop.org/show_bug.cgi?id=108147
  [fdo#108228]: https://bugs.freedesktop.org/show_bug.cgi?id=108228
  [fdo#108303]: https://bugs.freedesktop.org/show_bug.cgi?id=108303
  [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
  [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
  [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133


Participating hosts (7 -> 6)
------------------------------

  Missing    (1): shard-iclb 


Build changes
-------------

    * Linux: CI_DRM_5400 -> Patchwork_11278

  CI_DRM_5400: 4982584021f0b046841f196efc25735bd71ebdcf @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_4759: 478452fece3997dfacaa4d6babe6b8bf6fef784f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
  Patchwork_11278: 7058ad33b999acee846993440df70c8759da6c5e @ git://anongit.freedesktop.org/gfx-ci/linux
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/dp_mst: Fix some formatting in drm_dp_add_port()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_payload_send_msg()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_allocate_vcpi()
Okay!

Commit: drm/dp_mst: Fix some formatting in drm_dp_mst_deallocate_vcpi()
Okay!

Commit: drm/dp_mst: Rename drm_dp_mst_get_validated_(port|mstb)_ref and friends
Okay!

Commit: drm/dp_mst: Introduce new refcounting scheme for mstbs and ports
Okay!

Commit: drm/dp_mst: Restart last_connected_port_and_mstb() if topology ref fails
Okay!

Commit: drm/dp_mst: Stop releasing VCPI when removing ports from topology
Okay!

Commit: drm/dp_mst: Fix payload deallocation on hotplugs using malloc refs
Okay!

Commit: drm/i915: Keep malloc references to MST ports
Okay!

Commit: drm/amdgpu/display: Keep malloc ref to MST port
Okay!

Commit: drm/nouveau: Remove bogus cleanup in nv50_mstm_add_connector()
Okay!

Commit: drm/nouveau: Remove unnecessary VCPI checks in nv50_msto_cleanup()
Okay!

Commit: drm/nouveau: Keep malloc references to MST ports
Okay!

Commit: drm/nouveau: Stop unsetting mstc->port, use malloc refs
Okay!

Commit: drm/nouveau: Grab payload lock in nv50_msto_payload()
Okay!

Commit: drm/dp_mst: Add some atomic state iterator macros
Okay!

Commit: drm/dp_mst: Start tracking per-port VCPI allocations
Okay!

Commit: drm/dp_mst: Check payload count in drm_dp_mst_atomic_check()
Okay!

Commit: drm/nouveau: Use atomic VCPI helpers for MST
Okay!