Crash in render engine due to directx device hanging up #7395

Closed
opened 2026-01-31 01:02:47 +00:00 by claunia · 2 comments
Owner

Originally created by @ghost on GitHub (Apr 13, 2020).

Originally assigned to: @DHowett-MSFT on GitHub.

VSExtensibility team got a watson logged internally with a crash coming from the terminal. After looking into the watson it appears that the crash is coming from this line:

ca33d895a3/src/renderer/base/renderer.cpp (L81)

It looks like an intentional fail fast but it would be better to surface the error and allow the application hosting the rendering engine to decide if it should crash or not.

Originally created by @ghost on GitHub (Apr 13, 2020). Originally assigned to: @DHowett-MSFT on GitHub. VSExtensibility team got a watson logged internally with a crash coming from the terminal. After looking into the watson it appears that the crash is coming from this line: https://github.com/microsoft/terminal/blob/ca33d895a359664c337c6482b853489823c216d3/src/renderer/base/renderer.cpp#L81 It looks like an intentional fail fast but it would be better to surface the error and allow the application hosting the rendering engine to decide if it should crash or not.
Author
Owner

@DHowett-MSFT commented on GitHub (Apr 14, 2020):

I'm gonna take this one on. Two things:

  1. Implementing a backoff on retry, to increase our chances of having a successful render
  2. Implementing a "stop rendering for god's sake" when we fail 3 times, so the application can kick the render engine to life.
@DHowett-MSFT commented on GitHub (Apr 14, 2020): I'm gonna take this one on. Two things: 1. Implementing a backoff on retry, to increase our chances of having a successful render 2. Implementing a "stop rendering for god's sake" when we fail 3 times, so the application can kick the render engine to life.
Author
Owner

@ghost commented on GitHub (Apr 22, 2020):

:tada:This issue was addressed in #5353, which has now been successfully released as Windows Terminal Preview v0.11.1121.0.🎉

Handy links:

@ghost commented on GitHub (Apr 22, 2020): :tada:This issue was addressed in #5353, which has now been successfully released as `Windows Terminal Preview v0.11.1121.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v0.11.1121.0) * [Store Download](https://www.microsoft.com/store/apps/9n0dx20hk701?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#7395