This is an archive of the discontinued LLVM Phabricator instance.

OpenMP-Offload Add VA Management for Record-Replay Mechanism
Needs ReviewPublic

Authored by koparasy on Aug 5 2023, 1:14 AM.

Details

Summary

Extends the record-replay mechanism to re use the same VA between record and replay runs. This provides consistent memory addresses across runs.

Diff Detail

Event Timeline

koparasy created this revision.Aug 5 2023, 1:14 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 5 2023, 1:14 AM
koparasy requested review of this revision.Aug 5 2023, 1:14 AM
jdoerfert added inline comments.Oct 9 2023, 12:36 PM
openmp/libomptarget/plugins-nextgen/common/PluginInterface/PluginInterface.cpp
63

utils::alignUp(Addr, MaxMemoryAllocation)
Or sth like that.

65

I don't get this. I would at least not do this if Addr < MaxMemoryAllocation. Even if Addr > MaxMemoryAllocation, the space before Addr could very well be allocated already. Would this not be a problem?

71–74

Clang format the commit.

95

Heuristic

316–318
openmp/libomptarget/plugins-nextgen/common/PluginInterface/PluginInterface.h
647
650

-cuda

651

+ actual number of bytes allocated by this function.
Mention if it can be lower/larger or both

openmp/libomptarget/plugins-nextgen/cuda/dynamic_cuda/cuda.h
114

Remove the comments. We don't have any elsewhere.

openmp/libomptarget/plugins-nextgen/cuda/src/rtl.cpp
564

utils::roundUp

602–604
983

-memory
s/handlers/handles

openmp/libomptarget/src/interface.cpp
342

alignment

openmp/libomptarget/src/private.h
44

ReqAddr