Skip to content

Commit

Permalink
Issue codeplaysoftware#41: Separate execution and memory resources.
Browse files Browse the repository at this point in the history
* Fixes codeplaysoftware#41 and codeplaysoftware#42.
* Introduce `memory_resource` to represent the memory component of a system topology.
* Remove `can_place_memory` and `can_place_agents` from the `execution_resource` as these are no longer required.
* Remove `memory_resource` and `allocator` from the `execution_context` as these no longer make sense.
* Update the wording to describe how execution resources and memory resources are structured.
* Refactor `affinity_query` to be between an `execution_resource` and a `memory_resource`.
* Make minor corrections.
  • Loading branch information
Gordon authored and Gordon committed Oct 7, 2018
1 parent 1fc49ea commit 2abd3fb
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 0 deletions.
Binary file added affinity/cpp-20/system-topology.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
39 changes: 39 additions & 0 deletions affinity/cpp-20/system-topology.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
digraph G {

subgraph cluster_0 {
style=filled;
color=lightgrey;
node [style=filled,color=white];
"Execution Root" -> "CPU0";
"CPU0" -> "Core0";
"CPU0" -> "Core1";
"CPU0" -> "Core2";
"CPU0" -> "Core3";
"Execution Root" -> "CPU1";
"CPU1" -> "Core4";
"CPU1" -> "Core5";
"CPU1" -> "Core6";
"CPU1" -> "Core7";
"Execution Root" -> "GPU";
"GPU" -> "Compute Units [N]";
label = "execution";
}

subgraph cluster_1 {
style=filled;
color=lightgrey;
node [style=filled,color=white];
"Memory Root" -> "NUMA0";
"Memory Root" -> "NUMA1";
"Memory Root" -> "GPU Global";
label = "memory";
}

"this_system::discover_topology()" -> "Execution Root" [color="green"];
"Execution Root" -> "Memory Root" [color="blue"];
"CPU0" -> "NUMA0" [color="blue"];
"CPU1" -> "NUMA1" [color="blue"];
"GPU" -> "GPU Global" [color="blue"];

"this_system::discover_topology()" [shape=Mdiamond];
}

0 comments on commit 2abd3fb

Please sign in to comment.