diff options
| author | Jordan Justen <jordan.l.justen@intel.com> | 2024-09-07 11:14:49 -0700 |
|---|---|---|
| committer | Jordan Justen <jordan.l.justen@intel.com> | 2024-09-09 13:54:22 -0700 |
| commit | f40a486013c28a155928c68fcfcfae4f62a36171 (patch) | |
| tree | 2a40025255166c7c15711b3b51aa33e70ffb6573 | |
| parent | 8a43b1e7da907c135953214de84126da4fcbd387 (diff) | |
intel/dev: Fix warning for max_threads_per_psd when devinfo->verx10 == 120fix-120-max-threads-warning
Although we don't want to rely on hwconfig for devinfo->verx10 == 120,
due to the dependence on closed source software, we do check to see if
hwconfig reports different values in the DEVINFO_HWCONFIG macro.
Matt was seeing this warning on 8086:a7a0:
> MESA: warning: INTEL_HWCONFIG_TOTAL_PS_THREADS (128) != devinfo->max_threads_per_psd (64)
Reported-by: Matt Turner <mattst88@gmail.com>
Fixes: 3e4f73b3a0e ("intel/dev: Update hwconfig => max_threads_per_psd for Xe2")
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
| -rw-r--r-- | src/intel/dev/intel_hwconfig.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/intel/dev/intel_hwconfig.c b/src/intel/dev/intel_hwconfig.c index 7ec8ac387eb..55b636e3cc4 100644 --- a/src/intel/dev/intel_hwconfig.c +++ b/src/intel/dev/intel_hwconfig.c @@ -219,7 +219,7 @@ apply_hwconfig_item(struct intel_device_info *devinfo, break; /* ignore */ case INTEL_HWCONFIG_TOTAL_PS_THREADS: { unsigned threads = item->val[0]; - if (devinfo->verx10 == 125) + if (devinfo->ver == 12) threads /= 2; DEVINFO_HWCONFIG(max_threads_per_psd, threads); break; |