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] fix "translate Xen-provided PIRQs" patch.

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] fix "translate Xen-provided PIRQs" patch.
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Tue, 16 Sep 2008 21:00:45 +0900
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 16 Sep 2008 05:01:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C4F5176B.1D399%keir.fraser@xxxxxxxxxxxxx>
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>
References: <20080916023832.GD1345%yamahata@xxxxxxxxxxxxx> <C4F5176B.1D399%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.6i
Thank you for the quick commit.
Here is the patch to make xenLinux/ia64 boot.

evnchn: fix to 669:0ba49e9e74f9.

To keep the previous behavior of event channel for ia64,
set identity_mapped_irq() true.

Signed-off-by: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>

diff -r 59d3dee69ddf drivers/xen/core/evtchn.c
--- a/drivers/xen/core/evtchn.c Tue Sep 16 20:43:24 2008 +0900
+++ b/drivers/xen/core/evtchn.c Tue Sep 16 21:03:27 2008 +0900
@@ -764,7 +764,7 @@
 #elif defined(CONFIG_X86)
 #define identity_mapped_irq(irq) (((irq) - PIRQ_BASE) < 16)
 #else
-#define identity_mapped_irq(irq) (0)
+#define identity_mapped_irq(irq) (1)
 #endif
 
 int evtchn_map_pirq(int irq, int xen_pirq)



On Tue, Sep 16, 2008 at 07:57:31AM +0100, Keir Fraser wrote:
> I'll apply something like your patch.
> 
>  -- Keir
> 
> On 16/9/08 03:38, "Isaku Yamahata" <yamahata@xxxxxxxxxxxxx> wrote:
> 
> > Hi Keir.
> > 
> > The c/s of 660:7886619f623e in linux-2.6.18-xen breaks ia64.
> > Should I make the related functions arch-specific moving them
> > to linux/arch/x86/kernel/io_apic-xen.c?
> > Please consider applying the following patch or give me a suggestion.
> > 
> > IA64 doesn't have IO APIC so that the macro, IO_APIC_IRQ(),
> > doesn't make sense.
> > Instead, it has IO SAPIC which differs from IO APIC.
> > 
> > thanks,
> > 
> > On Fri, Sep 12, 2008 at 07:09:48PM +0900, Isaku Yamahata wrote:
> >> fix "translate Xen-provided PIRQs" patch.
> >> 
> >> This patch fixes the ia64 breakage caused by 660:7886619f623e.
> >> The change set is very x86 specific. So the functions, evtchn_map_pirq(),
> >> evtchn_get_xen_pirq() should be arch specific.
> >> 
> >> However the pulling out those function out from evntchan would be
> >> very ugly because they use static functions in the file.
> >> Eventually the issue in the upstream linux will be addressed by pv_ops,
> >> and it doesn't matter much on 2.6.18-xen for ia64. So I keep this patch
> >> as small as possible for now.
> >> 
> >> Signed-off-by: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
> >> 
> >> diff -r 035670ec617c drivers/xen/core/evtchn.c
> >> --- a/drivers/xen/core/evtchn.c Tue Sep 09 15:13:29 2008 +0100
> >> +++ b/drivers/xen/core/evtchn.c Fri Sep 12 18:10:16 2008 +0900
> >> @@ -761,9 +761,13 @@
> >> irq_info[irq] = mk_irq_info(IRQT_PIRQ, irq, 0);
> >>  }
> >>  
> >> +#ifndef CONFIG_X86
> >> +#define IO_APIC_IRQ(irq) (0)
> >> +#else
> >>  #ifndef CONFIG_X86_IO_APIC
> >>  #undef IO_APIC_IRQ
> >>  #define IO_APIC_IRQ(irq) ((irq) >= pirq_to_irq(16))
> >> +#endif
> >>  #endif
> >>  
> >>  int evtchn_map_pirq(int irq, int xen_pirq)
> >> 
> >> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
> 

-- 
yamahata

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

<Prev in Thread] Current Thread [Next in Thread>