git: aac7c7ac5458 - main - callout: Remove a redundant parameter to callout_cc_add()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sun, 17 Jul 2022 17:58:39 UTC
The branch main has been updated by markj:
URL: https://cgit.FreeBSD.org/src/commit/?id=aac7c7ac545804889d3bc28fae4efbcc07bcb6d3
commit aac7c7ac545804889d3bc28fae4efbcc07bcb6d3
Author: Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2022-06-30 17:31:57 +0000
Commit: Mark Johnston <markj@FreeBSD.org>
CommitDate: 2022-07-17 17:58:19 +0000
callout: Remove a redundant parameter to callout_cc_add()
The passed cpuid is always equal to the one stored in the callout
structure. No functional change intended.
MFC after: 1 week
Sponsored by: The FreeBSD Foundation
---
sys/kern/kern_timeout.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/sys/kern/kern_timeout.c b/sys/kern/kern_timeout.c
index 23ddd03ae72e..2bd469f94080 100644
--- a/sys/kern/kern_timeout.c
+++ b/sys/kern/kern_timeout.c
@@ -593,7 +593,7 @@ callout_lock(struct callout *c)
static void
callout_cc_add(struct callout *c, struct callout_cpu *cc,
sbintime_t sbt, sbintime_t precision, void (*func)(void *),
- void *arg, int cpu, int flags)
+ void *arg, int flags)
{
int bucket;
@@ -626,7 +626,7 @@ callout_cc_add(struct callout *c, struct callout_cpu *cc,
sbt = c->c_time + c->c_precision;
if (sbt < cc->cc_firstevent) {
cc->cc_firstevent = sbt;
- cpu_new_callout(cpu, sbt, c->c_time);
+ cpu_new_callout(c->c_cpu, sbt, c->c_time);
}
}
@@ -803,7 +803,7 @@ skip:
new_cc = callout_cpu_switch(c, cc, new_cpu);
flags = (direct) ? C_DIRECT_EXEC : 0;
callout_cc_add(c, new_cc, new_time, new_prec, new_func,
- new_arg, new_cpu, flags);
+ new_arg, flags);
CC_UNLOCK(new_cc);
CC_LOCK(cc);
#else
@@ -1062,7 +1062,7 @@ callout_reset_sbt_on(struct callout *c, sbintime_t sbt, sbintime_t prec,
}
#endif
- callout_cc_add(c, cc, to_sbt, precision, ftn, arg, cpu, flags);
+ callout_cc_add(c, cc, to_sbt, precision, ftn, arg, flags);
CTR6(KTR_CALLOUT, "%sscheduled %p func %p arg %p in %d.%08x",
cancelled ? "re" : "", c, c->c_func, c->c_arg, (int)(to_sbt >> 32),
(u_int)(to_sbt & 0xffffffff));