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

Issue 620852 link

Starred by 8 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug


Sign in to add a comment

Reduce memory used by network stack on Android

Project Member Reported by mmenke@chromium.org, Jun 16 2016

Issue description

This is a tracking bug for efforts to reduce memory taken up by the network stack and related code on Android.

The goal is to reduce the frequency that Chrome is OOM killed when in the background, reduce the frequency of Chrome's renderer process being OOM killed when Chrome's in the foreground, and reduce the frequency of other apps being OOM killed when chrome is in memory (background or foreground).
 

Comment 1 by mmenke@chromium.org, Jun 16 2016

Blockedon: 620853
Blockedon: 617719
Cc: mariakho...@chromium.org
Labels: Memory-Reduction

Comment 3 by mmenke@chromium.org, Jun 17 2016

Blockedon: 621191

Comment 4 by mmenke@chromium.org, Jun 17 2016

Blockedon: 621197

Comment 5 by mmenke@chromium.org, Jun 17 2016

Blockedon: 524258

Comment 6 by mmenke@chromium.org, Jun 20 2016

Labels: Performance-Memory
Blockedon: 622069
Blockedon: 622073
Blockedon: 622091
Blockedon: 622096
Blockedon: 622100
Blockedon: 622101
Blockedon: 620466
Blockedon: 620770
Blockedon: 622334
Blockedon: 622363
Blockedon: 622444
Blockedon: 619649

Comment 19 Deleted

Comment 20 by ssid@chromium.org, Jul 14 2016

Blocking: 593486
Cc: mmenke@chromium.org
Owner: ----
Status: Available (was: Started)
Unassigning from myself - I may still tackle the SSL buffer one, and the idle sockets one, but I think we've proven there's a good chance I won't have time for too much this stuff.
Blockedon: 621597
Blockedon: 535601
Blockedon: 615542
Blockedon: 669108
Blockedon: 671366
Blockedon: 671420
Owner: xunji...@chromium.org
Status: Assigned (was: Available)
Linking  Issue 671420  here, so other folks can stay up to date. Short summary is that X509* is going away (together with its hundreds of small allocations). It is going to be replaced with CRYPTO_BUFFER. According to davidben@'s initial investigation, it should improve net/cert/ memory usage by 4x. See more details in the linked crbug. Thanks David and the boringssl team for the great work.
Blockedon: 684732
Blockedon: 684746
Blockedon: 685205
Blockedon: 685235
Blockedon: 687182
Blockedon: 689062
Blockedon: 690095
Is there still active progress on this?
Project Member

Comment 36 by bugdroid1@chromium.org, Sep 21 2017

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

commit a46c80057c897f53772701bbde45b551401b691e
Author: Helen Li <xunjieli@chromium.org>
Date: Thu Sep 21 21:57:32 2017

Deprecate Net.Socket.IdleSocketFate histogram

Idle socket reuse rate is high on Desktop (close to 50%) and on Mobile (~30% on
Android). Newer memory instrumentation data do not show socket pool as a hotspot. 
There have been a few projects to reduce the amount of memory an idle socket
retains (e.g. use CRYPTO_BUFFER  Issue 671420  and lazy buffer initialization
 Issue 524258 ).

I do not have any follow-up experiments that would use this histogram, so I am
removing it. The histogram was added in crrev.com/2450463002.

BUG:  620852 
Change-Id: I1a7f858bfcafbcaad708366fa4d2bd5e967e14b0
Reviewed-on: https://chromium-review.googlesource.com/673272
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#503576}
[modify] https://crrev.com/a46c80057c897f53772701bbde45b551401b691e/net/socket/client_socket_pool_base.cc
[modify] https://crrev.com/a46c80057c897f53772701bbde45b551401b691e/net/socket/client_socket_pool_base_unittest.cc
[modify] https://crrev.com/a46c80057c897f53772701bbde45b551401b691e/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/a46c80057c897f53772701bbde45b551401b691e/tools/metrics/histograms/histograms.xml

Status: Archived (was: Assigned)
All linked bugs are closed. I am archiving this metabug as there's no active development on this issue for a while now.

Sign in to add a comment