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
(2) |
4
(2) |
5
|
6
|
|
7
(1) |
8
|
9
|
10
(1) |
11
(1) |
12
(2) |
13
|
|
14
|
15
|
16
(1) |
17
(1) |
18
|
19
|
20
(13) |
|
21
(3) |
22
(2) |
23
(6) |
24
(2) |
25
(5) |
26
(2) |
27
(2) |
|
28
(6) |
|
|
|
|
|
|
|
From: Paul F. <pj...@wa...> - 2021-02-23 21:17:42
|
Ouf, as we say over here. 12 patches uploaded to bugzilla. Time for bed. A+ Paul |
|
From: Paul F. <pj...@wa...> - 2021-02-23 21:15:45
|
> On 23 Feb 2021, at 21:44, Bart Van Assche <bva...@ac...> wrote:
>
>
> Hi Paul,
>
> Thanks for the feedback. Please take a look at commit caf05d5ca941
> ("drd/tests/swapcontext: Improve the portability of this test further").
>
> Bart.
Hi Bart
I’ll take a look tomorrow.
A+
Paul
|
|
From: Bart V. A. <bva...@ac...> - 2021-02-23 20:44:27
|
On 2/22/21 2:35 AM, Paul Floyd wrote:
> This testcase is now closer to building, but still not quite there. The
> problem is now with 'pthread_getattr_np', which, as the manpage says
>
>
> CONFORMING TO
>
> This function is a nonstandard GNU extension; hence the suffix
> "_np" (nonportable) in the name.
>
>
> This function exists on Solaris 11.4 but not on the other platforms that
> I have tested (macOS 10.7.5, Solaris 11.3, FreeBSD 12.2 and FreeBSD 14)
>
> If I use pthread_attr_init() instead, then the testcase seems to run OK,
> but yet again there is a problem. On FreeBSD it seems as though the
> pthread_kill() with SIGINT is causing the application to return a SIGINT
> status, which stops tests/vg_regtest because of
>
> # if $keepunfiltered, copies $1 to $1.unfiltered.out
> # renames $0 tp $1
> sub filtered_rename($$)
> {
> if ($keepunfiltered == 1) {
> mysystem("cp $_[1] $_[1].unfiltered.out");
> }
> rename ($_[0], $_[1]);
> }
>
> So finally with the diff below, and an update to the expected, it should
> pass.
>
> However, I'm not sure that using pthread_attr_init rather than
> pthread_getattr_np changes the behaviour. I suspect not, since in this
> case the global default stack size should be the same as the individual
> stack sizes. Changing the stacks and sizes gets done after the call to
> __valgrind_register_current_stack
Hi Paul,
Thanks for the feedback. Please take a look at commit caf05d5ca941
("drd/tests/swapcontext: Improve the portability of this test further").
Bart.
|
|
From: Bart V. A. <bva...@so...> - 2021-02-23 20:43:13
|
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=caf05d5ca94155c793641b9f32bb6be8b9a77448 commit caf05d5ca94155c793641b9f32bb6be8b9a77448 Author: Bart Van Assche <bva...@ac...> Date: Tue Feb 23 11:49:14 2021 -0800 drd/tests/swapcontext: Improve the portability of this test further - Remove the VALGRIND_STACK_REGISTER() invocation for the initial thread stack since it is superfluous. Remove the pthread_attr_getstack() call that became superfluous by this change. - Change SIGINT into SIGALRM for FreeBSD since pthread_kill(..., SIGINT) causes the application to return a SIGINT status. - Reduce the stack size of the threads created by this test. Diff: --- drd/tests/swapcontext.c | 51 ++++++++++++++++------------------------ drd/tests/swapcontext.stderr.exp | 2 +- 2 files changed, 21 insertions(+), 32 deletions(-) diff --git a/drd/tests/swapcontext.c b/drd/tests/swapcontext.c index e692559b06..6257b6d930 100644 --- a/drd/tests/swapcontext.c +++ b/drd/tests/swapcontext.c @@ -2,7 +2,8 @@ #define _GNU_SOURCE -#include <poll.h> +#include <assert.h> +#include <limits.h> #include <pthread.h> #include <signal.h> #include <stdlib.h> @@ -13,6 +14,8 @@ #include <unistd.h> #include "valgrind.h" +#define STACKSIZE (PTHREAD_STACK_MIN + 4096) + typedef struct thread_local { ucontext_t uc[3]; size_t nrsw; @@ -21,7 +24,6 @@ typedef struct thread_local { static void f(void *data, int n) { enum { NR_SWITCHES = 200000 }; - struct pollfd pfd; thread_local_t *tlocal = data; while (1) { @@ -33,29 +35,9 @@ static void f(void *data, int n) } } -void __valgrind_register_current_stack(void) -{ - pthread_attr_t attr; - size_t stacksize; - void *stack; - - if (pthread_getattr_np(pthread_self(), &attr) != 0) - abort(); - - if (pthread_attr_getstack(&attr, &stack, &stacksize) != 0) - abort(); - - VALGRIND_STACK_REGISTER(stack, stack + stacksize); -} - -#define STACKSIZE 8192 - void *worker(void *data) { thread_local_t *tlocal = data; - struct itimerspec it; - - __valgrind_register_current_stack(); if (getcontext(&(tlocal->uc[1])) < 0) abort(); @@ -66,17 +48,17 @@ void *worker(void *data) tlocal->uc[1].uc_stack.ss_sp = malloc(STACKSIZE); tlocal->uc[1].uc_stack.ss_size = STACKSIZE; makecontext(&tlocal->uc[1], (void (*)(void))f, 2, tlocal, 1); - VALGRIND_STACK_REGISTER(tlocal->uc[1].uc_stack.ss_sp, - tlocal->uc[1].uc_stack.ss_sp + - tlocal->uc[1].uc_stack.ss_size); + (void)VALGRIND_STACK_REGISTER(tlocal->uc[1].uc_stack.ss_sp, + tlocal->uc[1].uc_stack.ss_sp + + tlocal->uc[1].uc_stack.ss_size); tlocal->uc[2].uc_link = &tlocal->uc[0]; tlocal->uc[2].uc_stack.ss_sp = malloc(STACKSIZE); tlocal->uc[2].uc_stack.ss_size = STACKSIZE; makecontext(&tlocal->uc[2], (void (*)(void))f, 2, tlocal, 2); - VALGRIND_STACK_REGISTER(tlocal->uc[2].uc_stack.ss_sp, - tlocal->uc[2].uc_stack.ss_sp + - tlocal->uc[2].uc_stack.ss_size); + (void)VALGRIND_STACK_REGISTER(tlocal->uc[2].uc_stack.ss_sp, + tlocal->uc[2].uc_stack.ss_sp + + tlocal->uc[2].uc_stack.ss_size); swapcontext(&tlocal->uc[0], &tlocal->uc[1]); return NULL; @@ -87,17 +69,24 @@ int main(int argc, char *argv[]) enum { NR = 32 }; thread_local_t tlocal[NR]; pthread_t thread[NR]; - int i; + pthread_attr_t attr; + int i, res; memset(tlocal, 0, sizeof(tlocal)); + pthread_attr_init(&attr); + res = pthread_attr_setstacksize(&attr, STACKSIZE); + assert(res == 0); + for (i = 0; i < NR; i++) - pthread_create(&thread[i], NULL, worker, &tlocal[i]); + pthread_create(&thread[i], &attr, worker, &tlocal[i]); + + pthread_attr_destroy(&attr); // Wait until the threads have been created. sleep(1); for (i = 0; i < NR; i++) - pthread_kill(thread[i], SIGINT); + pthread_kill(thread[i], SIGALRM); for (i = 0; i < NR; i++) pthread_join(thread[i], NULL); diff --git a/drd/tests/swapcontext.stderr.exp b/drd/tests/swapcontext.stderr.exp index 1ebc73a407..fcb5d5ed47 100644 --- a/drd/tests/swapcontext.stderr.exp +++ b/drd/tests/swapcontext.stderr.exp @@ -1,6 +1,6 @@ -Process terminating with default action of signal 2 (SIGINT) +Process terminating with default action of signal 14 (SIGALRM) at 0x........: swapcontext (in /...libc...) by 0x........: f (swapcontext.c:?) |
|
From: Mark W. <ma...@so...> - 2021-02-23 15:23:29
|
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=ea98cccb4d50a8740708507c4c72cfb1e6c88e38 commit ea98cccb4d50a8740708507c4c72cfb1e6c88e38 Author: Mark Wielaard <ma...@kl...> Date: Tue Feb 23 16:19:26 2021 +0100 Filter out unsupported instructions from HWCAP2 on powerpc. Valgrind currently doesn't support the DARN random number instruction and the SCV syscall instruction. Filter them out of HWCAP2 so glibc and applications don't try to use them when running under valgrind. Also suppress printing a log message for scv instructions in the instruction stream. Reported by: Florian Weimer <fw...@re...> DARN bug: https://bugs.kde.org/show_bug.cgi?id=411189 SCV bug: https://bugs.kde.org/show_bug.cgi?id=431157 Diff: --- VEX/priv/guest_ppc_toIR.c | 5 +++-- coregrind/m_initimg/initimg-linux.c | 25 +++++++++++++++++++++++++ 2 files changed, 28 insertions(+), 2 deletions(-) diff --git a/VEX/priv/guest_ppc_toIR.c b/VEX/priv/guest_ppc_toIR.c index 880a47ebb2..308c2fe762 100644 --- a/VEX/priv/guest_ppc_toIR.c +++ b/VEX/priv/guest_ppc_toIR.c @@ -10878,8 +10878,9 @@ static Bool dis_syslink ( UInt prefix, UInt theInstr, /* There is no prefixed version of these instructions. */ PREFIX_CHECK - if (theInstr != 0x44000002) { - vex_printf("dis_syslink(ppc)(theInstr)\n"); + if (theInstr != 0x44000002) { // sc + if (theInstr != 0x44000001) // scv + vex_printf("dis_syslink(ppc)(theInstr)\n"); return False; } diff --git a/coregrind/m_initimg/initimg-linux.c b/coregrind/m_initimg/initimg-linux.c index ba84fa6e9d..cb7902446a 100644 --- a/coregrind/m_initimg/initimg-linux.c +++ b/coregrind/m_initimg/initimg-linux.c @@ -750,6 +750,14 @@ Addr setup_client_stack( void* init_sp, PPC_FEATURE2_HAS_ISEL 0x08000000 PPC_FEATURE2_HAS_TAR 0x04000000 PPC_FEATURE2_HAS_VCRYPTO 0x02000000 + PPC_FEATURE2_HTM_NOSC 0x01000000 + PPC_FEATURE2_ARCH_3_00 0x00800000 + PPC_FEATURE2_HAS_IEEE128 0x00400000 + PPC_FEATURE2_DARN 0x00200000 + PPC_FEATURE2_SCV 0x00100000 + PPC_FEATURE2_HTM_NO_SUSPEND 0x00080000 + PPC_FEATURE2_ARCH_3_1 0x00040000 + PPC_FEATURE2_MMA 0x00020000 */ auxv_2_07 = (auxv->u.a_val & 0x80000000ULL) == 0x80000000ULL; hw_caps_2_07 = (vex_archinfo->hwcaps & VEX_HWCAPS_PPC64_ISA2_07) @@ -797,6 +805,23 @@ Addr setup_client_stack( void* init_sp, * matches the setting in VEX HWCAPS. */ vg_assert(auxv_3_1 == hw_caps_3_1); + + /* Mask unrecognized HWCAP bits. Only keep the bits that have + * explicit support in VEX. Filter out HTM bits since the + * transaction begin instruction (tbegin) is always failed in + * Valgrind causing the code to execute the failure path. + * Also filter out the DARN random number (bug #411189). + * And the SCV syscall (bug #431157). + */ + auxv->u.a_val &= (0x80000000ULL /* ARCH_2_07 */ + | 0x20000000ULL /* DSCR */ + | 0x10000000ULL /* EBB */ + | 0x08000000ULL /* ISEL */ + | 0x04000000ULL /* TAR */ + | 0x04000000ULL /* VEC_CRYPTO */ + | 0x00800000ULL /* ARCH_3_00 */ + | 0x00400000ULL /* HAS_IEEE128 */ + | 0x00040000ULL); /* ARCH_3_1 */ } break; |
|
From: Mark W. <ma...@so...> - 2021-02-23 11:01:33
|
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=9337b44f934b792eff0d4b354f85e41ad13d3791 commit 9337b44f934b792eff0d4b354f85e41ad13d3791 Author: Mark Wielaard <ma...@kl...> Date: Tue Feb 23 11:50:13 2021 +0100 gdbserver_tests/hgtls.vgtest: Make sure gdb is installed before running The other gdbserver_tests that need to run gdb make sure it is actually available before trying to run it, otherwise the test is skipped. Do the same to hgtls.vgtest by adding test -e gdb to the prereq. Diff: --- gdbserver_tests/hgtls.vgtest | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gdbserver_tests/hgtls.vgtest b/gdbserver_tests/hgtls.vgtest index e455013a26..44bc4e1ec1 100644 --- a/gdbserver_tests/hgtls.vgtest +++ b/gdbserver_tests/hgtls.vgtest @@ -1,7 +1,7 @@ # test tls addresses prog: ../none/tests/tls vgopts: --tool=helgrind --vgdb=yes --vgdb-error=0 --vgdb-prefix=./vgdb-prefix-hgtls -q -prereq: test -e ../none/tests/tls && grep '^#define HAVE_TLS 1' ../config.h > /dev/null && grep '^#define HAVE_DLINFO_RTLD_DI_TLS_MODID 1' ../config.h > /dev/null +prereq: test -e gdb && test -e ../none/tests/tls && grep '^#define HAVE_TLS 1' ../config.h > /dev/null && grep '^#define HAVE_DLINFO_RTLD_DI_TLS_MODID 1' ../config.h > /dev/null stdout_filter: filter_make_empty stderr_filter: filter_make_empty progB: gdb |