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, resend] replacement for noirqdebug hack

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH, resend] replacement for noirqdebug hack
From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
Date: Thu, 08 Jun 2006 09:00:40 +0200
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 08 Jun 2006 08:31:52 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <2d952ccdfba10b96dadf09495853dcdf@xxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <448028CD.76E4.0078.0@xxxxxxxxxx> <2d952ccdfba10b96dadf09495853dcdf@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> Keir Fraser <Keir.Fraser@xxxxxxxxxxxx> 07.06.06 17:57 >>>
>
>How about the following patch, which is a cleaner modification to Linux 
>and does not affect shared_info layout? Note that it depends on a Xen 
>patch that is currently in the staging tree (extends the irq info 
>hypercall to inform about sharing status), and that the patch is 
>against xen-unstable. The status check is much slower than reading from 
>shared memory, but ought to be on a rare path (unless you have a 
>high-rate interrupt line shared across guests, which is going to be 
>rather suckful anyway).

Yes, that looks fine to me (assuming that the other patch you mention is the 
one introducing
PHYSDEVOP_irq_status_query, and that this is a non-intrusive addition).

Jan

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