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

Issue 613125 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Disable black allocation to investigate memory bloat on dev and beta

Project Member Reported by hpayer@chromium.org, May 19 2016

Issue description

Memory consumption increased at around the same time where memory consumption increased. Since that, it never really recovered. Disable black allocation to investigate memory bloat on dev and beta.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 19 2016

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

commit fbb221c48e07b08cf65ac63167167f1a2c5c9f74
Author: hpayer <hpayer@chromium.org>
Date: Thu May 19 11:18:35 2016

Disable black allocation to investigate memory bloat.

BUG= chromium:613125 
LOG=n

Review-Url: https://codereview.chromium.org/1992193002
Cr-Commit-Position: refs/heads/master@{#36358}

[modify] https://crrev.com/fbb221c48e07b08cf65ac63167167f1a2c5c9f74/src/flag-definitions.h

Comment 2 by hpayer@chromium.org, May 20 2016

Labels: Merge-Request-51

Comment 3 by tin...@google.com, May 20 2016

Labels: -Merge-Request-51 Merge-Review-51 Hotlist-Merge-Review
[Automated comment] Less than 2 weeks to go before stable on M51, manual review required.

Comment 4 by gov...@chromium.org, May 20 2016

Cc: sshruthi@chromium.org
Is this bug applicable to specific OS or all os?

Also before we approve merge to M51, Could you please confirm whether this change is baked/verified in Canary and safe to merge?

Comment 5 by hpayer@chromium.org, May 23 2016

Labels: OS-All

Comment 6 by gov...@chromium.org, May 23 2016

Thank you hpayer@. Could you please confirm whether this change is verified in Canary and safe to merge?

Comment 7 Deleted

Labels: -Merge-Approved-51 Merge-Review-51
Let's wait until we have Dev validation that this fixes the increased memory consumption. 

Comment 9 by sshru...@google.com, May 24 2016

Cc: -sshruthi@chromium.org
Any updates here? We will be cutting a stable RC early next week.
Labels: -Merge-Review-51 Merge-Rejected-51
c#0 says this is for dev and beta. Rejecting this merge request to M51 which is currently on stable. Please re-request with more information if this is really needed in M51.
Status: WontFix (was: Assigned)
Obsolete.

Sign in to add a comment