On Fri, 23 Feb 2024 16:13:56 -0500 Steven Rostedt <[email protected]> wrote:
> From: "Steven Rostedt (Google)" <[email protected]> > > In preparation to remove the second parameter of __assign_str(), make sure > it is really a duplicate of __string() by adding a WARN_ON_ONCE(). > Looks good to me. So eventually this is removed when the second parameter is removed? Reviewed-by: Masami Hiramatsu (Google) <[email protected]> Thank you, > Signed-off-by: Steven Rostedt (Google) <[email protected]> > --- > Changes since v1: > https://lore.kernel.org/linux-trace-kernel/[email protected]/ > > - If src is NULL it is saved as NULL, no need to not test that too. > > include/trace/stages/stage6_event_callback.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/trace/stages/stage6_event_callback.h > b/include/trace/stages/stage6_event_callback.h > index 0c0f50bcdc56..935608971899 100644 > --- a/include/trace/stages/stage6_event_callback.h > +++ b/include/trace/stages/stage6_event_callback.h > @@ -35,6 +35,7 @@ > do { \ > char *__str__ = __get_str(dst); \ > int __len__ = __get_dynamic_array_len(dst) - 1; \ > + WARN_ON_ONCE((src) != __data_offsets.dst##_ptr_); \ > memcpy(__str__, __data_offsets.dst##_ptr_ ? : \ > EVENT_NULL_STR, __len__); \ > __str__[__len__] = '\0'; \ > -- > 2.43.0 > -- Masami Hiramatsu (Google) <[email protected]>
