src/server/util/thrpool.c

2 months ago

author
Olaf Wintermann <olaf.wintermann@gmail.com>
date
Sun, 26 Jan 2025 23:09:43 +0100 (2 months ago)
changeset 569
70bca6190669
parent 556
b036ccad4b49
child 570
f95868a8ec37
permissions
-rw-r--r--

add threadpool debug logging

1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
1 /*
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
2 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
3 *
44
3da1f7b6847f added some error messages
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 14
diff changeset
4 * Copyright 2013 Olaf Wintermann. All rights reserved.
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
5 *
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
6 * Redistribution and use in source and binary forms, with or without
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
7 * modification, are permitted provided that the following conditions are met:
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
8 *
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
9 * 1. Redistributions of source code must retain the above copyright
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
10 * notice, this list of conditions and the following disclaimer.
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
11 *
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
12 * 2. Redistributions in binary form must reproduce the above copyright
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
13 * notice, this list of conditions and the following disclaimer in the
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
14 * documentation and/or other materials provided with the distribution.
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
15 *
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
16 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
17 * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
18 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
19 * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
20 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
21 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
22 * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
23 * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
24 * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
25 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
26 * POSSIBILITY OF SUCH DAMAGE.
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
27 */
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
28
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
29 #include <stdio.h>
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
30 #include <stdlib.h>
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
31 #include <unistd.h>
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
32
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
33 #include "atomic.h"
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
34 #include "thrpool.h"
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
35
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
36 static threadpool_job kill_job;
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
37
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
38 threadpool_t* threadpool_new(int min, int max) {
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
39 log_ereport(LOG_INFORM, "new threadpool (min: %d, max: %d)", min, max);
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
40 threadpool_t *pool = malloc(sizeof(threadpool_t));
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
41 pool->queue = NULL;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
42 pool->queue_len = 0;
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
43 pool->num_idle = 0;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
44 pool->min_threads = min;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
45 pool->max_threads = max;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
46 pool->num_threads = 0;
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
47 pool->last_job = 0;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
48 pool->last_thread = -1;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
49
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
50 pool->threads = calloc(max, sizeof(pthread_t));
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
51 pool->thrstatus = calloc(max, sizeof(int));
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
52
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
53 pthread_mutex_init(&pool->queue_lock, NULL);
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
54 pthread_mutex_init(&pool->avlbl_lock, NULL);
357
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
55 pthread_cond_init(&pool->available, NULL);
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
56
357
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
57 return pool;
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
58 }
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
59
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
60 int threadpool_start(threadpool_t *pool) {
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
61 /* create pool threads */
357
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
62 for(int i=0;i<pool->min_threads;i++) {
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
63 if (pthread_create(pool->threads + i, NULL, threadpool_func, pool) != 0) {
408
56edda8701e0 replace perror() messages with log_ereport in thrpool.c
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 358
diff changeset
64 log_ereport(LOG_FAILURE, "threadpool_start: pthread_create failed: %s", strerror(errno));
357
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
65 return 1;
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
66 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
67 }
357
f45e962edf45 add separate threadpool_start function for creating initial threadpool threads
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 115
diff changeset
68 return 0;
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
69 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
70
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
71 void* threadpool_func(void *data) {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
72 threadpool_t *pool = (threadpool_t*)data;
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
73
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
74 pthread_t thr_self = pthread_self();
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
75 int thr_index = -1;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
76 for(int i=0;i<pool->max_threads;i++) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
77 if(pool->threads[i] == thr_self) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
78 thr_index = i;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
79 break;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
80 }
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
81 }
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
82
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
83 if(thr_index == -1) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
84 log_ereport(LOG_CATASTROPHE, "threadpool: cannot find thread index for thread %ull\n", (unsigned long long)thr_self);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
85 return NULL;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
86 }
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
87
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
88 ws_atomic_inc32(&pool->num_threads);
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
89 for(;;) {
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
90 threadpool_job *job = threadpool_get_job(pool, thr_index);
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
91 if(job == &kill_job) {
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
92 break;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
93 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
94
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
95 job->callback(job->data);
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
96
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
97 free(job);
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
98 }
556
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
99 uint32_t nthreads = ws_atomic_dec32(&pool->num_threads);
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
100 if(nthreads == 0) {
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
101 log_ereport(LOG_INFORM, "threadpool closed"); // TODO: log threadpool name
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
102 }
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
103 return NULL;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
104 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
105
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
106 threadpool_job* threadpool_get_job(threadpool_t *pool, int thread_index) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
107 struct timespec timeout;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
108 clock_gettime(CLOCK_REALTIME, &timeout);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
109 timeout.tv_sec += 30;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
110
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
111 while(pthread_mutex_timedlock(&pool->queue_lock, &timeout)) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
112 log_ereport(LOG_INFORM, "threadpool_get_job: mutex timeout");
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
113 timeout.tv_sec += 30;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
114 }
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
115
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
116 threadpool_job *job = NULL;
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
117 pool->num_idle++;
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
118 while(job == NULL) {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
119 if(pool->queue_len == 0) {
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
120 timeout.tv_sec += 30;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
121 while(pthread_cond_timedwait(&pool->available, &pool->queue_lock, &timeout)) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
122 log_ereport(LOG_DEBUG, "threadpool_get_job: cond timeout: thread: %d queue: %u", thread_index, (unsigned int)pool->queue_len);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
123 timeout.tv_sec += 60;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
124 }
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
125 continue;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
126 } else {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
127 pool_queue_t *q = pool->queue;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
128 job = q->job;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
129 pool->queue = q->next;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
130 pool->queue_len--;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
131 free(q);
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
132 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
133 }
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
134 pool->num_idle--;
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
135
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
136 pool->last_thread = thread_index;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
137 pool->last_job = time(NULL);
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
138
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
139 pthread_mutex_unlock(&pool->queue_lock);
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
140 return job;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
141 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
142
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
143 void threadpool_run(threadpool_t *pool, job_callback_f func, void *data) {
358
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
144 // TODO: handle errors
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
145
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
146 if(pool->num_threads == 0) {
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
147 threadpool_start(pool);
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
148 }
f3b490a2150c start threadpool in threadpool_run() if no threads are created yet
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 357
diff changeset
149
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
150 threadpool_job *job = malloc(sizeof(threadpool_job));
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
151 job->callback = func;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
152 job->data = data;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
153
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
154 struct timespec timeout;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
155 clock_gettime(CLOCK_REALTIME, &timeout);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
156 timeout.tv_sec += 30;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
157
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
158 while(pthread_mutex_timedlock(&pool->queue_lock, &timeout)) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
159 log_ereport(LOG_INFORM, "threadpool_run: mutex timeout");
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
160 timeout.tv_sec += 30;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
161 }
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
162 threadpool_enqueue_job(pool, job);
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
163
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
164 int create_thread = 0;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
165 int destroy_thread = 0;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
166 int diff = pool->queue_len - pool->num_idle;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
167 if(diff > 0 && pool->num_threads < pool->max_threads) {
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
168 create_thread = 1;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
169 } else if(diff < 0 && pool->num_threads > pool->min_threads) {
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
170 destroy_thread = 1;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
171 }
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
172
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
173 //if(pool->queue_len == 1) {
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
174 pthread_cond_signal(&pool->available);
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
175 //}
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
176
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
177 if(create_thread) {
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
178 pthread_t t;
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
179 if (pthread_create(&t, NULL, threadpool_func, pool) != 0) {
408
56edda8701e0 replace perror() messages with log_ereport in thrpool.c
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 358
diff changeset
180 log_ereport(LOG_FAILURE, "threadpool_run: pthread_create failed: %s", strerror(errno));
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
181 }
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
182 }
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
183 if(destroy_thread) {
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
184 threadpool_enqueue_job(pool, &kill_job);
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
185 pthread_cond_signal(&pool->available);
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
186 }
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
187
569
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
188 // some diagnostics:
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
189 // if the queue has multiple elements, but the last job was started
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
190 // over a minute ago, print some diagnostic message, because
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
191 // this does look wrong
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
192 if(pool->queue_len > 5 && pool->last_job != 0) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
193 // reuse timeout sec value, because we don't need the most accurate
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
194 // time value here
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
195 time_t current = timeout.tv_sec;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
196 if(pool->last_job + 60 < current) {
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
197 // looks like the threadpool is blocked
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
198 struct tm lastjob;
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
199 localtime_r(&pool->last_job, &lastjob);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
200 log_ereport(
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
201 LOG_WARN,
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
202 "high threadpool wait time: queue: %u lastjob: %02d:%02d:%02d",
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
203 (unsigned int)pool->queue_len,
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
204 lastjob.tm_hour, lastjob.tm_min, lastjob.tm_sec);
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
205 }
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
206 }
70bca6190669 add threadpool debug logging
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 556
diff changeset
207
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
208 pthread_mutex_unlock(&pool->queue_lock);
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
209 }
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
210
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
211 void threadpool_enqueue_job(threadpool_t *pool, threadpool_job *job) {
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
212 pool_queue_t *q = malloc(sizeof(pool_queue_t));
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
213 q->job = job;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
214 q->next = NULL;
67
50505dc3f8a6 dynamic thread pool
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 44
diff changeset
215
1
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
216 if(pool->queue == NULL) {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
217 pool->queue = q;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
218 } else {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
219 pool_queue_t *last_elem = pool->queue;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
220 while(last_elem->next != NULL) {
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
221 last_elem = last_elem->next;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
222 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
223 last_elem->next = q;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
224 }
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
225 pool->queue_len++;
3c066d52342d added source
Olaf Wintermann <olaf.wintermann@gmail.com>
parents:
diff changeset
226 }
556
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
227
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
228 void threadpool_shutdown(threadpool_t *pool) {
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
229 int nthreads = pool->max_threads;
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
230 for(int i=0;i<nthreads;i++) {
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
231 pthread_mutex_lock(&pool->queue_lock);
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
232 threadpool_enqueue_job(pool, &kill_job);
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
233 pthread_cond_signal(&pool->available);
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
234 pthread_mutex_unlock(&pool->queue_lock);
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
235 }
b036ccad4b49 improve webserver shutdown and free some stuff to make the valgrind output cleaner
Olaf Wintermann <olaf.wintermann@gmail.com>
parents: 408
diff changeset
236 }

mercurial