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 <[email protected]>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/37255
Maintainer: Bobby R. Bruce <[email protected]>
Tested-by: kokoro <[email protected]>
Reviewed-by: Srikant Bharadwaj <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Srikant Bharadwaj <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
