On 09/08/2010 02:40 AM, Liu Yu wrote:
It is unnecessary to keep shadow tlb.
first, shadow tlb keep fixed value in shadow, which make things unflexible.
second, remove shadow tlb can save a lot memory.
This patch remove shadow tlb and caculate the shadow tlb entry value
before we write it to hardware.
Also we use new struct tlbe_ref to trace the relation
between guest tlb entry and page.
Did you look at the performance impact?
Back in the day, we did essentially the same thing on 440. However,
rather than discard the whole TLB when context switching away from the
host (to be demand-faulted when the guest is resumed), we found a
noticeable performance improvement by preserving a shadow TLB across
context switches. We only use it in the vcpu_put/vcpu_load path.
Of course, our TLB was much smaller (64 entries), so the use model may
not be the same at all (e.g. it takes longer to restore a full guest TLB
working set, but maybe it's not really possible to use all 1024 TLB0
entries in one timeslice anyways).
--
Hollis Blanchard
Mentor Graphics, Embedded Systems Division
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
