From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: linux-xfs@vger.kernel.org
Cc: Dave Chinner <david@fromorbit.com>
Subject: Re: [PATCH v2 10/21] xfs: set up scrub cross-referencing helpers
Date: Tue, 16 Jan 2018 15:06:22 -0800 [thread overview]
Message-ID: <20180116230622.GZ5602@magnolia> (raw)
In-Reply-To: <20180105215451.GM5602@magnolia>
On Fri, Jan 05, 2018 at 01:54:51PM -0800, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Create some helper functions that we'll use later to deal with problems
> we might encounter while cross referencing metadata with other metadata.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> v2: pass corruption flags directly to helpers, reducing helper function count
Ping?
--D
> ---
> fs/xfs/scrub/btree.c | 71 +++++++++++++++++++++-----
> fs/xfs/scrub/btree.h | 9 +++
> fs/xfs/scrub/common.c | 135 +++++++++++++++++++++++++++++++++++++++++++++----
> fs/xfs/scrub/common.h | 16 ++++++
> fs/xfs/scrub/scrub.c | 10 ++++
> fs/xfs/scrub/trace.h | 22 ++++++++
> 6 files changed, 241 insertions(+), 22 deletions(-)
>
> diff --git a/fs/xfs/scrub/btree.c b/fs/xfs/scrub/btree.c
> index df07661..e17d154 100644
> --- a/fs/xfs/scrub/btree.c
> +++ b/fs/xfs/scrub/btree.c
> @@ -42,12 +42,14 @@
> * Check for btree operation errors. See the section about handling
> * operational errors in common.c.
> */
> -bool
> -xfs_scrub_btree_process_error(
> +static bool
> +__xfs_scrub_btree_process_error(
> struct xfs_scrub_context *sc,
> struct xfs_btree_cur *cur,
> int level,
> - int *error)
> + int *error,
> + __u32 errflag,
> + void *ret_ip)
> {
> if (*error == 0)
> return true;
> @@ -60,36 +62,80 @@ xfs_scrub_btree_process_error(
> case -EFSBADCRC:
> case -EFSCORRUPTED:
> /* Note the badness but don't abort. */
> - sc->sm->sm_flags |= XFS_SCRUB_OFLAG_CORRUPT;
> + sc->sm->sm_flags |= errflag;
> *error = 0;
> /* fall through */
> default:
> if (cur->bc_flags & XFS_BTREE_ROOT_IN_INODE)
> trace_xfs_scrub_ifork_btree_op_error(sc, cur, level,
> - *error, __return_address);
> + *error, ret_ip);
> else
> trace_xfs_scrub_btree_op_error(sc, cur, level,
> - *error, __return_address);
> + *error, ret_ip);
> break;
> }
> return false;
> }
>
> +bool
> +xfs_scrub_btree_process_error(
> + struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur,
> + int level,
> + int *error)
> +{
> + return __xfs_scrub_btree_process_error(sc, cur, level, error,
> + XFS_SCRUB_OFLAG_CORRUPT, __return_address);
> +}
> +
> +bool
> +xfs_scrub_btree_xref_process_error(
> + struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur,
> + int level,
> + int *error)
> +{
> + return __xfs_scrub_btree_process_error(sc, cur, level, error,
> + XFS_SCRUB_OFLAG_XFAIL, __return_address);
> +}
> +
> /* Record btree block corruption. */
> -void
> -xfs_scrub_btree_set_corrupt(
> +static void
> +__xfs_scrub_btree_set_corrupt(
> struct xfs_scrub_context *sc,
> struct xfs_btree_cur *cur,
> - int level)
> + int level,
> + __u32 errflag,
> + void *ret_ip)
> {
> - sc->sm->sm_flags |= XFS_SCRUB_OFLAG_CORRUPT;
> + sc->sm->sm_flags |= errflag;
>
> if (cur->bc_flags & XFS_BTREE_ROOT_IN_INODE)
> trace_xfs_scrub_ifork_btree_error(sc, cur, level,
> - __return_address);
> + ret_ip);
> else
> trace_xfs_scrub_btree_error(sc, cur, level,
> - __return_address);
> + ret_ip);
> +}
> +
> +void
> +xfs_scrub_btree_set_corrupt(
> + struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur,
> + int level)
> +{
> + __xfs_scrub_btree_set_corrupt(sc, cur, level, XFS_SCRUB_OFLAG_CORRUPT,
> + __return_address);
> +}
> +
> +void
> +xfs_scrub_btree_xref_set_corrupt(
> + struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur,
> + int level)
> +{
> + __xfs_scrub_btree_set_corrupt(sc, cur, level, XFS_SCRUB_OFLAG_XCORRUPT,
> + __return_address);
> }
>
> /*
> @@ -512,5 +558,6 @@ xfs_scrub_btree(
> }
>
> out:
> +
> return error;
> }
> diff --git a/fs/xfs/scrub/btree.h b/fs/xfs/scrub/btree.h
> index 4de825a6..e2b868e 100644
> --- a/fs/xfs/scrub/btree.h
> +++ b/fs/xfs/scrub/btree.h
> @@ -26,10 +26,19 @@
> bool xfs_scrub_btree_process_error(struct xfs_scrub_context *sc,
> struct xfs_btree_cur *cur, int level, int *error);
>
> +/* Check for btree xref operation errors. */
> +bool xfs_scrub_btree_xref_process_error(struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur, int level,
> + int *error);
> +
> /* Check for btree corruption. */
> void xfs_scrub_btree_set_corrupt(struct xfs_scrub_context *sc,
> struct xfs_btree_cur *cur, int level);
>
> +/* Check for btree xref discrepancies. */
> +void xfs_scrub_btree_xref_set_corrupt(struct xfs_scrub_context *sc,
> + struct xfs_btree_cur *cur, int level);
> +
> struct xfs_scrub_btree;
> typedef int (*xfs_scrub_btree_rec_fn)(
> struct xfs_scrub_btree *bs,
> diff --git a/fs/xfs/scrub/common.c b/fs/xfs/scrub/common.c
> index d5c37d8..35c47cf 100644
> --- a/fs/xfs/scrub/common.c
> +++ b/fs/xfs/scrub/common.c
> @@ -78,12 +78,14 @@
> */
>
> /* Check for operational errors. */
> -bool
> -xfs_scrub_process_error(
> +static bool
> +__xfs_scrub_process_error(
> struct xfs_scrub_context *sc,
> xfs_agnumber_t agno,
> xfs_agblock_t bno,
> - int *error)
> + int *error,
> + __u32 errflag,
> + void *ret_ip)
> {
> switch (*error) {
> case 0:
> @@ -95,24 +97,48 @@ xfs_scrub_process_error(
> case -EFSBADCRC:
> case -EFSCORRUPTED:
> /* Note the badness but don't abort. */
> - sc->sm->sm_flags |= XFS_SCRUB_OFLAG_CORRUPT;
> + sc->sm->sm_flags |= errflag;
> *error = 0;
> /* fall through */
> default:
> trace_xfs_scrub_op_error(sc, agno, bno, *error,
> - __return_address);
> + ret_ip);
> break;
> }
> return false;
> }
>
> -/* Check for operational errors for a file offset. */
> bool
> -xfs_scrub_fblock_process_error(
> +xfs_scrub_process_error(
> + struct xfs_scrub_context *sc,
> + xfs_agnumber_t agno,
> + xfs_agblock_t bno,
> + int *error)
> +{
> + return __xfs_scrub_process_error(sc, agno, bno, error,
> + XFS_SCRUB_OFLAG_CORRUPT, __return_address);
> +}
> +
> +bool
> +xfs_scrub_xref_process_error(
> + struct xfs_scrub_context *sc,
> + xfs_agnumber_t agno,
> + xfs_agblock_t bno,
> + int *error)
> +{
> + return __xfs_scrub_process_error(sc, agno, bno, error,
> + XFS_SCRUB_OFLAG_XFAIL, __return_address);
> +}
> +
> +/* Check for operational errors for a file offset. */
> +static bool
> +__xfs_scrub_fblock_process_error(
> struct xfs_scrub_context *sc,
> int whichfork,
> xfs_fileoff_t offset,
> - int *error)
> + int *error,
> + __u32 errflag,
> + void *ret_ip)
> {
> switch (*error) {
> case 0:
> @@ -124,17 +150,39 @@ xfs_scrub_fblock_process_error(
> case -EFSBADCRC:
> case -EFSCORRUPTED:
> /* Note the badness but don't abort. */
> - sc->sm->sm_flags |= XFS_SCRUB_OFLAG_CORRUPT;
> + sc->sm->sm_flags |= errflag;
> *error = 0;
> /* fall through */
> default:
> trace_xfs_scrub_file_op_error(sc, whichfork, offset, *error,
> - __return_address);
> + ret_ip);
> break;
> }
> return false;
> }
>
> +bool
> +xfs_scrub_fblock_process_error(
> + struct xfs_scrub_context *sc,
> + int whichfork,
> + xfs_fileoff_t offset,
> + int *error)
> +{
> + return __xfs_scrub_fblock_process_error(sc, whichfork, offset, error,
> + XFS_SCRUB_OFLAG_CORRUPT, __return_address);
> +}
> +
> +bool
> +xfs_scrub_fblock_xref_process_error(
> + struct xfs_scrub_context *sc,
> + int whichfork,
> + xfs_fileoff_t offset,
> + int *error)
> +{
> + return __xfs_scrub_fblock_process_error(sc, whichfork, offset, error,
> + XFS_SCRUB_OFLAG_XFAIL, __return_address);
> +}
> +
> /*
> * Handling scrub corruption/optimization/warning checks.
> *
> @@ -183,6 +231,16 @@ xfs_scrub_block_set_corrupt(
> trace_xfs_scrub_block_error(sc, bp->b_bn, __return_address);
> }
>
> +/* Record a corruption while cross-referencing. */
> +void
> +xfs_scrub_block_xref_set_corrupt(
> + struct xfs_scrub_context *sc,
> + struct xfs_buf *bp)
> +{
> + sc->sm->sm_flags |= XFS_SCRUB_OFLAG_XCORRUPT;
> + trace_xfs_scrub_block_error(sc, bp->b_bn, __return_address);
> +}
> +
> /*
> * Record a corrupt inode. The trace data will include the block given
> * by bp if bp is given; otherwise it will use the block location of the
> @@ -198,6 +256,17 @@ xfs_scrub_ino_set_corrupt(
> trace_xfs_scrub_ino_error(sc, ino, bp ? bp->b_bn : 0, __return_address);
> }
>
> +/* Record a corruption while cross-referencing with an inode. */
> +void
> +xfs_scrub_ino_xref_set_corrupt(
> + struct xfs_scrub_context *sc,
> + xfs_ino_t ino,
> + struct xfs_buf *bp)
> +{
> + sc->sm->sm_flags |= XFS_SCRUB_OFLAG_XCORRUPT;
> + trace_xfs_scrub_ino_error(sc, ino, bp ? bp->b_bn : 0, __return_address);
> +}
> +
> /* Record corruption in a block indexed by a file fork. */
> void
> xfs_scrub_fblock_set_corrupt(
> @@ -209,6 +278,17 @@ xfs_scrub_fblock_set_corrupt(
> trace_xfs_scrub_fblock_error(sc, whichfork, offset, __return_address);
> }
>
> +/* Record a corruption while cross-referencing a fork block. */
> +void
> +xfs_scrub_fblock_xref_set_corrupt(
> + struct xfs_scrub_context *sc,
> + int whichfork,
> + xfs_fileoff_t offset)
> +{
> + sc->sm->sm_flags |= XFS_SCRUB_OFLAG_XCORRUPT;
> + trace_xfs_scrub_fblock_error(sc, whichfork, offset, __return_address);
> +}
> +
> /*
> * Warn about inodes that need administrative review but is not
> * incorrect.
> @@ -588,3 +668,38 @@ xfs_scrub_setup_inode_contents(
> /* scrub teardown will unlock and release the inode for us */
> return error;
> }
> +
> +/*
> + * Predicate that decides if we need to evaluate the cross-reference check.
> + * If there was an error accessing the cross-reference btree, just delete
> + * the cursor and skip the check.
> + */
> +bool
> +xfs_scrub_should_xref(
> + struct xfs_scrub_context *sc,
> + int *error,
> + struct xfs_btree_cur **curpp)
> +{
> + if (*error == 0)
> + return true;
> +
> + if (curpp) {
> + /* If we've already given up on xref, just bail out. */
> + if (!*curpp)
> + return false;
> +
> + /* xref error, delete cursor and bail out. */
> + xfs_btree_del_cursor(*curpp, XFS_BTREE_ERROR);
> + *curpp = NULL;
> + }
> +
> + sc->sm->sm_flags |= XFS_SCRUB_OFLAG_XFAIL;
> + trace_xfs_scrub_xref_error(sc, *error, __return_address);
> +
> + /*
> + * Errors encountered during cross-referencing with another
> + * data structure should not cause this scrubber to abort.
> + */
> + *error = 0;
> + return false;
> +}
> diff --git a/fs/xfs/scrub/common.h b/fs/xfs/scrub/common.h
> index fe12053..11a5bd0 100644
> --- a/fs/xfs/scrub/common.h
> +++ b/fs/xfs/scrub/common.h
> @@ -56,6 +56,11 @@ bool xfs_scrub_process_error(struct xfs_scrub_context *sc, xfs_agnumber_t agno,
> bool xfs_scrub_fblock_process_error(struct xfs_scrub_context *sc, int whichfork,
> xfs_fileoff_t offset, int *error);
>
> +bool xfs_scrub_xref_process_error(struct xfs_scrub_context *sc,
> + xfs_agnumber_t agno, xfs_agblock_t bno, int *error);
> +bool xfs_scrub_fblock_xref_process_error(struct xfs_scrub_context *sc,
> + int whichfork, xfs_fileoff_t offset, int *error);
> +
> void xfs_scrub_block_set_preen(struct xfs_scrub_context *sc,
> struct xfs_buf *bp);
> void xfs_scrub_ino_set_preen(struct xfs_scrub_context *sc, xfs_ino_t ino,
> @@ -68,6 +73,13 @@ void xfs_scrub_ino_set_corrupt(struct xfs_scrub_context *sc, xfs_ino_t ino,
> void xfs_scrub_fblock_set_corrupt(struct xfs_scrub_context *sc, int whichfork,
> xfs_fileoff_t offset);
>
> +void xfs_scrub_block_xref_set_corrupt(struct xfs_scrub_context *sc,
> + struct xfs_buf *bp);
> +void xfs_scrub_ino_xref_set_corrupt(struct xfs_scrub_context *sc, xfs_ino_t ino,
> + struct xfs_buf *bp);
> +void xfs_scrub_fblock_xref_set_corrupt(struct xfs_scrub_context *sc,
> + int whichfork, xfs_fileoff_t offset);
> +
> void xfs_scrub_ino_set_warning(struct xfs_scrub_context *sc, xfs_ino_t ino,
> struct xfs_buf *bp);
> void xfs_scrub_fblock_set_warning(struct xfs_scrub_context *sc, int whichfork,
> @@ -76,6 +88,10 @@ void xfs_scrub_fblock_set_warning(struct xfs_scrub_context *sc, int whichfork,
> void xfs_scrub_set_incomplete(struct xfs_scrub_context *sc);
> int xfs_scrub_checkpoint_log(struct xfs_mount *mp);
>
> +/* Are we set up for a cross-referencing operation? */
> +bool xfs_scrub_should_xref(struct xfs_scrub_context *sc, int *error,
> + struct xfs_btree_cur **curpp);
> +
> /* Setup functions */
> int xfs_scrub_setup_fs(struct xfs_scrub_context *sc, struct xfs_inode *ip);
> int xfs_scrub_setup_ag_allocbt(struct xfs_scrub_context *sc,
> diff --git a/fs/xfs/scrub/scrub.c b/fs/xfs/scrub/scrub.c
> index cd46077..0ed2a12 100644
> --- a/fs/xfs/scrub/scrub.c
> +++ b/fs/xfs/scrub/scrub.c
> @@ -110,6 +110,16 @@
> * structure itself is corrupt, the CORRUPT flag will be set. If
> * the metadata is correct but otherwise suboptimal, the PREEN flag
> * will be set.
> + *
> + * We perform secondary validation of filesystem metadata by
> + * cross-referencing every record with all other available metadata.
> + * For example, for block mapping extents, we verify that there are no
> + * records in the free space and inode btrees corresponding to that
> + * space extent and that there is a corresponding entry in the reverse
> + * mapping btree. Inconsistent metadata is noted by setting the
> + * XCORRUPT flag; btree query function errors are noted by setting the
> + * XFAIL flag and deleting the cursor to prevent further attempts to
> + * cross-reference with a defective btree.
> */
>
> /*
> diff --git a/fs/xfs/scrub/trace.h b/fs/xfs/scrub/trace.h
> index c4ebfb5..81becf6 100644
> --- a/fs/xfs/scrub/trace.h
> +++ b/fs/xfs/scrub/trace.h
> @@ -491,6 +491,28 @@ DEFINE_EVENT(xfs_scrub_sbtree_class, name, \
> DEFINE_SCRUB_SBTREE_EVENT(xfs_scrub_btree_rec);
> DEFINE_SCRUB_SBTREE_EVENT(xfs_scrub_btree_key);
>
> +TRACE_EVENT(xfs_scrub_xref_error,
> + TP_PROTO(struct xfs_scrub_context *sc, int error, void *ret_ip),
> + TP_ARGS(sc, error, ret_ip),
> + TP_STRUCT__entry(
> + __field(dev_t, dev)
> + __field(int, type)
> + __field(int, error)
> + __field(void *, ret_ip)
> + ),
> + TP_fast_assign(
> + __entry->dev = sc->mp->m_super->s_dev;
> + __entry->type = sc->sm->sm_type;
> + __entry->error = error;
> + __entry->ret_ip = ret_ip;
> + ),
> + TP_printk("dev %d:%d type %u xref error %d ret_ip %pF",
> + MAJOR(__entry->dev), MINOR(__entry->dev),
> + __entry->type,
> + __entry->error,
> + __entry->ret_ip)
> +);
> +
> #endif /* _TRACE_XFS_SCRUB_TRACE_H */
>
> #undef TRACE_INCLUDE_PATH
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2018-01-16 23:06 UTC|newest]
Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-23 0:42 [PATCH v11 00/21] xfs: online scrub xref support Darrick J. Wong
2017-12-23 0:42 ` [PATCH 01/21] xfs: ignore agfl read errors when not scrubbing agfl Darrick J. Wong
2018-01-05 1:12 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 02/21] xfs: catch a few more error codes when scrubbing secondary sb Darrick J. Wong
2018-01-05 1:17 ` Dave Chinner
2018-01-05 1:24 ` Darrick J. Wong
2018-01-05 2:10 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 03/21] xfs: xfs_scrub_bmap should use for_each_xfs_iext Darrick J. Wong
2018-01-05 1:17 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 04/21] xfs: always grab transaction when scrubbing inode Darrick J. Wong
2018-01-05 1:18 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 05/21] xfs: distinguish between corrupt inode and invalid inum in xfs_scrub_get_inode Darrick J. Wong
2018-01-05 1:23 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 06/21] xfs: add scrub cross-referencing helpers for the free space btrees Darrick J. Wong
2018-01-05 1:29 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 07/21] xfs: add scrub cross-referencing helpers for the inode btrees Darrick J. Wong
2018-01-05 1:36 ` Dave Chinner
2018-01-05 2:19 ` Darrick J. Wong
2018-01-05 21:51 ` [PATCH v2 " Darrick J. Wong
2018-01-16 23:05 ` Darrick J. Wong
2018-01-17 0:36 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 08/21] xfs: add scrub cross-referencing helpers for the rmap btrees Darrick J. Wong
2018-01-05 1:40 ` Dave Chinner
2018-01-05 2:49 ` Darrick J. Wong
2018-01-05 3:38 ` Dave Chinner
2018-01-05 21:53 ` [PATCH v2 " Darrick J. Wong
2018-01-06 20:46 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 09/21] xfs: add scrub cross-referencing helpers for the refcount btrees Darrick J. Wong
2018-01-05 1:41 ` Dave Chinner
2017-12-23 0:43 ` [PATCH 10/21] xfs: set up scrub cross-referencing helpers Darrick J. Wong
2018-01-05 2:08 ` Dave Chinner
2018-01-05 3:05 ` Darrick J. Wong
2018-01-05 21:54 ` [PATCH v2 " Darrick J. Wong
2018-01-16 23:06 ` Darrick J. Wong [this message]
2018-01-17 0:41 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 11/21] xfs: fix a few erroneous process_error calls in the scrubbers Darrick J. Wong
2018-01-05 2:11 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 12/21] xfs: check btree block ownership with bnobt/rmapbt when scrubbing btree Darrick J. Wong
2018-01-05 2:24 ` Dave Chinner
2018-01-05 2:53 ` Darrick J. Wong
2018-01-05 3:39 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 13/21] xfs: introduce scrubber cross-referencing stubs Darrick J. Wong
2018-01-08 23:36 ` Dave Chinner
2018-01-08 23:59 ` Darrick J. Wong
2018-01-09 21:00 ` [PATCH v2 " Darrick J. Wong
2018-01-10 0:12 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 14/21] xfs: cross-reference with the bnobt Darrick J. Wong
2018-01-08 23:51 ` Dave Chinner
2018-01-09 0:34 ` Darrick J. Wong
2018-01-09 0:57 ` Dave Chinner
2018-01-09 21:15 ` [PATCH v2 " Darrick J. Wong
2018-01-10 0:15 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 15/21] xfs: cross-reference bnobt records with cntbt Darrick J. Wong
2018-01-08 23:55 ` Dave Chinner
2018-01-09 0:37 ` Darrick J. Wong
2018-01-09 21:20 ` [PATCH v2 " Darrick J. Wong
2018-01-10 0:19 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 16/21] xfs: cross-reference inode btrees during scrub Darrick J. Wong
2018-01-09 21:22 ` [PATCH v2 " Darrick J. Wong
2018-01-15 22:17 ` Dave Chinner
2018-01-16 6:30 ` Darrick J. Wong
2018-01-16 23:23 ` [PATCH v3 " Darrick J. Wong
2018-01-17 0:44 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 17/21] xfs: cross-reference reverse-mapping btree Darrick J. Wong
2018-01-09 21:24 ` [PATCH v2 " Darrick J. Wong
2018-01-15 23:04 ` Dave Chinner
2018-01-16 6:38 ` Darrick J. Wong
2018-01-16 23:25 ` [PATCH v3 " Darrick J. Wong
2018-01-17 0:52 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 18/21] xfs: cross-reference the rmapbt data with the refcountbt Darrick J. Wong
2018-01-09 21:25 ` [PATCH v2 " Darrick J. Wong
2018-01-15 23:49 ` Dave Chinner
2018-01-16 6:49 ` Darrick J. Wong
2018-01-16 19:47 ` Darrick J. Wong
2018-01-16 23:26 ` [PATCH v3 " Darrick J. Wong
2018-01-17 1:00 ` Dave Chinner
2018-01-17 1:11 ` Darrick J. Wong
2017-12-23 0:44 ` [PATCH 19/21] xfs: cross-reference refcount btree during scrub Darrick J. Wong
2018-01-09 21:25 ` [PATCH v2 " Darrick J. Wong
2018-01-16 2:44 ` Dave Chinner
2018-01-16 6:52 ` Darrick J. Wong
2018-01-16 20:26 ` Darrick J. Wong
2018-01-16 23:27 ` [PATCH v3 " Darrick J. Wong
2018-01-17 1:02 ` Dave Chinner
2017-12-23 0:44 ` [PATCH 20/21] xfs: cross-reference the realtime bitmap Darrick J. Wong
2018-01-09 21:26 ` [PATCH v2 " Darrick J. Wong
2018-01-16 2:57 ` Dave Chinner
2018-01-16 6:55 ` Darrick J. Wong
2018-01-16 23:27 ` [PATCH v3 " Darrick J. Wong
2018-01-17 1:03 ` Dave Chinner
2017-12-23 0:45 ` [PATCH 21/21] xfs: cross-reference the block mappings when possible Darrick J. Wong
2018-01-09 21:26 ` [PATCH v2 " Darrick J. Wong
2018-01-16 2:58 ` Dave Chinner
2018-01-16 6:55 ` Darrick J. Wong
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=20180116230622.GZ5602@magnolia \
--to=darrick.wong@oracle.com \
--cc=david@fromorbit.com \
--cc=linux-xfs@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).