From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/bridge: drm_bridge_attach: lock the encoder chain mutex during insertion Date: Wed, 25 Mar 2026 06:59:30 +1000 Message-ID: In-Reply-To: <20260324-drm-bridge-alloc-encoder-chain-mutex-v5-3-8bf786c5c7e6@bootlin.com> References: <20260324-drm-bridge-alloc-encoder-chain-mutex-v5-0-8bf786c5c7e6@bootlin.com> <20260324-drm-bridge-alloc-encoder-chain-mutex-v5-3-8bf786c5c7e6@bootlin.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review Correct. The mutex is taken only around the `list_add` operation and the er= ror-path `list_del`, not around the `bridge->funcs->attach()` callback. Thi= s is the right granularity =E2=80=94 it avoids holding the mutex during pot= entially slow or lock-taking driver callbacks. No issues. --- Generated by Claude Code Patch Reviewer