[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Qemu-arm] [PATCH] target-arm: Fix translation level on

From: Sergey Sorokin
Subject: Re: [Qemu-devel] [Qemu-arm] [PATCH] target-arm: Fix translation level on early translation faults
Date: Fri, 04 Mar 2016 18:03:58 +0300

03.03.2016, 19:54, "Peter Maydell" <address@hidden>:
>  On 3 March 2016 at 16:37, Sergey Fedorov <address@hidden> 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.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]