cc: Add cache purging to software image decode controller.

cc: Add cache purging to software image decode controller.

This patch adds code that reacts to memory state change signals to
purge unlocked discardable memory when we're throttled or suspended.

R=ericrk@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/ac08f57b9e35a48416126cedc6177beb257459f7
Cr-Commit-Position: refs/heads/master@{#420750}

[vmpstr, 2016-09-23 18:30:17]

Description was changed from

==========
cc: Add cache purging to software image decode controller.

This patch adds code that reacts to memory state change signals to
purge unlocked discardable memory when we're throttled or suspended.

R=ericrk@chromium.org
==========

to

==========
cc: Add cache purging to software image decode controller.

This patch adds code that reacts to memory state change signals to
purge unlocked discardable memory when we're throttled or suspended.

R=ericrk@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[vmpstr, 2016-09-23 18:33:06]

Please take a look.

[ericrk, 2016-09-23 20:18:52]

https://codereview.chromium.org/2364683004/diff/1/cc/tiles/software_image_dec...
File cc/tiles/software_image_decode_controller.cc (right):

https://codereview.chromium.org/2364683004/diff/1/cc/tiles/software_image_dec...
cc/tiles/software_image_decode_controller.cc:44: const size_t
kSuspectedMaxItemsInCache = 0;
I suspect you meant suspended :D

https://codereview.chromium.org/2364683004/diff/1/cc/tiles/software_image_dec...
cc/tiles/software_image_decode_controller.cc:1103: num_items_in_cache =
kThrottledMaxItemsInCache;
One potential issue is that, if we continue operating in a throttled state, the
cache will build up to the previous kMaxItemsInCache again. Should we store this
somewhere so we can continue enforcing the throttled limit until we change back
to normal?

[vmpstr, 2016-09-23 20:35:18]

PTAL

[ericrk, 2016-09-23 20:54:55]

lgtm

[vmpstr, 2016-09-23 20:55:19]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-09-23 20:56:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 22:41:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-23 22:48:06]

Description was changed from

==========
cc: Add cache purging to software image decode controller.

This patch adds code that reacts to memory state change signals to
purge unlocked discardable memory when we're throttled or suspended.

R=ericrk@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Add cache purging to software image decode controller.

This patch adds code that reacts to memory state change signals to
purge unlocked discardable memory when we're throttled or suspended.

R=ericrk@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/ac08f57b9e35a48416126cedc6177beb257459f7
Cr-Commit-Position: refs/heads/master@{#420750}
==========

[commit-bot: I haz the power, 2016-09-23 22:48:08]

Patchset 2 (id:??) landed as
https://crrev.com/ac08f57b9e35a48416126cedc6177beb257459f7
Cr-Commit-Position: refs/heads/master@{#420750}

[haraken, 2016-09-25 22:41:18]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2016-09-25 22:41:19]

LGTM