Re: Warning and BUG with btrfs and corrupted image

2009-02-05 Thread jim owens
Pavel Machek wrote: If you don't want it, don't compile it in. The Kconfig text is very clear. No, I'd not expect that option to panic systems. That's why I suggested: diff --git a/fs/xfs/Kconfig b/fs/xfs/Kconfig index 29228f5..b7ac847 100644 --- a/fs/xfs/Kconfig +++ b/fs/xfs/Kconfig @@ -77,4

Re: Warning and BUG with btrfs and corrupted image

2009-02-05 Thread Pavel Machek
On Thu 2009-02-05 08:02:39, Chris Mason wrote: > On Thu, 2009-02-05 at 10:02 +0100, Pavel Machek wrote: > > > > > CONFIG_*_DEBUG means include *debug* code there to help developers, > > > > > including adding additional failure tests into the kernel. Besides, > > > > > which bit of "don't turn it o

Re: Warning and BUG with btrfs and corrupted image

2009-02-05 Thread Chris Mason
On Thu, 2009-02-05 at 10:02 +0100, Pavel Machek wrote: > > > > CONFIG_*_DEBUG means include *debug* code there to help developers, > > > > including adding additional failure tests into the kernel. Besides, > > > > which bit of "don't turn it on unless you are an XFS developer" > > > > don't you un

Re: Warning and BUG with btrfs and corrupted image

2009-02-05 Thread Pavel Machek
> > > CONFIG_*_DEBUG means include *debug* code there to help developers, > > > including adding additional failure tests into the kernel. Besides, > > > which bit of "don't turn it on unless you are an XFS developer" > > > don't you understand? > > > > Yes, but DEBUG code is normally to help deb

Re: Warning and BUG with btrfs and corrupted image

2009-02-05 Thread Dave Chinner
On Wed, Feb 04, 2009 at 07:29:51PM +0100, Pavel Machek wrote: > On Sun 2009-02-01 12:40:50, Dave Chinner wrote: > > On Mon, Jan 26, 2009 at 05:27:11PM +0100, Pavel Machek wrote: > > > On Wed 2009-01-21 15:00:42, Dave Chinner wrote: > > > + Turning this option on will result in kernel panicking an