From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp2130.oracle.com ([156.151.31.86]:58286 "EHLO userp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933481AbeFKSUe (ORCPT ); Mon, 11 Jun 2018 14:20:34 -0400 Received: from pps.filterd (userp2130.oracle.com [127.0.0.1]) by userp2130.oracle.com (8.16.0.22/8.16.0.22) with SMTP id w5BIB4Xc079841 for ; Mon, 11 Jun 2018 18:20:33 GMT Received: from userv0022.oracle.com (userv0022.oracle.com [156.151.31.74]) by userp2130.oracle.com with ESMTP id 2jg6b1ensr-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Mon, 11 Jun 2018 18:20:33 +0000 Received: from userv0122.oracle.com (userv0122.oracle.com [156.151.31.75]) by userv0022.oracle.com (8.14.4/8.14.4) with ESMTP id w5BIKXfB032269 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Mon, 11 Jun 2018 18:20:33 GMT Received: from abhmp0012.oracle.com (abhmp0012.oracle.com [141.146.116.18]) by userv0122.oracle.com (8.14.4/8.14.4) with ESMTP id w5BIKW8s030004 for ; Mon, 11 Jun 2018 18:20:32 GMT Date: Mon, 11 Jun 2018 11:20:28 -0700 From: "Darrick J. Wong" Subject: Re: [PATCH v2 27/27] xfsprogs: Clean up old parent pointer definitions Message-ID: <20180611182028.GH22045@magnolia> References: <1528607272-11122-1-git-send-email-allison.henderson@oracle.com> <1528607272-11122-28-git-send-email-allison.henderson@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1528607272-11122-28-git-send-email-allison.henderson@oracle.com> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Allison Henderson Cc: linux-xfs@vger.kernel.org On Sat, Jun 09, 2018 at 10:07:52PM -0700, Allison Henderson wrote: > Existing parent pointer stubs and structs are not used in this > implementation. Remove uneeded definitions. > > Signed-off-by: Allison Henderson > --- > include/parent.h | 10 ---------- > libhandle/handle.c | 26 -------------------------- > libhandle/jdm.c | 20 -------------------- > 3 files changed, 56 deletions(-) > > diff --git a/include/parent.h b/include/parent.h > index 33f8d85..5665d64 100644 > --- a/include/parent.h > +++ b/include/parent.h > @@ -18,16 +18,6 @@ > #ifndef __PARENT_H__ > #define __PARENT_H__ > > -typedef struct parent { > - __u64 p_ino; > - __u32 p_gen; > - __u16 p_reclen; > -} parent_t; > - > -typedef struct parent_cursor { > - __u32 opaque[4]; /* an opaque cookie */ > -} parent_cursor_t; > - > struct path_list; > > typedef int (*walk_pptr_fn)(struct xfs_pptr_info *pi, struct xfs_parent_ptr *pptr, > diff --git a/libhandle/handle.c b/libhandle/handle.c > index a70fa32..862b943 100644 > --- a/libhandle/handle.c > +++ b/libhandle/handle.c > @@ -20,7 +20,6 @@ > #include "platform_defs.h" > #include "xfs.h" > #include "handle.h" > -#include "parent.h" > > /* just pick a value we know is more than big enough */ > #define MAXHANSIZ 64 > @@ -422,31 +421,6 @@ attr_list_by_handle( > } > > int > -parents_by_handle( > - void *hanp, > - size_t hlen, > - parent_t *buf, > - size_t bufsiz, > - unsigned int *count) > - > -{ > - errno = EOPNOTSUPP; > - return -1; > -} You probably need to scrape the declarations out of the header files because otherwise any program that thinks it wants to use this will compile ok but barf on unresolved symbols in the linker phase. There's a murkier question here about how do we handle deprecating these libhandle APIs that have never been usable on Linux? --D > - > -int > -parentpaths_by_handle( > - void *hanp, > - size_t hlen, > - parent_t *buf, > - size_t bufsiz, > - unsigned int *count) > -{ > - errno = EOPNOTSUPP; > - return -1; > -} > - > -int > fssetdm_by_handle( > void *hanp, > size_t hlen, > diff --git a/libhandle/jdm.c b/libhandle/jdm.c > index 821061b..0c2440b 100644 > --- a/libhandle/jdm.c > +++ b/libhandle/jdm.c > @@ -20,7 +20,6 @@ > #include "xfs.h" > #include "handle.h" > #include "jdm.h" > -#include "parent.h" > > /* internal fshandle - typecast to a void for external use */ > #define FSHANDLE_SZ 8 > @@ -178,22 +177,3 @@ jdm_attr_list( jdm_fshandle_t *fshp, > return rval; > } > > -int > -jdm_parents( jdm_fshandle_t *fshp, > - xfs_bstat_t *statp, > - parent_t *bufp, size_t bufsz, > - unsigned int *count) > -{ > - errno = EOPNOTSUPP; > - return -1; > -} > - > -int > -jdm_parentpaths( jdm_fshandle_t *fshp, > - xfs_bstat_t *statp, > - parent_t *bufp, size_t bufsz, > - unsigned int *count) > -{ > - errno = EOPNOTSUPP; > - return -1; > -} > -- > 2.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html