X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fhousekeeping.c;h=dae17c3eab220770de5ad614244704063adb3f7d;hb=7b92376d3c819cfe849b92dfc5c312084408af9c;hp=f54e3122f2375ab1907cd7274f863ddcafe7ddc5;hpb=ffe458c7fb6342ac60522b9750b2ba677bc4a0a8;p=citadel.git diff --git a/citadel/housekeeping.c b/citadel/housekeeping.c index f54e3122f..dae17c3ea 100644 --- a/citadel/housekeeping.c +++ b/citadel/housekeeping.c @@ -1,100 +1,197 @@ /* - * This file contains housekeeping tasks which periodically - * need to be executed. + * This file contains miscellaneous housekeeping tasks. * + * Copyright (c) 1987-2017 by the citadel.org team + * + * This program is open source software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License, version 3. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. */ -#include -#include #include -#include -#include -#include -#include -#include -#include -#include "citadel.h" -#include "server.h" -#include "citserver.h" -#include "config.h" -#include "housekeeping.h" -#include "sysdep_decls.h" +#include + +#include "ctdl_module.h" +#include "serv_extensions.h" #include "room_ops.h" +#include "internet_addressing.h" +#include "journaling.h" + +void check_sched_shutdown(void) { + if ((ScheduledShutdown == 1) && (ContextList == NULL)) { + syslog(LOG_NOTICE, "housekeeping: scheduled shutdown initiating"); + server_shutting_down = 1; + } +} + /* - * Terminate idle sessions. This function pounds through the session table - * comparing the current time to each session's time-of-last-command. If an - * idle session is found it is terminated, then the search restarts at the - * beginning because the pointer to our place in the list becomes invalid. + * Check (and fix) floor reference counts. This doesn't need to be done + * very often, since the counts should remain correct during normal operation. */ -void terminate_idle_sessions(void) { - struct CitContext *ccptr; - time_t now; - - time(&now); - for (ccptr = ContextList; ccptr != NULL; ccptr = ccptr->next) { - if ( (ccptr!=CC) - && (config.c_sleeping > 0) - && (now - (ccptr->lastcmd) > config.c_sleeping) ) { - lprintf(3, "Session %d timed out\n", ccptr->cs_pid); - kill_session(ccptr->cs_pid); - ccptr = ContextList; - } +void check_ref_counts_backend(struct ctdlroom *qrbuf, void *data) { + + int *new_refcounts; + + new_refcounts = (int *) data; + + ++new_refcounts[(int)qrbuf->QRfloor]; +} + + +void check_ref_counts(void) { + struct floor flbuf; + int a; + + int new_refcounts[MAXFLOORS]; + + syslog(LOG_DEBUG, "housekeeping: checking floor reference counts"); + for (a=0; a 0) { + flbuf.f_flags = flbuf.f_flags | QR_INUSE; } + else { + flbuf.f_flags = flbuf.f_flags & ~QR_INUSE; + } + lputfloor(&flbuf, a); + syslog(LOG_DEBUG, "housekeeping: floor %d has %d rooms", a, new_refcounts[a]); } +} /* - * Main housekeeping function. This gets run whenever a session terminates. + * This is the housekeeping loop. Worker threads come through here after + * processing client requests but before jumping back into the pool. We + * only allow housekeeping to execute once per minute, and we only allow one + * instance to run at a time. */ +static int housekeeping_in_progress = 0; +static time_t last_timer = 0L; void do_housekeeping(void) { + int do_housekeeping_now = 0; + int do_perminute_housekeeping_now = 0; + time_t now; - begin_critical_section(S_HOUSEKEEPING); /* - * Terminate idle sessions. + * We do it this way instead of wrapping the whole loop in an + * S_HOUSEKEEPING critical section because it eliminates the need to + * potentially have multiple concurrent mutexes in progress. */ - lprintf(7, "Calling terminate_idle_sessions()\n"); - terminate_idle_sessions(); + begin_critical_section(S_HOUSEKEEPING); + if (housekeeping_in_progress == 0) { + do_housekeeping_now = 1; + housekeeping_in_progress = 1; + } + end_critical_section(S_HOUSEKEEPING); - /* - * If the server is scheduled to shut down the next time all - * users are logged out, now's the time to do it. - */ - if ((ScheduledShutdown == 1) && (ContextList == NULL)) { - lprintf(3, "Scheduled shutdown initiating.\n"); - master_cleanup(); + now = time(NULL); + if (do_housekeeping_now == 0) { + if ( (now - last_timer) > (time_t)300 ) { + syslog(LOG_WARNING, + "housekeeping: WARNING: housekeeping loop has not run for %ld minutes. Is something stuck?", + ((now - last_timer) / 60) + ); } - end_critical_section(S_HOUSEKEEPING); + return; } + /* + * Ok, at this point we've made the decision to run the housekeeping + * loop. Everything below this point is real work. + */ + if ( (now - last_timer) > (time_t)60 ) { + do_perminute_housekeeping_now = 1; + last_timer = time(NULL); + } -/* - * Check (and fix) floor reference counts. This doesn't need to be done - * very often, since the counts should remain correct during normal operation. - * NOTE: this function pair should ONLY be called during startup. It is NOT - * thread safe. - */ -void check_ref_counts_backend(struct quickroom *qrbuf) { - struct floor flbuf; + /* First, do the "as often as needed" stuff... */ + JournalRunQueue(); + PerformSessionHooks(EVT_HOUSE); - lgetfloor(&flbuf, qrbuf->QRfloor); - ++flbuf.f_ref_count; - flbuf.f_flags = flbuf.f_flags | QR_INUSE; - lputfloor(&flbuf, qrbuf->QRfloor); + /* Then, do the "once per minute" stuff... */ + if (do_perminute_housekeeping_now) { + cdb_check_handles(); /* suggested by Justin Case */ + PerformSessionHooks(EVT_TIMER); /* Run any timer hooks */ } -void check_ref_counts(void) { - struct floor flbuf; - int a; + /* + * All done. + */ + begin_critical_section(S_HOUSEKEEPING); + housekeeping_in_progress = 0; + end_critical_section(S_HOUSEKEEPING); +} - for (a=0; a