On Mon, Apr 01, 2019 at 10:10:46AM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
>
> Use our newly expanded geometry structure to report the overall fs and
> realtime health status.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> ---
I kind of wonder whether it's possible (or makes sense) to make the core
sickness bits exportable to userspace and thus avoid some of the
translation code, but that aside this looks fine to me:
Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
> fs/xfs/libxfs/xfs_fs.h | 11 ++++++++++-
> fs/xfs/libxfs/xfs_health.h | 3 +++
> fs/xfs/xfs_health.c | 27 +++++++++++++++++++++++++++
> fs/xfs/xfs_ioctl.c | 3 +++
> 4 files changed, 43 insertions(+), 1 deletion(-)
>
>
> diff --git a/fs/xfs/libxfs/xfs_fs.h b/fs/xfs/libxfs/xfs_fs.h
> index 87226e00e7bd..ddbfde7ff79d 100644
> --- a/fs/xfs/libxfs/xfs_fs.h
> +++ b/fs/xfs/libxfs/xfs_fs.h
> @@ -199,9 +199,18 @@ typedef struct xfs_fsop_geom {
> __u32 rtsectsize; /* realtime sector size, bytes */
> __u32 dirblocksize; /* directory block size, bytes */
> __u32 logsunit; /* log stripe unit, bytes */
> - __u64 reserved[18]; /* reserved space */
> + __u32 health; /* o: unhealthy fs & rt metadata */
> + __u32 reserved32; /* reserved space */
> + __u64 reserved[17]; /* reserved space */
> } xfs_fsop_geom_t;
>
> +#define XFS_FSOP_GEOM_HEALTH_FS_COUNTERS (1 << 0) /* summary counters */
> +#define XFS_FSOP_GEOM_HEALTH_FS_UQUOTA (1 << 1) /* user quota */
> +#define XFS_FSOP_GEOM_HEALTH_FS_GQUOTA (1 << 2) /* group quota */
> +#define XFS_FSOP_GEOM_HEALTH_FS_PQUOTA (1 << 3) /* project quota */
> +#define XFS_FSOP_GEOM_HEALTH_RT_BITMAP (1 << 4) /* realtime bitmap */
> +#define XFS_FSOP_GEOM_HEALTH_RT_SUMMARY (1 << 5) /* realtime summary */
> +
> /* Output for XFS_FS_COUNTS */
> typedef struct xfs_fsop_counts {
> __u64 freedata; /* free data section blocks */
> diff --git a/fs/xfs/libxfs/xfs_health.h b/fs/xfs/libxfs/xfs_health.h
> index 269b124dc1d7..36736d54a3e3 100644
> --- a/fs/xfs/libxfs/xfs_health.h
> +++ b/fs/xfs/libxfs/xfs_health.h
> @@ -39,6 +39,7 @@
> struct xfs_mount;
> struct xfs_perag;
> struct xfs_inode;
> +struct xfs_fsop_geom;
>
> /* Observable health issues for metadata spanning the entire filesystem. */
> #define XFS_HEALTH_FS_COUNTERS (1 << 0) /* summary counters */
> @@ -200,4 +201,6 @@ xfs_inode_healthy(struct xfs_inode *ip)
> return xfs_inode_measure_sickness(ip) == 0;
> }
>
> +void xfs_fsop_geom_health(struct xfs_mount *mp, struct xfs_fsop_geom *geo);
> +
> #endif /* __XFS_HEALTH_H__ */
> diff --git a/fs/xfs/xfs_health.c b/fs/xfs/xfs_health.c
> index 6e2da858c356..151c98693bef 100644
> --- a/fs/xfs/xfs_health.c
> +++ b/fs/xfs/xfs_health.c
> @@ -249,3 +249,30 @@ xfs_inode_measure_sickness(
> spin_unlock(&ip->i_flags_lock);
> return ret;
> }
> +
> +/* Fill out fs geometry health info. */
> +void
> +xfs_fsop_geom_health(
> + struct xfs_mount *mp,
> + struct xfs_fsop_geom *geo)
> +{
> + unsigned int sick;
> +
> + geo->health = 0;
> +
> + sick = xfs_fs_measure_sickness(mp);
> + if (sick & XFS_HEALTH_FS_COUNTERS)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_FS_COUNTERS;
> + if (sick & XFS_HEALTH_FS_UQUOTA)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_FS_UQUOTA;
> + if (sick & XFS_HEALTH_FS_GQUOTA)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_FS_GQUOTA;
> + if (sick & XFS_HEALTH_FS_PQUOTA)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_FS_PQUOTA;
> +
> + sick = xfs_rt_measure_sickness(mp);
> + if (sick & XFS_HEALTH_RT_BITMAP)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_RT_BITMAP;
> + if (sick & XFS_HEALTH_RT_SUMMARY)
> + geo->health |= XFS_FSOP_GEOM_HEALTH_RT_SUMMARY;
> +}
> diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c
> index b5918ce656bd..f9bf11b6a055 100644
> --- a/fs/xfs/xfs_ioctl.c
> +++ b/fs/xfs/xfs_ioctl.c
> @@ -34,6 +34,7 @@
> #include "scrub/xfs_scrub.h"
> #include "xfs_sb.h"
> #include "xfs_ag.h"
> +#include "xfs_health.h"
>
> #include <linux/capability.h>
> #include <linux/cred.h>
> @@ -830,6 +831,8 @@ xfs_ioc_fsgeometry(
> if (error)
> return error;
>
> + xfs_fsop_geom_health(mp, &fsgeo);
> +
> if (copy_to_user(arg, &fsgeo, sizeof(fsgeo)))
> return -EFAULT;
> return 0;
>
