Comment 13 for bug 1886318

Revision history for this message
Mark Cave-Ayland (mark-cave-ayland) wrote :

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.