[Top] [All Lists]

Re: FAQ updated (was Re: XFS breakage...)

To: Nathan Scott <nathans@xxxxxxx>
Subject: Re: FAQ updated (was Re: XFS breakage...)
From: David Greaves <david@xxxxxxxxxxxx>
Date: Thu, 20 Jul 2006 14:28:32 +0100
Cc: Kasper Sandberg <lkml@xxxxxxxxxxx>, Justin Piszcz <jpiszcz@xxxxxxxxxxxxxxx>, Torsten Landschoff <torsten@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, cw@xxxxxxxx, ml@xxxxxxxx, radsaq@xxxxxxxxx
In-reply-to: <20060720171310.B1970528@xxxxxxxxxxxxxxxxxxxxxxxx>
References: <20060718222941.GA3801@xxxxxxxxxxxxxxx> <20060719085731.C1935136@xxxxxxxxxxxxxxxxxxxxxxxx> <1153304468.3706.4.camel@localhost> <20060720171310.B1970528@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird (X11/20060516)
Nathan Scott wrote:
> Correction there - no -stable exists with this yet, I guess that'll
> be once its out though.
>> what action do you suggest i do now?
> I've captured the state of this issue here, with options and ways
> to correct the problem:
>       http://oss.sgi.com/projects/xfs/faq.html#dir2
> Hope this helps.

It does, thanks :)

Does this problem exist in 2.16.6.x??

From various comments like:
  Unless 2.6.16.x is a dead-end could we please also have this patch put
  into there?
  a result (I believe) of the corruption bug that was in 2.6.16/17.
  I just want to confirm this bug as well and unfortunately it was my
  system disk too who had to take the hit. Im running 2.6.16
I assume it does.

But the FAQ says:
Q: What is the issue with directory corruption in Linux 2.6.17?
In the Linux kernel 2.6.17 release a subtle bug...

which implies it's not...


So given this is from
                         * One less used entry in the free table.
                        INT_MOD(free->hdr.nused, ARCH_CONVERT, -1);
                        xfs_dir2_free_log_header(tp, fbp);

and it looks awfully similar to the patch which says:

--- linux-
+++ linux-
@@ -970,7 +970,7 @@ xfs_dir2_leafn_remove(
                         * One less used entry in the free table.
-                       free->hdr.nused = cpu_to_be32(-1);
+                       be32_add(&free->hdr.nused, -1);
                        xfs_dir2_free_log_header(tp, fbp);

Should 2.6.16.x replace
  INT_MOD(free->hdr.nused, ARCH_CONVERT, -1);
  be32_add(&free->hdr.nused, -1);

I hope so because I assumed there simply wasn't a patch for 2.6.16 and
applied this 'best guess' to my servers and rebooted/remounted successfully.



<Prev in Thread] Current Thread [Next in Thread>