Mesa (gallium-0.2): cell: additional 'offset' checking in spe_lqd(), spe_stqd()
Brian Paul
brianp at kemper.freedesktop.org
Fri Oct 10 20:58:21 UTC 2008
Module: Mesa
Branch: gallium-0.2
Commit: f42ef6f39d213b4c6315ba95791c16ca2b1a4b21
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=f42ef6f39d213b4c6315ba95791c16ca2b1a4b21
Author: Brian Paul <brian.paul at tungstengraphics.com>
Date: Fri Oct 10 14:44:52 2008 -0600
cell: additional 'offset' checking in spe_lqd(), spe_stqd()
---
src/gallium/auxiliary/rtasm/rtasm_ppc_spe.c | 18 ++++++++++++++----
1 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/rtasm/rtasm_ppc_spe.c b/src/gallium/auxiliary/rtasm/rtasm_ppc_spe.c
index 5b0f6bd..d0bacd0 100644
--- a/src/gallium/auxiliary/rtasm/rtasm_ppc_spe.c
+++ b/src/gallium/auxiliary/rtasm/rtasm_ppc_spe.c
@@ -552,14 +552,19 @@ spe_comment(struct spe_function *p, int rel_indent, const char *s)
/**
* Load quad word.
- * NOTE: imm is in bytes and the least significant 4 bits must be zero!
+ * NOTE: offset is in bytes and the least significant 4 bits must be zero!
*/
void spe_lqd(struct spe_function *p, unsigned rT, unsigned rA, int offset)
{
const boolean pSave = p->print;
- p->print = FALSE;
+ /* offset must be a multiple of 16 */
assert(offset % 16 == 0);
+ /* offset must fit in 10-bit signed int field, after shifting */
+ assert((offset >> 4) <= 511);
+ assert((offset >> 4) >= -512);
+
+ p->print = FALSE;
emit_RI10(p, 0x034, rT, rA, offset >> 4, "spe_lqd");
p->print = pSave;
@@ -572,14 +577,19 @@ void spe_lqd(struct spe_function *p, unsigned rT, unsigned rA, int offset)
/**
* Store quad word.
- * NOTE: imm is in bytes and the least significant 4 bits must be zero!
+ * NOTE: offset is in bytes and the least significant 4 bits must be zero!
*/
void spe_stqd(struct spe_function *p, unsigned rT, unsigned rA, int offset)
{
const boolean pSave = p->print;
- p->print = FALSE;
+ /* offset must be a multiple of 16 */
assert(offset % 16 == 0);
+ /* offset must fit in 10-bit signed int field, after shifting */
+ assert((offset >> 4) <= 511);
+ assert((offset >> 4) >= -512);
+
+ p->print = FALSE;
emit_RI10(p, 0x024, rT, rA, offset >> 4, "spe_stqd");
p->print = pSave;
More information about the mesa-commit
mailing list