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

Issue 841322 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: May 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1%-2.2% regression in system_health.memory_mobile at 555727:555854

Project Member Reported by primiano@google.com, May 9 2018

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=841322

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=ae2873331e756cfd885c25f93b1b7f298b822f4e16953bde4c29846d643766ed


Bot(s) for this bug's original alert(s):

android-nexus5
android-nexus5X
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, May 10 2018

Cc: clemensh@chromium.org hpayer@chromium.org mlippautz@chromium.org cbruni@chromium.org machenb...@chromium.org
Owner: machenb...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16982dd7c40000

[heap] Clear the memory of pooled pages when allocating from the pool. by hpayer@chromium.org
https://chromium.googlesource.com/v8/v8/+/c280e7d4f4001ffe6da3d9a7b8658b2cd590e26a

Revert "[wasm] Maintain link from Instance to Module." by machenbach@chromium.org
https://chromium.googlesource.com/v8/v8/+/79a99dfc10a16e70eadd7fac9a5f83698d4f6689

Revert "[logging] Remove unused code" by machenbach@chromium.org
https://chromium.googlesource.com/v8/v8/+/ab814fb833a97378f73cbc81c31f68c4a81754ac

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 4 by hpayer@chromium.org, May 11 2018

Owner: hpayer@chromium.org
That must be [heap] Clear the memory of pooled pages when allocating from the pool.
Will revert to investigate offline.

Comment 5 by hpayer@chromium.org, May 15 2018

Fixed. The CL was reverted.

Comment 6 by hpayer@chromium.org, May 15 2018

Status: Fixed (was: Assigned)
The CL was reverted.

Sign in to add a comment