Skip to content

More validation of the surface texture lifetime#1892

Merged
kvark merged 1 commit intogfx-rs:masterfrom
kvark:swapchain-validate
Sep 1, 2021
Merged

More validation of the surface texture lifetime#1892
kvark merged 1 commit intogfx-rs:masterfrom
kvark:swapchain-validate

Conversation

@kvark
Copy link
Copy Markdown
Member

@kvark kvark commented Sep 1, 2021

Connections
Closes #1878

Description
Adds has_work flag to surface textures. This is temporary, until #1688 takes a better care of it. cc @Wumpf
Adds a few error conditions in addition to the "no work done":

  • if we finished recording a pass, but the surface texture is gone
  • if we are presenting a wrong frame to a wrong surface

Testing
Manually tested

@kvark kvark requested a review from cwfitzgerald September 1, 2021 06:07
Copy link
Copy Markdown
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got a micro-nit, but otherwise looks good!

@kvark kvark force-pushed the swapchain-validate branch from 09fecef to 2f63a3a Compare September 1, 2021 20:04
@kvark kvark enabled auto-merge (rebase) September 1, 2021 20:05
@kvark kvark merged commit 53d262c into gfx-rs:master Sep 1, 2021
@kvark kvark deleted the swapchain-validate branch September 2, 2021 00:55
@kvark kvark added PR: needs back-porting PR with a fix that needs to land on crates and removed PR: needs back-porting PR with a fix that needs to land on crates labels Sep 2, 2021
@kvark
Copy link
Copy Markdown
Member Author

kvark commented Sep 2, 2021

Removing the label - technically a breaking change.

waywardmonkeys added a commit to waywardmonkeys/wgpu that referenced this pull request Feb 5, 2024
When no work is submitted for a frame, presenting the surface results
in a timeout due to no work having been submitted.

Fixes gfx-rs#3189.

This flag was added in gfx-rs#1892 with a note that it was going to be
temporary until gfx-rs#1688 landed.
waywardmonkeys added a commit to waywardmonkeys/wgpu that referenced this pull request Feb 5, 2024
When no work is submitted for a frame, presenting the surface results
in a timeout due to no work having been submitted.

Fixes gfx-rs#3189.

This flag was added in gfx-rs#1892 with a note that it was going to be
temporary until gfx-rs#1688 landed.
waywardmonkeys added a commit to waywardmonkeys/wgpu that referenced this pull request Feb 5, 2024
When no work is submitted for a frame, presenting the surface results
in a timeout due to no work having been submitted.

Fixes gfx-rs#3189.

This flag was added in gfx-rs#1892 with a note that it was going to be
temporary until gfx-rs#1688 landed.
nical pushed a commit that referenced this pull request Feb 9, 2024
When no work is submitted for a frame, presenting the surface results
in a timeout due to no work having been submitted.

Fixes #3189.

This flag was added in #1892 with a note that it was going to be
temporary until #1688 landed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants