Re: [Xenomai-core] kernel BUG running cyclictest on powerpc 405

["Steven A. Falco" <sfalco@domain.hid>]

[-- Attachment #1.1: Type: text/plain, Size: 1388 bytes --]

Thanks!  I'll try your patch and report if I still have trouble.

I've attached my patch for PPC405.

    Steve

Philippe Gerum wrote:
> Steven A. Falco wrote:
>   
>> I have built a 2.6.23-rc7 kernel (from Denx git) with Xenomai 2.4-rc3. 
>> Architecture is powerpc, processor is a 405GP.
>>
>> I had to make some additions to arch/powerpc/kernel/head_40x.S, and I
>> can submit a patch if someone tells me where to post it.
>>
>>     
>
> Here would be nice, or to the Adeos list, indifferently - I would pick
> it from both anyway.
>
>   
>> However, while the kernel boots and runs, the xenomai user apps do not
>> work correctly.  Specifically, when running cyclictest, I get the following:
>>
>>     
>
> This patch should fix this issue.
>
> diff --git a/arch/powerpc/sysdev/uic.c b/arch/powerpc/sysdev/uic.c
> index eeb38e2..5a38086 100644
> --- a/arch/powerpc/sysdev/uic.c
> +++ b/arch/powerpc/sysdev/uic.c
> @@ -48,7 +48,7 @@ struct uic {
>  	int index;
>  	int dcrbase;
>
> -	spinlock_t lock;
> +	ipipe_spinlock_t lock;
>
>  	/* The remapper for this UIC */
>  	struct irq_host	*irqhost;
>   
>> How do I debug this?  I have done a little kernel hacking, but I am a
>> noob when it comes to Xenomai...
>>
>>     
>
> Well, actually, it's shame on me. I did not notice that the universal
> interrupt controller code was missing some bits in post-2.6.21 I-pipe
> patches.
>
>   

[-- Attachment #1.2: Type: text/html, Size: 1942 bytes --]

[-- Attachment #2: ipipe.patch --]
[-- Type: text/x-patch, Size: 1334 bytes --]

--- linux-2.6-denx.xenomai/arch/powerpc/kernel/head_40x.S	2007-09-28 13:40:46.000000000 -0400
+++ linux-2.6-denx/arch/powerpc/kernel/head_40x.S	2007-10-24 14:02:23.000000000 -0400
@@ -227,6 +227,12 @@
 	EXC_XFER_TEMPLATE(hdlr, n, MSR_KERNEL, NOCOPY, transfer_to_handler_full, \
 			  ret_from_except_full)
 
+#ifdef CONFIG_IPIPE
+#define EXC_XFER_IPIPE(n, hdlr)		\
+	EXC_XFER_TEMPLATE(hdlr, n+1, MSR_KERNEL, NOCOPY, transfer_to_handler, \
+			  __ipipe_ret_from_except)
+#endif /* CONFIG_IPIPE */
+
 #define EXC_XFER_LITE(n, hdlr)		\
 	EXC_XFER_TEMPLATE(hdlr, n+1, MSR_KERNEL, NOCOPY, transfer_to_handler, \
 			  ret_from_except)
@@ -395,7 +401,11 @@
 	EXC_XFER_EE_LITE(0x400, handle_page_fault)
 
 /* 0x0500 - External Interrupt Exception */
+#ifdef CONFIG_IPIPE
+	EXCEPTION(0x0500, HardwareInterrupt, __ipipe_grab_irq, EXC_XFER_IPIPE)
+#else /* !CONFIG_IPIPE */
 	EXCEPTION(0x0500, HardwareInterrupt, do_IRQ, EXC_XFER_LITE)
+#endif /* CONFIG_IPIPE */
 
 /* 0x0600 - Alignment Exception */
 	START_EXCEPTION(0x0600, Alignment)
@@ -433,7 +443,11 @@
 	lis	r0,TSR_PIS@h
 	mtspr	SPRN_TSR,r0		/* Clear the PIT exception */
 	addi	r3,r1,STACK_FRAME_OVERHEAD
+#ifdef CONFIG_IPIPE
+	EXC_XFER_IPIPE(0x1000, __ipipe_grab_timer)
+#else /* !CONFIG_IPIPE */
 	EXC_XFER_LITE(0x1000, timer_interrupt)
+#endif /* CONFIG_IPIPE */
 
 #if 0
 /* NOTE: