From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by smtp.subspace.kernel.org (Postfix) with ESMTPS id C723A3DB325
	for <linux-trace-kernel@vger.kernel.org>; Tue, 13 Oct 2026 08:15:28 +0000 (UTC)
Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124
ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;
	t=1791879331; cv=none; b=bWv7OMVsoxOsXrKBwzLvPdVqACsxXHaJ3TFu00QjSmZj4jBHoYihcEV/p0DogFVzbSfTeRnYOX4Ziko1nGNq7xc/ereq5M1SF6snG4m2b1y1HIz7Y4sbRqfco15+fgk/a6jAUjWxcVWyz9sq6y6YDAQ2bIpXDK/5uKtbFmhWHTc=
ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org;
	s=arc-20240116; t=1791879331; c=relaxed/simple;
	bh=0BbsB0ebjAEQFhp8u+tuohGvtTOC5fycH0bwk7JJrwM=;
	h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References:
	 MIME-Version:Content-Type; b=amedkNJB9wAvjW69IG755ij2tkwag0/+C8F0QmnkLxXWKYdW5VNxFRwDyeUe0WEmO/1X1nouws/eRgoom8J6Pabu7lMmmpZtSA1u1emk8Z5+dWKgCRslXzu12rl9+EiB4QLgog/eurCaM9jWhZn9Uq/HnoYSRnW7p2F05rNqsUg=
ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=iDbIKzLS; arc=none smtp.client-ip=170.10.133.124
Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com
Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com
Authentication-Results: smtp.subspace.kernel.org;
	dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="iDbIKzLS"
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;
	s=mimecast20190719; t=1791879327;
	h=from:from:reply-to:subject:subject:date:date:message-id:message-id:
	 to:to:cc:cc:mime-version:mime-version:content-type:content-type:
	 content-transfer-encoding:content-transfer-encoding:
	 in-reply-to:in-reply-to:references:references:autocrypt:autocrypt;
	bh=0BbsB0ebjAEQFhp8u+tuohGvtTOC5fycH0bwk7JJrwM=;
	b=iDbIKzLSxdDKSeH/mKCWtiho1gP5uJ8EZm6CbH5m/fj/f58M6FFrphQ68pCG48n8vvnMPK
	BwcogZCMdcHzGEONf/bW4Ibp+kz8TTEVGsKqJKupmhTwzIZvvTut528nLF5qTIffSCndtP
	2AACzYCabhRfRVkXgx23befjhXFT5eo=
Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com
 [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS
 (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id
 us-mta-296-kZowtSV1OeSPFujzsRo9Wg-1; Tue, 13 Oct 2026 08:15:26 +0000
X-MC-Unique: kZowtSV1OeSPFujzsRo9Wg-1
X-Mimecast-MFC-AGG-ID: kZowtSV1OeSPFujzsRo9Wg_1791879325
Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-48c4445512dso2167440f8f.2
        for <linux-trace-kernel@vger.kernel.org>; Tue, 13 Oct 2026 01:15:26 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20260707; t=1791879325; x=1792484125;
        h=mime-version:user-agent:content-transfer-encoding:content-type
         :autocrypt:references:in-reply-to:date:cc:to:from:subject:message-id
         :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id
         :reply-to:content-type;
        bh=ImKAcRelEC90byXajVp4krfoSiChsTHlVCrCTr80AKM=;
        b=xQ/XwVvpsfCcdqWxRfYx7yt0W6lIQ9BNmXTM1aHmlbDXxbVDyBCd1JSaS7iPwxFiIF
         9NfhcTBuixISj+GU55G7DkrZyUcArhMRFG+IpmaLbepZDOj0bWdL35/2isj3qqpV8GkO
         RPQ7XCI+Fp387VTdp4nYhviOy8lJ9ObpnNNDuGoUH8rP2gj1P7AFu5yfe6+EXvqfCEVX
         fucxkX3kEU72AWa6nC9yAdw9MNNLud8pc4jRIaDXjOOWQV9Vp7RbVh1qBHKEFNZde80m
         e0Kc+44NmC2g/eZ7DcY4eQ1TP90D9Q37RZMSKXrtobqeVFjVzEYsIRHBdO+ns/32D2q/
         3YZw==
X-Forwarded-Encrypted: i=1; AKwUvBx4OH3Ml916bmK8QqJ0K3SDJtURZQ7OaW4e281w5qf6Mwblrdwk5ueh5uRgpmp0PI3J2bDrEoaXcowyV78BMfIh494=@vger.kernel.org
X-Gm-Message-State: AFq9FYK3b9+MPvjOqGOMYfKcMfzG4lJ+hnvYdMrBZeWSKZl33SyYnjVY
	+OT7YRdaFO7Ki3bZVz0eE2GK9qgplwUSs1d3CyEjAIs6+iO8KQSAhl6iF42KTP4gvrs3Gcl4ADx
	eMfouF1agt7bivl0xgB1Xj3f4rDNy0XRfb+1n0LPG0NK+EAFObza2PkzQt0E0GRw0DixWCpBb8g
	==
X-Gm-Gg: AYBFou3VrdFEC1es4GTlIk2vu6R7l5xdzRN7KP7N1uqyOaul+XSRTz14CYfy0jHPtZr
	9trHC5YvpEnkX1qqYinsnli9xmtTWhJSe5NFzBFVBG7C1WXAaNJycyGfzzuE3n4sVShocEuvM87
	A6w0833xG0jSGtilXhoCEajK2CxULJsPmXm3N36arMcL8skn9bQDEcwaWGJ/OtRLTWiBWjrQNnl
	UG+PBYtC+BLNQjsgoKHcgHZWNSIFMrOoz1MixoCkk3+rVbol90hA62K9+9fNSyqT4ePqeTdUB1D
	L1DqGhIE4vKuxrDO96Q+p0WKl09YSkWhs0dQkqJZedu8OTzdjvz81svHwx4KwI1jY7UxvKaaTCR
	i99xDjdzJHcUD4OIYOw9bvGubL6Yx
X-Received: by 2002:adf:efc3:0:b0:48b:6f3:667f with SMTP id ffacd0b85a97d-48dbacef900mr17118170f8f.56.1791879324796;
        Tue, 13 Oct 2026 01:15:24 -0700 (PDT)
X-Received: by 2002:adf:efc3:0:b0:48b:6f3:667f with SMTP id ffacd0b85a97d-48dbacef900mr17118114f8f.56.1791879324220;
        Tue, 13 Oct 2026 01:15:24 -0700 (PDT)
Received: from gmonaco-thinkpadt14gen3.rmtit.csb ([195.174.135.130])
        by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-48de0ab5262sm3808098f8f.22.2026.10.13.01.15.22
        (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
        Tue, 13 Oct 2026 01:15:23 -0700 (PDT)
Message-ID: <b5d9041971e2d033df78ada10977fd1e566f1bef.camel@redhat.com>
Subject: Re: [PATCH v2 04/15] rv: Cast result of model_get_*_name()
From: Gabriele Monaco <gmonaco@redhat.com>
To: Nam Cao <namcao@linutronix.de>, linux-kernel@vger.kernel.org, 
	linux-trace-kernel@vger.kernel.org, bpf@vger.kernel.org, Steven Rostedt
	 <rostedt@goodmis.org>
Cc: Alexei Starovoitov <alexei.starovoitov@gmail.com>, Wen Yang
	 <wen.yang@linux.dev>, Tobias Schaffner <tobias.schaffner@siemens.com>, 
 Viktor Malik <vmalik@redhat.com>
Date: Tue, 13 Oct 2026 10:15:22 +0200
In-Reply-To: <87jynmgm94.fsf@yellow.woof>
References: <20261001152042.124445-1-gmonaco@redhat.com>
	 <20261001152042.124445-5-gmonaco@redhat.com> <87jynmgm94.fsf@yellow.woof>
Autocrypt: addr=gmonaco@redhat.com; prefer-encrypt=mutual;
 keydata=mDMEZuK5YxYJKwYBBAHaRw8BAQdAmJ3dM9Sz6/Hodu33Qrf8QH2bNeNbOikqYtxWFLVm0
 1a0JEdhYnJpZWxlIE1vbmFjbyA8Z21vbmFjb0BrZXJuZWwub3JnPoiZBBMWCgBBFiEEysoR+AuB3R
 Zwp6j270psSVh4TfIFAmjKX2MCGwMFCQWjmoAFCwkIBwICIgIGFQoJCAsCBBYCAwECHgcCF4AACgk
 Q70psSVh4TfIQuAD+JulczTN6l7oJjyroySU55Fbjdvo52xiYYlMjPG7dCTsBAMFI7dSL5zg98I+8
 cXY1J7kyNsY6/dcipqBM4RMaxXsOtCRHYWJyaWVsZSBNb25hY28gPGdtb25hY29AcmVkaGF0LmNvb
 T6InAQTFgoARAIbAwUJBaOagAULCQgHAgIiAgYVCgkICwIEFgIDAQIeBwIXgBYhBMrKEfgLgd0WcK
 eo9u9KbElYeE3yBQJoymCyAhkBAAoJEO9KbElYeE3yjX4BAJ/ETNnlHn8OjZPT77xGmal9kbT1bC1
 7DfrYVISWV2Y1AP9HdAMhWNAvtCtN2S1beYjNybuK6IzWYcFfeOV+OBWRDQ==
User-Agent: Evolution 3.60.2 (3.60.2-2.fc44)
Precedence: bulk
X-Mailing-List: linux-trace-kernel@vger.kernel.org
List-Id: <linux-trace-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-trace-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-trace-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-Mimecast-Spam-Score: 0
X-Mimecast-MFC-PROC-ID: C68-j2aJsoUuY-ov8oFCgICOg-YUhIyGr0lfaKQAoLM_1791879325
X-Mimecast-Originator: redhat.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, 2026-10-13 at 08:14 +0200, Nam Cao wrote:
> Gabriele Monaco <gmonaco@redhat.com> writes:
>=20
> > Functions like model_get_event_name() and model_get_state_name() are
> > shared with BPF monitors, however those programs don't play nice with
> > string pointers without fixed length and the event_name and state_name
> > arrays need to be defined differently. This gets the compiler to notice
> > those are const char and the above function discard the const qualifier=
.
> >=20
> > Drop the warning by casting the result to (char *). Note, a cleaner
> > solution would require to modify all callers, but that would require
> > changing every single monitor's tracepoint and the advantage is minimal=
.
>=20
> I prefer the cleaner solution. Just mass replace "char *" with
> "const char *", and manually fix up a few errors and we are done.

Right, no excuse for being lazy in the era of LLMs..

Thanks for the patch,
Gabriele

>=20
> diff --git a/include/rv/automata.h b/include/rv/automata.h
> index 4a4eb40cf09a..e4afd5ba1d06 100644
> --- a/include/rv/automata.h
> +++ b/include/rv/automata.h
> @@ -22,7 +22,7 @@
> =C2=A0/*
> =C2=A0 * model_get_state_name - return the (string) name of the given sta=
te
> =C2=A0 */
> -static char *model_get_state_name(enum states state)
> +static const char *model_get_state_name(enum states state)
> =C2=A0{
> =C2=A0=09if ((state < 0) || (state >=3D STATE_MAX))
> =C2=A0=09=09return "INVALID";
> @@ -33,7 +33,7 @@ static char *model_get_state_name(enum states state)
> =C2=A0/*
> =C2=A0 * model_get_event_name - return the (string) name of the given eve=
nt
> =C2=A0 */
> -static char *model_get_event_name(enum events event)
> +static const char *model_get_event_name(enum events event)
> =C2=A0{
> =C2=A0=09if ((event < 0) || (event >=3D EVENT_MAX))
> =C2=A0=09=09return "INVALID";
> diff --git a/include/rv/da_monitor.h b/include/rv/da_monitor.h
> index 6fc5ef8142ac..fc830c9f5046 100644
> --- a/include/rv/da_monitor.h
> +++ b/include/rv/da_monitor.h
> @@ -633,8 +633,8 @@ static inline void da_monitor_destroy(void)
> =C2=A0 */
> =C2=A0
> =C2=A0static inline void da_trace_event(struct da_monitor *da_mon,
> -=09=09=09=09=C2=A0 char *curr_state, char *event,
> -=09=09=09=09=C2=A0 char *next_state, bool is_final,
> +=09=09=09=09=C2=A0 const char *curr_state, const char *event,
> +=09=09=09=09=C2=A0 const char *next_state, bool is_final,
> =C2=A0=09=09=09=09=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_event_, MONITOR_NAME)(curr_state, event,
> next_state,
> @@ -642,7 +642,7 @@ static inline void da_trace_event(struct da_monitor
> *da_mon,
> =C2=A0}
> =C2=A0
> =C2=A0static inline void da_trace_error(struct da_monitor *da_mon,
> -=09=09=09=09=C2=A0 char *curr_state, char *event,
> +=09=09=09=09=C2=A0 const char *curr_state, const char *event,
> =C2=A0=09=09=09=09=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_error_, MONITOR_NAME)(curr_state, event);
> @@ -662,8 +662,8 @@ static inline da_id_type da_get_id(struct da_monitor
> *da_mon)
> =C2=A0 */
> =C2=A0
> =C2=A0static inline void da_trace_event(struct da_monitor *da_mon,
> -=09=09=09=09=C2=A0 char *curr_state, char *event,
> -=09=09=09=09=C2=A0 char *next_state, bool is_final,
> +=09=09=09=09=C2=A0 const char *curr_state, const char *event,
> +=09=09=09=09=C2=A0 const char *next_state, bool is_final,
> =C2=A0=09=09=09=09=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_event_, MONITOR_NAME)(id, curr_state, event,
> @@ -671,7 +671,7 @@ static inline void da_trace_event(struct da_monitor
> *da_mon,
> =C2=A0}
> =C2=A0
> =C2=A0static inline void da_trace_error(struct da_monitor *da_mon,
> -=09=09=09=09=C2=A0 char *curr_state, char *event,
> +=09=09=09=09=C2=A0 const char *curr_state, const char *event,
> =C2=A0=09=09=09=09=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_error_, MONITOR_NAME)(id, curr_state, event);
> diff --git a/include/rv/ha_monitor.h b/include/rv/ha_monitor.h
> index 6e1c7fe5449a..0952a3f161c3 100644
> --- a/include/rv/ha_monitor.h
> +++ b/include/rv/ha_monitor.h
> @@ -87,7 +87,7 @@ _Static_assert(offsetof(struct ha_monitor, da_mon) =3D=
=3D 0,
> =C2=A0 * environment constraint is false. This function can be called als=
o with no
> =C2=A0 * event from a timer (state constraints only).
> =C2=A0 */
> -static void ha_react(enum states curr_state, enum events event, char *en=
v)
> +static void ha_react(enum states curr_state, enum events event, const ch=
ar
> *env)
> =C2=A0{
> =C2=A0=09rv_react(&rv_this,
> =C2=A0=09=09 "rv: monitor %s does not allow event %s on state %s with env
> %s\n",
> @@ -98,13 +98,13 @@ static void ha_react(enum states curr_state, enum eve=
nts
> event, char *env)
> =C2=A0
> =C2=A0#else /* CONFIG_RV_REACTOR */
> =C2=A0
> -static void ha_react(enum states curr_state, enum events event, char *en=
v) {
> }
> +static void ha_react(enum states curr_state, enum events event, const ch=
ar
> *env) { }
> =C2=A0#endif
> =C2=A0
> =C2=A0/*
> =C2=A0 * model_get_state_name - return the (string) name of the given sta=
te
> =C2=A0 */
> -static char *model_get_env_name(enum envs env)
> +static const char *model_get_env_name(enum envs env)
> =C2=A0{
> =C2=A0=09if ((env < 0) || (env >=3D ENV_MAX))
> =C2=A0=09=09return "INVALID";
> @@ -227,7 +227,7 @@ static inline void ha_get_env_string(struct seq_buf *=
s,
> =C2=A0
> =C2=A0#if RV_MON_TYPE =3D=3D RV_MON_GLOBAL || RV_MON_TYPE =3D=3D RV_MON_P=
ER_CPU
> =C2=A0static inline void ha_trace_error_env(struct ha_monitor *ha_mon,
> -=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 char *curr_state, char *event=
, char
> *env,
> +=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 const char *curr_state, const=
 char
> *event, const char *env,
> =C2=A0=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_error_env_, MONITOR_NAME)(curr_state, event, e=
nv);
> @@ -237,7 +237,7 @@ static inline void ha_trace_error_env(struct ha_monit=
or
> *ha_mon,
> =C2=A0#define ha_get_target(ha_mon) da_get_target(&ha_mon->da_mon)
> =C2=A0
> =C2=A0static inline void ha_trace_error_env(struct ha_monitor *ha_mon,
> -=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 char *curr_state, char *event=
, char
> *env,
> +=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 const char *curr_state, const=
 char
> *event, const char *env,
> =C2=A0=09=09=09=09=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 da_id_type id)
> =C2=A0{
> =C2=A0=09CONCATENATE(trace_error_env_, MONITOR_NAME)(id, curr_state, even=
t,
> env);
> diff --git a/kernel/trace/rv/monitors/nomiss/nomiss.h
> b/kernel/trace/rv/monitors/nomiss/nomiss.h
> index 3d1b436194d7..0e74a27bc38c 100644
> --- a/kernel/trace/rv/monitors/nomiss/nomiss.h
> +++ b/kernel/trace/rv/monitors/nomiss/nomiss.h
> @@ -41,9 +41,9 @@ _Static_assert(env_max_stored_nomiss <=3D MAX_HA_ENV_LE=
N, "Not
> enough slots");
> =C2=A0#define HA_CLK_NS
> =C2=A0
> =C2=A0struct automaton_nomiss {
> -=09char *state_names[state_max_nomiss];
> -=09char *event_names[event_max_nomiss];
> -=09char *env_names[env_max_nomiss];
> +=09const char *state_names[state_max_nomiss];
> +=09const char *event_names[event_max_nomiss];
> +=09const char *env_names[env_max_nomiss];
> =C2=A0=09unsigned char function[state_max_nomiss][event_max_nomiss];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_nomiss];
> diff --git a/kernel/trace/rv/monitors/nomiss/nomiss_trace.h
> b/kernel/trace/rv/monitors/nomiss/nomiss_trace.h
> index 42e7efaca4e7..8ad43b6dde6a 100644
> --- a/kernel/trace/rv/monitors/nomiss/nomiss_trace.h
> +++ b/kernel/trace/rv/monitors/nomiss/nomiss_trace.h
> @@ -6,14 +6,14 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_NOMISS
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_nomiss,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_nomiss,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_env_da_monitor_id, error_env_nomiss,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *env),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *env),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, env));
> =C2=A0#endif /* CONFIG_RV_MON_NOMISS */
> diff --git a/kernel/trace/rv/monitors/nrp/nrp.h
> b/kernel/trace/rv/monitors/nrp/nrp.h
> index 3270d4c0139f..f6c1ec62233d 100644
> --- a/kernel/trace/rv/monitors/nrp/nrp.h
> +++ b/kernel/trace/rv/monitors/nrp/nrp.h
> @@ -26,8 +26,8 @@ enum events_nrp {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_nrp {
> -=09char *state_names[state_max_nrp];
> -=09char *event_names[event_max_nrp];
> +=09const char *state_names[state_max_nrp];
> +=09const char *event_names[event_max_nrp];
> =C2=A0=09unsigned char function[state_max_nrp][event_max_nrp];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_nrp];
> diff --git a/kernel/trace/rv/monitors/nrp/nrp_trace.h
> b/kernel/trace/rv/monitors/nrp/nrp_trace.h
> index 2e13497de3b6..bcdb44c8e1fa 100644
> --- a/kernel/trace/rv/monitors/nrp/nrp_trace.h
> +++ b/kernel/trace/rv/monitors/nrp/nrp_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_NRP
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_nrp,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_nrp,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0#endif /* CONFIG_RV_MON_NRP */
> diff --git a/kernel/trace/rv/monitors/opid/opid.h
> b/kernel/trace/rv/monitors/opid/opid.h
> index fb0aa4c28aa6..42680aa81f33 100644
> --- a/kernel/trace/rv/monitors/opid/opid.h
> +++ b/kernel/trace/rv/monitors/opid/opid.h
> @@ -30,9 +30,9 @@ enum envs_opid {
> =C2=A0_Static_assert(env_max_stored_opid <=3D MAX_HA_ENV_LEN, "Not enough=
 slots");
> =C2=A0
> =C2=A0struct automaton_opid {
> -=09char *state_names[state_max_opid];
> -=09char *event_names[event_max_opid];
> -=09char *env_names[env_max_opid];
> +=09const char *state_names[state_max_opid];
> +=09const char *event_names[event_max_opid];
> +=09const char *env_names[env_max_opid];
> =C2=A0=09unsigned char function[state_max_opid][event_max_opid];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_opid];
> diff --git a/kernel/trace/rv/monitors/opid/opid_trace.h
> b/kernel/trace/rv/monitors/opid/opid_trace.h
> index b04005b64208..daca1f4a1cd6 100644
> --- a/kernel/trace/rv/monitors/opid/opid_trace.h
> +++ b/kernel/trace/rv/monitors/opid/opid_trace.h
> @@ -6,14 +6,14 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_OPID
> =C2=A0DEFINE_EVENT(event_da_monitor, event_opid,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_opid,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_env_da_monitor, error_env_opid,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *env=
),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char *env),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, env));
> =C2=A0#endif /* CONFIG_RV_MON_OPID */
> diff --git a/kernel/trace/rv/monitors/pagefault/pagefault_trace.h
> b/kernel/trace/rv/monitors/pagefault/pagefault_trace.h
> index fe1f82597b1a..cd0915a42c85 100644
> --- a/kernel/trace/rv/monitors/pagefault/pagefault_trace.h
> +++ b/kernel/trace/rv/monitors/pagefault/pagefault_trace.h
> @@ -6,7 +6,7 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_PAGEFAULT
> =C2=A0DEFINE_EVENT(event_ltl_monitor_id, event_pagefault,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, char *sta=
tes, char *atoms,
> char *next),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, const cha=
r *states, const
> char *atoms, const char *next),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(task, states, atoms, next));
> =C2=A0DEFINE_EVENT(error_ltl_monitor_id, error_pagefault,
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task),
> diff --git a/kernel/trace/rv/monitors/sco/sco.h
> b/kernel/trace/rv/monitors/sco/sco.h
> index bac3beb51e72..65bcfcb136b7 100644
> --- a/kernel/trace/rv/monitors/sco/sco.h
> +++ b/kernel/trace/rv/monitors/sco/sco.h
> @@ -23,8 +23,8 @@ enum events_sco {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_sco {
> -=09char *state_names[state_max_sco];
> -=09char *event_names[event_max_sco];
> +=09const char *state_names[state_max_sco];
> +=09const char *event_names[event_max_sco];
> =C2=A0=09unsigned char function[state_max_sco][event_max_sco];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_sco];
> diff --git a/kernel/trace/rv/monitors/sco/sco_trace.h
> b/kernel/trace/rv/monitors/sco/sco_trace.h
> index b711cd9024ec..99bc63873f4a 100644
> --- a/kernel/trace/rv/monitors/sco/sco_trace.h
> +++ b/kernel/trace/rv/monitors/sco/sco_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SCO
> =C2=A0DEFINE_EVENT(event_da_monitor, event_sco,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_sco,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0#endif /* CONFIG_RV_MON_SCO */
> diff --git a/kernel/trace/rv/monitors/scpd/scpd.h
> b/kernel/trace/rv/monitors/scpd/scpd.h
> index d6329da2671b..e9742c2789f7 100644
> --- a/kernel/trace/rv/monitors/scpd/scpd.h
> +++ b/kernel/trace/rv/monitors/scpd/scpd.h
> @@ -24,8 +24,8 @@ enum events_scpd {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_scpd {
> -=09char *state_names[state_max_scpd];
> -=09char *event_names[event_max_scpd];
> +=09const char *state_names[state_max_scpd];
> +=09const char *event_names[event_max_scpd];
> =C2=A0=09unsigned char function[state_max_scpd][event_max_scpd];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_scpd];
> diff --git a/kernel/trace/rv/monitors/scpd/scpd_trace.h
> b/kernel/trace/rv/monitors/scpd/scpd_trace.h
> index 6b0f4aa4732e..d9013db67f94 100644
> --- a/kernel/trace/rv/monitors/scpd/scpd_trace.h
> +++ b/kernel/trace/rv/monitors/scpd/scpd_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SCPD
> =C2=A0DEFINE_EVENT(event_da_monitor, event_scpd,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_scpd,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0#endif /* CONFIG_RV_MON_SCPD */
> diff --git a/kernel/trace/rv/monitors/sleep/sleep_trace.h
> b/kernel/trace/rv/monitors/sleep/sleep_trace.h
> index 22eaf31da987..446c30c3af36 100644
> --- a/kernel/trace/rv/monitors/sleep/sleep_trace.h
> +++ b/kernel/trace/rv/monitors/sleep/sleep_trace.h
> @@ -6,7 +6,7 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SLEEP
> =C2=A0DEFINE_EVENT(event_ltl_monitor_id, event_sleep,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, char *sta=
tes, char *atoms,
> char *next),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, const cha=
r *states, const
> char *atoms, const char *next),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(task, states, atoms, next));
> =C2=A0DEFINE_EVENT(error_ltl_monitor_id, error_sleep,
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task),
> diff --git a/kernel/trace/rv/monitors/snep/snep.h
> b/kernel/trace/rv/monitors/snep/snep.h
> index 357520a5b3d1..fabc363684d2 100644
> --- a/kernel/trace/rv/monitors/snep/snep.h
> +++ b/kernel/trace/rv/monitors/snep/snep.h
> @@ -24,8 +24,8 @@ enum events_snep {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_snep {
> -=09char *state_names[state_max_snep];
> -=09char *event_names[event_max_snep];
> +=09const char *state_names[state_max_snep];
> +=09const char *event_names[event_max_snep];
> =C2=A0=09unsigned char function[state_max_snep][event_max_snep];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_snep];
> diff --git a/kernel/trace/rv/monitors/snep/snep_trace.h
> b/kernel/trace/rv/monitors/snep/snep_trace.h
> index 01aad49a949a..d758882e7607 100644
> --- a/kernel/trace/rv/monitors/snep/snep_trace.h
> +++ b/kernel/trace/rv/monitors/snep/snep_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SNEP
> =C2=A0DEFINE_EVENT(event_da_monitor, event_snep,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_snep,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0#endif /* CONFIG_RV_MON_SNEP */
> diff --git a/kernel/trace/rv/monitors/snroc/snroc.h
> b/kernel/trace/rv/monitors/snroc/snroc.h
> index 88b7328ad31a..12aa743e4f19 100644
> --- a/kernel/trace/rv/monitors/snroc/snroc.h
> +++ b/kernel/trace/rv/monitors/snroc/snroc.h
> @@ -23,8 +23,8 @@ enum events_snroc {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_snroc {
> -=09char *state_names[state_max_snroc];
> -=09char *event_names[event_max_snroc];
> +=09const char *state_names[state_max_snroc];
> +=09const char *event_names[event_max_snroc];
> =C2=A0=09unsigned char function[state_max_snroc][event_max_snroc];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_snroc];
> diff --git a/kernel/trace/rv/monitors/snroc/snroc_trace.h
> b/kernel/trace/rv/monitors/snroc/snroc_trace.h
> index 50114cef5122..aa9162a2d34d 100644
> --- a/kernel/trace/rv/monitors/snroc/snroc_trace.h
> +++ b/kernel/trace/rv/monitors/snroc/snroc_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SNROC
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_snroc,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_snroc,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0#endif /* CONFIG_RV_MON_SNROC */
> diff --git a/kernel/trace/rv/monitors/sssw/sssw.h
> b/kernel/trace/rv/monitors/sssw/sssw.h
> index 1a4b806061c3..ad8d7a18c52d 100644
> --- a/kernel/trace/rv/monitors/sssw/sssw.h
> +++ b/kernel/trace/rv/monitors/sssw/sssw.h
> @@ -31,8 +31,8 @@ enum events_sssw {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_sssw {
> -=09char *state_names[state_max_sssw];
> -=09char *event_names[event_max_sssw];
> +=09const char *state_names[state_max_sssw];
> +=09const char *event_names[event_max_sssw];
> =C2=A0=09unsigned char function[state_max_sssw][event_max_sssw];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_sssw];
> diff --git a/kernel/trace/rv/monitors/sssw/sssw_trace.h
> b/kernel/trace/rv/monitors/sssw/sssw_trace.h
> index 6c03cfc6960b..14d22f793a07 100644
> --- a/kernel/trace/rv/monitors/sssw/sssw_trace.h
> +++ b/kernel/trace/rv/monitors/sssw/sssw_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_SSSW
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_sssw,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_sssw,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0#endif /* CONFIG_RV_MON_SSSW */
> diff --git a/kernel/trace/rv/monitors/stall/stall.h
> b/kernel/trace/rv/monitors/stall/stall.h
> index 638520cb1082..ce38b6b2311e 100644
> --- a/kernel/trace/rv/monitors/stall/stall.h
> +++ b/kernel/trace/rv/monitors/stall/stall.h
> @@ -33,9 +33,9 @@ enum envs_stall {
> =C2=A0_Static_assert(env_max_stored_stall <=3D MAX_HA_ENV_LEN, "Not enoug=
h slots");
> =C2=A0
> =C2=A0struct automaton_stall {
> -=09char *state_names[state_max_stall];
> -=09char *event_names[event_max_stall];
> -=09char *env_names[env_max_stall];
> +=09const char *state_names[state_max_stall];
> +=09const char *event_names[event_max_stall];
> +=09const char *env_names[env_max_stall];
> =C2=A0=09unsigned char function[state_max_stall][event_max_stall];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_stall];
> diff --git a/kernel/trace/rv/monitors/stall/stall_trace.h
> b/kernel/trace/rv/monitors/stall/stall_trace.h
> index 6a7cc1b1d040..30896932a91c 100644
> --- a/kernel/trace/rv/monitors/stall/stall_trace.h
> +++ b/kernel/trace/rv/monitors/stall/stall_trace.h
> @@ -6,14 +6,14 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_STALL
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_stall,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_stall,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_env_da_monitor_id, error_env_stall,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *env),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *env),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, env));
> =C2=A0#endif /* CONFIG_RV_MON_STALL */
> diff --git a/kernel/trace/rv/monitors/sts/sts.h
> b/kernel/trace/rv/monitors/sts/sts.h
> index 6f7b2d9d72e6..74397c37cb0e 100644
> --- a/kernel/trace/rv/monitors/sts/sts.h
> +++ b/kernel/trace/rv/monitors/sts/sts.h
> @@ -31,8 +31,8 @@ enum events_sts {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_sts {
> -=09char *state_names[state_max_sts];
> -=09char *event_names[event_max_sts];
> +=09const char *state_names[state_max_sts];
> +=09const char *event_names[event_max_sts];
> =C2=A0=09unsigned char function[state_max_sts][event_max_sts];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_sts];
> diff --git a/kernel/trace/rv/monitors/sts/sts_trace.h
> b/kernel/trace/rv/monitors/sts/sts_trace.h
> index d78beb58d5b3..61e8e414627d 100644
> --- a/kernel/trace/rv/monitors/sts/sts_trace.h
> +++ b/kernel/trace/rv/monitors/sts/sts_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_STS
> =C2=A0DEFINE_EVENT(event_da_monitor, event_sts,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_sts,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0#endif /* CONFIG_RV_MON_STS */
> diff --git a/kernel/trace/rv/monitors/wakeup/wakeup_trace.h
> b/kernel/trace/rv/monitors/wakeup/wakeup_trace.h
> index 7e056183f920..79bfaf91c912 100644
> --- a/kernel/trace/rv/monitors/wakeup/wakeup_trace.h
> +++ b/kernel/trace/rv/monitors/wakeup/wakeup_trace.h
> @@ -6,7 +6,7 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_WAKEUP
> =C2=A0DEFINE_EVENT(event_ltl_monitor_id, event_wakeup,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, char *sta=
tes, char *atoms,
> char *next),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task, const cha=
r *states, const
> char *atoms, const char *next),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(task, states, atoms, next));
> =C2=A0DEFINE_EVENT(error_ltl_monitor_id, error_wakeup,
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(struct task_struct *task),
> diff --git a/kernel/trace/rv/monitors/wip/wip.h
> b/kernel/trace/rv/monitors/wip/wip.h
> index b4c3eea94c86..b65baeae5419 100644
> --- a/kernel/trace/rv/monitors/wip/wip.h
> +++ b/kernel/trace/rv/monitors/wip/wip.h
> @@ -23,8 +23,8 @@ enum events_wip {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_wip {
> -=09char *state_names[state_max_wip];
> -=09char *event_names[event_max_wip];
> +=09const char *state_names[state_max_wip];
> +=09const char *event_names[event_max_wip];
> =C2=A0=09unsigned char function[state_max_wip][event_max_wip];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_wip];
> diff --git a/kernel/trace/rv/monitors/wip/wip_trace.h
> b/kernel/trace/rv/monitors/wip/wip_trace.h
> index aa2162f47a4c..99cf368a4b03 100644
> --- a/kernel/trace/rv/monitors/wip/wip_trace.h
> +++ b/kernel/trace/rv/monitors/wip/wip_trace.h
> @@ -6,10 +6,10 @@
> =C2=A0
> =C2=A0#ifdef CONFIG_RV_MON_WIP
> =C2=A0DEFINE_EVENT(event_da_monitor, event_wip,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event, char *nex=
t_state, bool
> final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t, const char
> *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event, next_state, final=
_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor, error_wip,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(const char *state, const char *even=
t),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(state, event));
> =C2=A0#endif /* CONFIG_RV_MON_WIP */
> diff --git a/kernel/trace/rv/monitors/wwnr/wwnr.h
> b/kernel/trace/rv/monitors/wwnr/wwnr.h
> index a28006512c9b..06d724ac1182 100644
> --- a/kernel/trace/rv/monitors/wwnr/wwnr.h
> +++ b/kernel/trace/rv/monitors/wwnr/wwnr.h
> @@ -23,8 +23,8 @@ enum events_wwnr {
> =C2=A0};
> =C2=A0
> =C2=A0struct automaton_wwnr {
> -=09char *state_names[state_max_wwnr];
> -=09char *event_names[event_max_wwnr];
> +=09const char *state_names[state_max_wwnr];
> +=09const char *event_names[event_max_wwnr];
> =C2=A0=09unsigned char function[state_max_wwnr][event_max_wwnr];
> =C2=A0=09unsigned char initial_state;
> =C2=A0=09bool final_states[state_max_wwnr];
> diff --git a/kernel/trace/rv/monitors/wwnr/wwnr_trace.h
> b/kernel/trace/rv/monitors/wwnr/wwnr_trace.h
> index fc97ec7476ad..c36c8935dce5 100644
> --- a/kernel/trace/rv/monitors/wwnr/wwnr_trace.h
> +++ b/kernel/trace/rv/monitors/wwnr/wwnr_trace.h
> @@ -7,10 +7,10 @@
> =C2=A0#ifdef CONFIG_RV_MON_WWNR
> =C2=A0/* id is the pid of the task */
> =C2=A0DEFINE_EVENT(event_da_monitor_id, event_wwnr,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event, c=
har *next_state,
> bool final_state),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event, const
> char *next_state, bool final_state),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event, next_state, f=
inal_state));
> =C2=A0
> =C2=A0DEFINE_EVENT(error_da_monitor_id, error_wwnr,
> -=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, char *state, char *event),
> +=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_PROTO(int id, const char *state, const ch=
ar *event),
> =C2=A0=09=C2=A0=C2=A0=C2=A0=C2=A0 TP_ARGS(id, state, event));
> =C2=A0#endif /* CONFIG_RV_MON_WWNR */
> diff --git a/kernel/trace/rv/rv_monitors_test.c
> b/kernel/trace/rv/rv_monitors_test.c
> index 3ad11195e664..e70ae29da268 100644
> --- a/kernel/trace/rv/rv_monitors_test.c
> +++ b/kernel/trace/rv/rv_monitors_test.c
> @@ -21,7 +21,7 @@
> =C2=A0static struct rv_kunit_ctx *active_ctx;
> =C2=A0
> =C2=A0__printf(1, 0)
> -static void rv_kunit_mock_react(const char *msg, va_list args)
> +static void rv_kunit_mock_react(const const char *msg, va_list args)
> =C2=A0{
> =C2=A0=09if (active_ctx)
> =C2=A0=09=09++active_ctx->reactions;
> diff --git a/kernel/trace/rv/rv_reactors.c b/kernel/trace/rv/rv_reactors.=
c
> index 2f5fc8d18dea..538fd0d70fc7 100644
> --- a/kernel/trace/rv/rv_reactors.c
> +++ b/kernel/trace/rv/rv_reactors.c
> @@ -71,7 +71,7 @@
> =C2=A0 */
> =C2=A0static LIST_HEAD(rv_reactors_list);
> =C2=A0
> -static struct rv_reactor *get_reactor_rdef_by_name(char *name)
> +static struct rv_reactor *get_reactor_rdef_by_name(const char *name)
> =C2=A0{
> =C2=A0=09struct rv_reactor *r;
> =C2=A0
> diff --git a/kernel/trace/rv/rv_trace.h b/kernel/trace/rv/rv_trace.h
> index 2f8a932432c9..a756c7d11755 100644
> --- a/kernel/trace/rv/rv_trace.h
> +++ b/kernel/trace/rv/rv_trace.h
> @@ -11,7 +11,7 @@
> =C2=A0#ifdef CONFIG_DA_MON_EVENTS_IMPLICIT
> =C2=A0DECLARE_EVENT_CLASS(event_da_monitor,
> =C2=A0
> -=09TP_PROTO(char *state, char *event, char *next_state, bool
> final_state),
> +=09TP_PROTO(const char *state, const char *event, const char
> *next_state, bool final_state),
> =C2=A0
> =C2=A0=09TP_ARGS(state, event, next_state, final_state),
> =C2=A0
> @@ -38,7 +38,7 @@ DECLARE_EVENT_CLASS(event_da_monitor,
> =C2=A0
> =C2=A0DECLARE_EVENT_CLASS(error_da_monitor,
> =C2=A0
> -=09TP_PROTO(char *state, char *event),
> +=09TP_PROTO(const char *state, const char *event),
> =C2=A0
> =C2=A0=09TP_ARGS(state, event),
> =C2=A0
> @@ -68,7 +68,7 @@ DECLARE_EVENT_CLASS(error_da_monitor,
> =C2=A0/* For simplicity this class is marked as DA although relevant only=
 for HA */
> =C2=A0DECLARE_EVENT_CLASS(error_env_da_monitor,
> =C2=A0
> -=09TP_PROTO(char *state, char *event, char *env),
> +=09TP_PROTO(const char *state, const char *event, const char *env),
> =C2=A0
> =C2=A0=09TP_ARGS(state, event, env),
> =C2=A0
> @@ -100,7 +100,7 @@ DECLARE_EVENT_CLASS(error_env_da_monitor,
> =C2=A0#ifdef CONFIG_DA_MON_EVENTS_ID
> =C2=A0DECLARE_EVENT_CLASS(event_da_monitor_id,
> =C2=A0
> -=09TP_PROTO(int id, char *state, char *event, char *next_state, bool
> final_state),
> +=09TP_PROTO(int id, const char *state, const char *event, const char
> *next_state, bool final_state),
> =C2=A0
> =C2=A0=09TP_ARGS(id, state, event, next_state, final_state),
> =C2=A0
> @@ -130,7 +130,7 @@ DECLARE_EVENT_CLASS(event_da_monitor_id,
> =C2=A0
> =C2=A0DECLARE_EVENT_CLASS(error_da_monitor_id,
> =C2=A0
> -=09TP_PROTO(int id, char *state, char *event),
> +=09TP_PROTO(int id, const char *state, const char *event),
> =C2=A0
> =C2=A0=09TP_ARGS(id, state, event),
> =C2=A0
> @@ -162,7 +162,7 @@ DECLARE_EVENT_CLASS(error_da_monitor_id,
> =C2=A0/* For simplicity this class is marked as DA although relevant only=
 for HA */
> =C2=A0DECLARE_EVENT_CLASS(error_env_da_monitor_id,
> =C2=A0
> -=09TP_PROTO(int id, char *state, char *event, char *env),
> +=09TP_PROTO(int id, const char *state, const char *event, const char
> *env),
> =C2=A0
> =C2=A0=09TP_ARGS(id, state, event, env),
> =C2=A0
> @@ -197,7 +197,7 @@ DECLARE_EVENT_CLASS(error_env_da_monitor_id,
> =C2=A0#ifdef CONFIG_LTL_MON_EVENTS_ID
> =C2=A0DECLARE_EVENT_CLASS(event_ltl_monitor_id,
> =C2=A0
> -=09TP_PROTO(struct task_struct *task, char *states, char *atoms, char
> *next),
> +=09TP_PROTO(struct task_struct *task, const char *states, const char
> *atoms, const char *next),
> =C2=A0
> =C2=A0=09TP_ARGS(task, states, atoms, next),
> =C2=A0
> @@ -249,7 +249,7 @@ DECLARE_EVENT_CLASS(error_ltl_monitor_id,
> =C2=A0/* Tracepoint useful for monitors development, currenly only used i=
n DA */
> =C2=A0TRACE_EVENT(rv_retries_error,
> =C2=A0
> -=09TP_PROTO(char *name, char *event),
> +=09TP_PROTO(const char *name, const char *event),
> =C2=A0
> =C2=A0=09TP_ARGS(name, event),
> =C2=A0