Re: qemu-kvm.git unittest failures

[Avi Kivity <avi@redhat.com>]

On 07/03/2010 01:44 AM, Marcelo Tosatti wrote:
>
>> Config entry:
>>
>> [access]
>> file = access.flat
>>
>> Massive log, compressed and attached.
>>      
> run
> test pde.p user: FAIL: error code 5 expected 4
> test pte.rw pde.p user: FAIL: error code 5 expected 4
> test pte.user pde.p user: FAIL: error code 5 expected 4
> test pte.rw pte.user pde.p user: FAIL: error code 5 expected 4
> test pte.a pde.p user: FAIL: error code 5 expected 4
> test pte.rw pte.a pde.p user: FAIL: error code 5 expected 4
> test pte.user pte.a pde.p user: FAIL: error code 5 expected 4
>
> P flag (bit 0).
> This flag is 0 if there is no valid translation for the linear address
> because the P
> flag was 0 in one of the paging-structure entries used to translate that
> address.
>
> Avi, a walk ignoring access permissions should be done to properly set
> the P flag on error code. Does anybody care?
>    

Probably not, but best to be compatible with real hardware.

We could simply continue the existing walk, just set a flag indicating 
that it can't succeed.

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.