xfs
[Top] [All Lists]

Re: [PATCH, RFC] put xfs build features into the stats file

To: Eric Sandeen <sandeen@xxxxxxxxxxx>
Subject: Re: [PATCH, RFC] put xfs build features into the stats file
From: Nathan Scott <nscott@xxxxxxxxxx>
Date: Mon, 20 Apr 2009 12:59:28 +1000 (EST)
Cc: xfs-oss <xfs@xxxxxxxxxxx>
In-reply-to: <49EBDF30.90702@xxxxxxxxxxx>
Separate file?  That line uses a comma separator ... differs to
every other line in that stats file, painful for parsers.  And
no risk of breaking anything if its a separate file.

cheers.

----- "Eric Sandeen" <sandeen@xxxxxxxxxxx> wrote:

> Running xfstests on an xfs build w/o xattrs enabled, I realized
> there's no reliable way to see if the running xfs has various
> features other than debug (which is shown in /proc/fs/xfs/stat)
> 
> What do folks think of adding the BUILD_OPTIONS to this file
> as well?
> 
> # cat /proc/fs/xfs/stat
> extent_alloc 0 0 0 0
> ...
> xpc 0 0 0
> debug 0
> features security attributes, large block numbers, no debug 
> 
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
> ---
> 
> Index: linux-2.6/fs/xfs/linux-2.6/xfs_stats.c
> ===================================================================
> --- linux-2.6.orig/fs/xfs/linux-2.6/xfs_stats.c
> +++ linux-2.6/fs/xfs/linux-2.6/xfs_stats.c
> @@ -89,6 +89,8 @@ xfs_read_xfsstats(
>               0);
>  #endif
>  
> +     len += sprintf(buffer + len, "features " XFS_BUILD_OPTIONS "\n");
> +
>       if (offset >= len) {
>               *start = buffer;
>               *eof = 1;
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs

-- 
Nathan

<Prev in Thread] Current Thread [Next in Thread>