Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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
Copy link
Contributor

Fixes#2877.

@langston-barrett
Copy link
ContributorAuthor

langston-barrett commentedNov 2, 2025
edited
Loading

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
Copy link
Member

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.

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
Copy link
ContributorAuthor

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
Copy link
Member

We can't drop metadata without reloading it I think?
Add more swap? :P

@domenukk
Copy link
Member

Maybe we can make it an option for the time being until someone is bored enough to implement it right?

@langston-barrett
Copy link
ContributorAuthor

Alright, will reopen when I have a minute to be more thorough.

domenukk reacted with heart emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

OnDiskCorpus actually only evicts the input from memory, not the entire Testcase

2 participants

@langston-barrett@domenukk

[8]ページ先頭

©2009-2025 Movatter.jp