mem: When loading an image directly in memory, use the right CL size.
Some code was added fairly recently which would load a memory image into a memory directly in order to make it easier to set up ROMs. Unfortunately, that code accidentally used the image size instead of the cache line size when setting up the port proxy which would actually write the data. This happens to work when the image size is a power of two since that's all the proxy checks for, but there's no guarantee that every image will be sized that way. This change instead looks into the system object, retrieves the cache line size from it, and uses that to set up the port proxy. Change-Id: I227ac475b855d9516e1feb881769e12ec4e7d598 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/35155 Reviewed-by: Jason Lowe-Power <power.jg@gmail.com> Maintainer: Jason Lowe-Power <power.jg@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
@@ -93,7 +93,8 @@ AbstractMemory::initState()
|
||||
panic_if(!image_range.isSubset(range), "%s: memory image %s doesn't fit.",
|
||||
name(), file);
|
||||
|
||||
PortProxy proxy([this](PacketPtr pkt) { functionalAccess(pkt); }, size());
|
||||
PortProxy proxy([this](PacketPtr pkt) { functionalAccess(pkt); },
|
||||
system()->cacheLineSize());
|
||||
|
||||
panic_if(!image.write(proxy), "%s: Unable to write image.");
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user