diff options
author | Felix Kuehling <[email protected]> | 2022-11-01 15:02:48 -0400 |
---|---|---|
committer | Alex Deucher <[email protected]> | 2022-11-09 17:57:48 -0500 |
commit | b91c23e099f0b65d62159da13458c5eefa76083f (patch) | |
tree | 0b85c10adc5206cd246f6a40d8044d4b140f87ad /tools/perf/util/scripting-engines/trace-event-python.c | |
parent | 66f7903779fbbc620bf1040017e4833ef6a0b541 (diff) |
drm/amdkfd: Fix error handling in criu_checkpoint
Checkpoint BOs last. That way we don't need to close dmabuf FDs if
something else fails later. This avoids problematic access to user mode
memory in the error handling code path.
criu_checkpoint_bos has its own error handling and cleanup that does not
depend on access to user memory.
In the private data, keep BOs before the remaining objects. This is
necessary to restore things in the correct order as restoring events
depends on the events-page BO being restored first.
Fixes: be072b06c739 ("drm/amdkfd: CRIU export BOs as prime dmabuf objects")
Reported-by: Jann Horn <[email protected]>
CC: Rajneesh Bhardwaj <[email protected]>
Signed-off-by: Felix Kuehling <[email protected]>
Reviewed-and-tested-by: Rajneesh Bhardwaj <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
Cc: [email protected]
Diffstat (limited to 'tools/perf/util/scripting-engines/trace-event-python.c')
0 files changed, 0 insertions, 0 deletions