From a73773b24e2ec28fd6f2ca1a2705dca5f517ba64 Mon Sep 17 00:00:00 2001 From: Augustin Cavalier Date: Thu, 7 Jul 2022 14:57:23 -0400 Subject: [PATCH] freebsd_network: Clean up struct callout. * Remove unused flags field. * Rename internal members to have c_ at the beginning, like FreeBSD. --- src/libs/compat/freebsd_network/callout.cpp | 23 +++++++++---------- .../freebsd_network/compat/sys/_callout.h | 5 ++-- 2 files changed, 13 insertions(+), 15 deletions(-) diff --git a/src/libs/compat/freebsd_network/callout.cpp b/src/libs/compat/freebsd_network/callout.cpp index 8ecd727a42..303a98ebeb 100644 --- a/src/libs/compat/freebsd_network/callout.cpp +++ b/src/libs/compat/freebsd_network/callout.cpp @@ -52,12 +52,12 @@ callout_thread(void* /*data*/) if (c == NULL) break; - if (c->due < system_time()) { + if (c->c_due < system_time()) { struct mtx *mutex = c->c_mtx; // execute timer list_remove_item(&sTimers, c); - c->due = -1; + c->c_due = -1; sCurrentCallout = c; mutex_unlock(&sLock); @@ -79,8 +79,8 @@ callout_thread(void* /*data*/) // restart scanning as we unlocked the list } else { // calculate new timeout - if (c->due < timeout) - timeout = c->due; + if (c->c_due < timeout) + timeout = c->c_due; } } @@ -164,8 +164,7 @@ callout_init(struct callout *callout, int mpsafe) void callout_init_mtx(struct callout *c, struct mtx *mtx, int flags) { - c->due = 0; - c->flags = 0; + c->c_due = 0; c->c_arg = NULL; c->c_func = NULL; @@ -188,13 +187,13 @@ callout_reset(struct callout *c, int _ticks, void (*func)(void *), void *arg) if (_ticks >= 0) { // reschedule or add this timer - if (c->due <= 0) + if (c->c_due <= 0) list_add_item(&sTimers, c); - c->due = system_time() + TICKS_2_USEC(_ticks); + c->c_due = system_time() + TICKS_2_USEC(_ticks); // notify timer about the change if necessary - if (sTimeout > c->due) + if (sTimeout > c->c_due) release_sem(sWaitSem); } @@ -228,12 +227,12 @@ _callout_stop_safe(struct callout *c, int safe) return 0; } - if (c->due <= 0) + if (c->c_due <= 0) return -1; // this timer is scheduled, cancel it list_remove_item(&sTimers, c); - c->due = 0; + c->c_due = 0; return 1; } @@ -241,7 +240,7 @@ _callout_stop_safe(struct callout *c, int safe) int callout_pending(struct callout *c) { - return c->due > 0; + return c->c_due > 0; } diff --git a/src/libs/compat/freebsd_network/compat/sys/_callout.h b/src/libs/compat/freebsd_network/compat/sys/_callout.h index b7270d7350..acba322096 100644 --- a/src/libs/compat/freebsd_network/compat/sys/_callout.h +++ b/src/libs/compat/freebsd_network/compat/sys/_callout.h @@ -12,9 +12,8 @@ struct callout { - struct list_link link; - bigtime_t due; - uint32 flags; + struct list_link c_link; + bigtime_t c_due; void * c_arg; void (*c_func)(void *);