Bug 368461

Summary: mmapunmap test fails on ppc64
Product: [Developer tools] valgrind Reporter: Carl Love <cel>
Component: generalAssignee: Julian Seward <jseward>
Status: CLOSED FIXED    
Severity: normal CC: mark, philippe.waroquiers, will_schmidt
Priority: NOR    
Version First Reported In: 3.12 SVN   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: fix for mmapunmap test on ppc64

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.