New issue
Advanced search Search tips

Issue 876976 link

Starred by 1 user
Status: Fixed
Owner:
vovoy@chromium.org
Closed: Sep 7
Cc:
cmtm@chromium.org
yusukes@chromium.org
sonnyrao@chromium.org
semenzato@chromium.org
cylee@chromium.org
vovoy@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

oom_score_adj of ARC++ process shall not be -1000

Project Member Reported by vovoy@chromium.org, Aug 23 
oom_score_adj of some ARC++ processes are set to -1000 by https://cs.chromium.org/chromium/src/chrome/browser/resource_coordinator/tab_manager_delegate_chromeos.cc?rcl=d2cac823b636b11cc9182a5d21ba4c6824f54394&l=84

It shall be -100 instead.

Reference:
login: change default oom_score_adj of android container to -100
https://crrev.com/c/1161619
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 30 

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

commit 49d3354b914f2b4bfd44c3d309deec491a45fa3b
Author: Kuo-Hsin Yang <vovoy@chromium.org>
Date: Thu Aug 30 09:51:15 2018

Change the lowest oom_score_adj value for ARC++ process

oom_score_adj should not be set to -1000 for ARC++ processes.

Bug:  chromium:876976 
Change-Id: I4259d646755761dce5429170a45be5f8faf34b9f
Reviewed-on: https://chromium-review.googlesource.com/1186282
Commit-Queue: Vovo Yang <vovoy@chromium.org>
Reviewed-by: Cheng-Yu Lee <cylee@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587498}
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/chromeos/arc/process/arc_process.cc
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/chromeos/arc/process/arc_process.h
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/chromeos/arc/process/arc_process_unittest.cc
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/resource_coordinator/tab_manager_delegate_chromeos.cc
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/resource_coordinator/tab_manager_delegate_chromeos.h
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/resource_coordinator/tab_manager_delegate_chromeos_unittest.cc
[modify] https://crrev.com/49d3354b914f2b4bfd44c3d309deec491a45fa3b/chrome/browser/task_manager/providers/arc/arc_process_task.cc

Comment 2 by vovoy@chromium.org, Sep 7

Status: Fixed (was: Assigned)
►

Sign in to add a comment