In that case please disregard those patches. Can you try this diff below which will log any invalid accesses and see if anything appears on stderr?
diff --git a/memory.c b/memory.c index 9200b20130..5d1a6d4477 100644 --- a/memory.c +++ b/memory.c @@ -1354,10 +1354,12 @@ bool memory_region_access_valid(MemoryRegion *mr, { if (mr->ops->valid.accepts && !mr->ops->valid.accepts(mr->opaque, addr, size, is_write, attrs)) { + fprintf(stderr, "invalid accepts: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size); return false; }
if (!mr->ops->valid.unaligned && (addr & (size - 1))) { + fprintf(stderr, "invalid aligned: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size); return false; }
@@ -1368,6 +1370,7 @@ bool memory_region_access_valid(MemoryRegion *mr,
if (size > mr->ops->valid.max_access_size || size < mr->ops->valid.min_access_size) { + fprintf(stderr, "invalid size: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size); return false; } return true;
ATB,
Mark.
In that case please disregard those patches. Can you try this diff below which will log any invalid accesses and see if anything appears on stderr?
diff --git a/memory.c b/memory.c .5d1a6d4477 100644 region_ access_ valid(MemoryReg ion *mr, >valid. accepts >valid. accepts( mr->opaque, addr, size, is_write, attrs)) {
index 9200b20130.
--- a/memory.c
+++ b/memory.c
@@ -1354,10 +1354,12 @@ bool memory_
{
if (mr->ops-
&& !mr->ops-
+ fprintf(stderr, "invalid accepts: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size);
return false;
}
if (!mr->ops- >valid. unaligned && (addr & (size - 1))) {
+ fprintf(stderr, "invalid aligned: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size);
return false;
}
@@ -1368,6 +1370,7 @@ bool memory_ region_ access_ valid(MemoryReg ion *mr,
if (size > mr->ops- >valid. max_access_ size >valid. min_access_ size) {
|| size < mr->ops-
+ fprintf(stderr, "invalid size: %s addr %"PRIx64 " size: %d\n", mr->name, addr, size);
return false;
}
return true;
ATB,
Mark.