> + } else if (s->regs.mm_index > MM_DATA + 3) {
> val = ati_mm_read(s, s->regs.mm_index + addr - MM_DATA, size);
MM_INDEX is 0
MM_DATA is 4
"normal" registers start at 8.
So we want allow indirect access for offset 8 and above and deny offsets
0-7. mm_index is interpreted with an offset, see "- MM_DATA" in the
call above.
Not clear to me why this offset is 4, that doesn't make sense to me.
I'd expect either no offset or offset being 8. BALATON, can you
double-check that with the specs?
Assuming offset 4 is correct we must require mm_index being larger than
MM_DATA + MM_DATA + 3 ( == 11) to compensate for the offset.
take care,
Gerd
