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

report: report fuzzer fatal errors #3101

Merged
merged 1 commit into from
Apr 25, 2022
Merged

Conversation

avagin
Copy link
Collaborator

@avagin avagin commented Apr 22, 2022

Right now, all fuzzer fatal errors are hidden in "lost connection to test machine" or "no output from test machine" reports.

@avagin avagin requested a review from dvyukov April 22, 2022 00:38
@codecov
Copy link

codecov bot commented Apr 22, 2022

Codecov Report

Merging #3101 (2f3839d) into master (2738b39) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

Impacted Files Coverage Δ
pkg/log/log.go 62.8% <0.0%> (ø)
pkg/report/report.go 83.7% <ø> (ø)
syz-fuzzer/proc.go 0.0% <0.0%> (ø)
pkg/vcs/git.go 58.4% <0.0%> (-1.1%) ⬇️

@dvyukov
Copy link
Collaborator

dvyukov commented Apr 22, 2022

I afraid we can get 30 new bugs for each of 15 kernels over the next month. I would be more conservative and put them all into a single bucket for now. If there is a spike, it's possible to assess which ones are firing manually.

It's also possible to use the "AltTitle" thing to merge them all together.
If we do Title=full-message and AltTitle=SYZFATAL, then they will be merged into a single bug (grouped by AltTitle), but the full-message will be visible for each crash in the "Title" column on the dashboard.
The only downside is that the name of the bug will be a random full message of one of the crashes (the first one that will create the bug).

Signed-off-by: Andrei Vagin <avagin@google.com>
@avagin
Copy link
Collaborator Author

avagin commented Apr 22, 2022

@dvyukov I have added the alt title.

@dvyukov dvyukov merged commit f3819ca into google:master Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants