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] Re: [PATCH] irq: Exclude percpu IRQs from being fixed up

To: Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [PATCH] irq: Exclude percpu IRQs from being fixed up
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: Thu, 17 Feb 2011 09:09:25 +0000
Cc: Jeremy Fitzhardinge <jeremy@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Eddie Dong <eddie.dong@xxxxxxxxx>, Xin Li <xin.li@xxxxxxxxx>, Fengzhe Zhang <fengzhe.zhang@xxxxxxxxx>
Delivery-date: Thu, 17 Feb 2011 01:10:09 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D5CE1DE0200007800032557@xxxxxxxxxxxxxxxxxx>
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>
Organization: Citrix Systems, Inc.
References: <1A42CE6F5F474C41B63392A5F80372B2335E978D@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <4D5BF2FE02000078000322EB@xxxxxxxxxxxxxxxxxx> <4D5C68AF.3030807@xxxxxxxxx> <4D5CE1DE0200007800032557@xxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Thu, 2011-02-17 at 07:52 +0000, Jan Beulich wrote:
> >>> On 17.02.11 at 01:15, Fengzhe Zhang <fengzhe.zhang@xxxxxxxxx> wrote:
> > IRQ_PER_CPU switch is not found in current Kconfig. I'm not sure if this 
> 
> kernel/irq/Kconfig (introduced as a generic option in 2.6.38-rc2). In
> prior kernel you'd have to add a respective Kconfig item in
> drivers/xen/Kconfig.

Also this should be fixed in mainline _before_ being considered for
backporting to the xen/stable-2.6.32.x branch, otherwise it will simply
come back one day when the stable branch moves forward...

> > feature is going to be brought back in the short term. I remove the 
> > ifdef to set IRQ_PER_CPU flag in desc by default but still leave the IRQ 
> > handling logic unchanged. This is a temporary solution to fix system 
> > crash on poweroff. And this is the fix with minimum impact among the 
> > several solutions we tried.
> 
> But it's more a hack than a fix.

Agreed, it seems to take a very narrow approach to a specific failure
without looking at the bigger picture.

>  And making per-CPU IRQs properly
> treated as such isn't a bad idea in any case, I would say.

In addition changing generic code, which also has an impact on native,
in this way needs a lot more rationale in the commit message as to why
it is correct for both Xen and native. Most importantly it needs to go
via the x86 maintainers and not the Xen maintainers.

It also need to be made very clear why the semantics which are required
for this specific lock (lock_kicker_irq) are correct and desirable for
_every_  IRQ_PER_CPU (aka IRQF_PERCPU) lock on x86. The description of
this patch does not do this.

How does this change tie in with the existing mainline IRQF_NO_SUSPEND
flag (which Xen uses on these IPI IRQs) and the IRQF_FORCE_RESUME flag
currently in the tip tree (intended for 2.6.39, I believe)?

Ian.


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