Re: [PATCH v2 0/4] Fix accessed bit tracking

[Avi Kivity <avi@redhat.com>]

On 06/07/2010 11:43 AM, Lai Jiangshan wrote:
> Avi Kivity wrote:
>    
>> The kvm mmu synchronizes shadow ptes using the mmu lock, however the cpu
>> will happily ignore the lock when setting the accessed bit.  This can cause
>> the accessed bit to be lost.  Luckily this only results in incorrect page
>> selection for swap.
>>
>>      
> Atomic operation is heavy and slow, it hurts performance.
> Incorrect page selection for swap also hurts performance.
>    

We can avoid the exchange in most cases, for example if the new spte has 
the accessed bit set (already in the patch set) or if the page is 
already marked as accessed, or if we see the old spte has the accessed 
bit set (so no race can occur).  I'll update the patches to avoid 
atomics when possible.

I don't think atomics are that expensive, though, ~20 cycles on modern 
processors?

> I think there are very rare competitions happened and cause
> the accessed bit to be lost. Since there is no incorrect result
> when the accessed bit is lost. Is this problem over concern?
>    

The real concern is when we start using the dirty bit.  I'd like to 
fault read accesses with writeable sptes, but with the dirty bit clear.  
This way we can allow a guest to write to a page without a fault, but 
not cause it to swap too soon.

-- 
error compiling committee.c: too many arguments to function