|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking chang
At 11:46 +0100 on 27 Jun (1309175170), Tim Deegan wrote:
> This patch series tidies up a few bits ofthe nested p2m code.
> The main thing it does is reorganize the locking so that most of the
> changes to nested p2m tables happen only under the p2m lock, and the
> nestedp2m lock is only needed to reassign p2m tables to new cr3 values.
There are still a few things I'm not convinced about in the nested NPT
code:
- The function that allocates new nested p2ms probably needs an
overhaul, as I said in my last email.
- The flushing policy is a bit confusing: e.g., what exactly ought to
happen when the guest sets the tlb-control bits? AFAICS the nested-p2ms
are already kept in sync with host-p2m changes, and we flush all
TLBs when we update nested-p2ms, so can we skip this extra flush?
- Why is there a 10x increase in IPIs after this series? I don't see
what sequence of events sets the relevant cpumask bits to make this
happen.
Cheers,
Tim.
--
Tim Deegan <Tim.Deegan@xxxxxxxxxx>
Principal Software Engineer, Xen Platform Team
Citrix Systems UK Ltd. (Company #02937203, SL9 0BG)
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
<Prev in Thread] |
Current Thread |
[Next in Thread>
|
- [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes, Tim Deegan
- [Xen-devel] [PATCH 1 of 5] Nested p2m: implement "flush" as a first-class action, Tim Deegan
- [Xen-devel] [PATCH 2 of 5] Nested p2m: remove bogus check of CR3 value, Tim Deegan
- [Xen-devel] [PATCH 3 of 5] Nested p2m: clarify logic in p2m_get_nestedp2m(), Tim Deegan
- [Xen-devel] [PATCH 4 of 5] Nested p2m: flush only one p2m table when reallocating, Tim Deegan
- [Xen-devel] [PATCH 5 of 5] Nested p2m: rework locking around nested-p2m flushes and updates, Tim Deegan
- Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes,
Tim Deegan <=
- Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes, Tim Deegan
- Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes, Christoph Egger
- Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes, Christoph Egger
- Re: [Xen-devel] [PATCH 0 of 5] v2: Nested-p2m cleanups and locking changes, Tim Deegan
|
|
|
|
|