Re: [PATCH 1/5] rv/ltl: Prepare for other monitor types
From: Nam Cao <hidden>
Date: 2025-07-31 09:28:08
Also in:
lkml
On Thu, Jul 31, 2025 at 11:04:44AM +0200, Gabriele Monaco wrote:
I stole your solution to get rid of macros for the DA as well (might post it after this merge window or with the next changes) and I'm currently running with this:
Nice, glad you like it.
For global monitor, you could do
typdef struct {} monitor_target;
static monitor_target rv_global_target;
I didn't check clang, but gcc does not emit anything for this. So
effectively the compiled code does not have the "target" parameter.
quoted hunk ↗ jump to hunk
diff --git a/include/linux/rv.h b/include/linux/rv.h index 14410a42faef..6a7594080db1 100644 --- a/include/linux/rv.h +++ b/include/linux/rv.h@@ -13,6 +13,10 @@ #define MAX_DA_NAME_LEN 32 #define MAX_DA_RETRY_RACING_EVENTS 3 +#define RV_MON_GLOBAL 0 +#define RV_MON_PER_CPU 1 +#define RV_MON_PER_TASK 2 +The numbers don't really matter and you don't need to implement all, of course.
That makes sense, will do.
I'm not sure how are our patches going to coordinate,
Let's just post them. The one whose patches are not applied first will have to rebase. It is a trivial rebase anyway. Nam