summaryrefslogtreecommitdiff
path: root/src/pld/Makefile.am
diff options
context:
space:
mode:
authorMike Dunn <mikedunn@newsguy.com>2010-09-19 14:48:51 -0700
committerØyvind Harboe <oyvind.harboe@zylin.com>2010-09-20 09:21:12 +0200
commitebfb2f4f3715f264ae4474c1b2e78812d1625cdc (patch)
tree8b0039ebc6504c0d6d44a7d805bddeaa33846b8f /src/pld/Makefile.am
parent7e888741d13e66b6b343b8f0839621107c5a2962 (diff)
downloadopenocd_libswd-ebfb2f4f3715f264ae4474c1b2e78812d1625cdc.tar.gz
openocd_libswd-ebfb2f4f3715f264ae4474c1b2e78812d1625cdc.tar.bz2
openocd_libswd-ebfb2f4f3715f264ae4474c1b2e78812d1625cdc.tar.xz
openocd_libswd-ebfb2f4f3715f264ae4474c1b2e78812d1625cdc.zip
xscale: check that wp length does not exceed address
Hi everyone, A while back I sent in a patch that adds support for watchpoint lengths greater than four on xscale. It's been working well, until the other day, when it caused an unexpected debug exception. Looking into this I realized there is a case where it breaks: when the length arg is greater than the base address. This is a consequence of the way the hardware works. Don't see a work-around, so I added code to xscale_add_watchpoint() to check for and disallow this combination. Some more detail... xscale watchpoint hardware does not support a length directly. Instead, a mask value can be specified (not to be confused with the optional mask arg to the wp command, which xscale does not support). Any bits set in the mask are ignored when the watchpoint hardware compares the access address to the watchpoint address. So as long as the length is a power of two, setting the mask to length-1 effectively specifies the length. Or so I thought, until I realized that if the length exceeds the base address, *all* bits of the base address are ignored by the comaparator, and the watchpoint range effectively becomes 0 .. length. Questions, comments, criticisms gratefully received. Thanks, Mike Signed-off-by: Mike Dunn <mikedunn@newsguy.com>
Diffstat (limited to 'src/pld/Makefile.am')
0 files changed, 0 insertions, 0 deletions