Bug 368461 - mmapunmap test fails on ppc64
Summary: mmapunmap test fails on ppc64
Status: CLOSED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: general (show other bugs)
Version: 3.12 SVN
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Julian Seward
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-08 16:34 UTC by Carl Love
Modified: 2016-09-14 16:20 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
fix for mmapunmap test on ppc64 (2.28 KB, patch)
2016-09-08 16:38 UTC, Carl Love 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Carl Love 2016-09-08 16:34:51 UTC 
The test results in a post failure on ppc 64 machines

Reproducible: Always
Comment 1 Carl Love 2016-09-08 16:38:04 UTC 
Created attachment 100988 [details]
fix for mmapunmap test on ppc64

The attached patch fixes the post failure on ppc64.  The patch was submitted by Will Schmidt

Will post the patch to the developers mailing list for review as it touches arch independent files.
Comment 2 Philippe Waroquiers 2016-09-10 09:30:08 UTC 
(In reply to Carl Love from comment #1)
> Created attachment 100988 [details]
> fix for mmapunmap test on ppc64
> 
> The attached patch fixes the post failure on ppc64.  The patch was submitted
> by Will Schmidt
> 
> Will post the patch to the developers mailing list for review as it touches
> arch independent files.
This looks good to me.
Comment 3 Carl Love 2016-09-13 15:55:42 UTC 
Patch committed, valgrind commit  15950
Comment 4 Mark Wielaard 2016-09-14 10:29:42 UTC 
The mmapunmap.post.exp-ppc64 file seems to be missing in the commit.
make post-regtest-checks fails with
massif/tests/Makefile.am:1: error: mmapunmap.post.exp-ppc64 is in EXTRA_DIST but doesn't exist
Comment 5 Carl Love 2016-09-14 16:20:36 UTC 
committed the missing file massif/tests/mmapunmap.post.exp-ppc64 in commit 15955

Retested, issue is resolved.