Hi, On Fri, Sep 12, 2025 at 07:03:40PM +0200, Luca Ceresoli wrote: > This series shows removed bridges to the global <debugfs>/dri/bridges file. > Removed bridges are bridges after drm_bridges_remove() but before they are > eventually freed on the last drm_bridge_put(). > > This is part of the work towards removal of bridges from a still existing > DRM pipeline without use-after-free. The grand plan was discussed in [1]. > Here's the work breakdown (➜ marks the current series): > > This is part of the work towards removal of bridges from a still existing > DRM pipeline without use-after-free. The grand plan was discussed in [1]. > Here's the work breakdown (➜ marks the current series): > > 1. ➜ add refcounting to DRM bridges (struct drm_bridge) > (based on devm_drm_bridge_alloc() [0]) > A. ✔ add new alloc API and refcounting (v6.16) > B. ✔ convert all bridge drivers to new API (v6.17-rc1) > C. ✔ kunit tests (v6.17-rc1) > D. ✔ add get/put to drm_bridge_add/remove() + attach/detach() > and warn on old allocation pattern (v6.17-rc1) > E. … add get/put on drm_bridge accessors > 1. ✔ drm_bridge_chain_get_first_bridge() + add a cleanup action > (drm-misc-next) > 2. ✔ drm_bridge_get_prev_bridge() (drm-misc-next) > 3. …✔ drm_bridge_get_next_bridge() (partially in drm-misc-next) > 4. …✔ drm_for_each_bridge_in_chain() (partially in drm-misc-next) > 5. drm_bridge_connector_init > 6. of_drm_find_bridge > 7. drm_of_find_panel_or_bridge, *_of_get_bridge > F. ➜ debugfs improvements > 1. ✔ add top-level 'bridges' file (v6.16) > 2. ➜ show refcount and list removed bridges > 2. … handle gracefully atomic updates during bridge removal > 3. … DSI host-device driver interaction > 4. finish the hotplug bridge work, removing the "always-disconnected" > connector, moving code to the core and potentially removing the > hotplug-bridge itself (this needs to be clarified as points 1-3 are > developed) > > To show the removed bridges we need to keep track of them, thus add a new > global list to store them between drm_bridge_remove() and the eventual > free. This is bit tricky in case a bridge is removed and then re-added > before being freed. This is handled in patch 2.
Once the minor issue I've reported is fixed, Reviewed-by: Maxime Ripard <mrip...@kernel.org> Maxime
signature.asc
Description: PGP signature
