From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp2130.oracle.com ([156.151.31.86]:49784 "EHLO userp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728723AbfAHUmN (ORCPT ); Tue, 8 Jan 2019 15:42:13 -0500 Date: Tue, 8 Jan 2019 12:42:09 -0800 From: "Darrick J. Wong" Subject: [PATCH 6/5] libxfs(progs): fix attr include mess Message-ID: <20190108204209.GT12689@magnolia> References: <154697976479.2839.3584921201780682011.stgit@magnolia> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <154697976479.2839.3584921201780682011.stgit@magnolia> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: linux-xfs@vger.kernel.org, sandeen@sandeen.net From: Darrick J. Wong Remove all the userspace xfs_attr shim cruft so that we have one definition of ATTR_* macros so that we can actually use xfs_attr.c routines and include xfs_attr.h without problems. Signed-off-by: Darrick J. Wong --- include/libxfs.h | 10 +--------- libxfs/libxfs_api_defs.h | 5 +++++ libxfs/libxfs_priv.h | 8 -------- libxfs/xfs_attr.c | 1 + libxfs/xfs_attr_leaf.c | 1 + 5 files changed, 8 insertions(+), 17 deletions(-) diff --git a/include/libxfs.h b/include/libxfs.h index 2bdef706..32dbb1c6 100644 --- a/include/libxfs.h +++ b/include/libxfs.h @@ -214,14 +214,6 @@ libxfs_bmbt_disk_get_all( int libxfs_rtfree_extent(struct xfs_trans *, xfs_rtblock_t, xfs_extlen_t); bool libxfs_verify_rtbno(struct xfs_mount *mp, xfs_rtblock_t rtbno); -/* XXX: need parts of xfs_attr.h in userspace */ -#define LIBXFS_ATTR_ROOT 0x0002 /* use attrs in root namespace */ -#define LIBXFS_ATTR_SECURE 0x0008 /* use attrs in security namespace */ -#define LIBXFS_ATTR_CREATE 0x0010 /* create, but fail if attr exists */ -#define LIBXFS_ATTR_REPLACE 0x0020 /* set, but fail if attr not exists */ - -int xfs_attr_remove(struct xfs_inode *dp, const unsigned char *name, int flags); -int xfs_attr_set(struct xfs_inode *dp, const unsigned char *name, - unsigned char *value, int valuelen, int flags); +#include "xfs_attr.h" #endif /* __LIBXFS_H__ */ diff --git a/libxfs/libxfs_api_defs.h b/libxfs/libxfs_api_defs.h index e10d78cd..2d5ff88f 100644 --- a/libxfs/libxfs_api_defs.h +++ b/libxfs/libxfs_api_defs.h @@ -150,4 +150,9 @@ #define xfs_fs_geometry libxfs_fs_geometry #define xfs_init_local_fork libxfs_init_local_fork +#define LIBXFS_ATTR_ROOT ATTR_ROOT +#define LIBXFS_ATTR_SECURE ATTR_SECURE +#define LIBXFS_ATTR_CREATE ATTR_CREATE +#define LIBXFS_ATTR_REPLACE ATTR_REPLACE + #endif /* __LIBXFS_API_DEFS_H__ */ diff --git a/libxfs/libxfs_priv.h b/libxfs/libxfs_priv.h index b45d07eb..59cd3333 100644 --- a/libxfs/libxfs_priv.h +++ b/libxfs/libxfs_priv.h @@ -104,14 +104,6 @@ extern char *progname; */ #define PTR_FMT "%p" -/* XXX: need to push these out to make LIBXFS_ATTR defines */ -#define ATTR_ROOT 0x0002 -#define ATTR_SECURE 0x0008 -#define ATTR_CREATE 0x0010 -#define ATTR_REPLACE 0x0020 -#define ATTR_KERNOTIME 0 -#define ATTR_KERNOVAL 0 - #define XFS_IGET_CREATE 0x1 #define XFS_IGET_UNTRUSTED 0x2 diff --git a/libxfs/xfs_attr.c b/libxfs/xfs_attr.c index b8838302..170e64cf 100644 --- a/libxfs/xfs_attr.c +++ b/libxfs/xfs_attr.c @@ -20,6 +20,7 @@ #include "xfs_trans.h" #include "xfs_bmap.h" #include "xfs_bmap_btree.h" +#include "xfs_attr.h" #include "xfs_attr_leaf.h" #include "xfs_attr_remote.h" #include "xfs_trans_space.h" diff --git a/libxfs/xfs_attr_leaf.c b/libxfs/xfs_attr_leaf.c index 5683d294..48f4b28d 100644 --- a/libxfs/xfs_attr_leaf.c +++ b/libxfs/xfs_attr_leaf.c @@ -21,6 +21,7 @@ #include "xfs_bmap.h" #include "xfs_attr_sf.h" #include "xfs_attr_remote.h" +#include "xfs_attr.h" #include "xfs_attr_leaf.h" #include "xfs_trace.h" #include "xfs_cksum.h"