LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Juergen Pfeiffer <j.pfeiffer@seskion.de>
To: linux-kernel@vger.kernel.org
Subject: Re: long disable of Softirqs in br_forward_delay_timer_expired()
Date: Fri, 19 Jan 2007 10:35:21 +0100	[thread overview]
Message-ID: <45B090D9.7020602@seskion.de> (raw)
In-Reply-To: <45AF79AE.1060102@seskion.de>

Juergen Pfeiffer schrieb:
>
> I had problems in my implementation of Profibus-protocol, because my 
> FDL-State machine is implemented in tasklets and
> sometimes there were situations, where Soft-Irqs were disabled for 
> 20-40mS (Coldfire 5485 / 96MHz).
> After inserting some testpoints in kernels source, doing dump_stack(), 
> when the jiffie-time get longer then 20mS,
> i detected the place of the long Soft-Irq disable in function
>
> static void br_forward_delay_timer_expired(..)
> inside file "net/bridge/br_stp_timer.c"
>
> It does a
> spin_lock_bh(..);
> ... some functionality;
> spin_unlock_bh(..);


Hi

I found the reason for the long disabling of Soft-Irqs:

In-between the spin_lock_bh() and spin_unlock_bh() was a printk() going 
to serial console with 19200baud.
So it took easily 30mS for console-output.

No i start klogd an the log Messages go to a file in tmpfs in a short time.



Juergen Pfeiffer,
Seskion GmbH
Karlsruher Str. 11/1
70771 Leinfelden-Echterdingen
Germany

j.pfeiffer@seskion.de

www.seskion.de
------------------------------------------------------------------------

      reply	other threads:[~2007-01-19  9:35 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-18 13:44 Juergen Pfeiffer
2007-01-19  9:35 ` Juergen Pfeiffer [this message]

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=45B090D9.7020602@seskion.de \
    --to=j.pfeiffer@seskion.de \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: long disable of Softirqs in br_forward_delay_timer_expired()' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).