Skip to content

Flush staging init buffers#916

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:init-flush
Sep 2, 2020
Merged

Flush staging init buffers#916
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:init-flush

Conversation

@kvark
Copy link
Copy Markdown
Member

@kvark kvark commented Sep 2, 2020

Connections
Related to gfx-rs/gfx#3362
With the new use of write_buffer/write_texture, the results on D3D11 regressed.

Description
It just happened to be the case that all the backends returned coherent staging buffers, used internally for write_* and mapped_at_creation == true. However, D3D11 backend currently doesn't have the coherent memory at all, and it helped me to see that we were missing a flush() on the staging buffer.

Testing
Untested, but should work (tm).

@kvark kvark requested a review from cwfitzgerald September 2, 2020 20:09
@kvark kvark mentioned this pull request Sep 2, 2020
6 tasks
Copy link
Copy Markdown
Contributor

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

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

This is an autogenerated code review.

Checker summary (by rust_clippy):
The tool has found 0 warnings, 1 errors.

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.

LGTM
bors r+

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Sep 2, 2020

@bors bors bot merged commit bbb9b84 into gfx-rs:master Sep 2, 2020
@kvark kvark mentioned this pull request Sep 2, 2020
bors bot added a commit that referenced this pull request Sep 2, 2020
917: Patch 0.6.1 r=straightforward a=kvark

**Connections**
Closes #905 
Includes #916

**Description**
Just back-porting the fixes from master

**Testing**
playtested only

Co-authored-by: Nathan Adams <dinnerbone@dinnerbone.com>
Co-authored-by: Dzmitry Malyshau <dmalyshau@mozilla.com>
Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
@kvark kvark deleted the init-flush branch September 2, 2020 20:41
bors bot added a commit to gfx-rs/wgpu-rs that referenced this pull request Sep 2, 2020
548: wgpu update with correctness fixes r=kvark a=kvark

Picks up a few important correctness fixes, such as gfx-rs/wgpu#916 and gfx-rs/wgpu#912

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
kvark added a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
548: wgpu update with correctness fixes r=kvark a=kvark

Picks up a few important correctness fixes, such as gfx-rs#916 and gfx-rs#912

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
kvark added a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
916: Update capture API r=kvark a=kvark

Picks up gfx-rs#1401

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants