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
(8) |
2
(2) |
3
(1) |
4
(2) |
|
5
(1) |
6
(4) |
7
(6) |
8
(5) |
9
(3) |
10
(5) |
11
(1) |
|
12
(6) |
13
(4) |
14
(1) |
15
(4) |
16
(1) |
17
|
18
|
|
19
|
20
|
21
(2) |
22
(28) |
23
(17) |
24
(6) |
25
(4) |
|
26
(2) |
27
(2) |
28
|
29
(5) |
30
(8) |
31
(14) |
|
|
From: Ivo R. <ivo...@gm...> - 2015-07-31 23:13:16
|
I have come recently to very strange piece of code in function
VG_(load_ELF)(), when it iterates over program headers.
I think that all switch cases after PT_INTERP are misplaced.
That is, the closing bracket for case PT_INTERP is located
*after* the last "default" case.
So what happens with cases PT_GNU_STACK, PT_SUNWSTACK
and default is a mystery for me.
I dug into file history across renames
(coregrind/ume.c -> coregrind/m_ume.c -> coregrind/m_ume/elf.c)
and found that the default case was possibly added at a wrong
position at r2659. At r2643 everything looked ok.
r2643:
switch(ph->p_type) {
...
case PT_INTERP: {
...
for(j = 0; j < interp->e.e_phnum; j++) {
...
}
break;
} // for case PT_INTERP
} // for switch
r2659:
switch(ph->p_type) {
...
case PT_INTERP: {
...
for(j = 0; j < interp->e.e_phnum; j++) {
...
}
break;
default: <---- misplaced
// do nothing
break;
} // for case PT_INTERP
} // for switch
Do you agree with my findings?
What I don't understand is why compiler does not flag this as an error?
I.
|
|
From: <sv...@va...> - 2015-07-31 18:55:04
|
Author: iraisr
Date: Fri Jul 31 19:54:57 2015
New Revision: 15465
Log:
Update list of Solaris specific ignored files.
Modified:
trunk/auxprogs/ (props changed)
trunk/cachegrind/ (props changed)
trunk/callgrind/ (props changed)
trunk/drd/ (props changed)
trunk/exp-dhat/ (props changed)
trunk/exp-sgcheck/ (props changed)
trunk/gdbserver_tests/solaris/ (props changed)
trunk/helgrind/ (props changed)
trunk/lackey/ (props changed)
trunk/massif/ (props changed)
trunk/memcheck/ (props changed)
trunk/memcheck/tests/amd64-solaris/ (props changed)
trunk/memcheck/tests/solaris/ (props changed)
trunk/memcheck/tests/x86-solaris/ (props changed)
trunk/none/ (props changed)
trunk/none/tests/amd64-solaris/ (props changed)
trunk/none/tests/solaris/ (props changed)
trunk/none/tests/x86-solaris/ (props changed)
|
|
From: <sv...@va...> - 2015-07-31 16:12:56
|
Author: iraisr
Date: Fri Jul 31 17:12:44 2015
New Revision: 15464
Log:
Use handwritten memcheck assembly helpers on x86/Solaris in addition to {arm,x86}-linux.
Fixes BZ #350813.
Modified:
trunk/NEWS
trunk/memcheck/mc_main.c
Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Fri Jul 31 17:12:44 2015
@@ -271,6 +271,7 @@
349941 di_notify_mmap might create wrong start/size DebugInfoMapping
350809 Fix none/tests/async-sigs for Solaris
350811 Remove reference to --db-attach which has been removed.
+350813 Use handwritten memcheck assembly helpers on x86/Solaris in addition to {arm,x86}-linux
n-i-bz Provide implementations of certain compiler builtins to support
compilers who may not provide those
n-i-bz Old STABS code is still being compiled, but never used. Remove it.
Modified: trunk/memcheck/mc_main.c
==============================================================================
--- trunk/memcheck/mc_main.c (original)
+++ trunk/memcheck/mc_main.c Fri Jul 31 17:12:44 2015
@@ -4587,7 +4587,7 @@
);
#elif ENABLE_ASSEMBLY_HELPERS && defined(PERF_FAST_LOADV) \
- && defined(VGP_x86_linux)
+ && (defined(VGP_x86_linux) || defined(VGP_x86_solaris))
__asm__(
".text\n"
".align 16\n"
@@ -4622,7 +4622,7 @@
);
#else
-// Generic for all platforms except {arm32,x86}-linux
+// Generic for all platforms except {arm32,x86}-linux and x86-solaris
VG_REGPARM(1) ULong MC_(helperc_LOADV64le) ( Addr a )
{
return mc_LOADV64(a, False);
@@ -4785,7 +4785,7 @@
);
#elif ENABLE_ASSEMBLY_HELPERS && defined(PERF_FAST_LOADV) \
- && defined(VGP_x86_linux)
+ && (defined(VGP_x86_linux) || defined(VGP_x86_solaris))
__asm__(
".text\n"
".align 16\n"
@@ -4818,7 +4818,7 @@
);
#else
-// Generic for all platforms except {arm32,x86}-linux
+// Generic for all platforms except {arm32,x86}-linux and x86-solaris
VG_REGPARM(1) UWord MC_(helperc_LOADV32le) ( Addr a )
{
return mc_LOADV32(a, False);
@@ -4998,7 +4998,7 @@
);
#elif ENABLE_ASSEMBLY_HELPERS && defined(PERF_FAST_LOADV) \
- && defined(VGP_x86_linux)
+ && (defined(VGP_x86_linux) || defined(VGP_x86_solaris))
__asm__(
".text\n"
".align 16\n"
@@ -5043,7 +5043,7 @@
);
#else
-// Generic for all platforms except {arm32,x86}-linux
+// Generic for all platforms except {arm32,x86}-linux and x86-solaris
VG_REGPARM(1) UWord MC_(helperc_LOADV16le) ( Addr a )
{
return mc_LOADV16(a, False);
@@ -5192,7 +5192,7 @@
/* Non-generic assembly for x86-linux */
#elif ENABLE_ASSEMBLY_HELPERS && defined(PERF_FAST_LOADV) \
- && defined(VGP_x86_linux)
+ && (defined(VGP_x86_linux) || defined(VGP_x86_solaris))
__asm__(
".text\n"
".align 16\n"
@@ -5234,7 +5234,7 @@
);
#else
-// Generic for all platforms except {arm32,x86}-linux
+// Generic for all platforms except {arm32,x86}-linux and x86-solaris
VG_REGPARM(1)
UWord MC_(helperc_LOADV8) ( Addr a )
{
|
|
From: Julian S. <js...@ac...> - 2015-07-31 13:59:43
|
> I did not stress-test the range-analysis but we do something like the > above quite a bit in the insn decoders. If the DIP macros contain some kind of error then we are going to get a zillion complaints from gcc, as you observe, since there's essentially one DIP usage per instruction-decoding clause. J |
|
From: Florian K. <fl...@ei...> - 2015-07-31 13:53:05
|
On 31.07.2015 15:24, John Reiser wrote:
>> The option -Wformat-signedness should do what is needed here.
>
>> Did I say *many* places? A build on x86-64 produces 5131 warnings.
>> Good grief!
>
> About as many as running memcheck for the first time on a program of similar size ;-)
:)
To valgrind's rescue I should say that this was a build with primary and
secondary platform. So the actual number is about half. Which is, like,
nothing :)
I couldn't resist and looked at a few of those warnings. Nice stuff:
unsigned char c;
...
printf("%u\n", c);
No warning here. Good !
I did not stress-test the range-analysis but we do something like the
above quite a bit in the insn decoders.
Florian
|
|
From: John R. <jr...@bi...> - 2015-07-31 13:24:09
|
> The option -Wformat-signedness should do what is needed here. > Did I say *many* places? A build on x86-64 produces 5131 warnings. > Good grief! About as many as running memcheck for the first time on a program of similar size ;-) |
|
From: <sv...@va...> - 2015-07-31 13:06:24
|
Author: rhyskidd
Date: Fri Jul 31 14:06:14 2015
New Revision: 15463
Log:
Properly guard none/tests/amd64/avx-1. This resolves one failing regression test on OS X with modern hardware. n-i-bz.
Modified:
trunk/none/tests/amd64/avx-1.vgtest
Modified: trunk/none/tests/amd64/avx-1.vgtest
==============================================================================
--- trunk/none/tests/amd64/avx-1.vgtest (original)
+++ trunk/none/tests/amd64/avx-1.vgtest Fri Jul 31 14:06:14 2015
@@ -1,3 +1,3 @@
prog: avx-1
-prereq: ../../../tests/x86_amd64_features amd64-avx
+prereq: test -x avx-1 && ../../../tests/x86_amd64_features amd64-avx
vgopts: -q
|
|
From: Julian S. <js...@ac...> - 2015-07-31 09:43:47
|
On 31/07/15 11:29, Julian Seward wrote: > Yesterday a strange thing happened: as the list administrator, I received > around 80 email messages notifying of people unsubscribing from this list. Another 74 arrived just now. Some of them seem to be the same email addresses as the first group, though. J |
|
From: Julian S. <js...@ac...> - 2015-07-31 09:29:58
|
Hi, Yesterday a strange thing happened: as the list administrator, I received around 80 email messages notifying of people unsubscribing from this list. This strikes me as very strange. All of them are dated 30 July 2015, 11:49. It seems to me unlikely that 80 people decided to unsubscribe all at the same time. So I'm flagging it here just in case anybody has any insight as to what happened. Maybe the messages are fake -- but they contained plausible email addresses -- that is, at least some of the email addresses I recognised as being Valgrind-related. Maybe the mail system got hacked? Maybe Sourceforge mass-unsubscribed old or invalid email addresses following their recent downtime? I just don't know. J |
|
From: Florian K. <fl...@ei...> - 2015-07-31 07:36:09
|
On 31.07.2015 06:26, Matthias Schwarzott wrote: > Am 30.07.2015 um 23:46 schrieb Florian Krohm: >> On 23.07.2015 19:36, Matthias Schwarzott wrote: >>> An additional comment to memcheck profiling. >>> >>> The counter values should be printed unsigned. >> > Am 30.07.2015 um 23:46 schrieb Florian Krohm: >> How did you find this? There are *many* places in valgrind where an >> incorrect format is used to print unsigned values. rsa. > On 31.07.2015 06:26, Matthias Schwarzott wrote: > The option -Wformat-signedness should do what is needed here. > Yes, that option does the trick. Something like gcc 5.1 is needed. I use 4.8.4 be default so did not realise that option existed. Thanks for letting me know. Did I say *many* places? A build on x86-64 produces 5131 warnings. Good grief! Florian |
|
From: <sv...@va...> - 2015-07-31 06:58:23
|
Author: florian
Date: Fri Jul 31 07:58:16 2015
New Revision: 15462
Log:
Fix testcase such that it can be run under cron on Solaris.
The tescase depends on SIGHUP to be delivered but cron on Solaris
ignored the signal. So it needs to be enabled in child processes
after fork.
Patch by Ivo Raisr <iv...@iv...>. Fixes BZ #350809.
Modified:
trunk/NEWS
trunk/none/tests/async-sigs.c
Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Fri Jul 31 07:58:16 2015
@@ -269,6 +269,7 @@
349874 Fix typos in source code
349828 memcpy intercepts memmove causing src/dst overlap error (ppc64 ld.so)
349941 di_notify_mmap might create wrong start/size DebugInfoMapping
+350809 Fix none/tests/async-sigs for Solaris
350811 Remove reference to --db-attach which has been removed.
n-i-bz Provide implementations of certain compiler builtins to support
compilers who may not provide those
Modified: trunk/none/tests/async-sigs.c
==============================================================================
--- trunk/none/tests/async-sigs.c (original)
+++ trunk/none/tests/async-sigs.c Fri Jul 31 07:58:16 2015
@@ -22,6 +22,15 @@
{
}
+static void install_handler(int sig, void (*sig_handler)(int))
+{
+ struct sigaction sa;
+ sa.sa_handler = sig_handler;
+ sigemptyset(&sa.sa_mask);
+ sa.sa_flags = 0;
+ sigaction(sig, &sa, 0);
+}
+
/* Kill our child, but use a separate kill command. This is so that
it's running independently of Valgrind, and so is async with
respect to thread scheduling. */
@@ -87,11 +96,7 @@
// - otherwise, wait in client code (by spinning).
// The alarm() calls is so that if something breaks, we don't get stuck.
if (pid == 0) {
- struct sigaction sa;
- sa.sa_handler = handler;
- sigemptyset(&sa.sa_mask);
- sa.sa_flags = 0;
- sigaction(caughtsig, &sa, 0);
+ install_handler(caughtsig, handler);
alarm(10);
for (;;)
@@ -130,6 +135,9 @@
int main()
{
+ /* Restore default behaviour of SIGHUP when forked from cron. */
+ install_handler(SIGHUP, SIG_DFL);
+
test(/*non-blocked*/0, /* sync*/SIGSEGV, /* sync*/SIGBUS);
test(/*non-blocked*/0, /* sync*/SIGSEGV, /*async*/SIGHUP);
test(/*non-blocked*/0, /*async*/SIGUSR1, /* sync*/SIGBUS);
|
|
From: <sv...@va...> - 2015-07-31 06:42:26
|
Author: florian
Date: Fri Jul 31 07:42:19 2015
New Revision: 15461
Log:
Remove implicit reference to the --db-attach feature which was
removed in r15445. Patch by Ivo Raisr <iv...@iv...>.
Fixes BZ #350811.
Modified:
trunk/NEWS
trunk/README.solaris
Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Fri Jul 31 07:42:19 2015
@@ -269,6 +269,7 @@
349874 Fix typos in source code
349828 memcpy intercepts memmove causing src/dst overlap error (ppc64 ld.so)
349941 di_notify_mmap might create wrong start/size DebugInfoMapping
+350811 Remove reference to --db-attach which has been removed.
n-i-bz Provide implementations of certain compiler builtins to support
compilers who may not provide those
n-i-bz Old STABS code is still being compiled, but never used. Remove it.
Modified: trunk/README.solaris
==============================================================================
--- trunk/README.solaris (original)
+++ trunk/README.solaris Fri Jul 31 07:42:19 2015
@@ -112,9 +112,6 @@
- Performance comparison against other platforms.
- Prevent SIGPIPE when writing to a socket (coregrind/m_libcfile.c).
-- Implement support for attaching a debugger (coregrind/m_debugger.c). This
- is a low priority task because this feature is going to be removed in
- future versions of Valgrind in favor of using only gdbserver.
- Implement ticket locking for fair scheduling (--fair-sched=yes).
- Implement support in DRD and Helgrind tools for thr_join() with thread == 0.
- Add support for accessing thread-local variables via gdb (auxprogs/getoff.c).
|
|
From: Matthias S. <zz...@ge...> - 2015-07-31 04:26:32
|
Am 30.07.2015 um 23:46 schrieb Florian Krohm: > On 23.07.2015 19:36, Matthias Schwarzott wrote: >> An additional comment to memcheck profiling. >> >> The counter values should be printed unsigned. > > How did you find this? There are *many* places in valgrind where an > incorrect format is used to print unsigned values. It would be good to > clean that up. But neither gcc nor clang is of any help here even though > VG_(printf) has a printf-like attribute and we compile with -Wformat and > what not... > I googled for this and found this set of options for gcc: # -Wformat=2 # Enable -Wformat plus additional format checks. Currently equivalent to -Wformat -Wformat-nonliteral -Wformat-security -Wformat-y2k. # -Wformat-nonliteral # If -Wformat is specified, also warn if the format string is not a string literal and so cannot be checked, unless the format function takes its format arguments as a va_list. # -Wformat-security # If -Wformat is specified, also warn about uses of format functions that represent possible security problems. At present, this warns about calls to printf and scanf functions where the format string is not a string literal and there are no format arguments, as in printf (foo);. This may be a security hole if the format string came from untrusted input and contains ‘%n’. (This is currently a subset of what -Wformat-nonliteral warns about, but in future warnings may be added to -Wformat-security that are not included in -Wformat-nonliteral.) # -Wformat-signedness # If -Wformat is specified, also warn if the format string requires an unsigned argument and the argument is signed and vice versa. The option -Wformat-signedness should do what is needed here. Regards Matthias |
|
From: Matthias S. <zz...@ge...> - 2015-07-31 04:12:13
|
Am 30.07.2015 um 23:46 schrieb Florian Krohm:
> On 23.07.2015 19:36, Matthias Schwarzott wrote:
>> An additional comment to memcheck profiling.
>>
>> The counter values should be printed unsigned.
>
> How did you find this? There are *many* places in valgrind where an
> incorrect format is used to print unsigned values. It would be good to
> clean that up. But neither gcc nor clang is of any help here even though
> VG_(printf) has a printf-like attribute and we compile with -Wformat and
> what not...
Just as experiment I executed a small subset of the tests (around
2minutes) of an internal test-application (that originally has a
memcheck runtime of 4.5h).
Then I saw negative values being reported for these profiling counter
values.
In my case I am not even sure if there were wraparounds for some counters.
Regards
Matthias
>
> Florian
>
>> --- a/memcheck/mc_main.c
>> +++ b/memcheck/mc_main.c
>> @@ -6363,7 +6363,7 @@ static void done_prof_mem ( void )
>> }
>> if (MC_(event_ctr)[i] > 0) {
>> spaced = False;
>> - VG_(printf)( "prof mem event %3d: %9d %s\n",
>> + VG_(printf)( "prof mem event %3d: %9u %s\n",
>> i, MC_(event_ctr)[i],
>> MC_(event_ctr_name)[i]
>> ? MC_(event_ctr_name)[i] : "unnamed");
>>
>
>
|