On 11/30/18 11:57 AM, Aaron Lindsay wrote: >> Eh? But we've set overflow_mask based on PMCR.LC, so what you say here >> doesn't >> make sense. > > Sorry, I had an off-by-one-bit think-o I couldn't get past until I > started typing a concrete example to explain myself. I'll change this > line to be: > > if (!(env->cp15.c9_pmcr & PMCRLC)) > new_pmccntr &= 0xffffffff;
Ah, much better. Thanks. r~
