aboutsummaryrefslogtreecommitdiff
path: root/sim/bfin/dv-bfin_wp.c
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2015-12-26 19:02:07 -0500
committerMike Frysinger <vapier@gentoo.org>2015-12-26 19:09:43 -0500
commit466b619e95908dc073b78413f0d0d0b1cb97e4b5 (patch)
tree2b3a47fc43dab9497495d232fd6769945db63aff /sim/bfin/dv-bfin_wp.c
parentb72dd4c228d71b44e487ff2c53c0d2883653bce7 (diff)
downloadgdb-466b619e95908dc073b78413f0d0d0b1cb97e4b5.zip
gdb-466b619e95908dc073b78413f0d0d0b1cb97e4b5.tar.gz
gdb-466b619e95908dc073b78413f0d0d0b1cb97e4b5.tar.bz2
sim: bfin: push down mmr address/size checks
The bfin port is using the WITH_DEVICES framework for two reasons: - get access to the cpu making the request (if available) - check the alignment & size for core & system MMRs We addressed the first part with commit dea10706e9159ba6e94eab4c25010f3, and we handle the second part with this commit. Arguably this is more correct too because trying to do bad reads/writes directly (when devices support is disabled) often results in bad memory accesses. As part of this clean up, we also adjust all of the existing logic that would reject invalid accesses: the code was relying on the checks never returning, but that's not the case when things like gdb (via the user's commands) are making the requests. Thus we'd still end up with bad mem accesses, or sometimes gdb being hung due to while(1) loops. Now we can connect (most of) these models into any address and have them work correctly.
Diffstat (limited to 'sim/bfin/dv-bfin_wp.c')
-rw-r--r--sim/bfin/dv-bfin_wp.c15
1 files changed, 11 insertions, 4 deletions
diff --git a/sim/bfin/dv-bfin_wp.c b/sim/bfin/dv-bfin_wp.c
index 5292cb2..17dc979 100644
--- a/sim/bfin/dv-bfin_wp.c
+++ b/sim/bfin/dv-bfin_wp.c
@@ -77,6 +77,10 @@ bfin_wp_io_write_buffer (struct hw *me, const void *source, int space,
bu32 value;
bu32 *valuep;
+ /* Invalid access mode is higher priority than missing register. */
+ if (!dv_bfin_mmr_require_32 (me, addr, nr_bytes, true))
+ return 0;
+
value = dv_load_4 (source);
mmr_off = addr - wp->base;
valuep = (void *)((unsigned long)wp + mmr_base() + mmr_off);
@@ -99,7 +103,7 @@ bfin_wp_io_write_buffer (struct hw *me, const void *source, int space,
break;
default:
dv_bfin_mmr_invalid (me, addr, nr_bytes, true);
- break;
+ return 0;
}
return nr_bytes;
@@ -114,6 +118,10 @@ bfin_wp_io_read_buffer (struct hw *me, void *dest, int space,
bu32 value;
bu32 *valuep;
+ /* Invalid access mode is higher priority than missing register. */
+ if (!dv_bfin_mmr_require_32 (me, addr, nr_bytes, false))
+ return 0;
+
mmr_off = addr - wp->base;
valuep = (void *)((unsigned long)wp + mmr_base() + mmr_off);
@@ -131,9 +139,8 @@ bfin_wp_io_read_buffer (struct hw *me, void *dest, int space,
value = *valuep;
break;
default:
- while (1) /* Core MMRs -> exception -> doesn't return. */
- dv_bfin_mmr_invalid (me, addr, nr_bytes, false);
- break;
+ dv_bfin_mmr_invalid (me, addr, nr_bytes, false);
+ return 0;
}
dv_store_4 (dest, value);