linux/drivers/gpu/drm/amd/amdkfd, branch v4.8

drm/amdkfd: print doorbell offset as a hex value

2016-08-09T07:02:02Z

The doorbell offset is formatted with a 0x prefix to suggest it is a hexadecimal value, when in fact %d is being used and this is confusing. Use %X instead to match the proceeding 0x prefix. Signed-off-by: Colin Ian King Signed-off-by: Oded Gabbay

drm/amdkfd: destroy mutex if process creation fails

2016-07-03T05:05:45Z

Signed-off-by: Oded Gabbay

drm/amdkfd: Remove create_workqueue()

2016-07-03T05:05:45Z

alloc_workqueue replaces deprecated create_workqueue(). create_workqueue has been replaced with alloc_workqueue with max_active as 0 since there is no need for throttling the number of active work items. WQ_MEM_RECLAIM has not been set to because kfd_process_wq will not be used in memory reclaim path. kfd_process_wq is used for delay destruction. A work item embedded in kfd_process gets queued to kfd_process_wq and when it executes it destroys and frees the containing kfd_process and thus itself. This requires a dedicated workqueue because a work item once queued, may get freed at any point of time and any external entity cannot flush the work item. So, in order to wait for such a work item, it needs to be put on a dedicated workqueue. kfd_module_exit() calls kfd_process_destroy_wq which ensures that all pending work items are finished before the module is removed. flush_workqueue is unnecessary since destroy_workqueue() itself calls drain_workqueue() which flushes repeatedly till the workqueue becomes empty. Hence flush_workqueue has been removed. Signed-off-by: Bhaktipriya Shridhar Acked-by: Tejun Heo Signed-off-by: Oded Gabbay

Back-merge tag 'v4.7-rc5' into drm-next

2016-07-02T05:56:01Z

Linux 4.7-rc5 The fsl-dcu pull needs -rc3 so go to -rc5 for now.

drm/amdkfd: Clean up inline handling

2016-06-21T19:32:52Z

- inline functions need to be static inline, otherwise gcc can opt to not inline and the linker gets unhappy. - no forward decls for inline functions, just include the right headers. Cc: Oded Gabbay Cc: Ben Goz Reviewed-by: Oded Gabbay Signed-off-by: Daniel Vetter Link: http://patchwork.freedesktop.org/patch/msgid/1466500235-21282-2-git-send-email-daniel.vetter@ffwll.ch

drm/amdkfd: print once about mem_banks truncation

2016-06-03T05:50:40Z

This print can really spam the kernel log in case we are truncating mem_banks, so just print this info once. It should also not be classified as warning. Signed-off-by: Oded Gabbay

drm/amdkfd: destroy dbgmgr in notifier release

2016-06-03T05:50:40Z

amdkfd need to destroy the debug manager in case amdkfd's notifier function is called before the unbind function, because in that case, the unbind function will exit without destroying debug manager. Signed-off-by: Oded Gabbay CC: Stable

drm/amdkfd: unbind only existing processes

2016-06-03T05:50:40Z

When unbinding a process from a device (initiated by amd_iommu_v2), the driver needs to make sure that process still exists in the process table. There is a possibility that amdkfd's own notifier handler - kfd_process_notifier_release() - was called before the unbind function and it already removed the process from the process table. v2: Because there can be only one process with the specified pasid, and because *p can't be NULL inside the hash_for_each_rcu macro, it is more reasonable to just put the whole code inside the if statement that compares the pasid value. That way, when we exit hash_for_each_rcu, we simply exit the function as well. Signed-off-by: Oded Gabbay CC: Stable

amdkfd: Trim unnescessary intermediate err var in kfd_chardev.c

2016-04-30T14:06:29Z

Found-By: Coccinelle Signed-off-by: Edward O'Callaghan Signed-off-by: Oded Gabbay

amdkfd: Trim off unnescessary semicolon from kfd_packet_manager.c

2016-04-30T14:06:28Z

Found-By: Coccinelle Signed-off-by: Edward O'Callaghan Signed-off-by: Oded Gabbay