<feed xmlns='http://www.w3.org/2005/Atom'>
<title>platform_hardware_google_pixel/powerstats, branch android10-s1-release</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/'/>
<entry>
<title>power.stats: Modify wlan data provider to not fail if wifi off</title>
<updated>2019-05-10T17:23:01+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-05-06T18:31:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=a1f868e78ee87bff55d10ad60f28f1e4a803043c'/>
<id>a1f868e78ee87bff55d10ad60f28f1e4a803043c</id>
<content type='text'>
Previous the WlanStateResidencyDataProvider was returning a failure when
it failed to open the wlan node to read the wlan power stats. This node
is not avaiable when wifi is off. Since this is intended behavior we
should not return a failure.

Bug: 130202935
Test: vts-tradefed run vts -m VtsHalPowerStatsV1_0Target
Change-Id: I9fa288bcb41c8ce07f40d580c1f82a809b3367f8
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Previous the WlanStateResidencyDataProvider was returning a failure when
it failed to open the wlan node to read the wlan power stats. This node
is not avaiable when wifi is off. Since this is intended behavior we
should not return a failure.

Bug: 130202935
Test: vts-tradefed run vts -m VtsHalPowerStatsV1_0Target
Change-Id: I9fa288bcb41c8ce07f40d580c1f82a809b3367f8
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Improve header finding in parser</title>
<updated>2019-03-26T22:11:24+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-03-26T22:07:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=f161635ebd34e6a1ddac03bb1ca0f7274ca5ded8'/>
<id>f161635ebd34e6a1ddac03bb1ca0f7274ca5ded8</id>
<content type='text'>
Bug: 129298740
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Change-Id: Ib5e9c16af41f62245cabda708df1dbeded737c51
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Bug: 129298740
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Change-Id: Ib5e9c16af41f62245cabda708df1dbeded737c51
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Rename column header for rail energy</title>
<updated>2019-03-18T23:13:55+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-03-18T23:09:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=fc0aa7b2d3fb3eb56926be9e678effaf9f03f1b1'/>
<id>fc0aa7b2d3fb3eb56926be9e678effaf9f03f1b1</id>
<content type='text'>
lshal debug output was not clear that rail energy is cumulative total
since boot. Renaming the column header to clarify this.

Bug: 124080758
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Change-Id: Ib7cff1e9bd9241eb2bbd73b4a5bc1604883a4178
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
lshal debug output was not clear that rail energy is cumulative total
since boot. Renaming the column header to clarify this.

Bug: 124080758
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Change-Id: Ib7cff1e9bd9241eb2bbd73b4a5bc1604883a4178
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Show rail energy data in lshal debug</title>
<updated>2019-03-18T18:31:30+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-03-15T20:16:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=6663d0afbf39ab46835e3d517ee60e3b5dd9f2f3'/>
<id>6663d0afbf39ab46835e3d517ee60e3b5dd9f2f3</id>
<content type='text'>
lshal debug of power.stats@1.0 service now shows rail energy data.

Example output:
============= PowerStats HAL 1.0 rail energy data ==============
          Subsys                 Rail               Energy
      &lt;example1&gt;          &lt;railName1&gt;         13087.34 mWs
      &lt;example2&gt;          &lt;railName2&gt;           267.46 mWs
========== End of PowerStats HAL 1.0 rail energy data ==========

Example output with "delta" option
============= PowerStats HAL 1.0 rail energy data ==============
Elapsed time: 86273 ms
          Subsys                 Rail               Energy (      Delta   )
      &lt;example1&gt;          &lt;railName1&gt;         18496.73 mWs (     6055.16)
      &lt;example2&gt;          &lt;railName2&gt;           757.76 mWs (      490.30)
========== End of PowerStats HAL 1.0 rail energy data ==========

Bug: 124080758
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Test: Tried on devices with and without rail energy support
Change-Id: I7421aedac07203e14322dc23712cb68f508fcb7c
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
lshal debug of power.stats@1.0 service now shows rail energy data.

Example output:
============= PowerStats HAL 1.0 rail energy data ==============
          Subsys                 Rail               Energy
      &lt;example1&gt;          &lt;railName1&gt;         13087.34 mWs
      &lt;example2&gt;          &lt;railName2&gt;           267.46 mWs
========== End of PowerStats HAL 1.0 rail energy data ==========

Example output with "delta" option
============= PowerStats HAL 1.0 rail energy data ==============
Elapsed time: 86273 ms
          Subsys                 Rail               Energy (      Delta   )
      &lt;example1&gt;          &lt;railName1&gt;         18496.73 mWs (     6055.16)
      &lt;example2&gt;          &lt;railName2&gt;           757.76 mWs (      490.30)
========== End of PowerStats HAL 1.0 rail energy data ==========

Bug: 124080758
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Test: Tried on devices with and without rail energy support
Change-Id: I7421aedac07203e14322dc23712cb68f508fcb7c
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Add AidlStateResidencyDataProvider</title>
<updated>2019-02-26T23:05:17+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-02-06T20:37:57+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=6317ff3771ad3cae10b6159463ab549d54a13741'/>
<id>6317ff3771ad3cae10b6159463ab549d54a13741</id>
<content type='text'>
Some drivers that we want to collect power stats from are pass-through
and would require significant changes in order to provide a sysfs node
that the current power stats framework can pull stats from.

These pass-through drivers typically have a userspace daemon that is
capable of keeping track of the power stats and providing them to the
power.stats HAL service, but currently have no way to do that.

The AidlStateResidencyDataProvider will provide a vendor binder service
that will allow data sources to register a callback that will get
invoked when stats are requested from power.stats HAL.

Bug: 117585786
Test: Created a dummy service to register a callback using that provides
data for mock entities. Tested that things continue to function when
killing either the dummy service or killing power.stats service. Also
tested cases where the dummy service provides data for unrecognized
entities, or states. Finally, I tested that an error gets logged in
logcat if the callback latency exceeds some threshold.

Change-Id: I090a33a4b87fc03fc258bece3ee38c90a3575d20
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Some drivers that we want to collect power stats from are pass-through
and would require significant changes in order to provide a sysfs node
that the current power stats framework can pull stats from.

These pass-through drivers typically have a userspace daemon that is
capable of keeping track of the power stats and providing them to the
power.stats HAL service, but currently have no way to do that.

The AidlStateResidencyDataProvider will provide a vendor binder service
that will allow data sources to register a callback that will get
invoked when stats are requested from power.stats HAL.

Bug: 117585786
Test: Created a dummy service to register a callback using that provides
data for mock entities. Tested that things continue to function when
killing either the dummy service or killing power.stats service. Also
tested cases where the dummy service provides data for unrecognized
entities, or states. Finally, I tested that an error gets logged in
logcat if the callback latency exceeds some threshold.

Change-Id: I090a33a4b87fc03fc258bece3ee38c90a3575d20
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats HAL: optional "delta" arg to debug</title>
<updated>2019-02-14T01:31:39+00:00</updated>
<author>
<name>Kelly Rossmoyer</name>
<email>krossmo@google.com</email>
</author>
<published>2019-02-13T22:52:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=741f76c727562eb2101615631d6da348ed9cb83f'/>
<id>741f76c727562eb2101615631d6da348ed9cb83f</id>
<content type='text'>
This CL adds a built-in delta tracking/calculation facility in
power.stats HAL's implementation of the standard debug() interface.
Right now, doing
    lshal debug android.hardware.power.stats@1.0::IPowerStats/default
only displays the current values of all supported power stats.  With
this CL in place, that command now looks for the optional "delta"
argument at the end of the line.  When it is present, behavior changes
as follows:
* a timestamp is captured for the current values
* a snapshot  is captured of the current values
* deltas from the previous timestamp and snapshot are displayed
* the very first time "delta" is used after boot, the calculated
  deltas show up as "none"

Note that no behavior is changed when the command is run without the
optional "delta" arg, and interspersing non-"delta" commands with
"delta" commands won't have any impact (the timestamp and snapshot are
not modified by non-"delta" commands).

See bug for example output.

Bug: 124388963
Test: Ran on multiple device types with various power consumption
configurations and verified that the output was both complete and
accurate.

Change-Id: Ib8d8ae7bf0e6000f4934d7b6caec13d3f5b3115b
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This CL adds a built-in delta tracking/calculation facility in
power.stats HAL's implementation of the standard debug() interface.
Right now, doing
    lshal debug android.hardware.power.stats@1.0::IPowerStats/default
only displays the current values of all supported power stats.  With
this CL in place, that command now looks for the optional "delta"
argument at the end of the line.  When it is present, behavior changes
as follows:
* a timestamp is captured for the current values
* a snapshot  is captured of the current values
* deltas from the previous timestamp and snapshot are displayed
* the very first time "delta" is used after boot, the calculated
  deltas show up as "none"

Note that no behavior is changed when the command is run without the
optional "delta" arg, and interspersing non-"delta" commands with
"delta" commands won't have any impact (the timestamp and snapshot are
not modified by non-"delta" commands).

See bug for example output.

Bug: 124388963
Test: Ran on multiple device types with various power consumption
configurations and verified that the output was both complete and
accurate.

Change-Id: Ib8d8ae7bf0e6000f4934d7b6caec13d3f5b3115b
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats adding OWNERS file</title>
<updated>2019-02-05T01:40:41+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-02-05T01:39:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=a4110e1466ea2b5c76f53444d744ffad3dabffa1'/>
<id>a4110e1466ea2b5c76f53444d744ffad3dabffa1</id>
<content type='text'>
Adding OWNERS file to power stats common code

Bug: 117284995
Test: NA
Change-Id: Ic1e6868d139c0657d96070e004599d84e8993161
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Adding OWNERS file to power stats common code

Bug: 117284995
Test: NA
Change-Id: Ic1e6868d139c0657d96070e004599d84e8993161
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Clean up pixel specific implementation</title>
<updated>2019-01-18T18:05:33+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-01-16T19:16:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=383b12972a37bead9c4b4a2a14dc0d7345960c54'/>
<id>383b12972a37bead9c4b4a2a14dc0d7345960c54</id>
<content type='text'>
- IStateResidencyDataProvider now uses std::unordered_map instead of
  std::map. This was originally intended to use an unordered_map.
- Changed PowerStats::debug header to "Entity" instead of "Subsystem"
- Removed dead code related to dumping of statistics for the old power
  HAL

Bug: 117887759
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Observed the following output on C2F2:
========== PowerStats HAL 1.0 state residencies ==========
          Entity            State         Total time     Total entries   Last entry timestamp
        Airbrush           Active               0 ms                 0               0 ms
        Airbrush            Sleep               0 ms                 0               0 ms
        Airbrush       Deep sleep               0 ms                 0               0 ms
        Airbrush          Suspend               0 ms                 0               0 ms
        Airbrush              Off               0 ms                 1             912 ms
        Airbrush          Unknown               0 ms                 0               0 ms
            WLAN           Active           31436 ms                 0               0 ms
            WLAN       Deep-Sleep               0 ms                 0               0 ms
             SoC             AOSD               0 ms                 0               0 ms
             SoC             CXSD               0 ms                 0               0 ms
             SoC              DDR               0 ms                 0               0 ms
            SLPI            Sleep           13747 ms               759           31038 ms
            CDSP            Sleep           15165 ms                98           26055 ms
            ADSP            Sleep           14327 ms               160           25012 ms
            MPSS            Sleep               5 ms                 4           22409 ms
            APSS            Sleep               0 ms                 0               0 ms
========== End of PowerStats HAL 1.0 state residencies ==========

Change-Id: I6a5bc139cfe568a3d5f8876dfc64e9c3725aeb82
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- IStateResidencyDataProvider now uses std::unordered_map instead of
  std::map. This was originally intended to use an unordered_map.
- Changed PowerStats::debug header to "Entity" instead of "Subsystem"
- Removed dead code related to dumping of statistics for the old power
  HAL

Bug: 117887759
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Observed the following output on C2F2:
========== PowerStats HAL 1.0 state residencies ==========
          Entity            State         Total time     Total entries   Last entry timestamp
        Airbrush           Active               0 ms                 0               0 ms
        Airbrush            Sleep               0 ms                 0               0 ms
        Airbrush       Deep sleep               0 ms                 0               0 ms
        Airbrush          Suspend               0 ms                 0               0 ms
        Airbrush              Off               0 ms                 1             912 ms
        Airbrush          Unknown               0 ms                 0               0 ms
            WLAN           Active           31436 ms                 0               0 ms
            WLAN       Deep-Sleep               0 ms                 0               0 ms
             SoC             AOSD               0 ms                 0               0 ms
             SoC             CXSD               0 ms                 0               0 ms
             SoC              DDR               0 ms                 0               0 ms
            SLPI            Sleep           13747 ms               759           31038 ms
            CDSP            Sleep           15165 ms                98           26055 ms
            ADSP            Sleep           14327 ms               160           25012 ms
            MPSS            Sleep               5 ms                 4           22409 ms
            APSS            Sleep               0 ms                 0               0 ms
========== End of PowerStats HAL 1.0 state residencies ==========

Change-Id: I6a5bc139cfe568a3d5f8876dfc64e9c3725aeb82
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "power.stats: Clean up pixel specific implementation"</title>
<updated>2019-01-17T00:53:31+00:00</updated>
<author>
<name>Ryan Mitchell</name>
<email>rtmitchell@google.com</email>
</author>
<published>2019-01-17T00:53:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=beac86e38326facacfdec102ec89c88a0dea6916'/>
<id>beac86e38326facacfdec102ec89c88a0dea6916</id>
<content type='text'>
This reverts commit 2562e2e62f1032c8dd9ec2844ba163ce734a2be5.

Reason for revert: &lt;Broke master build&gt;

Change-Id: I89da955ddf60c34023319954c3ab77f4ac0a5934
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reverts commit 2562e2e62f1032c8dd9ec2844ba163ce734a2be5.

Reason for revert: &lt;Broke master build&gt;

Change-Id: I89da955ddf60c34023319954c3ab77f4ac0a5934
</pre>
</div>
</content>
</entry>
<entry>
<title>power.stats: Clean up pixel specific implementation</title>
<updated>2019-01-16T19:24:46+00:00</updated>
<author>
<name>Benjamin Schwartz</name>
<email>bsschwar@google.com</email>
</author>
<published>2019-01-16T19:16:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_hardware_google_pixel/commit/?id=2562e2e62f1032c8dd9ec2844ba163ce734a2be5'/>
<id>2562e2e62f1032c8dd9ec2844ba163ce734a2be5</id>
<content type='text'>
- IStateResidencyDataProvider now uses std::unordered_map instead of
  std::map. This was originally intended to use an unordered_map.
- Changed PowerStats::debug header to "Entity" instead of "Subsystem"
- Removed dead code related to dumping of statistics for the old power
  HAL

Bug: 117887759
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Observed the following output on C2F2:
========== PowerStats HAL 1.0 state residencies ==========
          Entity            State         Total time     Total entries   Last entry timestamp
        Airbrush           Active               0 ms                 0               0 ms
        Airbrush            Sleep               0 ms                 0               0 ms
        Airbrush       Deep sleep               0 ms                 0               0 ms
        Airbrush          Suspend               0 ms                 0               0 ms
        Airbrush              Off               0 ms                 1             912 ms
        Airbrush          Unknown               0 ms                 0               0 ms
            WLAN           Active           31436 ms                 0               0 ms
            WLAN       Deep-Sleep               0 ms                 0               0 ms
             SoC             AOSD               0 ms                 0               0 ms
             SoC             CXSD               0 ms                 0               0 ms
             SoC              DDR               0 ms                 0               0 ms
            SLPI            Sleep           13747 ms               759           31038 ms
            CDSP            Sleep           15165 ms                98           26055 ms
            ADSP            Sleep           14327 ms               160           25012 ms
            MPSS            Sleep               5 ms                 4           22409 ms
            APSS            Sleep               0 ms                 0               0 ms
========== End of PowerStats HAL 1.0 state residencies ==========

Change-Id: I85947fa745ba64f8f3c8ec54c0d61c0e19487289
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- IStateResidencyDataProvider now uses std::unordered_map instead of
  std::map. This was originally intended to use an unordered_map.
- Changed PowerStats::debug header to "Entity" instead of "Subsystem"
- Removed dead code related to dumping of statistics for the old power
  HAL

Bug: 117887759
Test: adb shell "lshal debug android.hardware.power.stats@1.0::IPowerStats/default"
Observed the following output on C2F2:
========== PowerStats HAL 1.0 state residencies ==========
          Entity            State         Total time     Total entries   Last entry timestamp
        Airbrush           Active               0 ms                 0               0 ms
        Airbrush            Sleep               0 ms                 0               0 ms
        Airbrush       Deep sleep               0 ms                 0               0 ms
        Airbrush          Suspend               0 ms                 0               0 ms
        Airbrush              Off               0 ms                 1             912 ms
        Airbrush          Unknown               0 ms                 0               0 ms
            WLAN           Active           31436 ms                 0               0 ms
            WLAN       Deep-Sleep               0 ms                 0               0 ms
             SoC             AOSD               0 ms                 0               0 ms
             SoC             CXSD               0 ms                 0               0 ms
             SoC              DDR               0 ms                 0               0 ms
            SLPI            Sleep           13747 ms               759           31038 ms
            CDSP            Sleep           15165 ms                98           26055 ms
            ADSP            Sleep           14327 ms               160           25012 ms
            MPSS            Sleep               5 ms                 4           22409 ms
            APSS            Sleep               0 ms                 0               0 ms
========== End of PowerStats HAL 1.0 state residencies ==========

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