radv/amdgpu: remove legacy code for querying context status
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22521>
This commit is contained in:

committed by
Marge Bot

parent
767a9324b9
commit
c221bfbd85
@@ -617,8 +617,6 @@ radv_check_status(struct vk_device *vk_device)
|
||||
return vk_device_set_lost(&device->vk, "GPU hung detected in this process");
|
||||
if (status == RADV_INNOCENT_CONTEXT_RESET)
|
||||
return vk_device_set_lost(&device->vk, "GPU hung triggered by other process");
|
||||
if (status == RADV_UNKNOWN_CONTEXT_RESET)
|
||||
return vk_device_set_lost(&device->vk, "GPU hung triggered by unknown source");
|
||||
}
|
||||
}
|
||||
|
||||
|
@@ -106,7 +106,6 @@ enum radv_reset_status {
|
||||
RADV_NO_RESET,
|
||||
RADV_GUILTY_CONTEXT_RESET,
|
||||
RADV_INNOCENT_CONTEXT_RESET,
|
||||
RADV_UNKNOWN_CONTEXT_RESET,
|
||||
};
|
||||
|
||||
struct radeon_cmdbuf {
|
||||
|
@@ -1324,41 +1324,21 @@ radv_amdgpu_ctx_query_reset_status(struct radeon_winsys_ctx *rwctx)
|
||||
{
|
||||
int ret;
|
||||
struct radv_amdgpu_ctx *ctx = (struct radv_amdgpu_ctx *)rwctx;
|
||||
uint64_t flags;
|
||||
|
||||
if (ctx->ws->info.drm_minor >= 24) {
|
||||
uint64_t flags;
|
||||
ret = amdgpu_cs_query_reset_state2(ctx->ctx, &flags);
|
||||
ret = amdgpu_cs_query_reset_state2(ctx->ctx, &flags);
|
||||
if (ret) {
|
||||
fprintf(stderr, "radv/amdgpu: amdgpu_cs_query_reset_state2 failed. (%i)\n", ret);
|
||||
return RADV_NO_RESET;
|
||||
}
|
||||
|
||||
if (ret) {
|
||||
fprintf(stderr, "radv/amdgpu: amdgpu_cs_query_reset_state2 failed. (%i)\n", ret);
|
||||
return RADV_NO_RESET;
|
||||
}
|
||||
|
||||
if (flags & AMDGPU_CTX_QUERY2_FLAGS_RESET) {
|
||||
if (flags & AMDGPU_CTX_QUERY2_FLAGS_GUILTY) {
|
||||
/* Some job from this context once caused a GPU hang */
|
||||
return RADV_GUILTY_CONTEXT_RESET;
|
||||
} else {
|
||||
/* Some job from other context caused a GPU hang */
|
||||
return RADV_INNOCENT_CONTEXT_RESET;
|
||||
}
|
||||
}
|
||||
} else {
|
||||
uint32_t result, hangs;
|
||||
|
||||
ret = amdgpu_cs_query_reset_state(ctx->ctx, &result, &hangs);
|
||||
if (ret) {
|
||||
fprintf(stderr, "radv/amdgpu: amdgpu_cs_query_reset_state failed. (%i)\n", ret);
|
||||
return RADV_NO_RESET;
|
||||
}
|
||||
|
||||
switch (result) {
|
||||
case AMDGPU_CTX_GUILTY_RESET:
|
||||
if (flags & AMDGPU_CTX_QUERY2_FLAGS_RESET) {
|
||||
if (flags & AMDGPU_CTX_QUERY2_FLAGS_GUILTY) {
|
||||
/* Some job from this context once caused a GPU hang */
|
||||
return RADV_GUILTY_CONTEXT_RESET;
|
||||
case AMDGPU_CTX_INNOCENT_RESET:
|
||||
} else {
|
||||
/* Some job from other context caused a GPU hang */
|
||||
return RADV_INNOCENT_CONTEXT_RESET;
|
||||
case AMDGPU_CTX_UNKNOWN_RESET:
|
||||
return RADV_UNKNOWN_CONTEXT_RESET;
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user