Bug 433479 - FreeBSD support, part 7
Summary: FreeBSD support, part 7
Status: RESOLVED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: helgrind (show other bugs)
Version: unspecified
Platform: Other FreeBSD
: NOR normal
Target Milestone: ---
Assignee: Julian Seward
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-23 11:23 UTC by Paul Floyd
Modified: 2021-10-06 21:35 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Helgrind tests patch (71.29 KB, patch)
2021-02-23 11:23 UTC, Paul Floyd
Details
diff in helgrind/tests (71.58 KB, patch)
2021-09-23 19:26 UTC, Paul Floyd
Details
freebsd7 (72.33 KB, text/plain)
2021-09-30 13:26 UTC, Mark Wielaard
Details
freebsd7 (74.78 KB, text/plain)
2021-09-30 13:39 UTC, Mark Wielaard
Details
test directory (11.40 KB, patch)
2021-10-05 20:51 UTC, Paul Floyd
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Floyd 2021-02-23 11:23:01 UTC
Created attachment 136080 [details]
Helgrind tests patch

Helgrind tests changes

Makefile.am - new expecteds and filters
bar_bad - bogus pthread barrier was causing a crash, so made that conditional, also requires a new expected
tc09 - as above but with a mutex
tc23 - as above but with condition signal
filter_freebsd - adds "@*" to thread function names, reduces requirements for more expecteds
filter_stderr_freebsd - as above
filter_stderr - more filters, needs work ro make sed a configure variable
tc07 and tc08 and tc11 - add FreeBSD platform macros
Comment 1 Paul Floyd 2021-09-23 19:26:28 UTC
Created attachment 141838 [details]
diff in helgrind/tests
Comment 2 Mark Wielaard 2021-09-30 13:19:57 UTC
As far as I can see this looks good (I cannot really comment on th freebsd specific filters and exp files). But the modifications to the tests look OK.

This is missing the .gitignore and CONFIG configure.ac entries for helgrind/tests/filter_stderr

With that it builds and all tests under helgrind/tests PASS for me on fedora 34 x86_64. It also makes all tests under drd/tests PASS. 

I recommend merging this patch with the freebsd5.patch for the drd tests, since they share some source files.
Comment 3 Mark Wielaard 2021-09-30 13:26:33 UTC
Created attachment 142030 [details]
freebsd7

This is the patch as I tested it (only changes are the additions to .gitignore and configure.ac)
Comment 4 Mark Wielaard 2021-09-30 13:39:28 UTC
Created attachment 142031 [details]
freebsd7

Sorry, that patch didn't include helgrind/tests/filter_stderr.in
This one does.
Comment 5 Paul Floyd 2021-10-05 12:04:22 UTC
I've merged this with patch 5 (drd tests) in my working repo. Aiming to finish all by tonight (2021-20-05).
Comment 6 Paul Floyd 2021-10-05 20:51:38 UTC
Created attachment 142186 [details]
test directory

Hijacking this from hg tests to be just the test directory
Comment 7 Paul Floyd 2021-10-06 21:35:49 UTC
Code committed as
commit b128b20e66adbf00e87cf43e00e07ffe04f2ba8b