On 10.07.2015 05:13, Emil Velikov wrote:
> The original code only half considered hyperz as an option. As per
> previous commit "major != 2 cannot occur" we can simply things, and
> allow users to set the option if they choose to do so.
>
> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com>
> ---
> src/mesa/drivers/dri/r200/r200_context.c | 10 ++--------
> src/mesa/drivers/dri/radeon/radeon_context.c | 9 ++-------
> 2 files changed, 4 insertions(+), 15 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/r200/r200_context.c
> b/src/mesa/drivers/dri/r200/r200_context.c
> index 40cc50a..2a42ab3 100644
> --- a/src/mesa/drivers/dri/r200/r200_context.c
> +++ b/src/mesa/drivers/dri/r200/r200_context.c
> @@ -225,14 +225,8 @@ GLboolean r200CreateContext( gl_api api,
> rmesa->radeon.initialMaxAnisotropy =
> driQueryOptionf(&rmesa->radeon.optionCache,
> "def_max_anisotropy");
>
> - 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 );
> - else
> - rmesa->using_hyperz = GL_TRUE;
> - }
This code only set rmesa->using_hyperz = GL_TRUE if
sPriv->drm_version.major == 1. It was disabled for KMS in commit
e541845959761e9f47d14ade6b58a32db04ef7e4 ("r200: Fix piglit paths test.").
> + if (driQueryOptionb( &rmesa->radeon.optionCache, "hyperz"))
> + rmesa->using_hyperz = GL_TRUE;
This enables it again for KMS. Maybe that's okay though, especially if
the driconf option is disabled by default.
The rest of the series is
Reviewed-by: Michel Dänzer <michel.daen...@amd.com>
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
