public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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



      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