Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash introduced in [d3d9] Upload DYNAMIC+SYSMEM vertex and index data for each draw (!5312ef1) #3860

Closed
R1chterScale opened this issue Feb 20, 2024 · 3 comments
Assignees
Labels

Comments

@R1chterScale
Copy link

Running a copy of New Vegas using the New Vegas Reloaded graphics mod and noticed a replicable crash was present when starting a new game (and in a few other areas) when on the latest commit. Git bisect revealed it to be [d3d9] Upload DYNAMIC+SYSMEM vertex and index data for each draw (!5312ef1)

Software information

Fallout New Vegas with the New Vegas Reloaded mod (along with other general libraries and QoL mods, but NVR is the one that causes it, specifically the Interior Shadows setting).

System information

  • GPU: RX 7900XT
  • Driver: Mesa-git (RADV)
  • Wine version: 8.0 (from Proton-GE, but crashes on other versions)
  • DXVK version: Built from commit 5312ef1

Apitrace file(s)

FalloutNVout.trace
Worth noting is it had to be made with DXVK, WineD3D was not an option

Log files

game.log

@K0bin K0bin self-assigned this Feb 20, 2024
@K0bin K0bin added the d3d9 label Feb 20, 2024
@Blisto91
Copy link
Contributor

Blisto91 commented Feb 20, 2024

Hello and thank you for the well made issue.

While i was reproducing ingame i took the liberty of making a trace with a older version of dxvk so it gets past the crash, if that shows to be useful to the devs. It still crashes when replayed with the offending commit.
https://drive.proton.me/urls/NDRYZRMZAR#NhFx277Hu4YE

@K0bin
Copy link
Collaborator

K0bin commented Feb 20, 2024

The mod ends up unbinding the index buffer before drawing with it. So it might accidentally rely on the behavior of #3852.

@R1chterScale
Copy link
Author

Thanks again for looking at this and finding a fix so quickly :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants