From: "David Mosberger-Tang" <dmosberger@gmail.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [PATCH] get_wchan on running task sometimes MCAs the machine.
Date: Sat, 19 May 2007 02:26:38 +0000 [thread overview]
Message-ID: <ed5aea430705181926q1f29ddfpefac75221d7b472b@mail.gmail.com> (raw)
In-Reply-To: <20070517111651.GA760@lnx-holt.americas.sgi.com>
I don't think the original patch solves the problem. The issue is
that even if a task is stopped at the time unwinding is initiated, it
may be resumed later on (e.g., by another CPU), so the unwinding MUST
be safe even on a running task. get_wchan() unwinding the currently
running task is a special case, but fixing just that special case will
only make it harder to debug and fix the fundamental bug later on.
August 2005 sounds about right, I'm afraid. I'd love to update the
patch, but with Summer travel coming up etc., it won't happen before
July, for sure. It would be best if somebody else could champion the
libunwind-in-the-kernel effort. It's really needed but my time
available for this project will remain spotty for a while...
--david
On 5/18/07, Luck, Tony <tony.luck@intel.com> wrote:
> If the problem *only* occurs when trying to get_wchan on the
> current task (a futile endevour) then the original patch
> should be fine all by itself ... there are no races involved
> the current task is going to remain in TASK_RUNNING state at
> any point that it happens to look at itself.
>
> A quick scan of my e-mail archives shows an updated libunwind
> patch from you dated August 2005 ... which is a lot more than
> a couple of months ago :-) I assume that I've missed some more
> recent posting.
>
> -Tony
>
--
Mosberger Consulting LLC, http://www.mosberger-consulting.com/
next prev parent reply other threads:[~2007-05-19 2:26 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-17 11:16 [PATCH] get_wchan on running task sometimes MCAs the machine Robin Holt
2007-05-17 11:38 ` Keith Owens
2007-05-17 13:00 ` Robin Holt
2007-05-17 13:05 ` Keith Owens
2007-05-17 14:16 ` David Mosberger-Tang
2007-05-18 3:02 ` Robin Holt
2007-05-18 18:23 ` David Mosberger-Tang
2007-05-18 18:35 ` Robin Holt
2007-05-18 23:01 ` Luck, Tony
2007-05-19 2:08 ` Robin Holt
2007-05-19 2:26 ` David Mosberger-Tang [this message]
2007-05-23 3:32 ` Nick Piggin
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=ed5aea430705181926q1f29ddfpefac75221d7b472b@mail.gmail.com \
--to=dmosberger@gmail.com \
--cc=linux-ia64@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox