-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable serialization and shared memory #410
Changes from 36 commits
53bc81a
bc4b50a
3e9e9f7
073dbe5
c9ac614
b11a8de
8424317
f84ed1d
36d9211
e6289d8
44822df
d438ce2
f7bb9fc
8dab338
2abb8b8
d8adfa3
3699b42
1505a33
0f0c5bb
93f302c
1fbf448
0e302d8
528294c
2774533
dab9759
ba09841
8e1be78
2159368
bfc8ed5
d89aec3
50dc6f3
2942ac1
1d93b72
a73245d
263bea6
7e5cf15
9d2b76c
d8f92c9
ade9791
7856aa5
9f24343
1214961
da45bb2
f190325
44650e6
85d5119
b16e9f5
ee7b62e
b7cd6cd
a0cc47a
5d7cb38
8375015
b403596
00a191b
a8c43bd
6e3ca63
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,11 +32,21 @@ constexpr unsigned long all_values = (1 << 7) - 1; | |
} // namespace thermalqs | ||
|
||
constexpr size_t MAX_NUM_LAMBDAS = 3; | ||
enum class DataStatus { Deallocated = 0, OnDevice = 1, OnHost = 2 }; | ||
enum class DataStatus { Deallocated = 0, OnDevice = 1, OnHost = 2, UnManaged = 3 }; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we have sufficient coverage in our testing to make sure we are catching the potential issues? I recall this took up many of @Yurlungur's cycles to get this to work correctly back in the day. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I believe so. I think our tests should catch this if there's an issue. But I can't promise that. The |
||
enum class TableStatus { OnTable = 0, OffBottom = 1, OffTop = 2 }; | ||
constexpr Real ROOM_TEMPERATURE = 293; // K | ||
constexpr Real ATMOSPHERIC_PRESSURE = 1e6; | ||
|
||
struct SharedMemSettings { | ||
SharedMemSettings() = default; | ||
SharedMemSettings(char *data_, bool is_domain_root_) | ||
: data(data_), is_domain_root(is_domain_root_) {} | ||
bool CopyNeeded() const { return (data != nullptr) && is_domain_root; } | ||
char *data = nullptr; | ||
bool is_domain_root = false; // default true or false? | ||
}; | ||
const SharedMemSettings DEFAULT_SHMEM_STNGS = SharedMemSettings(); | ||
|
||
} // namespace singularity | ||
|
||
#endif // SINGULARITY_EOS_BASE_CONSTANTS_HPP_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know
SHMEM
is something else but hopefully next toEOSPAC
it's clear what this means