New issue
Advanced search Search tips

Issue 762963 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-09-20
OS: Linux , Android , Windows , Mac
Pri: 1
Type: Bug

Blocking:
issue 698746


Participants' hotlists:
I-TF-Launch


Sign in to add a comment

Regression: JavaScript/V8 - Array Buffer readouts give unpredictable results

Reported by d...@acmer.me, Sep 7 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.79 Safari/537.36

Steps to reproduce the problem:
1. Open https://www.blend4web.com/tmp/blend4web_ce/apps_dev/tutorials/web_page_integration/example.html
2. Press "Click here" button
3. Balloon jerks

What is the expected behavior?
Balloon moves smoothly.

What went wrong?
After some search we found
1) Wrong behavior is reproduced on +59.0.3071.
Everything is ok on 58.0.3029.
2) We can't reproduce the bug in little example.
3) If we comment 586-587 lines of https://www.blend4web.com/tmp/blend4web_ce/src/data.js and restart demo, balloon moves smoothly.

Did this work before? N/A 

Chrome version: 61.0.3163.79  Channel: stable
OS Version: Ubuntu 16.04
Flash Version:
 

Comment 1 by woxxom@gmail.com, Sep 7 2017

Bisect info: 455719 (good) - 455720 (bad)
https://chromium.googlesource.com/chromium/src/+log/772559ed..5e921f82?pretty=fuller
The only CL is r455720 "Update V8 to version 5.9.34.2 (Switch on I+TF)"
Landed in 59.0.3037.0

Components: -Blink Blink>JavaScript
Cc: hablich@chromium.org
Labels: M-63 OS-Android OS-Mac OS-Windows
Since this been there since M59 not tagging any blocker labels tagging with M63.
Status: Untriaged (was: Unconfirmed)
Untriaging it so that it gets addressed.
Components: -Blink>JavaScript Blink>JavaScript>Interpreter Blink>JavaScript>Compiler
Labels: -Pri-2 Pri-1
Status: Available (was: Untriaged)
Thanks, compiler & ignition folks, please investigate and retriage.
Blocking: 698746
Project Member

Comment 7 by sheriffbot@chromium.org, Sep 17

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
NextAction: 2018-09-20
Looks like this works now without jank?
It's still broken in 69 and Canary.
The bisect still points to the same CL.
Expected: the balloon slowly moves at its initial position.
Observed: the balloon position is reset every second to the bottom of the viewport.
What is the purpose of

for (var k = 0; k < 10000000; k++)
                    var blahblahblah = 100 * 312;

It does not seem to do anything except looping?
Hm, tight loop like that could mean an OSR issue. I don't have time to take this on at the moment, but that would be my first guess.
I've tried commenting out or deleting these two lines and I still see the bug.
Used a locally edited version served via Requestly extension instead of https://www.blend4web.com/tmp/blend4web_ce/src/data.js
This reminds me of those bug reports where seemingly irrelevant changes to the code produce/hide some bug.
Might be helpful to do a bisect prior to I+TF was shipped but I don't know how to force-enable it in older builds.
Try running older builds with "--ignition --turbo". It would also be helpful to figure out which function(s?) have incorrect data, by e.g. printing inputs and/or return values with console.log and diffing them. I realise that's somewhat of a tough request though.

How do other browsers behave?
re 13 part a:

Bisect info: 410370 (good) - 410383 (bad)
https://chromium.googlesource.com/chromium/src/+log/9d009539..ba4eb17e?pretty=fuller
Suspecting r410383 "Update V8 to version 5.4.373"
Landed in 54.0.2824.0

V8 log: https://chromium.googlesource.com/v8/v8/+log/d4fa8ea7..d67f12f7
There are only two TF commits but it might be helpful if someone does a per-revision bisect of V8.

Notes:
* Used the original page code.
* Command line: --js-flags="--ignition --turbo"
* In these old builds the balloon appears after a ~15sec pause.

re 13 part b: that's probably up to the reporter.
Stab in the dark would be https://codereview.chromium.org/2220973002 and some sort of weird incorrect NaN behaviour.

Does the issue repro with "--no-opt"? Might have to use "--no-crankshaft" for older builds.
--js-flags=--no-opt fixes the bug. Tested in r410383, r589108.
Cc: leszeks@chromium.org ahaas@chromium.org
Components: -Blink>JavaScript>Interpreter
Owner: jarin@chromium.org
Status: Assigned (was: Untriaged)
Cool, looks like a TurboFan issue then, assigning to jarin@ for further investigation.

+ahaas: your CL came up in a bisect of this issue. 
The NextAction date has arrived: 2018-09-20

Sign in to add a comment