From: Matt Fredrickson <creslin@digium.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Device Driver Etiquette
Date: Sun, 3 Jun 2007 01:20:37 -0500 (CDT) [thread overview]
Message-ID: <13097765.160991180851637904.JavaMail.root@jupiler.digium.com> (raw)
In-Reply-To: <17916027.160971180851607681.JavaMail.root@jupiler.digium.com>
----- "Arjan van de Ven" <arjan@infradead.org> wrote:
> On Fri, 2007-06-01 at 12:47 -0500, Matthew Fredrickson wrote:
>
> > My question is this: is there a way to either work around the
> problem I
> > am seeing with the stack without recompiling the kernel with 8K
> stack
> > size or without disabling irqs for such a long period of time (which
> I
> > think is not a nice thing to do either) OR is it acceptable
> (although
> > not nice) to simply fix it this way, by disabling irqs while it
> loads
> > the firmware?
>
> I wonder if you're chasing ghosts; 4K stack kernels have a seperate
> stack for interrupts so... those should be safe.
>
> Btw, you forgot to post a pointer to the source code of your driver,
> so
> it's a lot harder for us (read: impossible) to give you good advice..
As someone mentioned in another post, I believe what is causing this problem is a combination of factors. The triggering of the softlockup detector seems to be what pushes it over the edge. I think I will try as suggested to change the timer for it so that it does not trigger while the card is initializing.
If you'd like to see the source, here's a link:
http://svn.digium.com/view/zaptel/trunk/wct4xxp/
I can't give you the stack trace right now, since I'm not in the office, but it begins in the vpm450_init function and goes into the Octasic API functions, in the ChannelOpen routines as well as the chip initialization routines (the ones that load the firmware.
---
Matthew Fredrickson
Kernel Developer
Digium, Inc.
next parent reply other threads:[~2007-06-03 6:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <17916027.160971180851607681.JavaMail.root@jupiler.digium.com>
2007-06-03 6:20 ` Matt Fredrickson [this message]
[not found] <12467610.159201180797054019.JavaMail.root@jupiler.digium.com>
2007-06-02 15:12 ` Device Driver Etiquette Matt Fredrickson
2007-06-03 11:22 ` Daniel J Blueman
2007-06-01 23:16 Daniel J Blueman
-- strict thread matches above, loose matches on Subject: below --
2007-06-01 17:47 Matthew Fredrickson
2007-06-01 18:38 ` Lee Revell
2007-06-01 22:28 ` H. Peter Anvin
2007-06-02 15:05 ` Satyam Sharma
2007-06-02 23:22 ` Arjan van de Ven
2007-06-03 1:45 ` Satyam Sharma
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=13097765.160991180851637904.JavaMail.root@jupiler.digium.com \
--to=creslin@digium.com \
--cc=arjan@infradead.org \
--cc=linux-kernel@vger.kernel.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.