Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2060043002:
Enable zero-copy DXGI video by default. (Closed)
DescriptionEnable zero-copy DXGI video by default.
On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor.
BUG=574292
Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589
Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9
Committed: https://crrev.com/8e03ba3e620bd06b4ec46056020febf1f8151f74
Cr-Original-Original-Commit-Position: refs/heads/master@{#400269}
Cr-Original-Commit-Position: refs/heads/master@{#401724}
Cr-Commit-Position: refs/heads/master@{#409098}
Patch Set 1 #Patch Set 2 : merge master #
Messages
Total messages: 31 (16 generated)
Description was changed from ========== Enable zero-copy DXGI video by default. ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 ==========
jbauman@chromium.org changed reviewers: + piman@chromium.org
lgtm
The CQ bit was checked by jbauman@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2060043002/1
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269}
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2076293002/ by jbauman@chromium.org. The reason for reverting is: Possibly causing crashes in Intel driver. BUG=621048.
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ==========
The CQ bit was checked by jbauman@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2060043002/1
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by jbauman@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2060043002/1
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Cr-Commit-Position: refs/heads/master@{#400269} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Commit-Position: refs/heads/master@{#401724}
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2091293003/ by jbauman@chromium.org. The reason for reverting is: There seems to be a small increase in GPU hangs (and occasionally some corruption) on AMD GPUs. BUG=623029.
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ==========
The CQ bit was checked by jbauman@chromium.org
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/2060043002/#ps20001 (title: "merge master")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ==========
Message was sent while issue was closed.
Committed patchset #2 (id:20001)
Message was sent while issue was closed.
Description was changed from ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Cr-Original-Commit-Position: refs/heads/master@{#400269} Cr-Commit-Position: refs/heads/master@{#401724} ========== to ========== Enable zero-copy DXGI video by default. On systems where we use D3D11 for both hardware video decoding and ANGLE this will allow us to give the NV12 video frame from the decoder directly to the compositor. BUG=574292 Committed: https://crrev.com/6384f843ccb11bfc9ccd3b183115ac2e2ca10589 Committed: https://crrev.com/a25a1312b17284a3464185819e83c2452e133ec9 Committed: https://crrev.com/8e03ba3e620bd06b4ec46056020febf1f8151f74 Cr-Original-Original-Commit-Position: refs/heads/master@{#400269} Cr-Original-Commit-Position: refs/heads/master@{#401724} Cr-Commit-Position: refs/heads/master@{#409098} ==========
Message was sent while issue was closed.
Patchset 2 (id:??) landed as https://crrev.com/8e03ba3e620bd06b4ec46056020febf1f8151f74 Cr-Commit-Position: refs/heads/master@{#409098} |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
