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

Issue 710232 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Browser memory usage has increased in 58

Project Member Reported by abodenha@chromium.org, Apr 10 2017

Issue description

Chrome Version: 58.0.3029.51
OS: Chrome OS

Memory.Browser.Large2 increased from 146MB in 57.0.2987.115 to 154MB in 58.0.3029.51 on arm. 

There was also an increase on Intel in the same time period but much smaller magnitude.

rkc@, do you think this is something wzang@ could investigate?
 

Comment 2 by r...@chromium.org, Apr 10 2017

Cc: -wzang@chromium.org
Owner: sonnysasaka@chromium.org
Status: Assigned (was: Untriaged)
Since Sonny has worked on similar stuff before, let's put it in his plate.

Cc: zelidrag@chromium.org josa...@chromium.org
Labels: ReleaseBlock-Stable M-58
The increase on arm is clear enough that I think this justifies being RBS
Issue 710279 has been merged into this issue.

Comment 5 by igo@chromium.org, Apr 18 2017

Cc: igo@chromium.org
We are targeting a stable RC on Wednesday, meaning any fixes have to be in by Tuesday night, is there any chance we can get a fix in before tomorrow afternoon?

Has this been looked at yet?
Labels: -ReleaseBlock-Stable
Root cause is still not understood. We should def keep this as high priority, but right now I don't think holding stable on it is the right move.

Comment 8 by r...@chromium.org, Apr 24 2017

Status: Started (was: Assigned)
Sonny is looking at this now to see if we can find any low hanging fruit. I don't think merging to 58 is a good idea though. I am fine with this being on 59.

Cc: r...@chromium.org
I found this CL to be a cause: http://crrev.com/2640263003.
That CL caused around 3% increase in browser process memory after startup and idle. There may be other causes but it's hard to find because the memory footprint increase could be specific to a particular usage.
Owner: laszio@chromium.org

Comment 11 by r...@chromium.org, Apr 28 2017

Cc: abodenha@chromium.org
Labels: -M-58 ReleaseBlock-Beta M-59
We should get a resolution on this well before 59 goes to stable.
#11: Was the intent to mark this as Release Block Stable or Beta? Looks more like stable blocker
Labels: -ReleaseBlock-Beta ReleaseBlock-Stable
May I know how to measure the memory usage / how to run the test locally?
Owner: sonnysasaka@chromium.org
On the other hand, the CL (http://crrev.com/2640263003) has nothing to do with R59. It is only effective with gcc. In R59, chrome is built by clang.

I'll keep investigating but the outcome will not resolve the release blocker. Reassigning to Sonny.
#14: I used chrome://tracing to measure memory footprint, as described in the "Platform Independent" section here: https://docs.google.com/document/d/15mBOu_uZbgP5bpdHZJXEnF9csSRq7phUWXnZcteVr0o/edit#heading=h.9nj5ra4vpzyi

If R59 is no longer using GCC then that CL won't affect anything to R59.
Cc: llozano@chromium.org
Labels: -ReleaseBlock-Stable -M-59
Since R59 is no longer using GCC, this bug shouldn't affect 59 and should no longer be blocking release 59.
The bug turns out to be only in 58 and not in 59 and above.
Are we gonna fix this in 58?
59 is stable in under two weeks, we are not planning any further 58 releases, so there is not much point in fixing 58 (chance of emergency respin is not super high). 
Status: WontFix (was: Started)

Sign in to add a comment