Messages in this thread | | | From | Kalle Valo <> | Subject | Re: [PATCH 04/13] tracing/brcm: Use the new __vstring() helper | Date | Fri, 08 Jul 2022 08:02:31 +0300 |
| |
Steven Rostedt <rostedt@goodmis.org> writes:
> From: "Steven Rostedt (Google)" <rostedt@goodmis.org> > > 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 <aspriel@gmail.com> > Cc: Franky Lin <franky.lin@broadcom.com> > Cc: Hante Meuleman <hante.meuleman@broadcom.com> > Cc: Kalle Valo <kvalo@kernel.org> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Eric Dumazet <edumazet@google.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Paolo Abeni <pabeni@redhat.com> > Cc: linux-wireless@vger.kernel.org > Cc: brcm80211-dev-list.pdl@broadcom.com > Cc: SHA-cyfmac-dev-list@infineon.com > Cc: netdev@vger.kernel.org > Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org> > --- > .../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 <kvalo@kernel.org>
-- https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
| |