blob: 2e59b9fb246181d7a7f1f4957caf5b01b383e8d2 [file] [log] [blame]
bart922304f2011-03-13 12:02:44 +00001/* -*- mode: C; c-basic-offset: 3; indent-tabs-mode: nil; -*- */
sewardjaf44c822007-11-25 14:01:38 +00002/*
bart86562bd2009-02-16 19:43:56 +00003 This file is part of drd, a thread error detector.
sewardjaf44c822007-11-25 14:01:38 +00004
bart922304f2011-03-13 12:02:44 +00005 Copyright (C) 2006-2011 Bart Van Assche <bvanassche@acm.org>.
sewardjaf44c822007-11-25 14:01:38 +00006
7 This program is free software; you can redistribute it and/or
8 modify it under the terms of the GNU General Public License as
9 published by the Free Software Foundation; either version 2 of the
10 License, or (at your option) any later version.
11
12 This program is distributed in the hope that it will be useful, but
13 WITHOUT ANY WARRANTY; without even the implied warranty of
14 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
15 General Public License for more details.
16
17 You should have received a copy of the GNU General Public License
18 along with this program; if not, write to the Free Software
19 Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
20 02111-1307, USA.
21
22 The GNU General Public License is contained in the file COPYING.
23*/
24
25
bart28230a32008-02-29 17:27:03 +000026#include "drd_clientobj.h"
sewardjaf44c822007-11-25 14:01:38 +000027#include "drd_cond.h"
28#include "drd_error.h"
29#include "drd_mutex.h"
bart46b5fce2008-06-28 13:01:30 +000030#include "pub_tool_errormgr.h" /* VG_(maybe_record_error)() */
31#include "pub_tool_libcassert.h" /* tl_assert() */
bart850f1992010-05-29 18:43:21 +000032#include "pub_tool_libcbase.h" /* VG_(memcmp)() */
bart46b5fce2008-06-28 13:01:30 +000033#include "pub_tool_libcprint.h" /* VG_(printf)() */
34#include "pub_tool_machine.h" /* VG_(get_IP)() */
bart46b5fce2008-06-28 13:01:30 +000035#include "pub_tool_threadstate.h" /* VG_(get_running_tid)() */
sewardjaf44c822007-11-25 14:01:38 +000036
37
bart46b5fce2008-06-28 13:01:30 +000038/* Local functions. */
bart28230a32008-02-29 17:27:03 +000039
bartdc1ef032009-02-15 14:18:02 +000040static void DRD_(cond_cleanup)(struct cond_info* p);
bart28230a32008-02-29 17:27:03 +000041
42
bart46b5fce2008-06-28 13:01:30 +000043/* Local variables. */
bart28230a32008-02-29 17:27:03 +000044
bartdc1ef032009-02-15 14:18:02 +000045static Bool DRD_(s_report_signal_unlocked) = True;
46static Bool DRD_(s_trace_cond);
sewardjaf44c822007-11-25 14:01:38 +000047
48
bart46b5fce2008-06-28 13:01:30 +000049/* Function definitions. */
bart28230a32008-02-29 17:27:03 +000050
bartdc1ef032009-02-15 14:18:02 +000051void DRD_(cond_set_report_signal_unlocked)(const Bool r)
bart764dea22009-02-15 13:16:52 +000052{
bartbedfd232009-03-26 19:07:15 +000053 DRD_(s_report_signal_unlocked) = r;
bart764dea22009-02-15 13:16:52 +000054}
55
bartdc1ef032009-02-15 14:18:02 +000056void DRD_(cond_set_trace)(const Bool trace_cond)
sewardjaf44c822007-11-25 14:01:38 +000057{
bartbedfd232009-03-26 19:07:15 +000058 DRD_(s_trace_cond) = trace_cond;
sewardjaf44c822007-11-25 14:01:38 +000059}
60
61static
bartdc1ef032009-02-15 14:18:02 +000062void DRD_(cond_initialize)(struct cond_info* const p, const Addr cond)
sewardjaf44c822007-11-25 14:01:38 +000063{
bartbedfd232009-03-26 19:07:15 +000064 tl_assert(cond != 0);
bart62cc2322010-03-07 10:54:21 +000065 tl_assert(p->a1 == cond);
66 tl_assert(p->type == ClientCondvar);
sewardjaf44c822007-11-25 14:01:38 +000067
bartbedfd232009-03-26 19:07:15 +000068 p->cleanup = (void(*)(DrdClientobj*))(DRD_(cond_cleanup));
69 p->delete_thread = 0;
70 p->waiter_count = 0;
71 p->mutex = 0;
sewardjaf44c822007-11-25 14:01:38 +000072}
73
bart195e41f2009-02-15 11:34:57 +000074/**
75 * Free the memory that was allocated by cond_initialize(). Called by
76 * DRD_(clientobj_remove)().
bart28230a32008-02-29 17:27:03 +000077 */
bartdc1ef032009-02-15 14:18:02 +000078static void DRD_(cond_cleanup)(struct cond_info* p)
bart28230a32008-02-29 17:27:03 +000079{
bartbedfd232009-03-26 19:07:15 +000080 tl_assert(p);
81 if (p->mutex)
82 {
83 struct mutex_info* q;
84 q = &(DRD_(clientobj_get)(p->mutex, ClientMutex)->mutex);
bartbedfd232009-03-26 19:07:15 +000085 {
bart62cc2322010-03-07 10:54:21 +000086 CondDestrErrInfo cde = {
87 DRD_(thread_get_running_tid)(),
88 p->a1,
89 q ? q->a1 : 0,
90 q ? q->owner : DRD_INVALID_THREADID
91 };
bartbedfd232009-03-26 19:07:15 +000092 VG_(maybe_record_error)(VG_(get_running_tid)(),
93 CondDestrErr,
94 VG_(get_IP)(VG_(get_running_tid)()),
95 "Destroying condition variable that is being"
96 " waited upon",
97 &cde);
98 }
99 }
bart28230a32008-02-29 17:27:03 +0000100}
101
bart62cc2322010-03-07 10:54:21 +0000102/**
103 * Report that the synchronization object at address 'addr' is of the
104 * wrong type.
105 */
106static void wrong_type(const Addr addr)
107{
108 GenericErrInfo gei = {
109 .tid = DRD_(thread_get_running_tid)(),
110 .addr = addr,
111 };
112 VG_(maybe_record_error)(VG_(get_running_tid)(),
113 GenericErr,
114 VG_(get_IP)(VG_(get_running_tid)()),
115 "wrong type of synchronization object",
116 &gei);
117}
118
bartd45d9952009-05-31 18:53:54 +0000119static struct cond_info* cond_get_or_allocate(const Addr cond)
sewardjaf44c822007-11-25 14:01:38 +0000120{
bartbedfd232009-03-26 19:07:15 +0000121 struct cond_info *p;
bart28230a32008-02-29 17:27:03 +0000122
bartbedfd232009-03-26 19:07:15 +0000123 tl_assert(offsetof(DrdClientobj, cond) == 0);
124 p = &(DRD_(clientobj_get)(cond, ClientCondvar)->cond);
bart62cc2322010-03-07 10:54:21 +0000125 if (p)
126 return p;
127
128 if (DRD_(clientobj_present)(cond, cond + 1))
bartbedfd232009-03-26 19:07:15 +0000129 {
bart62cc2322010-03-07 10:54:21 +0000130 wrong_type(cond);
131 return 0;
bartbedfd232009-03-26 19:07:15 +0000132 }
bart62cc2322010-03-07 10:54:21 +0000133
134 p = &(DRD_(clientobj_add)(cond, ClientCondvar)->cond);
135 DRD_(cond_initialize)(p, cond);
bartbedfd232009-03-26 19:07:15 +0000136 return p;
sewardjaf44c822007-11-25 14:01:38 +0000137}
138
bartd45d9952009-05-31 18:53:54 +0000139struct cond_info* DRD_(cond_get)(const Addr cond)
bart28230a32008-02-29 17:27:03 +0000140{
bartbedfd232009-03-26 19:07:15 +0000141 tl_assert(offsetof(DrdClientobj, cond) == 0);
142 return &(DRD_(clientobj_get)(cond, ClientCondvar)->cond);
bart28230a32008-02-29 17:27:03 +0000143}
144
145/** Called before pthread_cond_init(). */
bartdc1ef032009-02-15 14:18:02 +0000146void DRD_(cond_pre_init)(const Addr cond)
sewardjaf44c822007-11-25 14:01:38 +0000147{
bartbedfd232009-03-26 19:07:15 +0000148 struct cond_info* p;
bart72b751c2008-03-01 13:44:24 +0000149
bartbedfd232009-03-26 19:07:15 +0000150 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000151 DRD_(trace_msg)("[%d] cond_init cond 0x%lx\n",
152 DRD_(thread_get_running_tid)(), cond);
bart72b751c2008-03-01 13:44:24 +0000153
bartbedfd232009-03-26 19:07:15 +0000154 p = DRD_(cond_get)(cond);
bart72b751c2008-03-01 13:44:24 +0000155
bartbedfd232009-03-26 19:07:15 +0000156 if (p)
157 {
bartd45d9952009-05-31 18:53:54 +0000158 CondErrInfo cei = { .tid = DRD_(thread_get_running_tid)(), .cond = cond };
bartbedfd232009-03-26 19:07:15 +0000159 VG_(maybe_record_error)(VG_(get_running_tid)(),
160 CondErr,
161 VG_(get_IP)(VG_(get_running_tid)()),
162 "initialized twice",
163 &cei);
164 }
bart72b751c2008-03-01 13:44:24 +0000165
bartd45d9952009-05-31 18:53:54 +0000166 p = cond_get_or_allocate(cond);
sewardjaf44c822007-11-25 14:01:38 +0000167}
168
bart28230a32008-02-29 17:27:03 +0000169/** Called after pthread_cond_destroy(). */
bartdc1ef032009-02-15 14:18:02 +0000170void DRD_(cond_post_destroy)(const Addr cond)
sewardjaf44c822007-11-25 14:01:38 +0000171{
bartbedfd232009-03-26 19:07:15 +0000172 struct cond_info* p;
bart72b751c2008-03-01 13:44:24 +0000173
bartbedfd232009-03-26 19:07:15 +0000174 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000175 DRD_(trace_msg)("[%d] cond_destroy cond 0x%lx\n",
176 DRD_(thread_get_running_tid)(), cond);
sewardjaf44c822007-11-25 14:01:38 +0000177
bartbedfd232009-03-26 19:07:15 +0000178 p = DRD_(cond_get)(cond);
179 if (p == 0)
180 {
bartd45d9952009-05-31 18:53:54 +0000181 CondErrInfo cei = { .tid = DRD_(thread_get_running_tid)(), .cond = cond };
bartbedfd232009-03-26 19:07:15 +0000182 VG_(maybe_record_error)(VG_(get_running_tid)(),
183 CondErr,
184 VG_(get_IP)(VG_(get_running_tid)()),
185 "not a condition variable",
186 &cei);
187 return;
188 }
sewardjaf44c822007-11-25 14:01:38 +0000189
bartbedfd232009-03-26 19:07:15 +0000190 if (p->waiter_count != 0)
191 {
bartd45d9952009-05-31 18:53:54 +0000192 CondErrInfo cei = { .tid = DRD_(thread_get_running_tid)(), .cond = cond };
bartbedfd232009-03-26 19:07:15 +0000193 VG_(maybe_record_error)(VG_(get_running_tid)(),
194 CondErr,
195 VG_(get_IP)(VG_(get_running_tid)()),
196 "destruction of condition variable being waited"
197 " upon",
198 &cei);
199 }
bart72b751c2008-03-01 13:44:24 +0000200
bartbedfd232009-03-26 19:07:15 +0000201 DRD_(clientobj_remove)(p->a1, ClientCondvar);
sewardjaf44c822007-11-25 14:01:38 +0000202}
203
bart62cc2322010-03-07 10:54:21 +0000204/**
205 * Called before pthread_cond_wait(). Note: before this function is called,
206 * mutex_unlock() has already been called from drd_clientreq.c.
bart08e6d6a2008-06-28 16:28:49 +0000207 */
bart62cc2322010-03-07 10:54:21 +0000208void DRD_(cond_pre_wait)(const Addr cond, const Addr mutex)
sewardjaf44c822007-11-25 14:01:38 +0000209{
bartbedfd232009-03-26 19:07:15 +0000210 struct cond_info* p;
211 struct mutex_info* q;
sewardjaf44c822007-11-25 14:01:38 +0000212
bartbedfd232009-03-26 19:07:15 +0000213 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000214 DRD_(trace_msg)("[%d] cond_pre_wait cond 0x%lx\n",
215 DRD_(thread_get_running_tid)(), cond);
bart3b1ee452008-02-29 19:28:15 +0000216
bartd45d9952009-05-31 18:53:54 +0000217 p = cond_get_or_allocate(cond);
bart62cc2322010-03-07 10:54:21 +0000218 if (!p)
219 {
220 CondErrInfo cei = { .tid = DRD_(thread_get_running_tid)(), .cond = cond };
221 VG_(maybe_record_error)(VG_(get_running_tid)(),
222 CondErr,
223 VG_(get_IP)(VG_(get_running_tid)()),
224 "not a condition variable",
225 &cei);
226 return;
227 }
bart3b1ee452008-02-29 19:28:15 +0000228
bartbedfd232009-03-26 19:07:15 +0000229 if (p->waiter_count == 0)
230 {
231 p->mutex = mutex;
232 }
233 else if (p->mutex != mutex)
234 {
235 CondWaitErrInfo cwei
bartd45d9952009-05-31 18:53:54 +0000236 = { .tid = DRD_(thread_get_running_tid)(),
237 .cond = cond, .mutex1 = p->mutex, .mutex2 = mutex };
bartbedfd232009-03-26 19:07:15 +0000238 VG_(maybe_record_error)(VG_(get_running_tid)(),
239 CondWaitErr,
240 VG_(get_IP)(VG_(get_running_tid)()),
241 "Inconsistent association of condition variable"
242 " and mutex",
243 &cwei);
244 }
245 tl_assert(p->mutex);
246 q = DRD_(mutex_get)(p->mutex);
247 if (q
248 && q->owner == DRD_(thread_get_running_tid)() && q->recursion_count > 0)
249 {
250 const ThreadId vg_tid = VG_(get_running_tid)();
bartd45d9952009-05-31 18:53:54 +0000251 MutexErrInfo MEI = { DRD_(thread_get_running_tid)(),
252 q->a1, q->recursion_count, q->owner };
bartbedfd232009-03-26 19:07:15 +0000253 VG_(maybe_record_error)(vg_tid,
254 MutexErr,
255 VG_(get_IP)(vg_tid),
256 "Mutex locked recursively",
257 &MEI);
258 }
259 else if (q == 0)
260 {
261 DRD_(not_a_mutex)(p->mutex);
262 }
bart08e6d6a2008-06-28 16:28:49 +0000263
bart62cc2322010-03-07 10:54:21 +0000264 ++p->waiter_count;
sewardjaf44c822007-11-25 14:01:38 +0000265}
266
bart62cc2322010-03-07 10:54:21 +0000267/**
268 * Called after pthread_cond_wait().
269 */
270void DRD_(cond_post_wait)(const Addr cond)
sewardjaf44c822007-11-25 14:01:38 +0000271{
bartbedfd232009-03-26 19:07:15 +0000272 struct cond_info* p;
sewardjaf44c822007-11-25 14:01:38 +0000273
bartbedfd232009-03-26 19:07:15 +0000274 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000275 DRD_(trace_msg)("[%d] cond_post_wait cond 0x%lx\n",
276 DRD_(thread_get_running_tid)(), cond);
bart3b1ee452008-02-29 19:28:15 +0000277
bartbedfd232009-03-26 19:07:15 +0000278 p = DRD_(cond_get)(cond);
bart62cc2322010-03-07 10:54:21 +0000279 if (!p)
bartbedfd232009-03-26 19:07:15 +0000280 {
bart7a2cc3c2011-04-30 07:27:41 +0000281 CondDestrErrInfo cde = {
282 DRD_(thread_get_running_tid)(), cond, 0, DRD_INVALID_THREADID
283 };
284 VG_(maybe_record_error)(VG_(get_running_tid)(),
285 CondDestrErr,
286 VG_(get_IP)(VG_(get_running_tid)()),
287 "condition variable has been destroyed while"
288 " being waited upon",
289 &cde);
bart62cc2322010-03-07 10:54:21 +0000290 return;
bartbedfd232009-03-26 19:07:15 +0000291 }
bart62cc2322010-03-07 10:54:21 +0000292
293 if (p->waiter_count > 0)
294 {
295 --p->waiter_count;
296 if (p->waiter_count == 0)
297 {
298 p->mutex = 0;
299 }
300 }
sewardjaf44c822007-11-25 14:01:38 +0000301}
302
bart62cc2322010-03-07 10:54:21 +0000303static void cond_signal(const DrdThreadId tid, struct cond_info* const cond_p)
sewardjaf44c822007-11-25 14:01:38 +0000304{
bartbedfd232009-03-26 19:07:15 +0000305 const ThreadId vg_tid = VG_(get_running_tid)();
306 const DrdThreadId drd_tid = DRD_(VgThreadIdToDrdThreadId)(vg_tid);
bart3b1ee452008-02-29 19:28:15 +0000307
bart62cc2322010-03-07 10:54:21 +0000308 tl_assert(cond_p);
309
310 if (cond_p->waiter_count > 0)
bartbedfd232009-03-26 19:07:15 +0000311 {
312 if (DRD_(s_report_signal_unlocked)
bart62cc2322010-03-07 10:54:21 +0000313 && ! DRD_(mutex_is_locked_by)(cond_p->mutex, drd_tid))
bartbedfd232009-03-26 19:07:15 +0000314 {
bart62cc2322010-03-07 10:54:21 +0000315 /*
316 * A signal is sent while the associated mutex has not been locked.
317 * This can indicate but is not necessarily a race condition.
318 */
319 CondRaceErrInfo cei = { .tid = DRD_(thread_get_running_tid)(),
320 .cond = cond_p->a1,
321 .mutex = cond_p->mutex,
322 };
323 VG_(maybe_record_error)(vg_tid,
324 CondRaceErr,
325 VG_(get_IP)(vg_tid),
326 "CondErr",
327 &cei);
bartbedfd232009-03-26 19:07:15 +0000328 }
329 }
330 else
331 {
bart62cc2322010-03-07 10:54:21 +0000332 /*
333 * No other thread is waiting for the signal, hence the signal will
334 * be lost. This is normal in a POSIX threads application.
335 */
bartbedfd232009-03-26 19:07:15 +0000336 }
sewardjaf44c822007-11-25 14:01:38 +0000337}
338
bart62cc2322010-03-07 10:54:21 +0000339static void not_initialized(Addr const cond)
340{
341 CondErrInfo cei = { .tid = DRD_(thread_get_running_tid)(), .cond = cond };
342 VG_(maybe_record_error)(VG_(get_running_tid)(),
343 CondErr,
344 VG_(get_IP)(VG_(get_running_tid)()),
345 "condition variable has not been initialized",
346 &cei);
347}
348
bart3b1ee452008-02-29 19:28:15 +0000349/** Called before pthread_cond_signal(). */
bartdc1ef032009-02-15 14:18:02 +0000350void DRD_(cond_pre_signal)(Addr const cond)
bart3b1ee452008-02-29 19:28:15 +0000351{
bart62cc2322010-03-07 10:54:21 +0000352 struct cond_info* p;
353
354 p = DRD_(cond_get)(cond);
bartbedfd232009-03-26 19:07:15 +0000355 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000356 DRD_(trace_msg)("[%d] cond_signal cond 0x%lx\n",
357 DRD_(thread_get_running_tid)(), cond);
bart3b1ee452008-02-29 19:28:15 +0000358
bart850f1992010-05-29 18:43:21 +0000359 tl_assert(DRD_(pthread_cond_initializer));
360 if (!p && VG_(memcmp)((void*)cond, (void*)DRD_(pthread_cond_initializer),
361 DRD_(pthread_cond_initializer_size)) != 0)
bart62cc2322010-03-07 10:54:21 +0000362 {
363 not_initialized(cond);
364 return;
365 }
366
bart850f1992010-05-29 18:43:21 +0000367 if (!p)
368 p = cond_get_or_allocate(cond);
369
bart62cc2322010-03-07 10:54:21 +0000370 cond_signal(DRD_(thread_get_running_tid)(), p);
bart3b1ee452008-02-29 19:28:15 +0000371}
372
bart28230a32008-02-29 17:27:03 +0000373/** Called before pthread_cond_broadcast(). */
bartdc1ef032009-02-15 14:18:02 +0000374void DRD_(cond_pre_broadcast)(Addr const cond)
sewardjaf44c822007-11-25 14:01:38 +0000375{
bart62cc2322010-03-07 10:54:21 +0000376 struct cond_info* p;
377
bartbedfd232009-03-26 19:07:15 +0000378 if (DRD_(s_trace_cond))
bartb92ff0f2011-10-08 08:29:29 +0000379 DRD_(trace_msg)("[%d] cond_broadcast cond 0x%lx\n",
380 DRD_(thread_get_running_tid)(), cond);
bart3b1ee452008-02-29 19:28:15 +0000381
bart62cc2322010-03-07 10:54:21 +0000382 p = DRD_(cond_get)(cond);
bart850f1992010-05-29 18:43:21 +0000383 tl_assert(DRD_(pthread_cond_initializer));
384 if (!p && VG_(memcmp)((void*)cond, (void*)DRD_(pthread_cond_initializer),
385 DRD_(pthread_cond_initializer_size)) != 0)
bart62cc2322010-03-07 10:54:21 +0000386 {
387 not_initialized(cond);
388 return;
389 }
390
bart850f1992010-05-29 18:43:21 +0000391 if (!p)
392 p = cond_get_or_allocate(cond);
393
bart62cc2322010-03-07 10:54:21 +0000394 cond_signal(DRD_(thread_get_running_tid)(), p);
sewardjaf44c822007-11-25 14:01:38 +0000395}