From: Linda Walsh <xfs@tlinx.org>
To: xfs-oss <xfs@oss.sgi.com>, LKML <linux-kernel@vger.kernel.org>,
Eric Sandeen <sandeen@sandeen.net>,
Dave Chinner <david@fromorbit.com>
Subject: Re: XFS Lock debugging noise or real problem?
Date: Wed, 13 Aug 2008 18:04:16 -0700 [thread overview]
Message-ID: <48A38490.7090604@tlinx.org> (raw)
In-Reply-To: <20080814004101.GE6119@disturbed>
Dave Chinner wrote:
> I've asked the lockdep ppl to treat stuff like memory reclaim and
> the iprune_mutex specially because of this recursive calling nature
> of memory reclaim, but so far nothing has happened....
---
So it's really a kernel bug, not an XFS bug...(?)
> FWIW, I think that recent changes have resulted in the xfs_fsr case
> (swap_extents) being annotated properly so that one should go
> away.
---
If it was limited to xfs_fsr, that'd be tolerable -- but its
cropping up in random user-level-apps (imaps, sort, et al).
> Well, any debugging code is really designed for test and dev systems,
> not for production systems.....
---
The lock-correctness code is described as a feature to provide
"provability". It's not called "debugging" and I don't regard that as
"debugging" -- but something that any production system that wants
operational integrity over a minor 'speed hit', would "theoretically"
want.
If it is "debug" code, it should be labeled as such -- but
code that can mathematically guarantee that parts of the kernel operate
correctly seems like a _reliability_ feature, not a debugging feature.
Thanks for the insight -- very appreciated.
linda
prev parent reply other threads:[~2008-08-14 1:04 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-11 19:31 Lock debugging noise or real problem? Linda Walsh
2008-08-11 20:10 ` Eric Sandeen
2008-08-12 2:33 ` Linda Walsh
2008-08-12 2:48 ` Eric Sandeen
2008-08-12 3:35 ` Linda Walsh
2008-08-12 22:28 ` XFS " Linda A. Walsh
2008-08-13 0:58 ` Dave Chinner
2008-08-13 22:05 ` Linda Walsh
2008-08-14 0:41 ` Dave Chinner
2008-08-14 1:04 ` Linda Walsh [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=48A38490.7090604@tlinx.org \
--to=xfs@tlinx.org \
--cc=david@fromorbit.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sandeen@sandeen.net \
--cc=xfs@oss.sgi.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