mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-27 12:57:53 +00:00
drm/amdkfd: range check cp bad op exception interrupts
[ Upstream commit 0cac183b98
]
Due to a CP interrupt bug, bad packet garbage exception codes are raised.
Do a range check so that the debugger and runtime do not receive garbage
codes.
Update the user api to guard exception code type checking as well.
Signed-off-by: Jonathan Kim <jonathan.kim@amd.com>
Tested-by: Jesse Zhang <jesse.zhang@amd.com>
Reviewed-by: Felix Kuehling <felix.kuehling@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
77c76b8c12
commit
b6735bfe94
4 changed files with 20 additions and 6 deletions
|
@ -336,7 +336,8 @@ static void event_interrupt_wq_v10(struct kfd_node *dev,
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
kfd_signal_event_interrupt(pasid, context_id0 & 0x7fffff, 23);
|
kfd_signal_event_interrupt(pasid, context_id0 & 0x7fffff, 23);
|
||||||
} else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE) {
|
} else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE &&
|
||||||
|
KFD_DBG_EC_TYPE_IS_PACKET(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0))) {
|
||||||
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
||||||
KFD_DEBUG_DOORBELL_ID(context_id0),
|
KFD_DEBUG_DOORBELL_ID(context_id0),
|
||||||
KFD_EC_MASK(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0)),
|
KFD_EC_MASK(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0)),
|
||||||
|
|
|
@ -325,7 +325,8 @@ static void event_interrupt_wq_v11(struct kfd_node *dev,
|
||||||
/* CP */
|
/* CP */
|
||||||
if (source_id == SOC15_INTSRC_CP_END_OF_PIPE)
|
if (source_id == SOC15_INTSRC_CP_END_OF_PIPE)
|
||||||
kfd_signal_event_interrupt(pasid, context_id0, 32);
|
kfd_signal_event_interrupt(pasid, context_id0, 32);
|
||||||
else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE)
|
else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE &&
|
||||||
|
KFD_DBG_EC_TYPE_IS_PACKET(KFD_CTXID0_CP_BAD_OP_ECODE(context_id0)))
|
||||||
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
||||||
KFD_CTXID0_DOORBELL_ID(context_id0),
|
KFD_CTXID0_DOORBELL_ID(context_id0),
|
||||||
KFD_EC_MASK(KFD_CTXID0_CP_BAD_OP_ECODE(context_id0)),
|
KFD_EC_MASK(KFD_CTXID0_CP_BAD_OP_ECODE(context_id0)),
|
||||||
|
|
|
@ -385,7 +385,8 @@ static void event_interrupt_wq_v9(struct kfd_node *dev,
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
kfd_signal_event_interrupt(pasid, sq_int_data, 24);
|
kfd_signal_event_interrupt(pasid, sq_int_data, 24);
|
||||||
} else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE) {
|
} else if (source_id == SOC15_INTSRC_CP_BAD_OPCODE &&
|
||||||
|
KFD_DBG_EC_TYPE_IS_PACKET(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0))) {
|
||||||
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
kfd_set_dbg_ev_from_interrupt(dev, pasid,
|
||||||
KFD_DEBUG_DOORBELL_ID(context_id0),
|
KFD_DEBUG_DOORBELL_ID(context_id0),
|
||||||
KFD_EC_MASK(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0)),
|
KFD_EC_MASK(KFD_DEBUG_CP_BAD_OP_ECODE(context_id0)),
|
||||||
|
|
|
@ -912,14 +912,25 @@ enum kfd_dbg_trap_exception_code {
|
||||||
KFD_EC_MASK(EC_DEVICE_NEW))
|
KFD_EC_MASK(EC_DEVICE_NEW))
|
||||||
#define KFD_EC_MASK_PROCESS (KFD_EC_MASK(EC_PROCESS_RUNTIME) | \
|
#define KFD_EC_MASK_PROCESS (KFD_EC_MASK(EC_PROCESS_RUNTIME) | \
|
||||||
KFD_EC_MASK(EC_PROCESS_DEVICE_REMOVE))
|
KFD_EC_MASK(EC_PROCESS_DEVICE_REMOVE))
|
||||||
|
#define KFD_EC_MASK_PACKET (KFD_EC_MASK(EC_QUEUE_PACKET_DISPATCH_DIM_INVALID) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_DISPATCH_GROUP_SEGMENT_SIZE_INVALID) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_DISPATCH_CODE_INVALID) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_RESERVED) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_UNSUPPORTED) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_DISPATCH_WORK_GROUP_SIZE_INVALID) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_DISPATCH_REGISTER_INVALID) | \
|
||||||
|
KFD_EC_MASK(EC_QUEUE_PACKET_VENDOR_UNSUPPORTED))
|
||||||
|
|
||||||
/* Checks for exception code types for KFD search */
|
/* Checks for exception code types for KFD search */
|
||||||
|
#define KFD_DBG_EC_IS_VALID(ecode) (ecode > EC_NONE && ecode < EC_MAX)
|
||||||
#define KFD_DBG_EC_TYPE_IS_QUEUE(ecode) \
|
#define KFD_DBG_EC_TYPE_IS_QUEUE(ecode) \
|
||||||
(!!(KFD_EC_MASK(ecode) & KFD_EC_MASK_QUEUE))
|
(KFD_DBG_EC_IS_VALID(ecode) && !!(KFD_EC_MASK(ecode) & KFD_EC_MASK_QUEUE))
|
||||||
#define KFD_DBG_EC_TYPE_IS_DEVICE(ecode) \
|
#define KFD_DBG_EC_TYPE_IS_DEVICE(ecode) \
|
||||||
(!!(KFD_EC_MASK(ecode) & KFD_EC_MASK_DEVICE))
|
(KFD_DBG_EC_IS_VALID(ecode) && !!(KFD_EC_MASK(ecode) & KFD_EC_MASK_DEVICE))
|
||||||
#define KFD_DBG_EC_TYPE_IS_PROCESS(ecode) \
|
#define KFD_DBG_EC_TYPE_IS_PROCESS(ecode) \
|
||||||
(!!(KFD_EC_MASK(ecode) & KFD_EC_MASK_PROCESS))
|
(KFD_DBG_EC_IS_VALID(ecode) && !!(KFD_EC_MASK(ecode) & KFD_EC_MASK_PROCESS))
|
||||||
|
#define KFD_DBG_EC_TYPE_IS_PACKET(ecode) \
|
||||||
|
(KFD_DBG_EC_IS_VALID(ecode) && !!(KFD_EC_MASK(ecode) & KFD_EC_MASK_PACKET))
|
||||||
|
|
||||||
|
|
||||||
/* Runtime enable states */
|
/* Runtime enable states */
|
||||||
|
|
Loading…
Reference in a new issue