Mesa (staging/20.2): ac: Don't negate strstr return values in ac_query_gpu_info

[GitLab Mirror]

Module: Mesa
Branch: staging/20.2
Commit: 493400cf1e2bf5bf81d927ea637d1c5f55c6aae7
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=493400cf1e2bf5bf81d927ea637d1c5f55c6aae7

Author: Michel Dänzer <mdaenzer at redhat.com>
Date:   Wed Nov 18 17:54:19 2020 +0100

ac: Don't negate strstr return values in ac_query_gpu_info

strstr returns a pointer to the needle sub-string within the haystack
string if the latter contains the former, or NULL otherwise. So this
essentially always set info->is_pro_graphics = true, since probably no
marketing name ever contains all of these sub-strings.

Fixes: b635dff25620 "ac: fix detection of Pro graphics"
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7675>

---

 src/amd/common/ac_gpu_info.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c
index 8f3562cf425..3bfa1259ef1 100644
--- a/src/amd/common/ac_gpu_info.c
+++ b/src/amd/common/ac_gpu_info.c
@@ -441,9 +441,9 @@ bool ac_query_gpu_info(int fd, void *dev_p,
 	info->chip_external_rev = amdinfo->chip_external_rev;
 	info->marketing_name = amdgpu_get_marketing_name(dev);
 	info->is_pro_graphics = info->marketing_name &&
-				(!strstr(info->marketing_name, "Pro") ||
-				 !strstr(info->marketing_name, "PRO") ||
-				 !strstr(info->marketing_name, "Frontier"));
+				(strstr(info->marketing_name, "Pro") ||
+				 strstr(info->marketing_name, "PRO") ||
+				 strstr(info->marketing_name, "Frontier"));
 
 	/* Set which chips have dedicated VRAM. */
 	info->has_dedicated_vram =