refactor: [EXC-1724] Refactor task_queue (#1490)

Refactor SystemState::task_queue to be a new struct with an additional field on_low_wasm_memory_hook_status, to ensure that whenever available first task that will be poped from the task queue is OnLowWasmMemoryHook (excluding paused executions). In the [follow-up](https://dfinity.atlassian.net/browse/EXC-1725) we will add logic in which when doing operations on task queue, in the case there are no paused executions in the queue, it is checked if `on_low_wasm_memory_hook_status` is `Ready`, and if so it will be returned as it is on the front of the queue. That will ensure that the hook is executed right after the execution of the message/task that triggered the hook condition. --------- Co-authored-by: Alin Sinpalean <[email protected]> Co-authored-by: Dimitris Sarlis <[email protected]>
dfinity · Oct 4, 2024 · 5127f04 · 5127f04
1 parent fcad095
commit 5127f04
Show file tree

Hide file tree

Showing 15 changed files with 461 additions and 256 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/rs/canister_sandbox/src/sandbox_server.rs b/rs/canister_sandbox/src/sandbox_server.rs
@@ -153,10 +153,7 @@ mod tests {
     use ic_limits::SMALL_APP_SUBNET_MAX_SIZE;
     use ic_logger::replica_logger::no_op_logger;
     use ic_registry_subnet_type::SubnetType;
-    use ic_replicated_state::{
-        canister_state::system_state::OnLowWasmMemoryHookStatus, Global, NumWasmPages, PageIndex,
-        PageMap,
-    };
+    use ic_replicated_state::{Global, NumWasmPages, PageIndex, PageMap};
     use ic_system_api::{
         sandbox_safe_system_state::{CanisterStatusView, SandboxSafeSystemState},
         ApiType, ExecutionParameters, InstructionLimits,
@@ -231,7 +228,6 @@ mod tests {
             RequestMetadata::new(0, Time::from_nanos_since_unix_epoch(0)),
             caller,
             0,
-            OnLowWasmMemoryHookStatus::ConditionNotSatisfied,
         )
     }