Commit eed41975 authored by Tao Zhou's avatar Tao Zhou Committed by Alex Deucher

drm/amdkfd: refine event_interrupt_poison_consumption

Combine reading and setting poison flag as one atomic operation
and add print message for the function.
Signed-off-by: default avatarTao Zhou <tao.zhou1@amd.com>
Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
Reviewed-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent c5b26681
...@@ -93,20 +93,19 @@ enum SQ_INTERRUPT_ERROR_TYPE { ...@@ -93,20 +93,19 @@ enum SQ_INTERRUPT_ERROR_TYPE {
static void event_interrupt_poison_consumption(struct kfd_dev *dev, static void event_interrupt_poison_consumption(struct kfd_dev *dev,
uint16_t pasid, uint16_t source_id) uint16_t pasid, uint16_t source_id)
{ {
int ret = -EINVAL; int old_poison, ret = -EINVAL;
struct kfd_process *p = kfd_lookup_process_by_pasid(pasid); struct kfd_process *p = kfd_lookup_process_by_pasid(pasid);
if (!p) if (!p)
return; return;
/* all queues of a process will be unmapped in one time */ /* all queues of a process will be unmapped in one time */
if (atomic_read(&p->poison)) { old_poison = atomic_cmpxchg(&p->poison, 0, 1);
kfd_unref_process(p); kfd_unref_process(p);
if (old_poison)
return; return;
}
atomic_set(&p->poison, 1); pr_warn("RAS poison consumption handling\n");
kfd_unref_process(p);
switch (source_id) { switch (source_id) {
case SOC15_INTSRC_SQ_INTERRUPT_MSG: case SOC15_INTSRC_SQ_INTERRUPT_MSG:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment