Attachment 41038 Details for Bug 32325 – [PATCH] Don't even consider enabling HyperZ or microtiling unless DRM major version is 1.

[patch] [PATCH] Don't even consider enabling HyperZ or microtiling unless DRM major version is 1.

radeon.diff (text/plain), 1.10 KB, created by Chris Rankin on 2010-12-12 06:07:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Rankin

Created: 2010-12-12 06:07:52 UTC

Size: 1.10 KB

patch

obsolete

>diff --git a/src/mesa/drivers/dri/radeon/radeon_context.c b/src/mesa/drivers/dri/radeon/radeon_context.c
>index e3de534..f52b7ca 100644
>--- a/src/mesa/drivers/dri/radeon/radeon_context.c
>+++ b/src/mesa/drivers/dri/radeon/radeon_context.c
>@@ -246,7 +246,8 @@ r100CreateContext( gl_api api,
>    rmesa->radeon.initialMaxAnisotropy = driQueryOptionf(&rmesa->radeon.optionCache,
>                                                  "def_max_anisotropy");
> 
>-   if ( driQueryOptionb( &rmesa->radeon.optionCache, "hyperz" ) ) {
>+   if ( sPriv->drm_version.major == 1
>+       && driQueryOptionb( &rmesa->radeon.optionCache, "hyperz" ) ) {
>       if ( sPriv->drm_version.minor < 13 )
> 	 fprintf( stderr, "DRM version 1.%d too old to support HyperZ, "
> 			  "disabling.\n", sPriv->drm_version.minor );
>@@ -254,7 +255,7 @@ r100CreateContext( gl_api api,
> 	 rmesa->using_hyperz = GL_TRUE;
>    }
> 
>-   if ( sPriv->drm_version.minor >= 15 )
>+   if ( (sPriv->drm_version.major == 1) && (sPriv->drm_version.minor >= 15) )
>       rmesa->texmicrotile = GL_TRUE;
> 
>    /* Init default driver functions then plug in our Radeon-specific functions

Actions: View | Splinter Review

Attachments on bug 32325: 41020 | 41038

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.