Commit 58d5f0d4 authored by Gabriele Monaco's avatar Gabriele Monaco Committed by Steven Rostedt (Google)
Browse files

rv: Return init error when registering monitors 

Monitors generated with dot2k have their registration function (the one
called during monitor initialisation) return always 0, even if the
registration failed on RV side.
This can hide potential errors.

Return the value returned by the RV register function.

Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Tomas Glozar <tglozar@redhat.com>
Cc: Juri Lelli <jlelli@redhat.com>
Cc: Clark Williams <williams@redhat.com>
Cc: John Kacur <jkacur@redhat.com>
Link: https://lore.kernel.org/20250723161240.194860-6-gmonaco@redhat.com


Reviewed-by: default avatarNam Cao <namcao@linutronix.de>
Signed-off-by: default avatarGabriele Monaco <gmonaco@redhat.com>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
parent 560473f2

kernel/trace/rv/monitors/sched/sched.c

+1 −2
Original line number Diff line number Diff line
@@ -21,8 +21,7 @@ struct rv_monitor rv_sched = { 


static int __init register_sched(void)

{

	rv_register_monitor(&rv_sched, NULL);

	return 0;

	return rv_register_monitor(&rv_sched, NULL);

}



static void __exit unregister_sched(void)

kernel/trace/rv/monitors/sco/sco.c

+1 −2
Original line number Diff line number Diff line
@@ -71,8 +71,7 @@ static struct rv_monitor rv_sco = { 


static int __init register_sco(void)

{

	rv_register_monitor(&rv_sco, &rv_sched);

	return 0;

	return rv_register_monitor(&rv_sco, &rv_sched);

}



static void __exit unregister_sco(void)

kernel/trace/rv/monitors/scpd/scpd.c

+1 −2
Original line number Diff line number Diff line
@@ -79,8 +79,7 @@ static struct rv_monitor rv_scpd = { 


static int __init register_scpd(void)

{

	rv_register_monitor(&rv_scpd, &rv_sched);

	return 0;

	return rv_register_monitor(&rv_scpd, &rv_sched);

}



static void __exit unregister_scpd(void)

kernel/trace/rv/monitors/sncid/sncid.c

+1 −2
Original line number Diff line number Diff line
@@ -79,8 +79,7 @@ static struct rv_monitor rv_sncid = { 


static int __init register_sncid(void)

{

	rv_register_monitor(&rv_sncid, &rv_sched);

	return 0;

	return rv_register_monitor(&rv_sncid, &rv_sched);

}



static void __exit unregister_sncid(void)

kernel/trace/rv/monitors/snep/snep.c

+1 −2
Original line number Diff line number Diff line
@@ -79,8 +79,7 @@ static struct rv_monitor rv_snep = { 


static int __init register_snep(void)

{

	rv_register_monitor(&rv_snep, &rv_sched);

	return 0;

	return rv_register_monitor(&rv_snep, &rv_sched);

}



static void __exit unregister_snep(void)