Re: [PATCH v6] drm/drv: Convert wedged event string building to seq_buf
Raag Jadav <[email protected]>
| Newsgroups | org.freedesktop.lists.dri-devel |
|---|---|
| Message-ID | <[email protected]> |
On Sun, Aug 23, 2026 at 06:03:50AM +0530, Mallesh Koujalagi wrote: > event_string[] has a fixed size of WEDGE_STR_LEN (32) bytes. > The original scnprintf()-based loop required a manual pre-flight > bounds check. > > Replace the manual bookkeeping with seq_buf, which tracks overflow > internally. seq_buf_printf() writes each "method," token into the > buffer. > > On overflow, a warning is issued, len retains the position of the > last successful write, so the trailing comma is stripped cleanly > without including any partial method name in the uevent payload. > > Assisted-by: Claude:claude-Sonnet-4.6 > Signed-off-by: Mallesh Koujalagi <[email protected]> Reviewed-by: Raag Jadav <[email protected]>