WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] VMX virtual-apic: Update the p2m mapping alongsi

To: Tim Deegan <Tim.Deegan@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] VMX virtual-apic: Update the p2m mapping alongside the control bit
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Wed, 26 May 2010 08:22:43 +0100
Cc:
Delivery-date: Wed, 26 May 2010 00:23:42 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <fef1e7dc74e64cf25d81.1274798527@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acr8GP6Q+Ltie9wMQO2chRNHHZAFswAizyK4
Thread-topic: [Xen-devel] [PATCH] VMX virtual-apic: Update the p2m mapping alongside the control bit
User-agent: Microsoft-Entourage/12.24.0.100205
On 25/05/2010 15:42, "Tim Deegan" <Tim.Deegan@xxxxxxxxxx> wrote:

> Otherwise a VM migrated from a system which supports the feature to one
> which doesn't ends up with a data page where its vlapic should be.

I don't see that this problem exists, let alone that this patch would fix it
if it did.

The dummy data page should not be migrated because it has p2m type
mmio_direct, which will fail the p2m_is_ram() check in mod_l1_entry() when
dom0 attempts to map the page to save/migrate it.

Even if that weren't the case, the dummy page will be mapped into the
guest's p2m unless the apic is disabled or relocated from its default
physaddr location. Neither of these is especially likely to be true during
save/migration.

The only thing I can see is this patch avoids a dummy page hanging around in
the p2m when HW APIC virt is disabled. But the dummy page is pretty benign
in this situation, so I don't see that we care.

 -- Keir

> Signed-off-by: Tim Deegan <Tim.Deegan@xxxxxxxxxx>
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel