archive mirror
 help / color / mirror / Atom feed
Subject: [Bug 214077] [xfstests xfs/168] xfs_repair failed with shrinking 776672
Date: Mon, 23 Aug 2021 16:14:21 +0000	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

Gao Xiang ( changed:

           What    |Removed                     |Added
                 CC|                            |

--- Comment #5 from Gao Xiang ( ---

I tried several times without any luck in my own VM,
FSTYP         -- xfs (debug)
PLATFORM      -- Linux/x86_64 AliYun 5.13.0-rc4+ #1 SMP Mon Aug 23 20:17:12 CST
MKFS_OPTIONS  -- -f -m crc=1,finobt=1,reflink=1,rmapbt=1,bigtime=1,inobtcount=1
-b size=1024 /dev/loop1
MOUNT_OPTIONS -- /dev/loop1 /fs/loop1

xfs/168 197s ...  225s
Ran: xfs/168
Passed all 1 tests

But with my own understanding, I guess that may be due to the following

                 * Roll the transaction before trying to re-init the per-ag 
                 * reservation. The new transaction is clean so it will cancel 
                 * without any side effects. 
                error = xfs_defer_finish(tpp);  <--- here we unlock the AGF,
new allocation allowed.
                if (error) 
                        return error; 

                error = -ENOSPC; 
                goto resv_init_out; 
        xfs_ialloc_log_agi(*tpp, agibp, XFS_AGI_LENGTH); 
        xfs_alloc_log_agf(*tpp, agfbp, XFS_AGF_LENGTH);
        return 0;
        err2 = xfs_ag_resv_init(agibp->b_pag, *tpp); <--- so resv init can be
failed here.
        if (!err2)
                return error;
        xfs_warn(mp, "Error %d reserving per-AG metadata reserve pool.", err2);
        xfs_force_shutdown(mp, SHUTDOWN_CORRUPT_INCORE);
        return err2;

As I mentioned before, I'm not sure if growfs has similiar race like this.

You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.

  parent reply	other threads:[~2021-08-23 16:14 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-16 12:43 [Bug 214077] New: [xfstests xfs/168] xfs_repair failed with shrinking 776672 bugzilla-daemon
2021-08-20 13:20 ` [Bug 214077] " bugzilla-daemon
2021-08-23  3:08 ` bugzilla-daemon
2021-08-23  3:17 ` bugzilla-daemon
2021-08-23  3:37 ` bugzilla-daemon
2021-08-23 16:14 ` bugzilla-daemon [this message]
2021-08-24  3:34 ` bugzilla-daemon

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:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \

* 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;
as well as URLs for NNTP newsgroup(s).