From: Claude Code Review Bot <claude-review@example.com>
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 [thread overview]
Message-ID: <review-patch4-20260526133123.691465-5-w15303746062@163.com> (raw)
In-Reply-To: <20260526133123.691465-5-w15303746062@163.com>
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 refresh rate rather than ASAP, so with low refresh rates the 100ms timeout could be hit before a vblank arrives. 1000ms matches the timeout already used in `drm_wait_one_vblank()` after commit `f050da08a4ed`.
**Patch ordering note:** This patch is numbered 4/5 (fixing a consequence of patch 1/3) and patch 5/5 is also a fix for patch 1. Both are correctly placed 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 Subject lines and is correct — the mbox ordering follows arrival time, not semantic order. The series applies in Subject-line order.
**No issues.**
---
Generated by Claude Code Patch Reviewer
next prev parent reply other threads:[~2026-05-27 4:49 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-26 13:31 [PATCH v2 6.18.y 0/5] drm/vkms: Backport generic vblank timer to fix ABBA deadlock w15303746062
2026-05-26 13:31 ` [PATCH v2 6.18.y 1/5] drm/vblank: Add vblank timer w15303746062
2026-05-27 4:49 ` Claude review: " Claude Code Review Bot
2026-05-26 13:31 ` [PATCH v2 6.18.y 2/5] drm/vblank: Add CRTC helpers for simple use cases w15303746062
2026-05-27 4:49 ` Claude review: " Claude Code Review Bot
2026-05-26 13:31 ` [PATCH v2 6.18.y 3/5] drm/vkms: Convert to DRM's vblank timer w15303746062
2026-05-27 4:49 ` Claude review: " Claude Code Review Bot
2026-05-26 13:31 ` [PATCH v2 6.18.y 4/5] drm/atomic: Increase timeout in drm_atomic_helper_wait_for_vblanks() w15303746062
2026-05-27 4:49 ` Claude Code Review Bot [this message]
2026-05-26 13:31 ` [PATCH v2 6.18.y 5/5] drm/vblank: Fix kernel docs for vblank timer w15303746062
2026-05-27 4:49 ` Claude review: " Claude Code Review Bot
2026-05-27 4:49 ` Claude review: drm/vkms: Backport generic vblank timer to fix ABBA deadlock Claude Code Review Bot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=review-patch4-20260526133123.691465-5-w15303746062@163.com \
--to=claude-review@example.com \
--cc=dri-devel-reviews@example.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox