All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Lukasz Zemla <Lukasz.Zemla@woodward.com>
Cc: "Xenomai@xenomai.org" <Xenomai@xenomai.org>
Subject: Re: [Xenomai] Xenomai installation on P1020RDB
Date: Sun, 29 Jul 2012 16:30:53 +0200	[thread overview]
Message-ID: <5015491D.80907@xenomai.org> (raw)
In-Reply-To: <5012BF81.90404@xenomai.org>

On 07/27/2012 06:19 PM, Philippe Gerum wrote:
> On 07/27/2012 06:12 PM, Lukasz Zemla wrote:
>>
>>> -----Original Message-----
>>> From: Philippe Gerum [mailto:rpm@xenomai.org]
>>> Sent: Friday, July 27, 2012 6:09 PM
>>> To: Gilles Chanteperdrix
>>> Cc: Lukasz Zemla; Xenomai@xenomai.org
>>> Subject: Re: [Xenomai] Xenomai installation on P1020RDB
>>>
>>> On 07/27/2012 06:00 PM, Gilles Chanteperdrix wrote:
>>>> On 07/27/2012 05:53 PM, Lukasz Zemla wrote:
>>>>
>>>>> What I discovered, the source of the '76s problem' could be
>>> preemption model. Now, when I set:
>>>>>    - CONFIG_PREEMPT - the lag exists exactly after eth0
>>> initialization.
>>>>>    - CONFIG_PREEMPT_VOLUNTARY - the boot process stops for a while
>>> (about 50-70s) in different places.
>>>>>    - CONFIG_PREEMPT_NONE then all is fine, I cannot observe any
>>> described problems.
>>>>
>>>>
>>>> CONFIG_PREEMPT or PREEMPT_VOLUNTARY. Try this patch:
>>>>
>>>> diff --git a/arch/powerpc/kernel/ipipe.c
>>> b/arch/powerpc/kernel/ipipe.c
>>>> index 45302c1..5b706da 100644
>>>> --- a/arch/powerpc/kernel/ipipe.c
>>>> +++ b/arch/powerpc/kernel/ipipe.c
>>>> @@ -344,8 +344,10 @@ static void __ipipe_do_IRQ(unsigned int irq,
>>> void
>>>> *cookie)
>>>>
>>>>    static void __ipipe_do_timer(unsigned int irq, void *cookie)
>>>>    {
>>>> +    irq_enter();
>>>>        check_stack_overflow();
>>>>        timer_interrupt(__this_cpu_ptr(&ipipe_percpu.tick_regs));
>>>> +    irq_exit();
>>>>    }
>>>>
>>>>    asmlinkage int __ipipe_grab_timer(struct pt_regs *regs)
>>>>
>>>> Or to remove the #ifdef CONFIG_IPIPE / #endif around the calls to
>>>> irq_enter/irq_exit in arch/powerpc/kernel/time.c
>>>>
>>>
>>> Mm, no. On ppc, we use a virtual irq to map the decrementer, so the
>>> pipeline core already took care of marking the irq entry before calling
>>> the __ipipe_do_timer handler. The issue may be in the log syncer
>>> instead.
>>
>> Maybe I have some symptom which I forgot to write. When the system
>> with CONFIG_PREEMPT stayed unused for last night I saw following errors:
>
> I should be able to reproduce this issue on some hw I have access to.
> I'll have a look and let you know. Thanks.
>

Unlucky. I can't reproduce this issue, enabling preemption on a P1022 
board. So we'll need to chase this bug differently.

Did you try disabling CONFIG_XENOMAI, while leaving CONFIG_IPIPE 
enabled? This would tells us whether the issue is triggered by having a 
high priority domain stacked.

Also, if you intend to debug further, I'd suggest that you enable 
CONFIG_IPIPE_DEBUG_CONTEXT and CONFIG_IPIPE_DEBUG_INTERNAL. Since you 
have CONFIG_SERIAL_8250_CONSOLE enabled already, you would have access 
to the __ipipe_serial_debug() service, which is a replacement for 
printk() in desperate cases, which does brute force output to the uart, 
and therefore does not suffer from any delay/buffering (*).

TIA,

(*) caution: this service only works after the 16650 MMIO area has been 
mapped in, so don't try it before the early serial devices have initialized.

-- 
Philippe.


  reply	other threads:[~2012-07-29 14:30 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-20 14:31 [Xenomai] Xenomai installation on P1020RDB Lukasz Zemla
2012-07-20 18:25 ` Wolfgang Grandegger
2012-07-23  8:48   ` Richard Cochran
2012-07-23 12:01     ` Wolfgang Grandegger
2012-07-25 16:18   ` Lukasz Zemla
2012-07-25 17:11     ` Gilles Chanteperdrix
2012-07-25 17:36       ` Gilles Chanteperdrix
2012-07-25 20:15         ` Lukasz Zemla
2012-07-25 20:42           ` Gilles Chanteperdrix
2012-07-26 11:20             ` Lukasz Zemla
2012-07-26 12:00               ` Gilles Chanteperdrix
2012-07-27 15:53                 ` Lukasz Zemla
2012-07-27 16:00                   ` Gilles Chanteperdrix
2012-07-27 16:08                     ` Philippe Gerum
2012-07-27 16:12                       ` Lukasz Zemla
2012-07-27 16:19                         ` Philippe Gerum
2012-07-29 14:30                           ` Philippe Gerum [this message]
2012-07-29 14:10         ` Philippe Gerum
2012-07-25 19:26     ` Wolfgang Grandegger
2012-07-25 20:26       ` Lukasz Zemla
2012-07-25 20:55         ` Wolfgang Grandegger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5015491D.80907@xenomai.org \
    --to=rpm@xenomai.org \
    --cc=Lukasz.Zemla@woodward.com \
    --cc=Xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.