[Mesa-dev] [PATCH 6/6] st/dri: list __DRI2_FENCE extension only where needed

Alex Deucher alexdeucher at gmail.com
Fri Jul 7 19:10:58 UTC 2017


On Fri, Jul 7, 2017 at 2:47 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
>
> The extension should be present (if applicable) in the list returned by
> getExtensions(). AFAICT no loader has ever looked for it in
> __driDriverExtensions/__driDriverGetExtensions.
>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>

Not really an expert, but the series seems sane to me:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  src/gallium/state_trackers/dri/dri2.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers/dri/dri2.c
> index 6a977636ea8..3bde9b35c02 100644
> --- a/src/gallium/state_trackers/dri/dri2.c
> +++ b/src/gallium/state_trackers/dri/dri2.c
> @@ -2241,7 +2241,6 @@ const __DRIextension *galliumdrm_driver_extensions[] = {
>      &driImageDriverExtension.base,
>      &driDRI2Extension.base,
>      &gallium_config_options.base,
> -    &dri2FenceExtension.base,
>      NULL
>  };
>
> --
> 2.13.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list