This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Do not consider releasing unallocated pages
ClosedPublic

Authored by fabio-d on Nov 28 2022, 5:04 AM.

Details

Summary

We already know that there are no free blocks above Region->AllocatedUser.
This results in a smaller RegionPageMap and faster releaseFreeMemoryToOS.

Diff Detail

Event Timeline

fabio-d created this revision.Nov 28 2022, 5:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2022, 5:04 AM
Herald added a subscriber: Enna1. · View Herald Transcript
fabio-d requested review of this revision.Nov 28 2022, 5:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 28 2022, 5:04 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
Chia-hungDuan accepted this revision.Nov 28 2022, 10:10 AM

Thanks for the fix!

This revision is now accepted and ready to land.Nov 28 2022, 10:10 AM
This revision was landed with ongoing or failed builds.Nov 29 2022, 6:35 PM
This revision was automatically updated to reflect the committed changes.