Commit 97547bc
src: lock-free init_process_flags
Fix #45152
PR-URL: #45221
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>1 parent 738144c commit 97547bc
2 files changed
Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -429,7 +429,7 @@ void ResetSignalHandlers() { | |||
| 429 | 429 | #endif // __POSIX__ | |
| 430 | 430 | } | |
| 431 | 431 | ||
| 432 | - static std::atomic<uint64_t> init_process_flags = 0; | ||
| 432 | + static std::atomic<uint32_t> init_process_flags = 0; | ||
| 433 | 433 | ||
| 434 | 434 | static void PlatformInit(ProcessInitializationFlags::Flags flags) { | |
| 435 | 435 | // init_process_flags is accessed in ResetStdio(), | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -228,6 +228,8 @@ class MultiIsolatePlatform; | |||
| 228 | 228 | class InitializationResultImpl; | |
| 229 | 229 | ||
| 230 | 230 | namespace ProcessFlags { | |
| 231 | + // TODO(addaleax): Switch to uint32_t to match std::atomic<uint32_t> | ||
| 232 | + // init_process_flags in node.cc | ||
| 231 | 233 | enum Flags : uint64_t { | |
| 232 | 234 | kNoFlags = 0, | |
| 233 | 235 | // Enable stdio inheritance, which is disabled by default. | |
0 commit comments