This is an archive of the discontinued LLVM Phabricator instance.

sanitizer_common: bump default file max size to 256MB
ClosedPublic

Authored by dvyukov on Oct 29 2021, 2:48 AM.

Details

Summary

I am hitting some cases where /proc/self/maps does not fit into 64MB.
256MB is lots of memory, but it's not radically more than the current 64MB.
Ideally we should read/parse these huge files incrementally,
but that's lots of work for a debugging/introspection interface.
So for now just bump the limit.

Depends on D112793.

Diff Detail

Event Timeline

dvyukov requested review of this revision.Oct 29 2021, 2:48 AM
dvyukov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 29 2021, 2:48 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
melver accepted this revision.Oct 29 2021, 3:37 AM
melver added inline comments.
compiler-rt/lib/sanitizer_common/sanitizer_common.h
724

Is 64MB more likely to be exceeded on 64-bit systems?

If so, maybe make the size conditional on bit-ness of the system? Because 64 vs 256 MB might be a lot for some 32-bit system.

This revision is now accepted and ready to land.Oct 29 2021, 3:37 AM
dvyukov updated this revision to Diff 383315.Oct 29 2021, 4:59 AM

use smaller limit for 32-bit arches

dvyukov marked an inline comment as done.Oct 29 2021, 4:59 AM
This revision was landed with ongoing or failed builds.Oct 29 2021, 5:03 AM
This revision was automatically updated to reflect the committed changes.