On 30/03/2015 13:45, Peter Crosthwaite wrote: > Can the address_space_translate_address() length clamp be made > conditional on non-MMIO access as the RC fix? I submitted > c3c1bb99d1c11978d9ce94d1bdcf0705378c1459 as I think its the right > thing to do regardless of memory type, but in reality it only fixes a > bug I encountered with RAM memory regions. The original code ignores > address_space_translate_internal() return-by-pointer length value > absolutely and the new code uses it absolutely. Should we just if the > whole thing, old vs new behaviour on MMIO vs non-MMIO? > > Happy to submit that fixup if that's the accepted plan.
I submitted what I think is the right fix (sorry Mark for misspelling your email address). I think you're patch is correct, so I'd rather not introduce hacks for the release; we can either revert it, or fix ioport.c. Paolo