Skip to content

Commit cf6b720

Browse files
author
Dan Terpstra
committed
Update Change Log and release notes for 421 release.
1 parent 25a4e6f commit cf6b720

File tree

2 files changed

+88
-1
lines changed

2 files changed

+88
-1
lines changed

ChangeLogP421.txt

+70
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,73 @@
1+
2012-02-13
2+
3+
* src/components/net/linux-net.c: Repairing more coverity warnings.
4+
5+
2012-02-11
6+
7+
* src/windows-common.c: Missed an instance of CPUs yesterday.
8+
9+
* src/: papi_internal.c, threads.c: This changes fixes two race
10+
conditions that are probably the cause of the pthrtough
11+
double-free error.
12+
13+
When freeing a thread, we remove and free all eventsets belonging
14+
to that thread. This could race with the thread itself removing
15+
the evenset, causing some ESI fields to be freed twice.
16+
17+
The problem was found by using the Valgrind 3.8 Helgrind tool
18+
19+
valgrind --tool=helgrind --free-is-write=yes ctests/pthrtough
20+
21+
In order for Helgrind to work, I had to temporarily modify PAPI
22+
to use POSIX pthread mutexes for locking. Is there any reason we
23+
don't use these all the time?
24+
25+
2012-02-10
26+
27+
* src/utils/: avail.c, component.c, event_chooser.c,
28+
native_avail.c: ix one more case of "CPU's" in the print header
29+
code.
30+
31+
Also remove the extraneous The following correspond to fields
32+
in the PAPI_event_info_t structure. message
33+
34+
* src/: testlib/papi_test.h, testlib/test_utils.c,
35+
ctests/all_native_events.c, ctests/calibrate.c,
36+
ctests/code2name.c, ctests/hwinfo.c: Fix one more case of "CPU's"
37+
in the print header code.
38+
39+
Also remove the extraneous The following correspond to fields
40+
in the PAPI_event_info_t structure. message
41+
42+
* src/buildbot_configure_with_components.sh: take infiniband out of
43+
the buildbot test.
44+
45+
* src/: x86_cache_info.c, components/coretemp/linux-coretemp.c,
46+
components/lmsensors/linux-lmsensors.c,
47+
components/lustre/linux-lustre.c, components/net/linux-net.c,
48+
utils/event_chooser.c: Fix coverity errors reported by Will
49+
Cohen.
50+
51+
* src/: aix.c, any-proc-null.c, linux-common.c, papi.c, papi.h,
52+
papivi.h, solaris-niagara2.c, solaris-ultra.c,
53+
ctests/clockres_pthreads.c: Address Redhat bug 785975. The
54+
plural of CPU appears to be CPUs
55+
56+
* src/Makefile.inc: Patch to cleanup dependencies, allowing for
57+
parallel makes. Patch due to Will Cohen from redhat
58+
59+
2012-02-09
60+
61+
* src/buildbot_configure_with_components.sh: Add infiniband and mx
62+
component to buildbot component tests.
63+
64+
* src/components/net/tests/: net_values_by_code.c,
65+
net_values_by_name.c: Apply patch suggested by Will Cohen to
66+
check for system return values.
67+
68+
* src/components/lmsensors/linux-lmsensors.h: Added missing string
69+
header
70+
171
2012-02-08
272

373
* man/... : update man pages one more time for 4.2.1

RELEASENOTES.txt

+18-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ For details on installing PAPI on your machine, consult the INSTALL.txt file
55
in this directory.
66

77
===============================================================================
8-
PAPI 4.2.1 RELEASE NOTES 08 Feb 2012
8+
PAPI 4.2.1 RELEASE NOTES 13 Feb 2012
99
===============================================================================
1010
For specific and detailed information on changes made in this release, grep
1111
the ChangeLogP421.txt file for keywords of interest or go directly to the PAPI
@@ -59,6 +59,23 @@ Bug Fixes:
5959
* src/components/lmsensors/: Rules.lmsensors, configure.in: Fixed
6060
configure error message and rules link error for shared object
6161
linking. Thanks Will Cohen.
62+
* src/components/lmsensors/linux-lmsensors.h: Added missing string
63+
header
64+
* src/components/net/tests/: net_values_by_code.c,
65+
net_values_by_name.c: Apply patch suggested by Will Cohen to
66+
check for system return values.
67+
* src/Makefile.inc: Patch to cleanup dependencies, allowing for
68+
parallel makes. Patch due to Will Cohen from redhat
69+
* src/: papi_internal.c, threads.c: Fix two race
70+
conditions that are probably the cause of the pthrtough
71+
double-free error.
72+
When freeing a thread, we remove and free all eventsets belonging
73+
to that thread. This could race with the thread itself removing
74+
the evenset, causing some ESI fields to be freed twice.
75+
The problem was found by using the Valgrind 3.8 Helgrind tool
76+
valgrind --tool=helgrind --free-is-write=yes ctests/pthrtough
77+
In order for Helgrind to work, I had to temporarily modify PAPI
78+
to use POSIX pthread mutexes for locking.
6279

6380
Enhancements:
6481
-------------

0 commit comments

Comments
 (0)