From: Dale Farnsworth <dale@farnsworth.org>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linuxppc-dev@ozlabs.org, Benjamin Herrenschmidt <benh@ozlabs.org>
Subject: Re: [RFC/PATCH] powerpc: Add irqtrace support to 32-bit powerpc
Date: Fri, 4 Apr 2008 14:29:52 -0700 [thread overview]
Message-ID: <20080404212952.GA15454@farnsworth.org> (raw)
In-Reply-To: <1207337598.19189.77.camel@johannes.berg>
On Fri, Apr 04, 2008 at 09:33:18PM +0200, Johannes Berg wrote:
> On Wed, 2008-04-02 at 16:15 +0200, Johannes Berg wrote:
> > On Fri, 2008-02-01 at 12:50 -0700, Dale Farnsworth wrote:
> > > This adds the low level irq tracing hooks for 32-bit powerpc.
> > > This patch applies on top of Benjamin Herrenschmidt's patch that
> > > provides similar support for 64-bit powerpc. See
> > > <http://ozlabs.org/pipermail/linuxppc-dev/2007-October/044361.html>
> > >
> > > It seems to work on my prpmc2800 board with a ppc7447.
> >
> > At first it appears to work on my powerbook (with Ben's latest version
> > of the 64-bit patch), but then it seems to cause random corruption, I've
> > had a crash in the ext3 code and one in console_callback(), both of
> > which I cannot otherwise explain.
>
> FWIW, same kernel without the patch works fine.
My apologies, Johannes. I made some unwarranted assumptions about
register usage in the lowest-level exception handlers. While it worked
on my 7447 board, I saw similar random failures on the 85xx (booke) system
I tried.
Thanks for testing this.
I'll follow up with a revised patch.
-Dale
next prev parent reply other threads:[~2008-04-04 21:30 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-01 19:50 [RFC/PATCH] powerpc: Add irqtrace support to 32-bit powerpc Dale Farnsworth
2008-02-01 20:09 ` Johannes Berg
2008-04-02 14:15 ` Johannes Berg
2008-04-04 19:33 ` Johannes Berg
2008-04-04 21:29 ` Dale Farnsworth [this message]
2008-04-04 21:35 ` Johannes Berg
2008-04-04 22:08 ` Benjamin Herrenschmidt
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=20080404212952.GA15454@farnsworth.org \
--to=dale@farnsworth.org \
--cc=benh@ozlabs.org \
--cc=johannes@sipsolutions.net \
--cc=linuxppc-dev@ozlabs.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.