linux.git/drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c, branch v4.12

drm/amdgpu: Avoid using signed integer to store pointer value

2017-04-06T17:28:07+00:00

Signed-off-by: Alex Xie 
Reviewed-by: Alex Deucher 
Signed-off-by: Alex Deucher

idr: Return the deleted entry from idr_remove

2017-02-14T02:44:03+00:00

It is a relatively common idiom (8 instances) to first look up an IDR
entry, and then remove it from the tree if it is found, possibly doing
further operations upon the entry afterwards.  If we change idr_remove()
to return the removed object, all of these users can save themselves a
walk of the IDR tree.

Signed-off-by: Matthew Wilcox

drm/amdgpu: Attach exclusive fence to prime exported bo's. (v5)

2016-11-11T15:20:04+00:00

External clients which import our bo's wait only
for exclusive dmabuf-fences, not on shared ones,
ditto for bo's which we import from external
providers and write to.

Therefore attach exclusive fences on prime shared buffers
if our exported buffer gets imported by an external
client, or if we import a buffer from an external
exporter.

See discussion in thread:
https://lists.freedesktop.org/archives/dri-devel/2016-October/122370.html

Prime export tested on Intel iGPU + AMD Tonga dGPU as
DRI3/Present Prime render offload, and with the Tonga
standalone as primary gpu.

v2: Add a wait for all shared fences before prime export,
    as suggested by Christian Koenig.

v3: - Mark buffer prime_exported in amdgpu_gem_prime_pin,
    so we only use the exclusive fence when exporting a
    bo to external clients like a separate iGPU, but not
    when exporting/importing from/to ourselves as part of
    regular DRI3 fd passing.

    - Propagate failure of reservation_object_wait_rcu back
    to caller.

v4: - Switch to a prime_shared_count counter instead of a
      flag, which gets in/decremented on prime_pin/unpin, so
      we can switch back to shared fences if all clients
      detach from our exported bo.

    - Also switch to exclusive fence for prime imported bo's.

v5: - Drop lret, instead use int ret -> long ret, as proposed
      by Christian.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95472
Tested-by: Mike Lothian  (v1)
Signed-off-by: Mario Kleiner 
Reviewed-by: Christian König .
Cc: Christian König 
Cc: Michel Dänzer 
Signed-off-by: Alex Deucher 
Cc: stable@vger.kernel.org

drm/amd/amdgpu : adding new tracepoints to track memory information.

2016-07-07T18:51:29+00:00

 - adding amdgpu_cs_bo_status to track total size and
   total entry count of bo for each submission.
 - adding amdgpu_ttm_bo_move to track the bo eviction
   including the size of bo and the location before/after the move

Signed-off-by: David Mao 
Reviewed-by: Christian König 
Reviewed-by: Chunming Zhou 
Signed-off-by: Alex Deucher

drm: Remove unused drm_device from drm_gem_object_lookup()

2016-05-17T06:47:30+00:00

drm_gem_object_lookup() has never required the drm_device for its file
local translation of the user handle to the GEM object. Let's remove the
unused parameter and save some space.

Signed-off-by: Chris Wilson 
Cc: dri-devel@lists.freedesktop.org
Cc: Dave Airlie 
Cc: Daniel Vetter 
[danvet: Fixup kerneldoc too.]
Signed-off-by: Daniel Vetter

drm/amd: cleanup remaining spaces and tabs v2

2016-05-11T16:31:20+00:00

This is the result of running the following commands:
find drivers/gpu/drm/amd/ -name "*.h" -exec sed -i 's/[ \t]\+$//' {} \;
find drivers/gpu/drm/amd/ -name "*.c" -exec sed -i 's/[ \t]\+$//' {} \;
find drivers/gpu/drm/amd/ -name "*.h" -exec sed -i 's/ \+\t/\t/' {} \;
find drivers/gpu/drm/amd/ -name "*.c" -exec sed -i 's/ \+\t/\t/' {} \;

v2: drop changes to DAL and internal headers

Signed-off-by: Christian König 
Reviewed-by: Alex Deucher 
Signed-off-by: Alex Deucher

drm/amdgpu: move get_user_pages out of amdgpu_ttm_tt_pin_userptr v6

2016-03-08T16:01:50+00:00

That avoids lock inversion between the BO reservation lock
and the anon_vma lock.

v2:
* Changed amdgpu_bo_list_entry.user_pages to an array of pointers
* Lock mmap_sem only for get_user_pages
* Added invalidation of unbound userpointer BOs
* Fixed memory leak and page reference leak

v3 (chk):
* Revert locking mmap_sem only for_get user_pages
* Revert adding invalidation of unbound userpointer BOs
* Sanitize and fix error handling

v4 (chk):
* Init userpages pointer everywhere.
* Fix error handling when get_user_pages() fails.
* Add invalidation of unbound userpointer BOs again.

v5 (chk):
* Add maximum number of tries.

v6 (chk):
* Fix error handling when we run out of tries.

Signed-off-by: Christian König 
Signed-off-by: Felix Kuehling 
Reviewed-by: Felix Kuehling  (v4)
Acked-by: Alex Deucher

drm/amdgpu: group userptr in the BO list v2

2016-03-08T16:01:45+00:00

We need them together with the next patch.

v2: Don't take bo reference twice

Signed-off-by: Christian König 
Signed-off-by: Felix Kuehling 
Reviewed-by: Felix Kuehling 
Signed-off-by: Alex Deucher

drm/amdgpu: fix error handling in amdgpu_bo_list_set

2016-02-26T06:50:18+00:00

Don't leak BOs in case of some error.

Signed-off-by: Christian König 
Reviewed-by: Alex Deucher

drm/amdgpu: check userptrs mm earlier

2016-02-10T19:17:16+00:00

Instead of when we try to bind it check the usermm when
we try to use it in the IOCTLs.

Signed-off-by: Christian König 
Acked-by: Alex Deucher