[gem5-dev] Change in gem5/gem5[master]: sim-se: Fixed initialization array size

2019-03-22 Thread Giacomo Travaglini (Gerrit)
Giacomo Travaglini has submitted this change and it was merged. (  
https://gem5-review.googlesource.com/c/public/gem5/+/17088 )


Change subject: sim-se: Fixed initialization array size
..

sim-se: Fixed initialization array size

Doubled the size of the zeroed auxiliary vector since 2 * intSize on
aarch64 > sizeof(uint64).

Change-Id: I5196b000a696e9ea3f2b5daa5d5bb071794369aa
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17088
Reviewed-by: Ciro Santilli 
Maintainer: Brandon Potter 
---
M src/arch/arm/process.cc
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Ciro Santilli: Looks good to me, approved
  Brandon Potter: Looks good to me, approved



diff --git a/src/arch/arm/process.cc b/src/arch/arm/process.cc
index bc44959..ba9fbfa 100644
--- a/src/arch/arm/process.cc
+++ b/src/arch/arm/process.cc
@@ -442,10 +442,10 @@
   (uint8_t*)&(auxv[x].getAuxVal()),
   intSize);
 }
-//Write out the terminating zeroed auxilliary vector
-const uint64_t zero = 0;
+//Write out the terminating zeroed auxillary vector
+const IntType zero[2] = {0, 0};
 initVirtMem.writeBlob(auxv_array_base + 2 * intSize * auxv.size(),
-(uint8_t*), 2 * intSize);
+(uint8_t*)zero, 2 * intSize);

 copyStringArray(envp, envp_array_base, env_data_base, initVirtMem);
 copyStringArray(argv, argv_array_base, arg_data_base, initVirtMem);

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17088
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I5196b000a696e9ea3f2b5daa5d5bb071794369aa
Gerrit-Change-Number: 17088
Gerrit-PatchSet: 5
Gerrit-Owner: Giacomo Travaglini 
Gerrit-Reviewer: Andreas Sandberg 
Gerrit-Reviewer: Brandon Potter 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-MessageType: merged
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in gem5/gem5[master]: sim-se: Fixed initialization array size

2019-03-12 Thread Giacomo Travaglini (Gerrit)

Hello Andreas Sandberg, Brandon Potter, Ciro Santilli,

I'd like you to reexamine a change. Please visit

https://gem5-review.googlesource.com/c/public/gem5/+/17088

to look at the new patch set (#3).

Change subject: sim-se: Fixed initialization array size
..

sim-se: Fixed initialization array size

Doubled the size of the zeroed auxiliary vector since 2 * intSize on
aarch64 > sizeof(uint64).

Change-Id: I5196b000a696e9ea3f2b5daa5d5bb071794369aa
---
M src/arch/arm/process.cc
1 file changed, 3 insertions(+), 3 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17088
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I5196b000a696e9ea3f2b5daa5d5bb071794369aa
Gerrit-Change-Number: 17088
Gerrit-PatchSet: 3
Gerrit-Owner: Giacomo Travaglini 
Gerrit-Reviewer: Andreas Sandberg 
Gerrit-Reviewer: Brandon Potter 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-MessageType: newpatchset
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev