From: Simon Kagstrom <simon.kagstrom@netinsight.net>
To: ebiederm@xmission.com (Eric W. Biederman)
Cc: "Andrew Morton" <akpm@linux-foundation.org>,
dedekind1@gmail.com, "Américo Wang" <xiyou.wangcong@gmail.com>,
linux-kernel@vger.kernel.org,
"David Woodhouse" <dwmw2@infradead.org>,
"Ingo Molnar" <mingo@elte.hu>
Subject: Re: [PATCH] Provide ways of crashing the kernel through debugfs
Date: Tue, 2 Feb 2010 09:16:57 +0100 [thread overview]
Message-ID: <20100202091657.124e6b0c@marrow.netinsight.se> (raw)
In-Reply-To: <m1eil4i85t.fsf@fess.ebiederm.org>
On Mon, 01 Feb 2010 20:16:46 -0800
ebiederm@xmission.com (Eric W. Biederman) wrote:
> Andrew Morton <akpm@linux-foundation.org> writes:
> > Yes, it's the sort of thing which lots of people have written
> > throw-away ad-hoc versions of. It probably makes sense to do it once,
> > do it right to save people from having to rererereinvent that wheel.
> >
> > What do others think?
>
> I think it makes sense, and in fact we have already merged one attempt
> at doing this generically. drivers/misc/lkdtm.c
So this functionality was already there - it would appear that I didn't do my
research good enough then (I did actually look for this kind of thing).
If something, I guess it shows that the kernel source is getting big!
To add to the irony, I work for a company that works on "DTM", although
it's not related to this functionality :-)
> I think Simon's patch adds some additional interesting failure modes.
> write_after_free, corrupt_stack_write, unaligned_load_store.
>
> Simon is there any chance you can change your patch to an enhancement of lkdtm?
>
> lkdtm actually digs into the interesting failure points with a jprobe
> to trigger the harder to reproduce scenarios. Like stack overflow in
> an interrupt handler.
Yes, this would be a better way of supporting the crash functionality.
I'll take a look on integrating the extra tests into lkdtm.
I think it would also be good to provide a debugfs interface to lkdtm
similar to what provoke-crash has. Right now you have to set it up via
module parameters, and for some tests (write_after_free) I think that's
a limitation since it often doesn't crash immediately.
// Simon
next prev parent reply other threads:[~2010-02-02 8:17 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-26 9:56 Simon Kagstrom
2010-01-26 10:08 ` Américo Wang
2010-01-26 10:18 ` Simon Kagstrom
2010-01-27 2:53 ` Américo Wang
2010-01-27 7:09 ` Simon Kagstrom
2010-01-28 14:38 ` Artem Bityutskiy
2010-01-29 6:13 ` Simon Kagstrom
2010-01-29 10:33 ` Andrew Morton
2010-02-02 4:16 ` Eric W. Biederman
2010-02-02 8:16 ` Simon Kagstrom [this message]
2010-02-03 8:52 ` [PATCH] lkdtm: Add debugfs access and loosen KPROBE ties Simon Kagstrom
2010-02-03 22:55 ` Andrew Morton
2010-02-04 9:06 ` [PATCH v2] " Simon Kagstrom
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=20100202091657.124e6b0c@marrow.netinsight.se \
--to=simon.kagstrom@netinsight.net \
--cc=akpm@linux-foundation.org \
--cc=dedekind1@gmail.com \
--cc=dwmw2@infradead.org \
--cc=ebiederm@xmission.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=xiyou.wangcong@gmail.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
all inboxes | Powered by JetHome®