On Fri, 2009-11-06 at 10:20 +0100, Richard Cochran wrote:
> On Fri, Nov 06, 2009 at 09:26:58AM +0100, Philippe Gerum wrote:
> > Ouch. I just can't believe this went unnoticed for that long... Well, no
> > wonder why then, the critical IPI never gets registered, so never
> > detected by the pipeline core in __ipipe_grab_irq. Thanks for the heads
> > up.
> > 
> > This may make things work a little better:
> 
> Yes, works fine now. Thanks for your help.

Well, thank you for digging this issue, first.

> 
> Richard
> 
> > diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c
> > index 8968b24..a4fe229 100644
> > --- a/arch/powerpc/kernel/smp.c
> > +++ b/arch/powerpc/kernel/smp.c
> > @@ -164,16 +164,16 @@ int smp_request_message_ipi(int virq, int msg)
> >     if (msg < 0 || msg > PPC_MSG_DEBUGGER_BREAK) {
> >             return -EINVAL;
> >     }
> > -#if !defined(CONFIG_DEBUGGER) && !defined(CONFIG_KEXEC)
> > -   if (msg == PPC_MSG_DEBUGGER_BREAK) {
> > -           return 1;
> > -   }
> > -#endif
> >  #ifdef CONFIG_IPIPE
> >     if (msg == PPC_MSG_DEBUGGER_BREAK)
> >             /* Piggyback the debugger IPI for the I-pipe. */
> >             __ipipe_register_ipi(virq);
> >  #endif
> > +#if !defined(CONFIG_DEBUGGER) && !defined(CONFIG_KEXEC)
> > +   if (msg == PPC_MSG_DEBUGGER_BREAK) {
> > +           return 1;
> > +   }
> > +#endif
> >  
> >     err = request_irq(virq, smp_ipi_action[msg], IRQF_DISABLED|IRQF_PERCPU,
> >                       smp_ipi_name[msg], 0);
> > 
> > -- 
> > Philippe.
> > 
> > 


-- 
Philippe.



_______________________________________________
Xenomai-core mailing list
Xenomai-core@gna.org
https://mail.gna.org/listinfo/xenomai-core

Reply via email to