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

strange warning message on windows using opengl on nvidia GTX 670 #480

Closed
totaam opened this issue Dec 20, 2013 · 26 comments
Closed

strange warning message on windows using opengl on nvidia GTX 670 #480

totaam opened this issue Dec 20, 2013 · 26 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Dec 20, 2013

Issue migrated from trac ticket # 480

component: client | priority: minor | resolution: fixed

2013-12-20 01:52:35: aradtech created the issue


Windows 7 64 Nvidia GTX 670 Latest drivers

2013-12-19 17:48:17,118 src 8246 type 8251 id 20084 severity 9148 length 119 message \
    Texture state usage warning: Waste of memory: Texture 0 has mipmaps, while its min filter is inconsistent with mipmaps.
2013-12-19 17:48:17,132 src 8246 type 8251 id 20084 severity 9148 length 119 message \
    Texture state usage warning: Waste of memory: Texture 0 has mipmaps, while its min filter is inconsistent with mipmaps.

is repeated over and over in console window.

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 01:52:48: aradtech uploaded file xpra-error.txt (5.2 KiB)

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:24:00: totaam changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:24:00: totaam changed owner from antoine to totaam

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:24:00: totaam edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:24:00: totaam commented


According to this: [https://qt.gitorious.org/qt/qtdeclarative/commit/4fa67b9ef3201214db0b7e6d61288faa33d6ea8d], it is a bogus warning message flood on nVidia driver with the KHR_debug extension active - nVidia seem unwilling to fix it in the driver.

We can probably apply the same fix here.

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:39:35: totaam uploaded file nvidia-dumbmipmapwarning.patch (1.1 KiB)

try to avoid the mipmap warning by explicitly telling nvidia we don't use mipmaps

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:42:10: totaam changed status from assigned to new

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:42:10: totaam changed owner from totaam to aradtech

@totaam
Copy link
Collaborator Author

totaam commented Dec 20, 2013

2013-12-20 02:42:10: totaam commented


Does the [/attachment/ticket/480/nvidia-dumbmipmapwarning.patch] help?

With r5010 and later, does this make the warning disappear? (patch not needed then)

set XPRA_OPENGL_DEBUG=0
xpra_cmd attach ...

@totaam
Copy link
Collaborator Author

totaam commented Dec 23, 2013

2013-12-23 22:21:47: aradtech commented


set XPRA_OPENGL_DEBUG=0 still have warning using r5016

@totaam
Copy link
Collaborator Author

totaam commented Jan 8, 2014

2014-01-08 19:03:22: aradtech commented


Warning still shows with patch applied on r5153.

@totaam
Copy link
Collaborator Author

totaam commented Jan 9, 2014

2014-01-09 07:01:52: totaam commented


Damn, all this unnecessary logging will slow us down and may cause/hide other problems.

Please try the [/attachment/ticket/480/nvidia-dumbmipmapwarning-v2.patch updated patch]. If this still does not solve the problem, please attach the full client log output both with XPRA_OPENGL_DEBUG=0 and with XPRA_OPENGL_DEBUG=1.

@totaam
Copy link
Collaborator Author

totaam commented Jan 9, 2014

2014-01-09 07:37:57: totaam uploaded file nvidia-dumbmipmapwarning-v2.patch (2.2 KiB)

updated patch ensuring we use the same texture identifier

@totaam
Copy link
Collaborator Author

totaam commented Jan 10, 2014

2014-01-10 03:17:51: aradtech commented


That patch fix's the problem :) Good work !

@totaam
Copy link
Collaborator Author

totaam commented Jan 10, 2014

2014-01-10 03:52:31: totaam changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jan 10, 2014

2014-01-10 03:52:31: totaam changed resolution from ** to fixed

@totaam
Copy link
Collaborator Author

totaam commented Jan 10, 2014

2014-01-10 03:52:31: totaam commented


Thanks! Merged in r5158, will backport.

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 05:52:14: antoine changed status from closed to reopened

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 05:52:14: antoine changed resolution from fixed to **

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 05:52:14: antoine commented


Grrr. Causes GL errors on Fedora 20 in gl_window_backing.py at line 256: invalid value

Possibly because the GL driver doesn't like the first parameter: GL_TEXTURE_RECTANGLE_ARB? The spec doesn't list it as being a valid argument... Just because it works on win32 doesn't mean it's right!

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 06:01:04: antoine changed status from reopened to new

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 06:01:04: antoine commented


r5166 uses GL_RECTANGLE_2D instead and works with mesa (tested in a Fedora 20 virtual machine)

Please check that it doesn't cause any regressions on win32: that GL is enabled, that the warning does not re-appear and that it still displays correctly.

Backport for v0.10.x is in 5170.

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2014

2014-01-13 22:30:30: aradtech commented


Version 5178 Tested on Windows with opengl , no messages showing and all seems to be working well.

@totaam
Copy link
Collaborator Author

totaam commented Jan 14, 2014

2014-01-14 01:46:55: totaam changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jan 14, 2014

2014-01-14 01:46:55: totaam changed resolution from ** to fixed

@totaam
Copy link
Collaborator Author

totaam commented Jan 14, 2014

2014-01-14 01:46:55: totaam commented


Thanks - closing again.

@totaam totaam closed this as completed Jan 14, 2014
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

No branches or pull requests

1 participant