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/
Home Products Support Community News


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

To: Tim Deegan <Tim.Deegan@xxxxxxxxxxxxx>
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 10:46:01 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Wed, 26 May 2010 02:49:04 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20100526085754.GE15976@xxxxxxxxxxxxxxxxxxxxxxx>
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: Acr8sYlAndjoUJO7TjGgvcl/dMtp1QABraoB
Thread-topic: [Xen-devel] [PATCH] VMX virtual-apic: Update the p2m mapping alongside the control bit
User-agent: Microsoft-Entourage/
On 26/05/2010 09:57, "Tim Deegan" <Tim.Deegan@xxxxxxxxxxxxx> wrote:

>> 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.
> Ah, so it will; I hadn't seen that.  The full failure mode is actually
> on migrating from Xen 3.3 (where mod_l1_entry() didn't have that check)
> to a newer Xen on a machine where the HW APIC virt isn't supported.  The
> sender sends the contents of the dummy page (incorrectly) and the
> receiver doesn't do anything (assuming that there'll be a hole in the
> p2m where the APIC should be).

Well, even so, the patch would still appear to only clear the p2m entry if
the APIC is disabled or relocated. Why would either be the case during

Anyway, somewhat academic given we agree it's not needed at all in 3.4+

 -- Keir

Xen-devel mailing list