summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabor Dozsa <gabor.dozsa@arm.com>2019-02-04 14:39:08 +0000
committerGiacomo Gabrielli <giacomo.gabrielli@arm.com>2019-07-28 16:28:57 +0000
commita3ac8caba9dba103d61d4b75b1ea3f8061cfabf4 (patch)
tree21fe4dfe984be7f430496d7e40124f945d8bafa3
parent5a9fb5a2bffe37bbfc525137946e6bc0809f6578 (diff)
downloadgem5-a3ac8caba9dba103d61d4b75b1ea3f8061cfabf4.tar.xz
cpu: Fix the type of the effective mem request size
A memory request size can be larger than 255 bytes (e.g. SVE with 2048-bit vector length) which could cause overflow in the 'uint8_t effSize' variable. Change-Id: I77e0d02a49ea7f81cacfa5be7e4ae40434af3109 Reviewed-by: Giacomo Gabrielli <giacomo.gabrielli@arm.com> Signed-off-by: Giacomo Gabrielli <giacomo.gabrielli@arm.com> Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19175 Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com> Reviewed-by: Jason Lowe-Power <jason@lowepower.com> Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
-rw-r--r--src/cpu/base_dyn_inst.hh2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cpu/base_dyn_inst.hh b/src/cpu/base_dyn_inst.hh
index 6f9555a38..de76559fb 100644
--- a/src/cpu/base_dyn_inst.hh
+++ b/src/cpu/base_dyn_inst.hh
@@ -221,7 +221,7 @@ class BaseDynInst : public ExecContext, public RefCounted
short asid;
/** The size of the request */
- uint8_t effSize;
+ unsigned effSize;
/** Pointer to the data for the memory access. */
uint8_t *memData;