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/atomic: Increase timeout in drm_atomic_helper_wait_for_vblanks() Date: Wed, 27 May 2026 14:49:45 +1000 Message-ID: In-Reply-To: <20260526133123.691465-5-w15303746062@163.com> References: <20260526133123.691465-1-w15303746062@163.com> <20260526133123.691465-5-w15303746062@163.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 Cherry-pick of upstream `79ae8510b5b81b9500370f89c619b50ca9c0990f`. A one-line change: ```c - msecs_to_jiffies(100) + msecs_to_jiffies(1000) ``` This is a follow-on fix to patch 1. The vblank timer fires at the display r= efresh rate rather than ASAP, so with low refresh rates the 100ms timeout c= ould be hit before a vblank arrives. 1000ms matches the timeout already use= d in `drm_wait_one_vblank()` after commit `f050da08a4ed`. **Patch ordering note:** This patch is numbered 4/5 (fixing a consequence o= f patch 1/3) and patch 5/5 is also a fix for patch 1. Both are correctly pl= aced after the patches they fix. The ordering within the mbox has patches 3= and 4 swapped by Message-Id sequence (patch 3/5 is `691465-4` while patch = 4/5 is `691465-5`), but this matches the git-send-email numbering in the Su= bject lines and is correct =E2=80=94 the mbox ordering follows arrival time= , not semantic order. The series applies in Subject-line order. **No issues.** --- Generated by Claude Code Patch Reviewer