[Mesa-dev] [PATCH 1/4] radeon, r200: remove unused variable texmicrotile

Roland Scheidegger sroland at vmware.com
Thu Jul 9 19:54:29 PDT 2015


Ohhh ripping out some more of my _very_ old code, how dare you :-).
Too bad some parts (fast z clear for instance) never made it past UMS
for these chips, quake3 never was as fast again ;-). I wonder how well
things actually work these days with enabled hyperz, it has some, let's
say, limitations...

Reviewed-by: Roland Scheidegger <sroland at vmware.com>



Am 09.07.2015 um 22:13 schrieb Emil Velikov:
> Dead since at least 2009 with commit ccf7814a315(radeon: major cleanups
> removing old dead codepaths.)
> 
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  src/mesa/drivers/dri/r200/r200_context.c     | 3 ---
>  src/mesa/drivers/dri/r200/r200_context.h     | 1 -
>  src/mesa/drivers/dri/radeon/radeon_context.c | 3 ---
>  src/mesa/drivers/dri/radeon/radeon_context.h | 1 -
>  4 files changed, 8 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/r200/r200_context.c b/src/mesa/drivers/dri/r200/r200_context.c
> index fb15082..40cc50a 100644
> --- a/src/mesa/drivers/dri/r200/r200_context.c
> +++ b/src/mesa/drivers/dri/r200/r200_context.c
> @@ -234,9 +234,6 @@ GLboolean r200CreateContext( gl_api api,
>  	 rmesa->using_hyperz = GL_TRUE;
>     }
>   
> -   if ( sPriv->drm_version.minor >= 15 )
> -      rmesa->texmicrotile = GL_TRUE;
> -
>     /* Init default driver functions then plug in our R200-specific functions
>      * (the texture functions are especially important)
>      */
> diff --git a/src/mesa/drivers/dri/r200/r200_context.h b/src/mesa/drivers/dri/r200/r200_context.h
> index eb498f7..2ca1c7a 100644
> --- a/src/mesa/drivers/dri/r200/r200_context.h
> +++ b/src/mesa/drivers/dri/r200/r200_context.h
> @@ -618,7 +618,6 @@ struct r200_context {
>     struct r200_swtcl_info swtcl;
>  
>     GLboolean using_hyperz;
> -   GLboolean texmicrotile;
>  
>    struct ati_fragment_shader *afs_loaded;
>  };
> diff --git a/src/mesa/drivers/dri/radeon/radeon_context.c b/src/mesa/drivers/dri/radeon/radeon_context.c
> index d4d1935..edb154c 100644
> --- a/src/mesa/drivers/dri/radeon/radeon_context.c
> +++ b/src/mesa/drivers/dri/radeon/radeon_context.c
> @@ -199,9 +199,6 @@ r100CreateContext( gl_api api,
>  	 rmesa->using_hyperz = GL_TRUE;
>     }
>  
> -   if ( sPriv->drm_version.minor >= 15 )
> -      rmesa->texmicrotile = GL_TRUE;
> -
>     /* Init default driver functions then plug in our Radeon-specific functions
>      * (the texture functions are especially important)
>      */
> diff --git a/src/mesa/drivers/dri/radeon/radeon_context.h b/src/mesa/drivers/dri/radeon/radeon_context.h
> index 4032532..badabd9 100644
> --- a/src/mesa/drivers/dri/radeon/radeon_context.h
> +++ b/src/mesa/drivers/dri/radeon/radeon_context.h
> @@ -426,7 +426,6 @@ struct r100_context {
>  	struct r100_swtcl_info swtcl;
>  
>  	GLboolean using_hyperz;
> -	GLboolean texmicrotile;
>  
>  	/* Performance counters
>  	 */
> 



More information about the mesa-dev mailing list