From: Dave Chinner <david@fromorbit.com>
To: Mark Tinguely <tinguely@sgi.com>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH] xfs: add the inode directory type support to XFS_IOC_FSGEOM
Date: Thu, 19 Sep 2013 07:54:45 +1000 [thread overview]
Message-ID: <20130918215445.GA9901@dastard> (raw)
In-Reply-To: <20130917144733.819109222@sgi.com>
On Tue, Sep 17, 2013 at 09:47:22AM -0500, Mark Tinguely wrote:
> Signed-off-by: Mark Tinguely <tinguely@sgi.com>
> ---
> fs/xfs/xfs_fs.h | 32 ++++++++++++++++----------------
> fs/xfs/xfs_fsops.c | 4 +++-
> 2 files changed, 19 insertions(+), 17 deletions(-)
>
> Index: b/fs/xfs/xfs_fs.h
> ===================================================================
> --- a/fs/xfs/xfs_fs.h
> +++ b/fs/xfs/xfs_fs.h
> @@ -222,22 +222,22 @@ typedef struct xfs_fsop_resblks {
>
> #define XFS_FSOP_GEOM_VERSION 0
>
> -#define XFS_FSOP_GEOM_FLAGS_ATTR 0x0001 /* attributes in use */
> -#define XFS_FSOP_GEOM_FLAGS_NLINK 0x0002 /* 32-bit nlink values */
> -#define XFS_FSOP_GEOM_FLAGS_QUOTA 0x0004 /* quotas enabled */
> -#define XFS_FSOP_GEOM_FLAGS_IALIGN 0x0008 /* inode alignment */
> -#define XFS_FSOP_GEOM_FLAGS_DALIGN 0x0010 /* large data alignment */
> -#define XFS_FSOP_GEOM_FLAGS_SHARED 0x0020 /* read-only shared */
> -#define XFS_FSOP_GEOM_FLAGS_EXTFLG 0x0040 /* special extent flag */
> -#define XFS_FSOP_GEOM_FLAGS_DIRV2 0x0080 /* directory version 2 */
> -#define XFS_FSOP_GEOM_FLAGS_LOGV2 0x0100 /* log format version 2 */
> -#define XFS_FSOP_GEOM_FLAGS_SECTOR 0x0200 /* sector sizes >1BB */
> -#define XFS_FSOP_GEOM_FLAGS_ATTR2 0x0400 /* inline attributes rework */
> -#define XFS_FSOP_GEOM_FLAGS_PROJID32 0x0800 /* 32-bit project IDs */
> -#define XFS_FSOP_GEOM_FLAGS_DIRV2CI 0x1000 /* ASCII only CI names */
> -#define XFS_FSOP_GEOM_FLAGS_LAZYSB 0x4000 /* lazy superblock counters */
> -#define XFS_FSOP_GEOM_FLAGS_V5SB 0x8000 /* version 5 superblock */
> -
> +#define XFS_FSOP_GEOM_FLAGS_ATTR 0x00001 /* attributes in use */
> +#define XFS_FSOP_GEOM_FLAGS_NLINK 0x00002 /* 32-bit nlink values */
> +#define XFS_FSOP_GEOM_FLAGS_QUOTA 0x00004 /* quotas enabled */
> +#define XFS_FSOP_GEOM_FLAGS_IALIGN 0x00008 /* inode alignment */
> +#define XFS_FSOP_GEOM_FLAGS_DALIGN 0x00010 /* large data alignment */
> +#define XFS_FSOP_GEOM_FLAGS_SHARED 0x00020 /* read-only shared */
> +#define XFS_FSOP_GEOM_FLAGS_EXTFLG 0x00040 /* special extent flag */
> +#define XFS_FSOP_GEOM_FLAGS_DIRV2 0x00080 /* directory version 2 */
> +#define XFS_FSOP_GEOM_FLAGS_LOGV2 0x00100 /* log format version 2 */
> +#define XFS_FSOP_GEOM_FLAGS_SECTOR 0x00200 /* sector sizes >1BB */
> +#define XFS_FSOP_GEOM_FLAGS_ATTR2 0x00400 /* inline attributes rework */
> +#define XFS_FSOP_GEOM_FLAGS_PROJID32 0x00800 /* 32-bit project IDs */
> +#define XFS_FSOP_GEOM_FLAGS_DIRV2CI 0x01000 /* ASCII only CI names */
> +#define XFS_FSOP_GEOM_FLAGS_LAZYSB 0x04000 /* lazy superblock counters */
> +#define XFS_FSOP_GEOM_FLAGS_V5SB 0x08000 /* version 5 superblock */
> +#define XFS_FSOP_GEOM_FLAGS_FTYPE 0x10000 /* inode directory types */
There is no need to do this. Can you drop these hunks as this simply
adds more work to synchronise with userspace.
> Index: b/fs/xfs/xfs_fsops.c
> ===================================================================
> --- a/fs/xfs/xfs_fsops.c
> +++ b/fs/xfs/xfs_fsops.c
> @@ -101,7 +101,9 @@ xfs_fs_geometry(
> (xfs_sb_version_hasprojid32bit(&mp->m_sb) ?
> XFS_FSOP_GEOM_FLAGS_PROJID32 : 0) |
> (xfs_sb_version_hascrc(&mp->m_sb) ?
> - XFS_FSOP_GEOM_FLAGS_V5SB : 0);
> + XFS_FSOP_GEOM_FLAGS_V5SB : 0) |
> + (xfs_sb_version_hasftype(&mp->m_sb) ?
> + XFS_FSOP_GEOM_FLAGS_FTYPE : 0);
> geo->logsectsize = xfs_sb_version_hassector(&mp->m_sb) ?
> mp->m_sb.sb_logsectsize : BBSIZE;
> geo->rtsectsize = mp->m_sb.sb_blocksize;
That looks good - can you post the patches to userspace that use
this so we can test it properly?
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2013-09-18 21:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20130917144721.917667018@sgi.com>
2013-09-17 14:47 ` [PATCH] xfs: add the inode directory type support to XFS_IOC_FSGEOM Mark Tinguely
2013-09-18 21:54 ` Dave Chinner [this message]
2013-10-02 21:00 Mark Tinguely
2013-10-03 14:37 ` Carlos Maiolino
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=20130918215445.GA9901@dastard \
--to=david@fromorbit.com \
--cc=tinguely@sgi.com \
--cc=xfs@oss.sgi.com \
/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