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] Dom0 Kernel - Fixes for saving/restoring MSI/MSI

To: "Shan, Haitao" <haitao.shan@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Dom0 Kernel - Fixes for saving/restoring MSI/MSI-X across Dom0 S3
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Mon, 24 Nov 2008 14:15:00 +0000
Cc: "'xen-devel@xxxxxxxxxxxxxxxxxxx'" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 24 Nov 2008 06:15:28 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <61563CE63B4F854986A895DA7AD3C17701FC57FF@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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: AclOJEJR+OwpiDmMQBGSwUTLsM7WbAAGsbQt
Thread-topic: [Xen-devel] [PATCH] Dom0 Kernel - Fixes for saving/restoring MSI/MSI-X across Dom0 S3
User-agent: Microsoft-Entourage/
On 24/11/08 11:03, "Shan, Haitao" <haitao.shan@xxxxxxxxx> wrote:

> With this patch, MSI/MSI-X capabilities and tables are saved in Dom0 when S3
> and restored when resume. Actually, this is also the approach that kernel
> takes. The only concern is that Dom0 should not touch MSI/MSI-X, they are
> owned by VMM itself. Maybe adding a hypercall to instruct Xen to do the
> saving/restoring is good. I wonder whether the reason is strong enough for
> adding a hypercall for such purpose.
> So Keir, maybe you can tell what is your prefer?

I think the approach in your patch is okay for now.

 -- Keir

Xen-devel mailing list