ae4b0ea...
by
Akeem G Abodunrin <email address hidden>
drm/i915/dg2: Add new LRI reg offsets
New LRI register offsets were introduced for DG2, this patch adds
those extra registers, and create new register table for setting offsets
to compare with HW generated context image - especially for gt_lrc test.
Also updates general purpose register with scratch offset for DG2, in
order to use it for live_lrc_fixed selftest.
Cc: Chris P Wilson <email address hidden>
Cc: Prathap Kumar Valsan <email address hidden>
Signed-off-by: Akeem G Abodunrin <email address hidden>
Signed-off-by: Matt Roper <email address hidden>
Reviewed-by: Caz Yokoyama <email address hidden>
Link: https://patchwork.freedesktop<email address hidden>
5e07652...
by
Matthew Brost <email address hidden>
drm/i915/selftests: Increase timeout in i915_gem_contexts selftests
Like in the case of several other selftests, generating lots of requests
in a loop takes a bit longer with GuC submission. Increase a timeout in
i915_gem_contexts selftest to take this into account.
Signed-off-by: Matthew Brost <email address hidden>
Reviewed-by: John Harrison <email address hidden>
Signed-off-by: John Harrison <email address hidden>
Link: https://patchwork.<email address hidden>
6321a72...
by
=?utf-8?b?VmlsbGUgU3lyasOkbMOk?= <email address hidden>
drm/i915: s/0/NULL/
Use NULL where appropriate.
drivers/gpu/drm/i915/gt/intel_ring_submission.c:1210:24: warning: Using plain integer as NULL pointer
Signed-off-by: Ville Syrjälä <email address hidden>
Link: https://patchwork.<email address hidden>
Reviewed-by: Matthew Auld <email address hidden>
ac5a2df...
by
Nathan Chancellor <email address hidden>
drm/i915/selftest: Fix use of err in igt_reset_{fail, nop}_engine()
Clang warns:
In file included from drivers/gpu/drm/i915/gt/intel_reset.c:1514:
drivers/gpu/drm/i915/gt/selftest_hangcheck.c:465:62: warning: variable
'err' is uninitialized when used here [-Wuninitialized] pr_err("[%s] Create context failed: %d!\n", engine->name, err); ^~~
...
drivers/gpu/drm/i915/gt/selftest_hangcheck.c:580:62: warning: variable
'err' is uninitialized when used here [-Wuninitialized] pr_err("[%s] Create context failed: %d!\n", engine->name, err); ^~~
...
2 warnings generated.
This appears to be a copy and paste issue. Use ce directly using the %pe
specifier to pretty print the error code so that err is not used
uninitialized in these functions.
This was added in commit 05e265841f7e ("drm/i915/dg1: add initial DG-1
definitions") so we could continue to add support for DG1 without
risk to expose a broken UAPI. Now that we added DG1 to the PCI ID list
i915 may bind to, remove the leftover.
Fixes: d5ef86b38e4c ("drm/i915: Add pci ids and uapi for DG1")
Signed-off-by: Lucas De Marchi <email address hidden>
Reviewed-by: Maarten Lankhorst <email address hidden>
Link: https://patchwork.freedesktop<email address hidden>
ff12ce2...
by
Dan Carpenter <email address hidden>
drm/i915/gt: Potential error pointer dereference in pinned_context()
If the intel_engine_create_pinned_context() function returns an error
pointer, then dereferencing "ce" will Oops. Use "vm" instead of
"ce->vm".
Implement the debug hook for the buddy resource manager. For this we
want to print out the status of the memory manager, including how much
memory is still allocatable, what page sizes we have etc. This will be
triggered when TTM is unable to fulfil an allocation request for device
local-memory.
v2(Thomas):
- s/MB/MiB
- s/KB/KiB
Signed-off-by: Matthew Auld <email address hidden>
Cc: Thomas Hellström <email address hidden>
Reviewed-by: Thomas Hellström <email address hidden>
Link: https://patchwork.<email address hidden>
c9b6e94...
by
=?utf-8?q?Thomas_Hellstr=C3=B6m?= <email address hidden>
drm/i915: Ditch the i915_gem_ww_ctx loop member
It's only used by the for_i915_gem_ww() macro and we can use
the (typically) on-stack _err variable in its place.
v2:
- Don't clear the _err variable when entering the loop
(Matthew Auld, Maarten Lankhorst).
- Use parentheses around the _err macro argument.
- Fix up comment.
Cc: Matthew Auld <email address hidden>
Suggested-by: Maarten Lankhorst <email address hidden>
Signed-off-by: Thomas Hellström <email address hidden>
Reviewed-by: Matthew Auld <email address hidden>
Signed-off-by: Matthew Auld <email address hidden>
Link: https://patchwork.freedesktop<email address hidden>
faf8909...
by
Matthew Brost <email address hidden>
drm/i915: Fix syncmap memory leak
A small race exists between intel_gt_retire_requests_timeout and
intel_timeline_exit which could result in the syncmap not getting
free'd. Rather than work to hard to seal this race, simply cleanup the
syncmap on fini.