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

Eschatos: Debug runtime popup #3951

Closed
waspennator opened this issue Apr 12, 2024 · 10 comments · Fixed by #3955
Closed

Eschatos: Debug runtime popup #3951

waspennator opened this issue Apr 12, 2024 · 10 comments · Fixed by #3955
Labels

Comments

@waspennator
Copy link

waspennator commented Apr 12, 2024

Game seems to work fine, but testing on deck, if I run it with dxvk, this box pops up before launch, if I force wined3d it launches without the box. Same on my windows laptop running normally

320141813-79036bb2-aaef-41cd-b104-effec9f4a01b

Software information

Eschatos

System information

  • GPU: AMD Van Gogh
  • Driver: Mesa 23.1.3
  • Wine version: 9.0
  • DXVK version: Unsure what version is being used in proton

Apitrace file(s)

  • Put a link here

For instructions on how to use apitrace, see: https://github.com/doitsujin/dxvk/wiki/Using-Apitrace

Log files

Please attach Proton or Wine logs as a text file:

  • When using Proton, set the Steam launch options for your game to PROTON_LOG=1 %command% and attach the corresponding steam-xxxxx.log file in your home directory.
  • When using regular Wine, use wine game.exe > game.log 2>&1 and attach the resulting game.log file.

steam-378770.log

@Blisto91
Copy link
Contributor

Blisto91 commented Apr 12, 2024

Can confirm it shows the popup when dxvk is used on Windows also. dgVoodoo2 is fine.
Game uses d3d9Ex if that matters

@WinterSnowfall
Copy link
Contributor

Could you try with this build: dxvk-devel.zip, and see if that does anything?

@K0bin K0bin added the d3d9 label Apr 13, 2024
@waspennator
Copy link
Author

waspennator commented Apr 13, 2024

Testing on my windows laptop, tried with both regular DXVK and the build you provided, still results in the same pop up.

Update: Also providing a trace file incase that helps.
ESCHATOS.zip

@WinterSnowfall
Copy link
Contributor

Thanks for trying it out. Is the trace captured with dxvk, WineD3D or native Windows?

@waspennator
Copy link
Author

waspennator commented Apr 13, 2024

I'm guessing native windows, unsure cause I just downloaded the api trace files, slapped in the d3d9 one into the Eschatos game folder and ran it.

I can get the pop up to show up on both windows and Deck as long as i'm using DXVK

@WinterSnowfall
Copy link
Contributor

I don't see anything out of the ordinary in the trace, except an early call to: IDirect3DDevice9Ex::CreateQuery(this = 0xa5f5dc0, Type = D3DQUERYTYPE_VERTEXSTATS, ppQuery = ?) = D3DERR_NOTAVAILABLE, which apparently fails on native, but should work fine with dxvk.

Here's a dxvk build that returns D3DERR_NOTAVAILABLE for that call to test with: dxvk-devel-2.zip

@waspennator
Copy link
Author

Tried out that Devel-2 build and yep, that seems to fix it, Eschatos launched without that debug pop up box now.

@WinterSnowfall
Copy link
Contributor

Yeah, thought so. According to official specs VERTEXSTATS is only available in debug runtimes: https://learn.microsoft.com/en-us/windows/win32/direct3d9/queries , so it sort of makes sense.

Still, a very silly check for a game to implement. I'll PR something to remove it, but I'm not sure if Josh or others need those queries for debugging/perf. testing or anything else, so we'll see how it goes.

@K0bin
Copy link
Collaborator

K0bin commented Apr 13, 2024

but I'm not sure if Josh or others need those queries for debugging/perf

Would be news to me and if Josh says so, we can still put it behind an environment variable.

@WinterSnowfall
Copy link
Contributor

but I'm not sure if Josh or others need those queries for debugging/perf

Would be news to me and if Josh says so, we can still put it behind an environment variable.

I wouldn't bother if it's not explicitly needed then, since native doesn't support it.

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

Successfully merging a pull request may close this issue.

4 participants