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
|
2
|
|
3
|
4
|
5
|
6
|
7
(1) |
8
|
9
|
|
10
|
11
|
12
(6) |
13
(4) |
14
(3) |
15
(1) |
16
(2) |
|
17
(7) |
18
(1) |
19
(5) |
20
(3) |
21
(1) |
22
(5) |
23
|
|
24
(1) |
25
(3) |
26
(2) |
27
(1) |
28
(2) |
29
(1) |
30
|
|
31
|
|
|
|
|
|
|
|
From: Matthias S. <zz...@ge...> - 2016-01-13 09:51:10
|
Am 13.01.2016 um 08:18 schrieb Florian Krohm: > Given that we already have some support for MSC_VER in valgrind.h it > would be good to sort this out. But that file is already quite messy > with all the ifdeffery that is going on in there so we should be careful > adding more.. > The attached patch is a draft version of a different way to cleanup VALGRIND_PRINTF. Regards Matthias |
|
From: Matthias S. <zz...@ge...> - 2016-01-13 08:12:00
|
Hi Florian!
Ok, the line numbers relate to an older copy of valgrind.h but the issue
still exists. The first error points exactly on the definition of
VALGRIND_PRINTF (line 6759 in the current version of the file, see
below, the second on the equivalent definition of
VALGRIND_PRINTF_BACKTRACE).
The format argument is not used if NVALGRIND is defined (obvious).
For gcc the warning is disabled by adding a declaration containing the
attribute __unused__ for the format argument.
But for sure this does not apply to Visual Studio compiler.
To silence this warning one either needs to not assign a name to the
argument (only possible in c++) or only have the "..." argument (for a
function or macro). But I guess for better error checking the argument
should stay for gcc (to make use of the attribute format...).
Here is the code of VALGRIND_PRINTF of the latest valgrind.h file:
> 6749: #if defined(__GNUC__) || defined(__INTEL_COMPILER) && !defined(_MSC_VER)
> 6750: /* Modern GCC will optimize the static routine out if unused,
> 6751: and unused attribute will shut down warnings about it. */
> 6752: static int VALGRIND_PRINTF(const char *format, ...)
> 6753: __attribute__((format(__printf__, 1, 2), __unused__));
> 6754: #endif
> 6755: static int
> 6756: #if defined(_MSC_VER)
> 6757: __inline
> 6758: #endif
> 6759: VALGRIND_PRINTF(const char *format, ...)
> 6760: {
> 6761: #if defined(NVALGRIND)
> 6762: return 0;
> 6763: #else /* NVALGRIND */
> 6764: #if defined(_MSC_VER) || defined(__MINGW64__)
> 6765: uintptr_t _qzz_res;
> 6766: #else
> 6767: unsigned long _qzz_res;
> 6768: #endif
> 6769: va_list vargs;
> 6770: va_start(vargs, format);
> 6771: #if defined(_MSC_VER) || defined(__MINGW64__)
> 6772: _qzz_res = VALGRIND_DO_CLIENT_REQUEST_EXPR(0,
> 6773: VG_USERREQ__PRINTF_VALIST_BY_REF,
> 6774: (uintptr_t)format,
> 6775: (uintptr_t)&vargs,
> 6776: 0, 0, 0);
> 6777: #else
> 6778: _qzz_res = VALGRIND_DO_CLIENT_REQUEST_EXPR(0,
> 6779: VG_USERREQ__PRINTF_VALIST_BY_REF,
> 6780: (unsigned long)format,
> 6781: (unsigned long)&vargs,
> 6782: 0, 0, 0);
> 6783: #endif
> 6784: va_end(vargs);
> 6785: return (int)_qzz_res;
> 6786: #endif /* NVALGRIND */
> 6787: }
Regards
Matthias
Am 13.01.2016 um 08:18 schrieb Florian Krohm:
> Given that we already have some support for MSC_VER in valgrind.h it
> would be good to sort this out. But that file is already quite messy
> with all the ifdeffery that is going on in there so we should be careful
> adding more..
>
> I do not understand what the issue really is. The lines mentioned in the
> errors point into macro definitions and offer no clue. They are
> unrelated to VALGRIND_PRINTF. Can you provide some context? Assume that
> none of us has access to that particular compiler to play around with.
>
> Florian
>
>
> On 12.01.2016 23:14, Matthias Schwarzott wrote:
>> When compiling code that includes valgrind.h on visual studio I get
>> compiler errors about VALGRIND_PRINTF:
>> 1>include\Valgrind/valgrind.h(4493) : error C2220: warning treated as
>> error - no 'object' file generated
>> 1>include\Valgrind/valgrind.h(4493) : warning C4100: 'format' :
>> unreferenced formal parameter
>> 1>include\Valgrind/valgrind.h(4531) : warning C4100: 'format' :
>> unreferenced formal parameter
>>
>> See https://bugs.kde.org/show_bug.cgi?id=356817
>>
>> There are multiple ways to fix this issue.
>> 1. Complicated one: extend the existing ifdef code with special
>> NVALGRIND function versions for gcc and MSVC, see
>> https://bugs.kde.org/attachment.cgi?id=96139
>>
>> 2. Simple one: have a function or macro with only "..." argument in the
>> NVALGRIND case that does nothing.
>>
>> --- a/include/valgrind.h
>> +++ b/include/valgrind.h
>> @@ -6746,6 +6746,18 @@ typedef
>> is the number of characters printed, excluding the "**<pid>** " part
>> at the
>> start and the backtrace (if present). */
>>
>> +#if defined(_MSC_VER) && defined(NVALGRIND)
>> +static int __inline VALGRIND_PRINTF(...)
>> +{
>> + return 0;
>> +}
>> +
>> +static int __inline VALGRIND_PRINTF_BACKTRACE(...)
>> +{
>> + return 0;
>> +}
>> +
>> +#else /* _MSC_VER && NVALGRIND */
>> #if defined(__GNUC__) || defined(__INTEL_COMPILER) && !defined(_MSC_VER)
>> /* Modern GCC will optimize the static routine out if unused,
>> and unused attribute will shut down warnings about it. */
>> @@ -6823,7 +6835,7 @@ VALGRIND_PRINTF_BACKTRACE(const char *format, ...)
>> return (int)_qzz_res;
>> #endif /* NVALGRIND */
>> }
>> -
>> +#endif /* _MSC_VER && NVALGRIND */
>>
>> /* These requests allow control to move from the simulated CPU to the
>> real CPU, calling an arbitary function.
>>
>>
>> What do you think about this?
>>
>> Regards
>> Matthias
>>
>> ------------------------------------------------------------------------------
>> Site24x7 APM Insight: Get Deep Visibility into Application Performance
>> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
>> Monitor end-to-end web transactions and take corrective actions now
>> Troubleshoot faster and improve end-user experience. Signup Now!
>> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
>> _______________________________________________
>> Valgrind-developers mailing list
>> Val...@li...
>> https://lists.sourceforge.net/lists/listinfo/valgrind-developers
>>
>
>
|
|
From: Florian K. <fl...@ei...> - 2016-01-13 07:43:52
|
Given that we already have some support for MSC_VER in valgrind.h it would be good to sort this out. But that file is already quite messy with all the ifdeffery that is going on in there so we should be careful adding more.. I do not understand what the issue really is. The lines mentioned in the errors point into macro definitions and offer no clue. They are unrelated to VALGRIND_PRINTF. Can you provide some context? Assume that none of us has access to that particular compiler to play around with. Florian On 12.01.2016 23:14, Matthias Schwarzott wrote: > When compiling code that includes valgrind.h on visual studio I get > compiler errors about VALGRIND_PRINTF: > 1>include\Valgrind/valgrind.h(4493) : error C2220: warning treated as > error - no 'object' file generated > 1>include\Valgrind/valgrind.h(4493) : warning C4100: 'format' : > unreferenced formal parameter > 1>include\Valgrind/valgrind.h(4531) : warning C4100: 'format' : > unreferenced formal parameter > > See https://bugs.kde.org/show_bug.cgi?id=356817 > > There are multiple ways to fix this issue. > 1. Complicated one: extend the existing ifdef code with special > NVALGRIND function versions for gcc and MSVC, see > https://bugs.kde.org/attachment.cgi?id=96139 > > 2. Simple one: have a function or macro with only "..." argument in the > NVALGRIND case that does nothing. > > --- a/include/valgrind.h > +++ b/include/valgrind.h > @@ -6746,6 +6746,18 @@ typedef > is the number of characters printed, excluding the "**<pid>** " part > at the > start and the backtrace (if present). */ > > +#if defined(_MSC_VER) && defined(NVALGRIND) > +static int __inline VALGRIND_PRINTF(...) > +{ > + return 0; > +} > + > +static int __inline VALGRIND_PRINTF_BACKTRACE(...) > +{ > + return 0; > +} > + > +#else /* _MSC_VER && NVALGRIND */ > #if defined(__GNUC__) || defined(__INTEL_COMPILER) && !defined(_MSC_VER) > /* Modern GCC will optimize the static routine out if unused, > and unused attribute will shut down warnings about it. */ > @@ -6823,7 +6835,7 @@ VALGRIND_PRINTF_BACKTRACE(const char *format, ...) > return (int)_qzz_res; > #endif /* NVALGRIND */ > } > - > +#endif /* _MSC_VER && NVALGRIND */ > > /* These requests allow control to move from the simulated CPU to the > real CPU, calling an arbitary function. > > > What do you think about this? > > Regards > Matthias > > ------------------------------------------------------------------------------ > Site24x7 APM Insight: Get Deep Visibility into Application Performance > APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month > Monitor end-to-end web transactions and take corrective actions now > Troubleshoot faster and improve end-user experience. Signup Now! > http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140 > _______________________________________________ > Valgrind-developers mailing list > Val...@li... > https://lists.sourceforge.net/lists/listinfo/valgrind-developers > |
|
From: <sv...@va...> - 2016-01-13 05:37:45
|
Author: iraisr
Date: Wed Jan 13 05:37:36 2016
New Revision: 15758
Log:
Fix expected output of helgrind/tests/tc20_verifywrap on Solaris.
n-i-bz
Modified:
trunk/NEWS
trunk/helgrind/tests/tc20_verifywrap.stderr.exp-solaris
Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Wed Jan 13 05:37:36 2016
@@ -57,8 +57,8 @@
355455 stderr.exp of test cases wrapmalloc and wrapmallocstatic overconstrained
355454 do not intercept malloc related symbols from the runtime linker
356044 Dwarf line info reader misinterprets is_stmt register
-357887 Fix a file handle leak. VG_(fclose) did not close the file
357871 pthread_spin_destroy not properly wrapped
+357887 Fix a file handle leak. VG_(fclose) did not close the file
n-i-bz Fix incorrect (or infinite loop) unwind on RHEL7 x86 32 bits
n-i-bz massif --pages-as-heap=yes does not report peak caused by mmap+munmap
Modified: trunk/helgrind/tests/tc20_verifywrap.stderr.exp-solaris
==============================================================================
--- trunk/helgrind/tests/tc20_verifywrap.stderr.exp-solaris (original)
+++ trunk/helgrind/tests/tc20_verifywrap.stderr.exp-solaris Wed Jan 13 05:37:36 2016
@@ -231,7 +231,6 @@
by 0x........: main (tc20_verifywrap.c:232)
-
---------------- pthread_spin_* ----------------
|