<feed xmlns='http://www.w3.org/2005/Atom'>
<title>system_core/storaged, branch replicant-10</title>
<subtitle>Fork of system/core
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/'/>
<entry>
<title>Remove liblog's rate limiting</title>
<updated>2019-01-16T18:37:14+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2019-01-15T23:19:43+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=3b4bddc6386f40d57181f0e93cf8f2f37695f230'/>
<id>3b4bddc6386f40d57181f0e93cf8f2f37695f230</id>
<content type='text'>
After a few years of being available, there only ended up being one
user of this, so it is clear that logd's duplicate message mechanism
is the favored solution.

The one user of this rate limiting is questionable as is, since due to
the nature of storaged, the mainloop only runs once per minute by
default as is, so there is essentially nothing to be gained by rate
limiting any further.

Test: build
Change-Id: I0610d11efda1ce8b581b939bad11ff295cb2daa6
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
After a few years of being available, there only ended up being one
user of this, so it is clear that logd's duplicate message mechanism
is the favored solution.

The one user of this rate limiting is questionable as is, since due to
the nature of storaged, the mainloop only runs once per minute by
default as is, so there is essentially nothing to be gained by rate
limiting any further.

Test: build
Change-Id: I0610d11efda1ce8b581b939bad11ff295cb2daa6
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: emplace recent_perf in load_perf_history_proto</title>
<updated>2018-11-20T17:00:05+00:00</updated>
<author>
<name>Mark Salyzyn</name>
<email>salyzyn@google.com</email>
</author>
<published>2018-11-20T16:28:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=0ade88c83de7f207389d2c7a8385e927604dae70'/>
<id>0ade88c83de7f207389d2c7a8385e927604dae70</id>
<content type='text'>
If recent_perf contains existing content, overwrite in method
load_perf_history_proto.  If proto of recent_perf contains a lot of
history, the content would have grown with system_server crashes.

Test: setprop ro.storaged.flush_proto.interval 60
      while true;do sleep 5;stop;sleep 1;start;ps -A|grep storaged;done
      and look at value for rss should remain about 4MB over long term.
Bug: 119798824
Change-Id: I50aae7d61eb791a8fcddbbd829254baf8f708186
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If recent_perf contains existing content, overwrite in method
load_perf_history_proto.  If proto of recent_perf contains a lot of
history, the content would have grown with system_server crashes.

Test: setprop ro.storaged.flush_proto.interval 60
      while true;do sleep 5;stop;sleep 1;start;ps -A|grep storaged;done
      and look at value for rss should remain about 4MB over long term.
Bug: 119798824
Change-Id: I50aae7d61eb791a8fcddbbd829254baf8f708186
</pre>
</div>
</content>
</entry>
<entry>
<title>Move system/core/ off NO_ERROR.</title>
<updated>2018-10-08T18:15:52+00:00</updated>
<author>
<name>Elliott Hughes</name>
<email>enh@google.com</email>
</author>
<published>2018-10-08T18:10:11+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=643268f325e2bda64248df24913f8b68c842c95a'/>
<id>643268f325e2bda64248df24913f8b68c842c95a</id>
<content type='text'>
It causes trouble for Windows, and OK already exists.

Bug: N/A
Test: builds
Change-Id: Ida22fd658b0ebb259c710ba39049b07c9e495d9c
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It causes trouble for Windows, and OK already exists.

Bug: N/A
Test: builds
Change-Id: Ida22fd658b0ebb259c710ba39049b07c9e495d9c
</pre>
</div>
</content>
</entry>
<entry>
<title>Match src paths with aidl package name</title>
<updated>2018-09-15T04:27:31+00:00</updated>
<author>
<name>Dan Willemsen</name>
<email>dwillemsen@google.com</email>
</author>
<published>2018-09-15T04:27:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=c53f5c153ff96aafbede2402c1fa94fbd1a73156'/>
<id>c53f5c153ff96aafbede2402c1fa94fbd1a73156</id>
<content type='text'>
In order for the build system to track updates to the header files
during incremental builds, always specify the src files using the same
path as the package for C++ compilations.

Bug: 112114177
Test: m
Change-Id: Iec355bf4c11204a09bb164bbf14eaa1a148c5f36
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In order for the build system to track updates to the header files
during incremental builds, always specify the src files using the same
path as the package for C++ compilations.

Bug: 112114177
Test: m
Change-Id: Iec355bf4c11204a09bb164bbf14eaa1a148c5f36
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: change log level for health HAL calls</title>
<updated>2018-09-13T16:41:46+00:00</updated>
<author>
<name>Yifan Hong</name>
<email>elsk@google.com</email>
</author>
<published>2018-09-13T16:39:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=539ee58b4171448581e038b2d90d92f347349584'/>
<id>539ee58b4171448581e038b2d90d92f347349584</id>
<content type='text'>
Devices that does not implement getDiskStats and getStorageInfo
returns NOT_SUPPORTED as a value. Do not log to error stream in
this case.
Fixes: 114475810
Test: boots

Change-Id: I95e9ebe23b4dee451cb8f6e8dc0b60a224006600
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Devices that does not implement getDiskStats and getStorageInfo
returns NOT_SUPPORTED as a value. Do not log to error stream in
this case.
Fixes: 114475810
Test: boots

Change-Id: I95e9ebe23b4dee451cb8f6e8dc0b60a224006600
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: Cap io_history when loading stats from disk.</title>
<updated>2018-08-21T16:20:40+00:00</updated>
<author>
<name>David Anderson</name>
<email>dvander@google.com</email>
</author>
<published>2018-07-27T01:01:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=dec6a881ba504aca4a4d0fad857b2f802ace8599'/>
<id>dec6a881ba504aca4a4d0fad857b2f802ace8599</id>
<content type='text'>
Similar to add_record_locked, load_uid_io_proto should respect the
maximum number of records.

Bug: 111578975
Test: storaged_test.load_uid_io_proto
Change-Id: Ic3c5095cdd09d2184f436813b5ab50d3ee0007b2
Merged-In: Ic3c5095cdd09d2184f436813b5ab50d3ee0007b2
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Similar to add_record_locked, load_uid_io_proto should respect the
maximum number of records.

Bug: 111578975
Test: storaged_test.load_uid_io_proto
Change-Id: Ic3c5095cdd09d2184f436813b5ab50d3ee0007b2
Merged-In: Ic3c5095cdd09d2184f436813b5ab50d3ee0007b2
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: Don't duplicate uid_records from proto files.</title>
<updated>2018-08-21T16:20:33+00:00</updated>
<author>
<name>David Anderson</name>
<email>dvander@google.com</email>
</author>
<published>2018-07-26T21:30:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=0026a14ff2655fa1b5c77e00e58e0f3dceed2d87'/>
<id>0026a14ff2655fa1b5c77e00e58e0f3dceed2d87</id>
<content type='text'>
It is possible for storaged to load the same saved proto file multiple
times, for example, if system_server crashes. In this case we do not
want to fill io_history with duplicate entries. This patch elides
records for which an app+userid record already exists for the same
timestamp.

Bug: 111578975
Test: gtest storaged_test.load_uid_io_proto
Change-Id: I87bc3e37d6464079cece03b5852285d79067b935
Merged-In: I87bc3e37d6464079cece03b5852285d79067b935
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It is possible for storaged to load the same saved proto file multiple
times, for example, if system_server crashes. In this case we do not
want to fill io_history with duplicate entries. This patch elides
records for which an app+userid record already exists for the same
timestamp.

Bug: 111578975
Test: gtest storaged_test.load_uid_io_proto
Change-Id: I87bc3e37d6464079cece03b5852285d79067b935
Merged-In: I87bc3e37d6464079cece03b5852285d79067b935
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: Fix-up private variable names in uid_monitor.</title>
<updated>2018-08-21T16:20:29+00:00</updated>
<author>
<name>David Anderson</name>
<email>dvander@google.com</email>
</author>
<published>2018-07-26T20:20:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=3488d9f9ddd19655511ccd8782f45e4b630ba8d2'/>
<id>3488d9f9ddd19655511ccd8782f45e4b630ba8d2</id>
<content type='text'>
Bug: N/A
Test: storaged builds
Change-Id: Ifd21e25baa7b1c6ce41c5e0ec5247f47ba716e6e
Merged-In: Ifd21e25baa7b1c6ce41c5e0ec5247f47ba716e6e
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Bug: N/A
Test: storaged builds
Change-Id: Ifd21e25baa7b1c6ce41c5e0ec5247f47ba716e6e
Merged-In: Ifd21e25baa7b1c6ce41c5e0ec5247f47ba716e6e
</pre>
</div>
</content>
</entry>
<entry>
<title>storaged: fix headers.</title>
<updated>2018-07-17T22:36:09+00:00</updated>
<author>
<name>Yifan Hong</name>
<email>elsk@google.com</email>
</author>
<published>2018-05-04T21:20:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=acd2d30cec34c47d446b6c328dafdf98b3fd5be1'/>
<id>acd2d30cec34c47d446b6c328dafdf98b3fd5be1</id>
<content type='text'>
Add missing libbatteryservice_headers dependency.
It used to use global header include dirs, which is bad.

Add old transitive dependency from libbatteryservice_headers
as well.

Test: builds
Bug: 68724651

Change-Id: Idcc4452160dce70b4f8b48e2d809dcd5744b93b4
Merged-In: Idcc4452160dce70b4f8b48e2d809dcd5744b93b4
(cherry picked from commit 0b31908912d7a1802ce7e3d0f0857fe6b34d926b)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add missing libbatteryservice_headers dependency.
It used to use global header include dirs, which is bad.

Add old transitive dependency from libbatteryservice_headers
as well.

Test: builds
Bug: 68724651

Change-Id: Idcc4452160dce70b4f8b48e2d809dcd5744b93b4
Merged-In: Idcc4452160dce70b4f8b48e2d809dcd5744b93b4
(cherry picked from commit 0b31908912d7a1802ce7e3d0f0857fe6b34d926b)
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "storaged: lower capabilities in init" am: 183b575e78</title>
<updated>2018-04-12T16:41:00+00:00</updated>
<author>
<name>David Anderson</name>
<email>dvander@google.com</email>
</author>
<published>2018-04-12T16:41:00+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=8cc138e03f68f1fde8dc679d30749129bb355e56'/>
<id>8cc138e03f68f1fde8dc679d30749129bb355e56</id>
<content type='text'>
am: 04cc5c3616

Change-Id: I2b1f68bfc21febb0f5ade62d06815377b03c77eb
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
am: 04cc5c3616

Change-Id: I2b1f68bfc21febb0f5ade62d06815377b03c77eb
</pre>
</div>
</content>
</entry>
</feed>
