This is an archive of the discontinued LLVM Phabricator instance.

[Sanitizer] Fix GetRSS on Linux with non-4k pages
ClosedPublic

Authored by foad on Dec 18 2014, 1:37 AM.

Details

Summary

The numbers in /proc/self/statm are in pages, not in fixed 4k units.
This fixes Linux/hard_rss_limit_mb_test.cc on my PowerPC64 box which
has 64k pages.

Diff Detail

Event Timeline

foad updated this revision to Diff 17425.Dec 18 2014, 1:37 AM
foad retitled this revision from to [Sanitizer] Fix GetRSS on Linux with non-4k pages.
foad updated this object.
foad edited the test plan for this revision. (Show Details)
foad added reviewers: kcc, willschm.
foad added a subscriber: Unknown Object (MLST).
willschm accepted this revision.Dec 18 2014, 8:10 AM
willschm edited edge metadata.

This looks like a good change to me.
Thanks,
-Will

This revision is now accepted and ready to land.Dec 18 2014, 8:10 AM
foad closed this revision.Dec 18 2014, 8:24 AM