Re: [PATCH v25 10/12] xfs: Add larp debug option

[Allison Henderson <allison.henderson@xxxxxxxxxx>]

On 11/23/21 4:29 PM, Darrick J. Wong wrote:
> On Tue, Nov 16, 2021 at 09:13:41PM -0700, Allison Henderson wrote:
> > This patch adds a debug option to enable log attribute replay. Eventually
> > this can be removed when delayed attrs becomes permanent.
> >
> > Signed-off-by: Allison Henderson <allison.henderson@xxxxxxxxxx>
> > Reviewed-by: Chandan Babu R <chandanrlinux@xxxxxxxxx>
>
> LOL LARP
> Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx>
Great, thank you!
Allison

> --D
>
> > ---
> >   fs/xfs/libxfs/xfs_attr.h |  4 ++++
> >   fs/xfs/xfs_globals.c     |  1 +
> >   fs/xfs/xfs_sysctl.h      |  1 +
> >   fs/xfs/xfs_sysfs.c       | 24 ++++++++++++++++++++++++
> >   4 files changed, 30 insertions(+)
> >
> > diff --git a/fs/xfs/libxfs/xfs_attr.h b/fs/xfs/libxfs/xfs_attr.h
> > index 977434f343a1..6f5a150565c7 100644
> > --- a/fs/xfs/libxfs/xfs_attr.h
> > +++ b/fs/xfs/libxfs/xfs_attr.h
> > @@ -30,7 +30,11 @@ struct xfs_attr_list_context;
> >   
> >   static inline bool xfs_has_larp(struct xfs_mount *mp)
> >   {
> > +#ifdef DEBUG
> > +	return xfs_globals.larp;
> > +#else
> >   	return false;
> > +#endif
> >   }
> >   
> >   /*
> > diff --git a/fs/xfs/xfs_globals.c b/fs/xfs/xfs_globals.c
> > index f62fa652c2fd..4d0a98f920ca 100644
> > --- a/fs/xfs/xfs_globals.c
> > +++ b/fs/xfs/xfs_globals.c
> > @@ -41,5 +41,6 @@ struct xfs_globals xfs_globals = {
> >   #endif
> >   #ifdef DEBUG
> >   	.pwork_threads		=	-1,	/* automatic thread detection */
> > +	.larp			=	false,	/* log attribute replay */
> >   #endif
> >   };
> > diff --git a/fs/xfs/xfs_sysctl.h b/fs/xfs/xfs_sysctl.h
> > index 7692e76ead33..f78ad6b10ea5 100644
> > --- a/fs/xfs/xfs_sysctl.h
> > +++ b/fs/xfs/xfs_sysctl.h
> > @@ -83,6 +83,7 @@ extern xfs_param_t	xfs_params;
> >   struct xfs_globals {
> >   #ifdef DEBUG
> >   	int	pwork_threads;		/* parallel workqueue threads */
> > +	bool	larp;			/* log attribute replay */
> >   #endif
> >   	int	log_recovery_delay;	/* log recovery delay (secs) */
> >   	int	mount_delay;		/* mount setup delay (secs) */
> > diff --git a/fs/xfs/xfs_sysfs.c b/fs/xfs/xfs_sysfs.c
> > index 8608f804388f..02a0b55e26b3 100644
> > --- a/fs/xfs/xfs_sysfs.c
> > +++ b/fs/xfs/xfs_sysfs.c
> > @@ -227,6 +227,29 @@ pwork_threads_show(
> >   	return sysfs_emit(buf, "%d\n", xfs_globals.pwork_threads);
> >   }
> >   XFS_SYSFS_ATTR_RW(pwork_threads);
> > +
> > +static ssize_t
> > +larp_store(
> > +	struct kobject	*kobject,
> > +	const char	*buf,
> > +	size_t		count)
> > +{
> > +	ssize_t		ret;
> > +
> > +	ret = kstrtobool(buf, &xfs_globals.larp);
> > +	if (ret < 0)
> > +		return ret;
> > +	return count;
> > +}
> > +
> > +STATIC ssize_t
> > +larp_show(
> > +	struct kobject	*kobject,
> > +	char		*buf)
> > +{
> > +	return snprintf(buf, PAGE_SIZE, "%d\n", xfs_globals.larp);
> > +}
> > +XFS_SYSFS_ATTR_RW(larp);
> >   #endif /* DEBUG */
> >   
> >   static struct attribute *xfs_dbg_attrs[] = {
> > @@ -236,6 +259,7 @@ static struct attribute *xfs_dbg_attrs[] = {
> >   	ATTR_LIST(always_cow),
> >   #ifdef DEBUG
> >   	ATTR_LIST(pwork_threads),
> > +	ATTR_LIST(larp),
> >   #endif
> >   	NULL,
> >   };
> > --
> > 2.25.1