From: Timothy Shimmin <tes@sgi.com>
To: Michael Nishimoto <miken@agami.com>
Cc: XFS Mailing List <xfs@oss.sgi.com>
Subject: Re: inconsistent xfs log record
Date: Wed, 09 Apr 2008 14:38:07 +1000 [thread overview]
Message-ID: <47FC482F.7090408@sgi.com> (raw)
In-Reply-To: <47FAD04D.5080308@agami.com>
Michael Nishimoto wrote:
> I've just finished analyzing an xfs filesystem which won't recover.
> An inconsistent log record has 332 log operations but the num_logop field
> in the record header says 333 log operations. The result is that xfs
> recovery
> complains with "bad clientid" because recovery eventually attempts to
> decode
> garbage.
>
> The log record really has 332 log ops (I counted!).
>
> Looking through xlog_write(), I don't see any way that record_cnt can be
> bumped
> without also writing out a log operation.
>
> Does this issue ring a bell with anyone?
>
> Michael
>
Having a bit of a look at other bugs than the snapshot one...
nothing really helpful.
I've seen a few "bad clientid" but that, as you say, just reflects that
at some point we have crap in the log op header which we
notice when doing recovery.
I had one (pv#945899) where it seemed to have got the head of the log wrong -
you could see using "xfs_logprint -d" at the change of cycle#s - it didn't
match.
Yours appears different.
I also had another one (pv#971596) but I didn't narrow it down to the
wrong# of log ops but maybe I wasn't looking carefully enough at the time.
Okay, for that one there were 2 bugs in one, one for bad clientid and
one for bad transaction - for the bad transaction,
there was something like a 2nd startop without an intervening commit op
for the tid - I moved onto something else before getting anywhere further.
--Tim
prev parent reply other threads:[~2008-04-09 4:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-08 1:54 inconsistent xfs log record Michael Nishimoto
2008-04-08 15:50 ` David Chinner
2008-04-08 16:14 ` Michael Nishimoto
2008-04-08 21:36 ` Michael Nishimoto
2008-04-09 2:30 ` David Chinner
2008-04-09 4:38 ` Timothy Shimmin [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=47FC482F.7090408@sgi.com \
--to=tes@sgi.com \
--cc=miken@agami.com \
--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