New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 726557 link

Starred by 2 users
Status: Fixed
Owner:
█ stanisc@chromium.org
not on Chrome anymore
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

Significant stability regression on Win 8.0 with D3D V-sync experiment

Project Member Reported by stanisc@chromium.org, May 26 2017 
Finch data suggests that D3D V-sync experiment doesn't work well on Windows 8.0.

I suspect there is the same kind of internal locking in D3D calls that I saw on Windows 7. 

Comparing the crashes between the experiment and the control group I see some interesting crashes unique to the experiment, for example:

[Out of Memory] base::internal::IncomingTaskQueue::PostPendingTask.

I think that likely means that the GPU pipeline is chocking due to locking.
Some of the UMA metrics support that as well.
Project Member

Comment 1 by bugdroid1@chromium.org, May 26 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2fde6d35bd484cc0cb9a4b7964f0a5e2cf000a6a

commit 2fde6d35bd484cc0cb9a4b7964f0a5e2cf000a6a
Author: stanisc <stanisc@chromium.org>
Date: Fri May 26 16:36:38 2017

Limit D3D V-sync experiment to Win 8.1+

Previously D3D V-sync experiment included Win 8.0 but the initial
finch data suggests that it has issues on Win 8.0, for example GPU
crash rate is 8 times higher than in the control group.
I suspect there is the same internal D3D locking issue that I saw
on my Windows 7 workstation.

This patch changes the supported Windows version to start with Win 8.1.

BUG= 726557 
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/2907753002
Cr-Commit-Position: refs/heads/master@{#475029}

[modify] https://crrev.com/2fde6d35bd484cc0cb9a4b7964f0a5e2cf000a6a/content/browser/compositor/gpu_process_transport_factory.cc
[modify] https://crrev.com/2fde6d35bd484cc0cb9a4b7964f0a5e2cf000a6a/gpu/ipc/service/image_transport_surface_win.cc

Comment 2 by stanisc@chromium.org, May 30 2017

Status: Fixed (was: Started)
►

Sign in to add a comment