gpu: Add callback for yielding command buffer execution.

gpu: Add callback for pausing command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2763753002
Cr-Commit-Position: refs/heads/master@{#460479}
Committed: https://chromium.googlesource.com/chromium/src/+/0559d3c6822220e0e5d4e1b1e88d865fbb808ba1

[sunnyps, 2017-03-20 21:20:42]

Description was changed from

==========
gpu: Add callback for yielding command buffer execution.

CommandExecutor can take a TryYieldCallback to yield command execution.
This is where the GpuScheduler will hook into.

R=piman@chromium.org
BUG=514813
==========

to

==========
gpu: Add callback for yielding command buffer execution.

CommandExecutor can take a TryYieldCallback to yield command execution.
This is where the GpuScheduler will hook into.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[sunnyps, 2017-03-20 21:20:44]

The CQ bit was checked by sunnyps@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-20 21:21:31]

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

[sunnyps, 2017-03-20 21:44:45]

ptal

[commit-bot: I haz the power, 2017-03-20 23:01:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-20 23:01:42]

Dry run: This issue passed the CQ dry run.

[piman, 2017-03-21 00:11:52]

lgtm

[sunnyps, 2017-03-21 00:20:23]

The CQ bit was checked by sunnyps@chromium.org

[commit-bot: I haz the power, 2017-03-21 00:21:17]

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

[sunnyps, 2017-03-21 00:27:11]

The CQ bit was unchecked by sunnyps@chromium.org

[sunnyps, 2017-03-29 01:18:07]

Description was changed from

==========
gpu: Add callback for yielding command buffer execution.

CommandExecutor can take a TryYieldCallback to yield command execution.
This is where the GpuScheduler will hook into.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
gpu: Add callback for yielding command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[sunnyps, 2017-03-29 01:21:44]

Description was changed from

==========
gpu: Add callback for yielding command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
gpu: Add callback for pausing command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[sunnyps, 2017-03-29 01:24:14]

I renamed the callback from TryYieldCallback to PauseExectionCallback. I'll land
this now, please consider last change as TBR. Thanks!

[sunnyps, 2017-03-29 01:24:31]

The CQ bit was checked by sunnyps@chromium.org

[sunnyps, 2017-03-29 01:24:32]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2763753002/#ps60001
(title: "no need to !!pause")

[commit-bot: I haz the power, 2017-03-29 01:25:20]

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, 2017-03-29 03:27:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-29 03:27:18]

Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_TIMED_OUT, build hasn't
started yet, builder probably lacks capacity)

[piman, 2017-03-29 17:35:22]

lgtm

[sunnyps, 2017-03-29 17:56:30]

The CQ bit was checked by sunnyps@chromium.org

[commit-bot: I haz the power, 2017-03-29 17:57:04]

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, 2017-03-29 19:08:20]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490810190055110,
"parent_rev": "b032d00c1530480e08446ba7660a895c1d23c74e", "commit_rev":
"0559d3c6822220e0e5d4e1b1e88d865fbb808ba1"}

[commit-bot: I haz the power, 2017-03-29 19:09:05]

Description was changed from

==========
gpu: Add callback for pausing command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
gpu: Add callback for pausing command buffer execution.

CommandExecutor can take a PauseExectionCallback to pause command
execution. This is where the PreemptionFlag is hooked into and where
the scheduler will too.

R=piman@chromium.org
BUG=514813
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2763753002
Cr-Commit-Position: refs/heads/master@{#460479}
Committed:
https://chromium.googlesource.com/chromium/src/+/0559d3c6822220e0e5d4e1b1e88d...
==========

[commit-bot: I haz the power, 2017-03-29 19:09:10]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/0559d3c6822220e0e5d4e1b1e88d...