mem: Remove a check that the memory size is a multiple of the page size.
authorGabe Black <gabeblack@google.com>
Sat, 7 Mar 2020 23:51:50 +0000 (15:51 -0800)
committerGabe Black <gabeblack@google.com>
Tue, 24 Mar 2020 01:40:57 +0000 (01:40 +0000)
commit87cf5a99ee686dc947cf21d08b2655d3b70437ee
tree25e98ed7b07a801465456f4a1d8b66cd06d974dd
parent0864696b734d2d39b867060d95b3fe95e3b72f1f
mem: Remove a check that the memory size is a multiple of the page size.

There are a few problems with this check.

1. Many ISAs support multiple page sizes.
2. Memories (particularly small ROMs) may not actually be in multiples
   of the page size.
3. In a heterogenous environment, there won't be a single page size even
   if each ISA picks a canonical page size.
4. Other than catching some egregious configuration mistakes, there's
   nothing functionally wrong/different about a memory that isn't evenly
   coverable in pages, especially in systems or configurations that
   don't even use paging.

Change-Id: I3cd241657318d2e3fd5a1226cb54fdebbf172788
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26423
Maintainer: Gabe Black <gabeblack@google.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: Gem5 Cloud Project GCB service account <345032938727@cloudbuild.gserviceaccount.com>
src/mem/abstract_mem.cc
src/mem/abstract_mem.hh