New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 748981 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Correcntess fuzzer having duplicate explosion

Project Member Reported by machenb...@chromium.org, Jul 26 2017

Issue description

This morning there were 100+ new issues:
https://bugs.chromium.org/p/chromium/issues/list?can=2&q=label:v8-foozzie-failure&sort=-modified+-pri&colspec=ID%20Pri%20M%20Stars%20ReleaseBlock%20Component%20Status%20Owner%20Summary%20OS%20Modified

Something seems to have gone wrong with the clustering. To mitigate I reduced the number of test cases from 10000 to 500.
 
The bugs are all due to the asm switch:
https://chromium-review.googlesource.com/c/583655/

The default case switched on --validate-asm, so we now compare:
--validate-asm
with
--validate-asm --stress-validate-asm --suppress-asm-messages

@mstarzinger: Should also the default pass --suppress-asm-messages?
If we want to maintain comparing those two variants, then adding "--suppress-asm-messages" would do the trick. But then we would need to add "--suppress-asm-messages" to _all_ variants, otherwise the other variant combinations will start to fail.
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 26 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/0cda6ffed7de59b3a94d15ae36062df398cdf0c4

commit 0cda6ffed7de59b3a94d15ae36062df398cdf0c4
Author: Michael Achenbach <machenbach@chromium.org>
Date: Wed Jul 26 11:51:44 2017

[foozzie] Suppress asm messages in all comparison configurations

TBR=mstarzinger@chromium.org
NOTRY=true

Bug:  chromium:748981 
Change-Id: I2043d283bd4eb6d98082f202d88638b3ca96a7c8
Reviewed-on: https://chromium-review.googlesource.com/586687
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#46897}
[modify] https://crrev.com/0cda6ffed7de59b3a94d15ae36062df398cdf0c4/tools/foozzie/v8_foozzie.py

Labels: -Pri-0 Pri-1
The above commit will suppress all the bugs. We can wait out the problem. Tomorrow, the fixed testing will close all the bugs again...
All bugs got auto-closed again. We might wanna think about an alert when too many bugs are produced in a given time period. And maybe an automatic kill-switch that e.g. reduces the max test cases of certain jobs.

I'll keep the max low until the end of the week because we have another noisy and spurious problem.
Status: Verified (was: Assigned)
This is resolved. Switched back the max-testcases value for the job.

Sign in to add a comment