mirror of
https://github.com/torvalds/linux.git
synced 2026-05-28 17:13:52 +02:00
30984ccf31
The da_monitor helper functions are generated from macros of the type:
DECLARE_DA_FUNCTION(name, type) \
static void da_func_x_##name(type arg) {} \
static void da_func_y_##name(type arg) {} \
This is good to minimise code duplication but the long macros made of
skipped end of lines is rather hard to parse. Since functions are
static, the advantage of naming them differently for each monitor is
minimal.
Refactor the da_monitor.h file to minimise macros, instead of declaring
functions from macros, we simply declare them with the same name for all
monitors (e.g. da_func_x) and for any remaining reference to the monitor
name (e.g. tracepoints, enums, global variables) we use the CONCATENATE
macro.
In this way the file is much easier to maintain while keeping the same
generality.
Functions depending on the monitor types are now conditionally compiled
according to the value of RV_MON_TYPE, which must be defined in the
monitor source.
The monitor type can be specified as in the original implementation,
although it's best to keep the default implementation (unsigned char) as
not all parts of code support larger data types, and likely there's no
need.
We keep the empty macro definitions to ease review of this change with
diff tools, but cleanup is required.
Also adapt existing monitors to keep the build working.
Reviewed-by: Nam Cao <namcao@linutronix.de>
Link: https://lore.kernel.org/r/20251126104241.291258-2-gmonaco@redhat.com
Signed-off-by: Gabriele Monaco <gmonaco@redhat.com>
86 lines
2.0 KiB
C
86 lines
2.0 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
#include <linux/ftrace.h>
|
|
#include <linux/tracepoint.h>
|
|
#include <linux/kernel.h>
|
|
#include <linux/module.h>
|
|
#include <linux/init.h>
|
|
#include <linux/rv.h>
|
|
#include <rv/instrumentation.h>
|
|
|
|
#define MODULE_NAME "wip"
|
|
|
|
#include <rv_trace.h>
|
|
#include <trace/events/sched.h>
|
|
#include <trace/events/preemptirq.h>
|
|
|
|
#define RV_MON_TYPE RV_MON_PER_CPU
|
|
#include "wip.h"
|
|
#include <rv/da_monitor.h>
|
|
|
|
static void handle_preempt_disable(void *data, unsigned long ip, unsigned long parent_ip)
|
|
{
|
|
da_handle_event(preempt_disable_wip);
|
|
}
|
|
|
|
static void handle_preempt_enable(void *data, unsigned long ip, unsigned long parent_ip)
|
|
{
|
|
da_handle_start_event(preempt_enable_wip);
|
|
}
|
|
|
|
static void handle_sched_waking(void *data, struct task_struct *task)
|
|
{
|
|
da_handle_event(sched_waking_wip);
|
|
}
|
|
|
|
static int enable_wip(void)
|
|
{
|
|
int retval;
|
|
|
|
retval = da_monitor_init();
|
|
if (retval)
|
|
return retval;
|
|
|
|
rv_attach_trace_probe("wip", preempt_enable, handle_preempt_enable);
|
|
rv_attach_trace_probe("wip", sched_waking, handle_sched_waking);
|
|
rv_attach_trace_probe("wip", preempt_disable, handle_preempt_disable);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static void disable_wip(void)
|
|
{
|
|
rv_this.enabled = 0;
|
|
|
|
rv_detach_trace_probe("wip", preempt_disable, handle_preempt_disable);
|
|
rv_detach_trace_probe("wip", preempt_enable, handle_preempt_enable);
|
|
rv_detach_trace_probe("wip", sched_waking, handle_sched_waking);
|
|
|
|
da_monitor_destroy();
|
|
}
|
|
|
|
static struct rv_monitor rv_this = {
|
|
.name = "wip",
|
|
.description = "wakeup in preemptive per-cpu testing monitor.",
|
|
.enable = enable_wip,
|
|
.disable = disable_wip,
|
|
.reset = da_monitor_reset_all,
|
|
.enabled = 0,
|
|
};
|
|
|
|
static int __init register_wip(void)
|
|
{
|
|
return rv_register_monitor(&rv_this, NULL);
|
|
}
|
|
|
|
static void __exit unregister_wip(void)
|
|
{
|
|
rv_unregister_monitor(&rv_this);
|
|
}
|
|
|
|
module_init(register_wip);
|
|
module_exit(unregister_wip);
|
|
|
|
MODULE_LICENSE("GPL");
|
|
MODULE_AUTHOR("Daniel Bristot de Oliveira <bristot@kernel.org>");
|
|
MODULE_DESCRIPTION("wip: wakeup in preemptive - per-cpu sample monitor.");
|