On 2019/12/9 21:05, Beata Michalska wrote:
>> Here we set the FnV to not valid, not to set it to valid.
>> because Guest will use the physical address that recorded in APEI table.
>>
> To be precise : the FnV is giving the status of FAR - so what you are setting
> here is status of 0b0 which means FAR is valid, not FnV on it's own.
> And my point was that you are changing the prototype for syn_data_abort_no_iss
> just for this case only so I was just thinking that it might not be
> worth that, instead
> you could just set it here ... or to be more flexible , provide a way
> to set specific bits
> on demand.
No, I set the FnV to 0b1, not 0b0, the whole esr_el1's value is 0x96000410, as
shown below log:
I remember changing the prototype for syn_data_abort_no_iss is suggested by
Peter Maydell.
[1]:
[ 62.851830] Internal error: synchronous external abort: 96000410 [#1]
PREEMPT SMP
[ 62.854465] Modules linked in:
>
