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 02/14] Nested Virtualization: localevent

To: Christoph Egger <Christoph.Egger@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH 02/14] Nested Virtualization: localevent
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Tue, 10 Aug 2010 09:21:48 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 10 Aug 2010 01:22:32 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <201008101001.29449.Christoph.Egger@xxxxxxx>
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: Acs4ZRN8zoyViiYgA0maCpzQSYZxsg==
Thread-topic: [Xen-devel] [PATCH 02/14] Nested Virtualization: localevent
User-agent: Microsoft-Entourage/
On 10/08/2010 09:01, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

>> Ah, this is the crux of it. You shouldn't need to stgi from the vcpu
>> destructor. It makes no sense and doing it shouldn't leave you with a
>> zombie domain.
> I backed out the 'localevent' patch in my local tree, removed the stgi
> call in the vcpu destructor and run tests. A lot of things have been
> changed since the issue has been found and the real bug might have
> already been fixed in the meantime.
> I haven't seen any issues with that changes in my tests so my next
> patch series I send will have the localevent patch and the stgi call
> dropped.

Thanks. I applied the p2m infrastructure patch that Tim acked, by the way.
Also may as well give Intel a little longer to respond on the common
infrastructure patches.

 -- keir

Xen-devel mailing list