X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fhousekeeping.c;h=dae17c3eab220770de5ad614244704063adb3f7d;hb=7b92376d3c819cfe849b92dfc5c312084408af9c;hp=fc878b499280850cebfb204abd76c50071de5777;hpb=8b523510f3c2e34a3654582be2db17f4138582dd;p=citadel.git diff --git a/citadel/housekeeping.c b/citadel/housekeeping.c index fc878b499..dae17c3ea 100644 --- a/citadel/housekeeping.c +++ b/citadel/housekeeping.c @@ -1,182 +1,197 @@ /* - * This file contains housekeeping tasks which periodically - * need to be executed. It keeps a nice little queue... + * This file contains miscellaneous housekeeping tasks. * - * $Id$ + * 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 "sysdep.h" -#include -#include #include -#include -#include -#include -#include -#include -#include -#include -#include -#ifdef HAVE_SYS_SELECT_H -#include -#endif -#include "tools.h" -#include "citadel.h" -#include "server.h" -#include "citserver.h" -#include "config.h" -#include "housekeeping.h" -#include "sysdep_decls.h" -#include "room_ops.h" +#include +#include "ctdl_module.h" +#include "serv_extensions.h" +#include "room_ops.h" +#include "internet_addressing.h" +#include "journaling.h" -int housepipe[2]; /* This is the queue for housekeeping tasks */ +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; - int session_to_kill; +void check_ref_counts_backend(struct ctdlroom *qrbuf, void *data) { - now = time(NULL); - session_to_kill = 0; - begin_critical_section(S_SESSION_TABLE); - for (ccptr = ContextList; ccptr != NULL; ccptr = ccptr->next) { - if ( (ccptr!=CC) - && (config.c_sleeping > 0) - && (now - (ccptr->lastcmd) > config.c_sleeping) ) { - ccptr->kill_me = 1; - } - } - end_critical_section(S_SESSION_TABLE); + int *new_refcounts; + + new_refcounts = (int *) data; + + ++new_refcounts[(int)qrbuf->QRfloor]; } +void check_ref_counts(void) { + struct floor flbuf; + int a; -void check_sched_shutdown(void) { - if ((ScheduledShutdown == 1) && (ContextList == NULL)) { - lprintf(3, "Scheduled shutdown initiating.\n"); - master_cleanup(); + 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]); + } +} /* - * This is the main loop for the housekeeping thread. It remains active - * during the entire run of the server. + * 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. */ -void housekeeping_loop(void) { - long flags; - struct timeval tv; - fd_set readfds; - int did_something; - char house_cmd[256]; /* Housekeep cmds are always 256 bytes long */ - char cmd[256]; - - if (pipe(housepipe) != 0) { - lprintf(1, "FATAL ERROR: can't create housekeeping pipe: %s\n", - strerror(errno)); - exit(0); - } +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; - flags = (long) fcntl(housepipe[1], F_GETFL); - flags |= O_NONBLOCK; - fcntl(housepipe[1], F_SETFL, flags); - - while(1) { - do { - did_something = 0; - tv.tv_sec = HOUSEKEEPING_WAKEUP; - tv.tv_usec = 0; - FD_ZERO(&readfds); - FD_SET(housepipe[0], &readfds); - select(housepipe[0] + 1, &readfds, NULL, NULL, &tv); - if (FD_ISSET(housepipe[0], &readfds)) { - did_something = 1; - } - - if (did_something) { - read(housepipe[0], house_cmd, 256); - } - else { - memset(house_cmd, 0, 256); - strcpy(house_cmd, "MINUTE"); - } - - extract(cmd, house_cmd, 0); - - /* Do whatever this cmd requires */ - - /* Once-every-minute housekeeper */ - if (!strcmp(cmd, "MINUTE")) { - terminate_idle_sessions(); - } - - /* Scheduled shutdown housekeeper */ - else if (!strcmp(cmd, "SCHED_SHUTDOWN")) { - check_sched_shutdown(); - } - - /* Unknown */ - else { - lprintf(7, "Unknown housekeeping command\n"); - } - - } while (did_something); + /* + * 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. + */ + begin_critical_section(S_HOUSEKEEPING); + if (housekeeping_in_progress == 0) { + do_housekeeping_now = 1; + housekeeping_in_progress = 1; } -} - + end_critical_section(S_HOUSEKEEPING); + 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) + ); + } + 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); + } + /* First, do the "as often as needed" stuff... */ + JournalRunQueue(); + PerformSessionHooks(EVT_HOUSE); -void enter_housekeeping_cmd(char *cmd) { - char cmdbuf[256]; + /* 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 */ + } - lprintf(9, "enter_housekeeping_cmd(%s)\n", cmd); - safestrncpy(cmdbuf, cmd, 256); + /* + * All done. + */ begin_critical_section(S_HOUSEKEEPING); - write(housepipe[1], cmdbuf, 256); + housekeeping_in_progress = 0; end_critical_section(S_HOUSEKEEPING); - lprintf(9, "leaving enter_housekeeping_cmd()\n"); } - -/* - * 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, void *data) { - struct floor flbuf; +void CtdlDisableHouseKeeping(void) +{ + int ActiveBackgroundJobs; + int do_housekeeping_now = 0; + struct CitContext *nptr; + int nContexts, i; - getfloor(&flbuf, qrbuf->QRfloor); - ++flbuf.f_ref_count; - flbuf.f_flags = flbuf.f_flags | QR_INUSE; - putfloor(&flbuf, qrbuf->QRfloor); +retry_block_housekeeping: + syslog(LOG_INFO, "housekeeping: trying to disable services"); + begin_critical_section(S_HOUSEKEEPING); + if (housekeeping_in_progress == 0) { + do_housekeeping_now = 1; + housekeeping_in_progress = 1; } - -void check_ref_counts(void) { - struct floor flbuf; - int a; - - for (a=0; a