Daniel Carvalho has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/37255 )
Change subject: configs: Fix MemorySize division
......................................................................
configs: Fix MemorySize division
The memory size is expected to be an integer.
Jira: https://gem5.atlassian.net/browse/GEM5-806
Change-Id: I44b2d423a3478d2598950779222151f09970cbd8
Signed-off-by: Daniel R. Carvalho <oda...@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/37255
Maintainer: Bobby R. Bruce <bbr...@ucdavis.edu>
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Srikant Bharadwaj <srikant.bharad...@amd.com>
---
M configs/topologies/MeshDirCorners_XY.py
M configs/topologies/Mesh_XY.py
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Srikant Bharadwaj: Looks good to me, approved
Bobby R. Bruce: Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/topologies/MeshDirCorners_XY.py
b/configs/topologies/MeshDirCorners_XY.py
index 7d065de..e0aea52 100644
--- a/configs/topologies/MeshDirCorners_XY.py
+++ b/configs/topologies/MeshDirCorners_XY.py
@@ -218,6 +218,6 @@
for n in numa_nodes:
if n:
FileSystemConfig.register_node(n,
- MemorySize(options.mem_size) / num_numa_nodes, i)
+ MemorySize(options.mem_size) // num_numa_nodes, i)
i += 1
diff --git a/configs/topologies/Mesh_XY.py b/configs/topologies/Mesh_XY.py
index 64a8506..faec1e3 100644
--- a/configs/topologies/Mesh_XY.py
+++ b/configs/topologies/Mesh_XY.py
@@ -178,4 +178,4 @@
def registerTopology(self, options):
for i in range(options.num_cpus):
FileSystemConfig.register_node([i],
- MemorySize(options.mem_size) / options.num_cpus, i)
+ MemorySize(options.mem_size) // options.num_cpus, i)
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/37255
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: hotfix-memorysize-division
Gerrit-Change-Id: I44b2d423a3478d2598950779222151f09970cbd8
Gerrit-Change-Number: 37255
Gerrit-PatchSet: 3
Gerrit-Owner: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: Srikant Bharadwaj <srikant.bharad...@amd.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s