03.03.2016, 19:54, "Peter Maydell" <peter.mayd...@linaro.org>: > On 3 March 2016 at 16:37, Sergey Fedorov <serge.f...@gmail.com> wrote: >> On 03.03.2016 17:55, Peter Maydell wrote: >>> Yes, this patch is definitely fixing a bug; I'm just mentioning that other >>> code path because it seems to be the result of previously fixing the bug >>> for a particular special case... >> >> Ah, right, I think I understand you :) So we'd better remove these lines: >> >> /* AArch64 reports these as level 0 faults. >> * AArch32 reports these as level 1 faults. >> */ >> level = va_size == 64 ? 0 : 1; >> fault_type = translation_fault; > > Those lines come after some code which has set level to something > else, so you'd need to rearrange that code a bit so it didn't > set level before it had determined that there wasn't a fault. > > thanks > -- PMM
Do we really need to rearrange the code? In pseudo-code the level is set explicitly as well, in case of a fault.