Browser memory usage has increased in 58 |
||||||||||||||
Issue descriptionChrome 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?
,
Apr 10 2017
Since Sonny has worked on similar stuff before, let's put it in his plate.
,
Apr 10 2017
The increase on arm is clear enough that I think this justifies being RBS
,
Apr 11 2017
Issue 710279 has been merged into this issue.
,
Apr 18 2017
,
Apr 24 2017
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?
,
Apr 24 2017
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.
,
Apr 24 2017
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.
,
Apr 26 2017
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.
,
Apr 28 2017
,
Apr 28 2017
We should get a resolution on this well before 59 goes to stable.
,
Apr 28 2017
#11: Was the intent to mark this as Release Block Stable or Beta? Looks more like stable blocker
,
Apr 28 2017
,
Apr 28 2017
May I know how to measure the memory usage / how to run the test locally?
,
Apr 28 2017
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.
,
Apr 28 2017
#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.
,
May 1 2017
,
May 22 2017
Since R59 is no longer using GCC, this bug shouldn't affect 59 and should no longer be blocking release 59.
,
May 26 2017
The bug turns out to be only in 58 and not in 59 and above. Are we gonna fix this in 58?
,
May 26 2017
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).
,
May 26 2017
|
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by abodenha@chromium.org
, Apr 10 2017