diff options
| author | Gabor Dozsa <gabor.dozsa@arm.com> | 2019-02-04 14:39:08 +0000 |
|---|---|---|
| committer | Giacomo Gabrielli <giacomo.gabrielli@arm.com> | 2019-07-28 16:28:57 +0000 |
| commit | a3ac8caba9dba103d61d4b75b1ea3f8061cfabf4 (patch) | |
| tree | 21fe4dfe984be7f430496d7e40124f945d8bafa3 /src/cpu/base_dyn_inst.hh | |
| parent | 5a9fb5a2bffe37bbfc525137946e6bc0809f6578 (diff) | |
| download | gem5-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>
Diffstat (limited to 'src/cpu/base_dyn_inst.hh')
| -rw-r--r-- | src/cpu/base_dyn_inst.hh | 2 |
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; |