• Laurent Pinchart's avatar
    media: v4l: vsp1: Fix deadlock in VSPDL DRM pipelines · 8eb0e642
    Laurent Pinchart authored
    The VSP uses a lock to protect the BRU and BRS assignment when
    configuring pipelines. The lock is taken in vsp1_du_atomic_begin() and
    released in vsp1_du_atomic_flush(), as well as taken and released in
    vsp1_du_setup_lif(). This guards against multiple pipelines trying to
    assign the same BRU and BRS at the same time.
    
    The DRM framework calls the .atomic_begin() operations in a loop over
    all CRTCs included in an atomic commit. On a VSPDL (the only VSP type
    where this matters), a single VSP instance handles two CRTCs, with a
    single lock. This results in a deadlock when the .atomic_begin()
    operation is called on the second CRTC.
    
    The DRM framework serializes atomic commits that affect the same CRTCs,
    but doesn't know about two CRTCs sharing the same VSPDL. Two commits
    affecting the VSPDL LIF0 and LIF1 respectively can thus race each other,
    hence the need for a lock.
    
    This could be fixed on the DRM side by forcing serialization of commits
    affecting CRTCs backed by the same VSPDL, but that would negatively
    affect performances, as the locking is only needed when the BRU and BRS
    need to be reassigned, which is an uncommon case.
    
    The lock protects the whole .atomic_begin() to .atomic_flush() sequence.
    The only operation that can occur in-between is vsp1_du_atomic_update(),
    which doesn't touch the BRU and BRS, and thus doesn't need to be
    protected by the lock. We can thus only take the lock around the
    pipeline setup calls in vsp1_du_atomic_flush(), which fixes the
    deadlock.
    
    Fixes: f81f9adc ("media: v4l: vsp1: Assign BRU and BRS to pipelines dynamically")
    Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
    Reviewed-by: default avatarKieran Bingham <kieran.bingham+renesas@ideasonboard.com>
    Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
    8eb0e642
vsp1_drm.c 25.8 KB