Skip to content

Commit

Permalink
topology-aware: don't use HBM implicitly.
Browse files Browse the repository at this point in the history
Don't assign HBM memory implicitly for containers not asking
for any particular memory type. Ideally we either should make
this configurable, or teach the memory allocator to fall back
using special memory if the allocation would otherwise fail.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
  • Loading branch information
klihub authored and askervin committed Jun 18, 2024
1 parent 8444b27 commit b9649e5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/plugins/topology-aware/policy/resources.go
Original file line number Diff line number Diff line change
Expand Up @@ -977,7 +977,7 @@ func newRequest(container cache.Container) Request {
container.PrettyName(), cpuType, full, fraction, isolate, prio)

if mtype == memoryUnspec {
mtype = defaultMemoryType
mtype = defaultMemoryType &^ memoryHBM
}

if mtype&memoryPMEM != 0 && mtype&memoryDRAM != 0 {
Expand Down

0 comments on commit b9649e5

Please sign in to comment.