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

GT218, Nvidia 340 failure is back on Quadro NVS 300 #368

Closed
illwieckz opened this issue Sep 5, 2020 · 4 comments
Closed

GT218, Nvidia 340 failure is back on Quadro NVS 300 #368

illwieckz opened this issue Sep 5, 2020 · 4 comments

Comments

@illwieckz
Copy link
Member

illwieckz commented Sep 5, 2020

We hardcoded some string detection to fix the same issue with the Geforce 210 (See Unvanquished/Unvanquished#919), but the driver string detection does not match the Quadro NVS 300 and then the bug is reproduceable on current 0.52.0/sync branch.

We currently test for the Geforce string to apply the workaround and then NVS does not match.

I noticed both GPU are using the 340 version of the driver, it may be better to look for it instead.

@illwieckz
Copy link
Member Author

While waiting for a fix, players facing the issue can workaround it by running the game that way:

./daemon -set r_arb_texture_gather 0

@illwieckz
Copy link
Member Author

illwieckz commented Sep 5, 2020

freem owns two GPUs running with the nvidia 340 driver:

  • GeForce 8400 GS rev.2 (NV98, G98) from year 2007,
  • GeForce 8400 GS rev.3 (NVA8, GT218) from year 2010;

Only the GT218 based one reproduces the bug, while the G98 does not reproduce the bug with the same driver (same computer, same system, cards were swapped out).

Because we can assume all GT218 use the 340 driver, we may detect the 340 driver to apply the patch, but it will apply the patch for nothing on some hardware (preventing the use of some optimization). It would still be better to detect all Geforce and Quadro cards…

illwieckz added a commit to illwieckz/Daemon that referenced this issue Sep 6, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
@illwieckz
Copy link
Member Author

The G98-based GPU is also driven by the Nvidia 340 drivers but does not support ARB_texture_gather, so disabling this extension for all GPUs using this driver will not degrade experience for this card (this extension would be disabled because it's missing anyway).

It's very likely the condition to reproduce the bug is not the GT218 GPU itself, but the Nvidia 340 driver advertising the ARB_texture_gather extension.

We have to make sure the bug does not affect newer driver that supports ARB_texture_gather, I'll test a GPU using the Nvidia 390 driver soon. But well, I have a feeling we would have known about it before…

illwieckz added a commit to illwieckz/Daemon that referenced this issue Sep 6, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
@illwieckz illwieckz changed the title GT210, Nvidia 340 failure is back on Quadro NVS 300! GT218, Nvidia 340 failure is back on Quadro NVS 300! Sep 6, 2020
@illwieckz illwieckz changed the title GT218, Nvidia 340 failure is back on Quadro NVS 300! GT218, Nvidia 340 failure is back on Quadro NVS 300 Sep 6, 2020
illwieckz added a commit to illwieckz/Daemon that referenced this issue Sep 10, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
illwieckz added a commit to illwieckz/Daemon that referenced this issue Sep 12, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
illwieckz added a commit to illwieckz/Daemon that referenced this issue Sep 13, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
illwieckz added a commit to illwieckz/Daemon that referenced this issue Oct 12, 2020
…ix DaemonEngine#368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
illwieckz added a commit that referenced this issue Oct 12, 2020
…ix #368

This may also match Quadro cards and not apply the workaround to all
Nvidia products.
@illwieckz
Copy link
Member Author

Fixed by #370.

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

No branches or pull requests

1 participant