From: Kasper Dupont <kasperd@cpvhh.24.jul.2011.kasperd.net>
To: Francois Romieu <romieu@fr.zoreil.com>
Cc: ivecera@redhat.com, hayeswang@realtek.com, gregkh@suse.de,
netdev@vger.kernel.org
Subject: Re: r8169 driver crashes in 2.6.32.43
Date: Fri, 5 Aug 2011 16:40:17 +0200 [thread overview]
Message-ID: <20110805144017.GA20006@colin.search.kasperd.net> (raw)
In-Reply-To: <20110805140047.GA19758@colin.search.kasperd.net>
On 05/08/11 16.08, Kasper Dupont wrote:
> At that point it did not itterate through the loop again
> and it did not leave the interrupt handler either. I'll
> power cycle the machine and take a closer look on the
> source to see what could possible be happening at that
> point.
I looked at the source around the place where that lockup
happened. There was absolutely no I/O or loops happening
between the two printk calls. It seemed the only real
candidate for a culprit responsible for that lockup was
the printk calls themselves.
Is it plausible that in 2.6.32.43 it is not safe to call
printk from within an interrupt handler?
I added some more printk statements in an attempt to find
out how it was possible for the code to lock up between
the end of the loop and the exit from the interrupt handler.
I wasn't able to reproduce the lockup in the same spot, but
instead I saw a lockup inside the loop in the branch where
it does netif_stop_queue.
Right now I suspect those builds where I added printk
statements lockup due to the printk statements. But does
the plain 2.6.32.43 kernel then also lockup due to printk
statements in the interrupt handler, or is it something
else?
--
Kasper Dupont -- Rigtige mænd skriver deres egne backupprogrammer
#define _(_)"d.%.4s%."_"2s" /* This is my email address */
char*_="@2kaspner"_()"%03"_("4s%.")"t\n";printf(_+11,_+6,_,11,_+2,_+7,_+6);
prev parent reply other threads:[~2011-08-05 14:40 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20110724195831.GA8718@colin.search.kasperd.net>
[not found] ` <20110724201626.GB24418@zoreil.com>
2011-07-25 10:36 ` r8169 driver crashes in 2.6.32.43 Kasper Dupont
2011-07-28 7:04 ` Francois Romieu
2011-07-28 8:48 ` Kasper Dupont
2011-07-28 10:58 ` Francois Romieu
2011-07-28 11:43 ` Kasper Dupont
2011-07-28 11:59 ` Kasper Dupont
2011-07-28 12:23 ` Francois Romieu
2011-07-28 12:45 ` Kasper Dupont
2011-07-28 12:54 ` Kasper Dupont
2011-07-28 14:47 ` Francois Romieu
2011-07-28 21:01 ` Kasper Dupont
2011-08-05 14:08 ` Kasper Dupont
2011-08-05 14:40 ` Kasper Dupont [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=20110805144017.GA20006@colin.search.kasperd.net \
--to=kasperd@cpvhh.24.jul.2011.kasperd.net \
--cc=gregkh@suse.de \
--cc=hayeswang@realtek.com \
--cc=ivecera@redhat.com \
--cc=netdev@vger.kernel.org \
--cc=romieu@fr.zoreil.com \
/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 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).