Deactivate stack squeezing for `cacheprof`
AbandonedPublic

Authored by sgraf on Dec 18 2018, 10:51 AM.

Details

Reviewers
simonmar
osa1
nomeata
simonpj
bgamari
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

cacheprof had non-deterministic allocations for some time.
The symptoms aren't too different from Trac Trac #4450, so this seems like
the easiest way to fix Trac Trac #8611.

Diff Detail

Repository
rNOFIB nofib
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25503
Build 64787: arc lint + arc unit
sgraf created this revision.Dec 18 2018, 10:51 AM
Owners added a reviewer: Restricted Owners Package.Dec 18 2018, 10:51 AM
sgraf updated this revision to Diff 19186.Dec 18 2018, 10:56 AM

Clean up the diff to only include relevant changes

sgraf updated this revision to Diff 19192.Dec 19 2018, 6:49 AM

Use -V0 instead of -Z

I note that Trac #4450 had exactly the same diagnosis that we've just rediscovered for Trac #8611, 8 years ago. So to avoid this happening again for more benchmarks, why don't we just do +RTS -V0 for all the single-threaded benchmarks in nofib? That is, add it by default to all parts of nofib except nofib/smp and nofib/parallel.

sgraf added a comment.Dec 21 2018, 2:46 AM

I note that Trac #4450 had exactly the same diagnosis that we've just rediscovered for Trac #8611, 8 years ago. So to avoid this happening again for more benchmarks, why don't we just do +RTS -V0 for all the single-threaded benchmarks in nofib? That is, add it by default to all parts of nofib except nofib/smp and nofib/parallel.

I was thinking that +RTS -V0 was just a temporary workaround until we have a story for Trac #16065. But if we want to allow nondeterministic allocations in regular program execution then I'll close Trac #16065 as invalid and will prepare a patch that does +RTS -V0 for all single-threaded benchmarks.

Ah OK. Commented on Trac #16065.

sgraf abandoned this revision.Dec 21 2018, 7:59 AM

Closed in favor of D5470.