Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork412
CachedOnDiskCorpus: Evict metadata in addition to input#3471
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
langston-barrett commentedNov 2, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Ah, this probably also needs some changes to load the metadata back into memory. [EDIT]: I guess it depends on the metadata? For coverage, it actually is probablynot important to load the metadata back into memory. |
domenukk commentedNov 3, 2025
Is there a downside to loading it? You don't know what people do with the metadata, they may look at the coverage data for some random things later, too.. |
langston-barrett commentedNov 3, 2025
No, agreed that there's no downside to loading it. However, there's also not an existing API for it. The metadata path, for instance, isonly calculated just before storing it to disk. I think loading metadata from disk would be good, but the issue of keeping it in memory is a critical blocker for me right now - I just can't run my fuzzer for very long before it OOMs. I don't personally use metadata except for coverage in this fuzzer, and I don't use the coverage metadata outside of the coverage feedback. So for me, evicting the metadata is very high priority and loading it back in is less important. |
domenukk commentedNov 3, 2025
We can't drop metadata without reloading it I think? |
domenukk commentedNov 3, 2025
Maybe we can make it an option for the time being until someone is bored enough to implement it right? |
langston-barrett commentedNov 6, 2025
Alright, will reopen when I have a minute to be more thorough. |
Fixes#2877.