You can subscribe to this list here.
| 2002 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
(1) |
Oct
(122) |
Nov
(152) |
Dec
(69) |
|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 2003 |
Jan
(6) |
Feb
(25) |
Mar
(73) |
Apr
(82) |
May
(24) |
Jun
(25) |
Jul
(10) |
Aug
(11) |
Sep
(10) |
Oct
(54) |
Nov
(203) |
Dec
(182) |
| 2004 |
Jan
(307) |
Feb
(305) |
Mar
(430) |
Apr
(312) |
May
(187) |
Jun
(342) |
Jul
(487) |
Aug
(637) |
Sep
(336) |
Oct
(373) |
Nov
(441) |
Dec
(210) |
| 2005 |
Jan
(385) |
Feb
(480) |
Mar
(636) |
Apr
(544) |
May
(679) |
Jun
(625) |
Jul
(810) |
Aug
(838) |
Sep
(634) |
Oct
(521) |
Nov
(965) |
Dec
(543) |
| 2006 |
Jan
(494) |
Feb
(431) |
Mar
(546) |
Apr
(411) |
May
(406) |
Jun
(322) |
Jul
(256) |
Aug
(401) |
Sep
(345) |
Oct
(542) |
Nov
(308) |
Dec
(481) |
| 2007 |
Jan
(427) |
Feb
(326) |
Mar
(367) |
Apr
(255) |
May
(244) |
Jun
(204) |
Jul
(223) |
Aug
(231) |
Sep
(354) |
Oct
(374) |
Nov
(497) |
Dec
(362) |
| 2008 |
Jan
(322) |
Feb
(482) |
Mar
(658) |
Apr
(422) |
May
(476) |
Jun
(396) |
Jul
(455) |
Aug
(267) |
Sep
(280) |
Oct
(253) |
Nov
(232) |
Dec
(304) |
| 2009 |
Jan
(486) |
Feb
(470) |
Mar
(458) |
Apr
(423) |
May
(696) |
Jun
(461) |
Jul
(551) |
Aug
(575) |
Sep
(134) |
Oct
(110) |
Nov
(157) |
Dec
(102) |
| 2010 |
Jan
(226) |
Feb
(86) |
Mar
(147) |
Apr
(117) |
May
(107) |
Jun
(203) |
Jul
(193) |
Aug
(238) |
Sep
(300) |
Oct
(246) |
Nov
(23) |
Dec
(75) |
| 2011 |
Jan
(133) |
Feb
(195) |
Mar
(315) |
Apr
(200) |
May
(267) |
Jun
(293) |
Jul
(353) |
Aug
(237) |
Sep
(278) |
Oct
(611) |
Nov
(274) |
Dec
(260) |
| 2012 |
Jan
(303) |
Feb
(391) |
Mar
(417) |
Apr
(441) |
May
(488) |
Jun
(655) |
Jul
(590) |
Aug
(610) |
Sep
(526) |
Oct
(478) |
Nov
(359) |
Dec
(372) |
| 2013 |
Jan
(467) |
Feb
(226) |
Mar
(391) |
Apr
(281) |
May
(299) |
Jun
(252) |
Jul
(311) |
Aug
(352) |
Sep
(481) |
Oct
(571) |
Nov
(222) |
Dec
(231) |
| 2014 |
Jan
(185) |
Feb
(329) |
Mar
(245) |
Apr
(238) |
May
(281) |
Jun
(399) |
Jul
(382) |
Aug
(500) |
Sep
(579) |
Oct
(435) |
Nov
(487) |
Dec
(256) |
| 2015 |
Jan
(338) |
Feb
(357) |
Mar
(330) |
Apr
(294) |
May
(191) |
Jun
(108) |
Jul
(142) |
Aug
(261) |
Sep
(190) |
Oct
(54) |
Nov
(83) |
Dec
(22) |
| 2016 |
Jan
(49) |
Feb
(89) |
Mar
(33) |
Apr
(50) |
May
(27) |
Jun
(34) |
Jul
(53) |
Aug
(53) |
Sep
(98) |
Oct
(206) |
Nov
(93) |
Dec
(53) |
| 2017 |
Jan
(65) |
Feb
(82) |
Mar
(102) |
Apr
(86) |
May
(187) |
Jun
(67) |
Jul
(23) |
Aug
(93) |
Sep
(65) |
Oct
(45) |
Nov
(35) |
Dec
(17) |
| 2018 |
Jan
(26) |
Feb
(35) |
Mar
(38) |
Apr
(32) |
May
(8) |
Jun
(43) |
Jul
(27) |
Aug
(30) |
Sep
(43) |
Oct
(42) |
Nov
(38) |
Dec
(67) |
| 2019 |
Jan
(32) |
Feb
(37) |
Mar
(53) |
Apr
(64) |
May
(49) |
Jun
(18) |
Jul
(14) |
Aug
(53) |
Sep
(25) |
Oct
(30) |
Nov
(49) |
Dec
(31) |
| 2020 |
Jan
(87) |
Feb
(45) |
Mar
(37) |
Apr
(51) |
May
(99) |
Jun
(36) |
Jul
(11) |
Aug
(14) |
Sep
(20) |
Oct
(24) |
Nov
(40) |
Dec
(23) |
| 2021 |
Jan
(14) |
Feb
(53) |
Mar
(85) |
Apr
(15) |
May
(19) |
Jun
(3) |
Jul
(14) |
Aug
(1) |
Sep
(57) |
Oct
(73) |
Nov
(56) |
Dec
(22) |
| 2022 |
Jan
(3) |
Feb
(22) |
Mar
(6) |
Apr
(55) |
May
(46) |
Jun
(39) |
Jul
(15) |
Aug
(9) |
Sep
(11) |
Oct
(34) |
Nov
(20) |
Dec
(36) |
| 2023 |
Jan
(79) |
Feb
(41) |
Mar
(99) |
Apr
(169) |
May
(48) |
Jun
(16) |
Jul
(16) |
Aug
(57) |
Sep
(19) |
Oct
|
Nov
|
Dec
|
| S | M | T | W | T | F | S |
|---|---|---|---|---|---|---|
|
|
|
|
|
|
1
(1) |
2
|
|
3
(5) |
4
(7) |
5
(16) |
6
(7) |
7
(5) |
8
|
9
|
|
10
(3) |
11
(7) |
12
(7) |
13
(15) |
14
(4) |
15
(8) |
16
(10) |
|
17
(1) |
18
(7) |
19
(5) |
20
(17) |
21
(10) |
22
(5) |
23
|
|
24
|
25
|
26
(10) |
27
(21) |
28
(18) |
29
(7) |
30
(4) |
|
From: Christian B. <bor...@de...> - 2011-04-18 20:36:00
|
Nightly build on sless390 ( SUSE Linux Enterprise Server 11 SP1 gcc 4.3.4 on z10 (s390x) ) Started at 2011-04-18 22:10:01 CEST Ended at 2011-04-18 22:35:46 CEST Results unchanged from 24 hours ago Checking out valgrind source tree ... done Configuring valgrind ... done Building valgrind ... done Running regression tests ... failed Regression test results follow == 460 tests, 6 stderr failures, 0 stdout failures, 0 post failures == none/tests/faultstatus (stderr) helgrind/tests/tc06_two_races_xml (stderr) helgrind/tests/tc23_bogus_condwait (stderr) drd/tests/tc04_free_lock (stderr) drd/tests/tc09_bad_unlock (stderr) drd/tests/tc23_bogus_condwait (stderr) |
|
From: Christian B. <bor...@de...> - 2011-04-18 20:31:48
|
Nightly build on fedora390 ( Fedora 13/14/15 mix with gcc 3.5.3 on z196 (s390x) ) Started at 2011-04-18 22:10:01 CEST Ended at 2011-04-18 22:31:01 CEST Results unchanged from 24 hours ago Checking out valgrind source tree ... done Configuring valgrind ... done Building valgrind ... done Running regression tests ... failed Regression test results follow == 459 tests, 6 stderr failures, 0 stdout failures, 0 post failures == helgrind/tests/tc06_two_races_xml (stderr) helgrind/tests/tc20_verifywrap (stderr) helgrind/tests/tc23_bogus_condwait (stderr) drd/tests/tc04_free_lock (stderr) drd/tests/tc09_bad_unlock (stderr) drd/tests/tc23_bogus_condwait (stderr) |
|
From: <sv...@va...> - 2011-04-18 11:26:33
|
Author: sewardj
Date: 2011-04-18 12:26:25 +0100 (Mon, 18 Apr 2011)
New Revision: 11703
Log:
Reinstate the check that a thread doing
pthread_cond_{signal,broadcast} holds the associated mutex, if that is
known. Basically you'd be nuts not to hold to this rule, even though
POSIX doesn't mandate it.
Modified:
branches/HGDEV2/helgrind/hg_main.c
Modified: branches/HGDEV2/helgrind/hg_main.c
===================================================================
--- branches/HGDEV2/helgrind/hg_main.c 2011-04-18 10:37:56 UTC (rev 11702)
+++ branches/HGDEV2/helgrind/hg_main.c 2011-04-18 11:26:25 UTC (rev 11703)
@@ -2178,32 +2178,53 @@
// Hmm. POSIX doesn't actually say that it's an error to call
// pthread_cond_signal with the associated mutex being unlocked.
// Although it does say that it should be "if consistent scheduling
- // is desired."
+ // is desired." For that reason, print "dubious" if the lock isn't
+ // held by any thread. Skip the "dubious" if it is held by some
+ // other thread; that sounds straight-out wrong.
//
- // For the moment, disable these checks.
- //lk = map_locks_maybe_lookup(cvi->mx_ga);
- //if (lk == NULL || cvi->mx_ga == 0) {
- // HG_(record_error_Misc)( thr,
- // "pthread_cond_{signal,broadcast}: "
- // "no or invalid mutex associated with cond");
- //}
- ///* note: lk could be NULL. Be careful. */
- //if (lk) {
- // if (lk->kind == LK_rdwr) {
- // HG_(record_error_Misc)(thr,
- // "pthread_cond_{signal,broadcast}: associated lock is a rwlock");
- // }
- // if (lk->heldBy == NULL) {
- // HG_(record_error_Misc)(thr,
- // "pthread_cond_{signal,broadcast}: "
- // "associated lock is not held by any thread");
- // }
- // if (lk->heldBy != NULL && 0 == VG_(elemBag)(lk->heldBy, (Word)thr)) {
- // HG_(record_error_Misc)(thr,
- // "pthread_cond_{signal,broadcast}: "
- // "associated lock is not held by calling thread");
- // }
- //}
+ // Anybody who writes code that signals on a CV without holding
+ // the associated MX needs to be shipped off to a lunatic asylum
+ // ASAP, even though POSIX doesn't actually declare such behaviour
+ // illegal -- it makes code extremely difficult to understand/
+ // reason about. In particular it puts the signalling thread in
+ // a situation where it is racing against the released waiter
+ // as soon as the signalling is done, and so there needs to be
+ // some auxiliary synchronisation mechanism in the program that
+ // makes this safe -- or the race(s) need to be harmless, or
+ // probably nonexistent.
+ //
+ if (1) {
+ Lock* lk = NULL;
+ if (cvi->mx_ga != 0) {
+ lk = map_locks_maybe_lookup( (Addr)cvi->mx_ga );
+ }
+ /* note: lk could be NULL. Be careful. */
+ if (lk) {
+ if (lk->kind == LK_rdwr) {
+ HG_(record_error_Misc)(thr,
+ "pthread_cond_{signal,broadcast}: associated lock is a rwlock");
+ }
+ if (lk->heldBy == NULL) {
+ HG_(record_error_Misc)(thr,
+ "pthread_cond_{signal,broadcast}: dubious: "
+ "associated lock is not held by any thread");
+ }
+ if (lk->heldBy != NULL && 0 == VG_(elemBag)(lk->heldBy, (Word)thr)) {
+ HG_(record_error_Misc)(thr,
+ "pthread_cond_{signal,broadcast}: "
+ "associated lock is not held by calling thread");
+ }
+ } else {
+ /* Couldn't even find the damn thing. */
+ // But actually .. that's not necessarily an error. We don't
+ // know the (CV,MX) binding until a pthread_cond_wait or bcast
+ // shows us what it is, and if that may not have happened yet.
+ // So just keep quiet in this circumstance.
+ //HG_(record_error_Misc)( thr,
+ // "pthread_cond_{signal,broadcast}: "
+ // "no or invalid mutex associated with cond");
+ }
+ }
libhb_so_send( thr->hbthr, cvi->so, True/*strong_send*/ );
}
|
|
From: Bart V. A. <bva...@ac...> - 2011-04-18 10:40:35
|
On Sun, Apr 10, 2011 at 10:07 AM, Brad Hards <br...@fr...> wrote:
>
> This is a pretty simple ioctl, which shows up running qemu
> under valgrind with host USB device pass-through.
>
> Please review and apply.
>
> Brad
>
> Index: include/vki/vki-linux.h
> ===================================================================
> --- include/vki/vki-linux.h (revision 11686)
> +++ include/vki/vki-linux.h (working copy)
> @@ -2394,6 +2394,7 @@
> #define VKI_USBDEVFS_REAPURBNDELAY _VKI_IOW('U', 13, void *)
> #define VKI_USBDEVFS_CONNECTINFO _VKI_IOW('U', 17, struct vki_usbdevfs_connectinfo)
> #define VKI_USBDEVFS_IOCTL _VKI_IOWR('U', 18, struct vki_usbdevfs_ioctl)
> +#define VKI_USBDEVFS_RESET _VKI_IO('U', 20)
>
> #define VKI_USBDEVFS_URB_TYPE_ISO 0
> #define VKI_USBDEVFS_URB_TYPE_INTERRUPT 1
> Index: coregrind/m_syswrap/syswrap-linux.c
> ===================================================================
> --- coregrind/m_syswrap/syswrap-linux.c (revision 11686)
> +++ coregrind/m_syswrap/syswrap-linux.c (working copy)
> @@ -4777,6 +4777,8 @@
> }
> }
> break;
> + case VKI_USBDEVFS_RESET:
> + break;
>
> /* I2C (/dev/i2c-*) ioctls */
> case VKI_I2C_SLAVE:
Applied as r11702 on the trunk - please retest.
Note: the preferred way for contributing patches to the Valgrind
project is by creating a bugzilla entry and to attach the patch to it.
Bart.
|
|
From: <sv...@va...> - 2011-04-18 10:38:10
|
Author: bart
Date: 2011-04-18 11:37:56 +0100 (Mon, 18 Apr 2011)
New Revision: 11702
Log:
Linux/syswrap: add support for USBDEVFS_RESET (patch contributed by Brad Hards <br...@fr...>).
Modified:
trunk/coregrind/m_syswrap/syswrap-linux.c
trunk/include/vki/vki-linux.h
Modified: trunk/coregrind/m_syswrap/syswrap-linux.c
===================================================================
--- trunk/coregrind/m_syswrap/syswrap-linux.c 2011-04-18 09:41:08 UTC (rev 11701)
+++ trunk/coregrind/m_syswrap/syswrap-linux.c 2011-04-18 10:37:56 UTC (rev 11702)
@@ -4778,6 +4778,8 @@
}
}
break;
+ case VKI_USBDEVFS_RESET:
+ break;
/* I2C (/dev/i2c-*) ioctls */
case VKI_I2C_SLAVE:
Modified: trunk/include/vki/vki-linux.h
===================================================================
--- trunk/include/vki/vki-linux.h 2011-04-18 09:41:08 UTC (rev 11701)
+++ trunk/include/vki/vki-linux.h 2011-04-18 10:37:56 UTC (rev 11702)
@@ -2394,6 +2394,7 @@
#define VKI_USBDEVFS_REAPURBNDELAY _VKI_IOW('U', 13, void *)
#define VKI_USBDEVFS_CONNECTINFO _VKI_IOW('U', 17, struct vki_usbdevfs_connectinfo)
#define VKI_USBDEVFS_IOCTL _VKI_IOWR('U', 18, struct vki_usbdevfs_ioctl)
+#define VKI_USBDEVFS_RESET _VKI_IO('U', 20)
#define VKI_USBDEVFS_URB_TYPE_ISO 0
#define VKI_USBDEVFS_URB_TYPE_INTERRUPT 1
|
|
From: Brad H. <br...@fr...> - 2011-04-18 09:53:41
|
On Sun, 10 Apr 2011 06:07:32 pm Brad Hards wrote:
> Index: include/vki/vki-linux.h
> ===================================================================
> --- include/vki/vki-linux.h (revision 11686)
> +++ include/vki/vki-linux.h (working copy)
> @@ -2394,6 +2394,7 @@
> #define VKI_USBDEVFS_REAPURBNDELAY _VKI_IOW('U', 13, void *)
> #define VKI_USBDEVFS_CONNECTINFO _VKI_IOW('U', 17, struct
> vki_usbdevfs_connectinfo) #define VKI_USBDEVFS_IOCTL
> _VKI_IOWR('U', 18, struct vki_usbdevfs_ioctl) +#define VKI_USBDEVFS_RESET
> _VKI_IO('U', 20)
>
> #define VKI_USBDEVFS_URB_TYPE_ISO 0
> #define VKI_USBDEVFS_URB_TYPE_INTERRUPT 1
> Index: coregrind/m_syswrap/syswrap-linux.c
> ===================================================================
> --- coregrind/m_syswrap/syswrap-linux.c (revision 11686)
> +++ coregrind/m_syswrap/syswrap-linux.c (working copy)
> @@ -4777,6 +4777,8 @@
> }
> }
> break;
> + case VKI_USBDEVFS_RESET:
> + break;
>
> /* I2C (/dev/i2c-*) ioctls */
> case VKI_I2C_SLAVE:
Hi. I wondered if there was anything required on this patch? Any questions or
concerns?
Brad
|
|
From: <sv...@va...> - 2011-04-18 09:41:17
|
Author: sewardj
Date: 2011-04-18 10:41:08 +0100 (Mon, 18 Apr 2011)
New Revision: 11701
Log:
Fix ungrammatical error message.
Modified:
branches/HGDEV2/helgrind/hg_main.c
Modified: branches/HGDEV2/helgrind/hg_main.c
===================================================================
--- branches/HGDEV2/helgrind/hg_main.c 2011-04-15 21:21:27 UTC (rev 11700)
+++ branches/HGDEV2/helgrind/hg_main.c 2011-04-18 09:41:08 UTC (rev 11701)
@@ -2306,7 +2306,7 @@
it? If this happened it would surely be a bug in the threads
library. Or one of those fabled "spurious wakeups". */
HG_(record_error_Misc)( thr, "Bug in libpthread: pthread_cond_wait "
- "succeeded on"
+ "succeeded"
" without prior pthread_cond_post");
}
|