diff options
author | Ian Romanick <idr@us.ibm.com> | 2008-02-20 14:45:08 -0800 |
---|---|---|
committer | Ian Romanick <idr@us.ibm.com> | 2008-02-21 10:43:45 -0800 |
commit | 2d1f086c12b6d64f5c3fb80474f26775aeb71370 (patch) | |
tree | b3b95ed628b4e61ababa866f6af310bdbf6b75fb /src/gallium/drivers/cell/spu/spu_dcache.c | |
parent | 7c74037852a484a8a50e8bc540b954a624de4d33 (diff) |
Cell: Fix off-by-one error in spu_dcache_fetch_unaligned
An off-by-one error caused an extra qword to be fetched under certain
alignment / size combinations.
Diffstat (limited to 'src/gallium/drivers/cell/spu/spu_dcache.c')
-rw-r--r-- | src/gallium/drivers/cell/spu/spu_dcache.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/gallium/drivers/cell/spu/spu_dcache.c b/src/gallium/drivers/cell/spu/spu_dcache.c index 9e30e17880..68aa5c4ae8 100644 --- a/src/gallium/drivers/cell/spu/spu_dcache.c +++ b/src/gallium/drivers/cell/spu/spu_dcache.c @@ -22,6 +22,7 @@ * USE OR OTHER DEALINGS IN THE SOFTWARE. */ +#include "cell/common.h" #include "spu_main.h" #include "spu_dcache.h" @@ -50,8 +51,8 @@ spu_dcache_fetch_unaligned(qword *dst, unsigned ea, unsigned size) { const int shift = ea & 0x0f; const unsigned aligned_start_ea = ea & ~0x0f; - const unsigned aligned_end_ea = (ea + size) & ~0x0f; - const unsigned num_entries = ((aligned_end_ea - aligned_start_ea) / 16) + 1; + const unsigned aligned_end_ea = ROUNDUP16(ea + size); + const unsigned num_entries = (aligned_end_ea - aligned_start_ea) / 16; unsigned i; |