Skip to content

A
angle
Commit df304e7a by shannon.woods%transgaming.com@gtempaccount.com
Options

When CheckDeviceStats returns S_PRESENT_MODE_CHANGED, reset display mode and…

When CheckDeviceStats returns S_PRESENT_MODE_CHANGED, reset display mode and retest for device lost. Unless the display mode is reset, CheckDeviceLost can keep returning S_PRESENT_MODE_CHANGED, potentially masking a device lost on account of a GPU hang, causing ANGLE to fail to report context lost. Review URL: https://codereview.appspot.com/7381061 TRAC #22720 Signed-off-by: Geoff Lang Signed-off-by: Shannon Woods Author: Jamie Madill git-svn-id: https://angleproject.googlecode.com/svn/branches/dx11proto@2035 736b8ea6-26fd-11df-bfd4-992fa37f6226
parent b9aad2ce
Showing with 14 additions and 0 deletions
src/libGLESv2/renderer/Renderer9.cpp
...@@ -2042,6 +2042,20 @@ bool Renderer9::testDeviceLost(bool notify) ...@@ -2042,6 +2042,20 @@ bool Renderer9::testDeviceLost(bool notify)
if (mDeviceEx) if (mDeviceEx)
{ {
status = mDeviceEx->CheckDeviceState(NULL); status = mDeviceEx->CheckDeviceState(NULL);
if (status == S_PRESENT_MODE_CHANGED)
{
// Reset the device so that D3D stops reporting S_PRESENT_MODE_CHANGED. Otherwise it will report
// it continuously, potentially masking a lost device. D3D resources are not lost on a mode change with WDDM.
D3DPRESENT_PARAMETERS presentParameters = getDefaultPresentParameters();
mDeviceEx->Reset(&presentParameters);
// Reset will not always cause the device loss to be reported so issue a dummy present.
mDeviceEx->Present(NULL, NULL, NULL, NULL);
// Retest the device status to see if the mode change really indicated a lost device.
status = mDeviceEx->CheckDeviceState(NULL);
}
} }
else if (mDevice) else if (mDevice)
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment