linux-xfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Allison Henderson <allison.henderson@oracle.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: linux-xfs@vger.kernel.org
Subject: Re: [PATCH 02/22] xfs: add scrub tracepoints
Date: Sun, 23 Jul 2017 09:23:51 -0700	[thread overview]
Message-ID: <76b3b0e0-2751-4749-fdd5-e3bea584441e@oracle.com> (raw)
In-Reply-To: <150061192134.14732.12889749323660766797.stgit@magnolia>

Looks good.  When I had sets with multiple versions, I would add a quick 
version history in the commit, just to help reviewers find the updates. 
Up to you though.  You can mark down my review.

Reviewed by: Allison Henderson <allison.henderson@oracle.com>

On 7/20/2017 9:38 PM, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
>  fs/xfs/libxfs/xfs_types.h |    5 +
>  fs/xfs/xfs_trace.h        |  375 +++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 380 insertions(+)
>
>
> diff --git a/fs/xfs/libxfs/xfs_types.h b/fs/xfs/libxfs/xfs_types.h
> index 0220159..3ee2dba 100644
> --- a/fs/xfs/libxfs/xfs_types.h
> +++ b/fs/xfs/libxfs/xfs_types.h
> @@ -94,6 +94,11 @@ typedef int64_t		xfs_sfiloff_t;	/* signed block number in a file */
>  #define	XFS_ATTR_FORK	1
>  #define	XFS_COW_FORK	2
>
> +#define XFS_FORK_DESC \
> +	{ XFS_DATA_FORK,	"data" }, \
> +	{ XFS_ATTR_FORK,	"attr" }, \
> +	{ XFS_COW_FORK,		"CoW" }
> +
>  /*
>   * Min numbers of data/attr fork btree root pointers.
>   */
> diff --git a/fs/xfs/xfs_trace.h b/fs/xfs/xfs_trace.h
> index bcc3cdf..2e7e193 100644
> --- a/fs/xfs/xfs_trace.h
> +++ b/fs/xfs/xfs_trace.h
> @@ -42,6 +42,7 @@ struct xfs_btree_cur;
>  struct xfs_refcount_irec;
>  struct xfs_fsmap;
>  struct xfs_rmap_irec;
> +struct xfs_scrub_metadata;
>
>  DECLARE_EVENT_CLASS(xfs_attr_list_class,
>  	TP_PROTO(struct xfs_attr_list_context *ctx),
> @@ -3309,6 +3310,380 @@ DEFINE_GETFSMAP_EVENT(xfs_getfsmap_low_key);
>  DEFINE_GETFSMAP_EVENT(xfs_getfsmap_high_key);
>  DEFINE_GETFSMAP_EVENT(xfs_getfsmap_mapping);
>
> +/* scrub */
> +#define XFS_SCRUB_TYPE_DESC \
> +	{ 0, NULL }
> +DECLARE_EVENT_CLASS(xfs_scrub_class,
> +	TP_PROTO(struct xfs_inode *ip, struct xfs_scrub_metadata *sm,
> +		 int error),
> +	TP_ARGS(ip, sm, error),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_ino_t, ino)
> +		__field(int, type)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_ino_t, inum)
> +		__field(unsigned int, gen)
> +		__field(unsigned int, flags)
> +		__field(int, error)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = ip->i_mount->m_super->s_dev;
> +		__entry->ino = ip->i_ino;
> +		__entry->error = error;
> +	),
> +	TP_printk("dev %d:%d ino %llu type %s agno %u inum %llu gen %u flags 0x%x error %d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->ino,
> +		  __print_symbolic(__entry->type, XFS_SCRUB_TYPE_DESC),
> +		  __entry->agno,
> +		  __entry->inum,
> +		  __entry->gen,
> +		  __entry->flags,
> +		  __entry->error)
> +)
> +#define DEFINE_SCRUB_EVENT(name) \
> +DEFINE_EVENT(xfs_scrub_class, name, \
> +	TP_PROTO(struct xfs_inode *ip, struct xfs_scrub_metadata *sm, \
> +		 int error), \
> +	TP_ARGS(ip, sm, error))
> +
> +DEFINE_SCRUB_EVENT(xfs_scrub);
> +DEFINE_SCRUB_EVENT(xfs_scrub_done);
> +DEFINE_SCRUB_EVENT(xfs_scrub_deadlock_retry);
> +
> +DECLARE_EVENT_CLASS(xfs_scrub_sbtree_class,
> +	TP_PROTO(struct xfs_mount *mp, xfs_agnumber_t agno, xfs_agblock_t bno,
> +		 xfs_btnum_t btnum, int level, int nlevels, int ptr),
> +	TP_ARGS(mp, agno, bno, btnum, level, nlevels, ptr),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_btnum_t, btnum)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_agblock_t, bno)
> +		__field(int, level)
> +		__field(int, nlevels)
> +		__field(int, ptr)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__entry->agno = agno;
> +		__entry->btnum = btnum;
> +		__entry->bno = bno;
> +		__entry->level = level;
> +		__entry->nlevels = nlevels;
> +		__entry->ptr = ptr;
> +	),
> +	TP_printk("dev %d:%d agno %u agbno %u btnum %d level %d nlevels %d ptr %d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->agno,
> +		  __entry->bno,
> +		  __entry->btnum,
> +		  __entry->level,
> +		  __entry->nlevels,
> +		  __entry->ptr)
> +)
> +#define DEFINE_SCRUB_SBTREE_EVENT(name) \
> +DEFINE_EVENT(xfs_scrub_sbtree_class, name, \
> +	TP_PROTO(struct xfs_mount *mp, xfs_agnumber_t agno, xfs_agblock_t bno, \
> +		 xfs_btnum_t btnum, int level, int nlevels, int ptr), \
> +	TP_ARGS(mp, agno, bno, btnum, level, nlevels, ptr))
> +
> +DEFINE_SCRUB_SBTREE_EVENT(xfs_scrub_btree_rec);
> +DEFINE_SCRUB_SBTREE_EVENT(xfs_scrub_btree_key);
> +
> +TRACE_EVENT(xfs_scrub_op_error,
> +	TP_PROTO(struct xfs_mount *mp, xfs_agnumber_t agno, xfs_agblock_t bno,
> +		 const char *type, int error, const char *func,
> +		 int line),
> +	TP_ARGS(mp, agno, bno, type, error, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_agblock_t, bno)
> +		__string(type, type)
> +		__field(int, error)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__entry->agno = agno;
> +		__entry->bno = bno;
> +		__assign_str(type, type);
> +		__entry->error = error;
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d agno %u agbno %u type '%s' error %d fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->agno,
> +		  __entry->bno,
> +		  __get_str(type),
> +		  __entry->error,
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
> +TRACE_EVENT(xfs_scrub_file_op_error,
> +	TP_PROTO(struct xfs_inode *ip, int whichfork, xfs_fileoff_t offset,
> +		 const char *type, int error, const char *func,
> +		 int line),
> +	TP_ARGS(ip, whichfork, offset, type, error, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_ino_t, ino)
> +		__field(int, whichfork)
> +		__field(xfs_fileoff_t, offset)
> +		__string(type, type)
> +		__field(int, error)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = ip->i_mount->m_super->s_dev;
> +		__entry->ino = ip->i_ino;
> +		__entry->whichfork = whichfork;
> +		__entry->offset = offset;
> +		__assign_str(type, type);
> +		__entry->error = error;
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d ino %llu %s offset %llu type '%s' error %d fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->ino,
> +		  __print_symbolic(__entry->whichfork, XFS_FORK_DESC),
> +		  __entry->offset,
> +		  __get_str(type),
> +		  __entry->error,
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
> +DECLARE_EVENT_CLASS(xfs_scrub_block_error_class,
> +	TP_PROTO(struct xfs_mount *mp, xfs_agnumber_t agno, xfs_agblock_t bno,
> +		 const char *type, const char *check, const char *func,
> +		 int line),
> +	TP_ARGS(mp, agno, bno, type, check, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_agblock_t, bno)
> +		__string(type, type)
> +		__string(check, check)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__entry->agno = agno;
> +		__entry->bno = bno;
> +		__assign_str(type, type);
> +		__assign_str(check, check);
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d agno %u agbno %u type '%s' check '%s' fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->agno,
> +		  __entry->bno,
> +		  __get_str(type),
> +		  __get_str(check),
> +		  __get_str(func),
> +		  __entry->line)
> +)
> +
> +#define DEFINE_SCRUB_BLOCK_ERROR_EVENT(name) \
> +DEFINE_EVENT(xfs_scrub_block_error_class, name, \
> +	TP_PROTO(struct xfs_mount *mp, xfs_agnumber_t agno, xfs_agblock_t bno, \
> +		 const char *type, const char *check, const char *func, \
> +		 int line), \
> +	TP_ARGS(mp, agno, bno, type, check, func, line))
> +
> +DEFINE_SCRUB_BLOCK_ERROR_EVENT(xfs_scrub_block_error);
> +DEFINE_SCRUB_BLOCK_ERROR_EVENT(xfs_scrub_block_preen);
> +
> +DECLARE_EVENT_CLASS(xfs_scrub_ino_error_class,
> +	TP_PROTO(struct xfs_mount *mp, xfs_ino_t ino, xfs_agnumber_t agno, xfs_agblock_t bno,
> +		 const char *type, const char *check, const char *func,
> +		 int line),
> +	TP_ARGS(mp, ino, agno, bno, type, check, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_ino_t, ino)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_agblock_t, bno)
> +		__string(type, type)
> +		__string(check, check)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__entry->ino = ino;
> +		__entry->agno = agno;
> +		__entry->bno = bno;
> +		__assign_str(type, type);
> +		__assign_str(check, check);
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d ino %llu agno %u agbno %u type '%s' check '%s' fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->ino,
> +		  __entry->agno,
> +		  __entry->bno,
> +		  __get_str(type),
> +		  __get_str(check),
> +		  __get_str(func),
> +		  __entry->line)
> +)
> +
> +#define DEFINE_SCRUB_INO_ERROR_EVENT(name) \
> +DEFINE_EVENT(xfs_scrub_ino_error_class, name, \
> +	TP_PROTO(struct xfs_mount *mp, xfs_ino_t ino, xfs_agnumber_t agno, xfs_agblock_t bno, \
> +		 const char *type, const char *check, const char *func, \
> +		 int line), \
> +	TP_ARGS(mp, ino, agno, bno, type, check, func, line))
> +
> +DEFINE_SCRUB_INO_ERROR_EVENT(xfs_scrub_ino_error);
> +DEFINE_SCRUB_INO_ERROR_EVENT(xfs_scrub_ino_preen);
> +
> +DECLARE_EVENT_CLASS(xfs_scrub_data_error_class,
> +	TP_PROTO(struct xfs_inode *ip, int whichfork, xfs_fileoff_t offset,
> +		 const char *type, const char *check, const char *func,
> +		 int line),
> +	TP_ARGS(ip, whichfork, offset, type, check, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__field(xfs_ino_t, ino)
> +		__field(int, whichfork)
> +		__field(xfs_fileoff_t, offset)
> +		__string(type, type)
> +		__string(check, check)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = ip->i_mount->m_super->s_dev;
> +		__entry->ino = ip->i_ino;
> +		__entry->whichfork = whichfork;
> +		__entry->offset = offset;
> +		__assign_str(type, type);
> +		__assign_str(check, check);
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d ino %llu %s fork offset %llu type '%s' check '%s' fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __entry->ino,
> +		  __print_symbolic(__entry->whichfork, XFS_FORK_DESC),
> +		  __entry->offset,
> +		  __get_str(type),
> +		  __get_str(check),
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
> +#define DEFINE_SCRUB_DATA_ERROR_EVENT(name) \
> +DEFINE_EVENT(xfs_scrub_data_error_class, name, \
> +	TP_PROTO(struct xfs_inode *ip, int whichfork, xfs_fileoff_t offset, \
> +		 const char *type, const char *check, const char *func, \
> +		 int line), \
> +	TP_ARGS(ip, whichfork, offset, type, check, func, line))
> +
> +DEFINE_SCRUB_DATA_ERROR_EVENT(xfs_scrub_data_error);
> +DEFINE_SCRUB_DATA_ERROR_EVENT(xfs_scrub_data_warning);
> +
> +TRACE_EVENT(xfs_scrub_xref_error,
> +	TP_PROTO(struct xfs_mount *mp, const char *type, int error,
> +		 const char *func, int line),
> +	TP_ARGS(mp, type, error, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__string(type, type)
> +		__field(int, error)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__assign_str(type, type);
> +		__entry->error = error;
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d btree %s xref error %d fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __get_str(type),
> +		  __entry->error,
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
> +TRACE_EVENT(xfs_scrub_btree_error,
> +	TP_PROTO(struct xfs_mount *mp, const char *bt_type, const char *bt_ptr,
> +		 xfs_agnumber_t agno, xfs_agblock_t bno, const char *check,
> +		 const char *func, int line),
> +	TP_ARGS(mp, bt_type, bt_ptr, agno, bno, check, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__string(bt_type, bt_type)
> +		__string(bt_ptr, bt_ptr)
> +		__field(xfs_agnumber_t, agno)
> +		__field(xfs_agblock_t, bno)
> +		__string(check, check)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__assign_str(bt_type, bt_type);
> +		__assign_str(bt_ptr, bt_ptr);
> +		__entry->agno = agno;
> +		__entry->bno = bno;
> +		__assign_str(check, check);
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d %s %s agno %u agbno %u check '%s' fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __get_str(bt_type),
> +		  __get_str(bt_ptr),
> +		  __entry->agno,
> +		  __entry->bno,
> +		  __get_str(check),
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
> +TRACE_EVENT(xfs_scrub_incomplete,
> +	TP_PROTO(struct xfs_mount *mp, const char *type, const char *check,
> +		 const char *func, int line),
> +	TP_ARGS(mp, type, check, func, line),
> +	TP_STRUCT__entry(
> +		__field(dev_t, dev)
> +		__string(type, type)
> +		__string(check, check)
> +		__string(func, func)
> +		__field(int, line)
> +	),
> +	TP_fast_assign(
> +		__entry->dev = mp->m_super->s_dev;
> +		__assign_str(type, type);
> +		__assign_str(check, check);
> +		__assign_str(func, func);
> +		__entry->line = line;
> +	),
> +	TP_printk("dev %d:%d %s check '%s' fn %s:%d",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev),
> +		  __get_str(type),
> +		  __get_str(check),
> +		  __get_str(func),
> +		  __entry->line)
> +);
> +
>  #endif /* _TRACE_XFS_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
>

  reply	other threads:[~2017-07-23 16:23 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-21  4:38 [PATCH v8 00/22] xfs: online scrub support Darrick J. Wong
2017-07-21  4:38 ` [PATCH 01/22] xfs: query the per-AG reservation counters Darrick J. Wong
2017-07-23 16:16   ` Allison Henderson
2017-07-23 22:25   ` Dave Chinner
2017-07-24 19:07     ` Darrick J. Wong
2017-07-21  4:38 ` [PATCH 02/22] xfs: add scrub tracepoints Darrick J. Wong
2017-07-23 16:23   ` Allison Henderson [this message]
2017-07-21  4:38 ` [PATCH 03/22] xfs: create an ioctl to scrub AG metadata Darrick J. Wong
2017-07-23 16:37   ` Allison Henderson
2017-07-23 23:45   ` Dave Chinner
2017-07-24 21:14     ` Darrick J. Wong
2017-07-21  4:38 ` [PATCH 04/22] xfs: generic functions to scrub metadata and btrees Darrick J. Wong
2017-07-23 16:40   ` Allison Henderson
2017-07-24  1:05   ` Dave Chinner
2017-07-24 21:58     ` Darrick J. Wong
2017-07-24 23:15       ` Dave Chinner
2017-07-25  0:39         ` Darrick J. Wong
2017-07-21  4:39 ` [PATCH 05/22] xfs: scrub in-memory metadata buffers Darrick J. Wong
2017-07-23 16:48   ` Allison Henderson
2017-07-24  1:43   ` Dave Chinner
2017-07-24 22:36     ` Darrick J. Wong
2017-07-24 23:38       ` Dave Chinner
2017-07-25  0:14         ` Darrick J. Wong
2017-07-25  3:32           ` Dave Chinner
2017-07-25  5:27             ` Darrick J. Wong
2017-07-21  4:39 ` [PATCH 06/22] xfs: scrub the backup superblocks Darrick J. Wong
2017-07-23 16:50   ` Allison Henderson
2017-07-25  4:05   ` Dave Chinner
2017-07-25  5:42     ` Darrick J. Wong
2017-07-21  4:39 ` [PATCH 07/22] xfs: scrub AGF and AGFL Darrick J. Wong
2017-07-23 16:59   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 08/22] xfs: scrub the AGI Darrick J. Wong
2017-07-23 17:02   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 09/22] xfs: scrub free space btrees Darrick J. Wong
2017-07-23 17:09   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 10/22] xfs: scrub inode btrees Darrick J. Wong
2017-07-23 17:15   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 11/22] xfs: scrub rmap btrees Darrick J. Wong
2017-07-23 17:21   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 12/22] xfs: scrub refcount btrees Darrick J. Wong
2017-07-23 17:25   ` Allison Henderson
2017-07-21  4:39 ` [PATCH 13/22] xfs: scrub inodes Darrick J. Wong
2017-07-23 17:38   ` Allison Henderson
2017-07-24 20:02     ` Darrick J. Wong
2017-07-21  4:40 ` [PATCH 14/22] xfs: scrub inode block mappings Darrick J. Wong
2017-07-23 17:41   ` Allison Henderson
2017-07-24 20:05     ` Darrick J. Wong
2017-07-21  4:40 ` [PATCH 15/22] xfs: scrub directory/attribute btrees Darrick J. Wong
2017-07-23 17:45   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 16/22] xfs: scrub directory metadata Darrick J. Wong
2017-07-23 17:51   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 17/22] xfs: scrub directory freespace Darrick J. Wong
2017-07-23 17:55   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 18/22] xfs: scrub extended attributes Darrick J. Wong
2017-07-23 17:57   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 19/22] xfs: scrub symbolic links Darrick J. Wong
2017-07-23 17:59   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 20/22] xfs: scrub parent pointers Darrick J. Wong
2017-07-23 18:03   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 21/22] xfs: scrub realtime bitmap/summary Darrick J. Wong
2017-07-23 18:05   ` Allison Henderson
2017-07-21  4:40 ` [PATCH 22/22] xfs: scrub quota information Darrick J. Wong
2017-07-23 18:07   ` Allison Henderson

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=76b3b0e0-2751-4749-fdd5-e3bea584441e@oracle.com \
    --to=allison.henderson@oracle.com \
    --cc=darrick.wong@oracle.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).