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: >
