Subject: Re: [PATCH 04/13] tracing/brcm: Use the new __vstring() helper

From: Kalle Valo <[email-masked]>
Date: Fri, 08 Jul 2022 08:02:31 +0300

Steven Rostedt <[email-masked]> writes:

> From: "Steven Rostedt (Google)" <[email-masked]>
>
> Instead of open coding a __dynamic_array() with a fixed length (which
> defeats the purpose of the dynamic array in the first place). Use the new
> __vstring() helper that will use a va_list and only write enough of the
> string into the ring buffer that is needed.
>
> Cc: Arend van Spriel <[email-masked]>
> Cc: Franky Lin <[email-masked]>
> Cc: Hante Meuleman <[email-masked]>
> Cc: Kalle Valo <[email-masked]>
> Cc: "David S. Miller" <[email-masked]>
> Cc: Eric Dumazet <[email-masked]>
> Cc: Jakub Kicinski <[email-masked]>
> Cc: Paolo Abeni <[email-masked]>
> Cc: [email-masked]
> Cc: [email-masked]
> Cc: [email-masked]
> Cc: [email-masked]
> Signed-off-by: Steven Rostedt (Google) <[email-masked]>
> ---
>  .../broadcom/brcm80211/brcmfmac/tracepoint.h         | 12 ++++--------
>  .../brcm80211/brcmsmac/brcms_trace_brcmsmac_msg.h    | 12 ++++--------
>  2 files changed, 8 insertions(+), 16 deletions(-)

Feel free to take this via your tree:

Acked-by: Kalle Valo <[email-masked]>

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches