[Mesa-dev] [PATCH 4/6] st/dri: use correct __DRI2_CONFIG_QUERY extension

Emil Velikov emil.l.velikov at gmail.com
Fri Jul 7 18:47:49 UTC 2017


From: Emil Velikov <emil.velikov at collabora.com>

If device supports robustness (AKA PIPE_CAP_DEVICE_RESET_STATUS_QUERY)
then we're using dri_robust_screen_extensions.
Hence for such cases the DRI loader was not able to query the driver
options.

This went unnoticed since only r600/radeonsi drivers have the feature.
At the same time neither of them has a local option that the loader
cares about.

Fixes: ff2978b4494 ("st/dri: Allow dri users to query also driver
options")
Cc: Thomas Hellstrom <thellstrom at vmware.com>
Cc: Marek Olšák <marek.olsak at amd.com>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
Perhaps we might want to use it in swrast? If so we'll need to move the
extension to dri_extensions.[ch] first.
---
 src/gallium/state_trackers/dri/dri2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers/dri/dri2.c
index 60ec38d8e44..6a977636ea8 100644
--- a/src/gallium/state_trackers/dri/dri2.c
+++ b/src/gallium/state_trackers/dri/dri2.c
@@ -1995,7 +1995,7 @@ static const __DRIextension *dri_robust_screen_extensions[] = {
    &dri2FlushExtension.base,
    &dri2ImageExtension.base,
    &dri2RendererQueryExtension.base,
-   &dri2ConfigQueryExtension.base,
+   &dri2GalliumConfigQueryExtension.base,
    &dri2ThrottleExtension.base,
    &dri2FenceExtension.base,
    &dri2InteropExtension.base,
-- 
2.13.0



More information about the mesa-dev mailing list