~proton1980/ubuntu/+source/linux/+git/mirror-drm-intel:drm-intel-gt-next

Last commit made on 2021-08-25
Get this branch:
git clone -b drm-intel-gt-next https://git.launchpad.net/~proton1980/ubuntu/+source/linux/+git/mirror-drm-intel
Only Ghz can upload to this branch. If you are Ghz please log in for upload directions.

Branch merges

Branch information

Name:
drm-intel-gt-next
Repository:
lp:~proton1980/ubuntu/+source/linux/+git/mirror-drm-intel

Recent commits

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.

Fixes: 3a7b72665ea5 ("drm/i915/selftest: Bump selftest timeouts for hangcheck")
Reported-by: Dan Carpenter <email address hidden>
Signed-off-by: Nathan Chancellor <email address hidden>
Reviewed-by: Joonas Lahtinen <email address hidden>
Signed-off-by: Joonas Lahtinen <email address hidden>
Link: https://patchwork<email address hidden>

81a14be... by Lucas De Marchi

drm/i915/dg1: remove __maybe_unused leftover

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".

Fixes: cf586021642d ("drm/i915/gt: Pipelined page migration")
Signed-off-by: Dan Carpenter <email address hidden>
Reviewed-by: Thomas Hellström <email address hidden>
Signed-off-by: Rodrigo Vivi <email address hidden>
Link: https://patchwork.freedesktop.org/patch/msgid/20210813113600.GC30697@kili

8c3363c... by Matthew Auld <email address hidden>

drm/i915/debugfs: hook up ttm_resource_manager_debug

This should give a more complete view of the various bits of internal
resource manager state, for device local-memory.

v2(Thomas):
   - Move the region printing into a nice helper

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>

5359b74... by Matthew Auld <email address hidden>

drm/i915/buddy: add some pretty printing

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.

unreferenced object 0xffff88813bc53b18 (size 96):
  comm "gem_close_race", pid 5410, jiffies 4294917818 (age 1105.600s)
  hex dump (first 32 bytes):
    01 00 00 00 00 00 00 00 00 00 00 00 0a 00 00 00 ................
    00 00 00 00 00 00 00 00 6b 6b 6b 6b 06 00 00 00 ........kkkk....
  backtrace:
    [<00000000120b863a>] __sync_alloc_leaf+0x1e/0x40 [i915]
    [<00000000042f6959>] __sync_set+0x1bb/0x240 [i915]
    [<0000000090f0e90f>] i915_request_await_dma_fence+0x1c7/0x400 [i915]
    [<0000000056a48219>] i915_request_await_object+0x222/0x360 [i915]
    [<00000000aaac4ee3>] i915_gem_do_execbuffer+0x1bd0/0x2250 [i915]
    [<000000003c9d830f>] i915_gem_execbuffer2_ioctl+0x405/0xce0 [i915]
    [<00000000fd7a8e68>] drm_ioctl_kernel+0xb0/0xf0 [drm]
    [<00000000e721ee87>] drm_ioctl+0x305/0x3c0 [drm]
    [<000000008b0d8986>] __x64_sys_ioctl+0x71/0xb0
    [<0000000076c362a4>] do_syscall_64+0x33/0x80
    [<00000000eb7a4831>] entry_SYSCALL_64_after_hwframe+0x44/0xa9

Signed-off-by: Matthew Brost <email address hidden>
Fixes: 531958f6f357 ("drm/i915/gt: Track timeline activeness in enter/exit")
Cc: <email address hidden>
Reviewed-by: John Harrison <email address hidden>
Signed-off-by: John Harrison <email address hidden>
Link: https://patchwork.freedesktop<email address hidden>