Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Experiment for AFL++ cloning strategies using different map sizes #1231

Merged
merged 29 commits into from
Aug 26, 2021
Merged

Experiment for AFL++ cloning strategies using different map sizes #1231

merged 29 commits into from
Aug 26, 2021

Conversation

pietroborrello
Copy link
Contributor

We'd like to request an experiment to study the impact of the AFL++'s edge map size on context sensitivity achieved with cloning strategies.

As with PR #1202 it is unlikely that this PR will pass the CI since the builds require quite a lot of memory (8-16Gb on average) and Travis seems to provide limited RAM. However, we already tested the bug benchmarks locally and excluded the ones that do not build.

Also, it would be helpful if you could provide some approximate numbers on the RAM available for the VMs used as builders, so as to limit our technique to avoid filling memory during compilation. Especially for bigger binaries that we produce, some may die at linking stage (e.g. wireshark in experiment 2021-07-09-cloning). Similarly, knowing RAM and cache characteristics in the runners would let us tune the map sizes better.

@google-cla
Copy link

google-cla bot commented Aug 23, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label Aug 23, 2021
@andreafioraldi
Copy link
Contributor

@googlebot I consent.

@google-cla google-cla bot added cla: yes and removed cla: no labels Aug 23, 2021
@andreafioraldi
Copy link
Contributor

Hey @jonathanmetzman we are a bit in a hurry for the deadline (this is for a paper), can I ask you to review it ASAP if you have time?

@inferno-chromium inferno-chromium merged commit 245cab2 into google:master Aug 26, 2021
@andreafioraldi
Copy link
Contributor

The experiment https://www.fuzzbench.com/reports/experimental/2021-08-23-cloning/index.html is stuck on 15m from something like 2 days, @inferno-chromium do u know what happened? Did we break the infra 😅?

@pietroborrello
Copy link
Contributor Author

Maybe google/oss-fuzz#6180 has broken something? We were merged just before #1225 that was fixing it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants