From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 192167F93 for ; Tue, 10 Dec 2013 08:46:48 -0600 (CST) Received: from cuda.sgi.com (cuda1.sgi.com [192.48.157.11]) by relay1.corp.sgi.com (Postfix) with ESMTP id EEA4A8F8050 for ; Tue, 10 Dec 2013 06:46:47 -0800 (PST) Received: from sandeen.net (sandeen.net [63.231.237.45]) by cuda.sgi.com with ESMTP id Tszx3AnrdggSRA5k for ; Tue, 10 Dec 2013 06:46:46 -0800 (PST) Message-ID: <52A72930.4020404@sandeen.net> Date: Tue, 10 Dec 2013 08:46:08 -0600 From: Eric Sandeen MIME-Version: 1.0 Subject: Re: [PATCH] xfsdump: don't assume getdents exists References: <20131106213635.GC24712@redacted.bos.redhat.com> In-Reply-To: <20131106213635.GC24712@redacted.bos.redhat.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Kyle McMartin , sandeen@redhat.com Cc: xfs@oss.sgi.com On 11/6/13, 3:36 PM, Kyle McMartin wrote: > New Linux ports are using a standard syscall list that does not include > deprecated syscalls where 64-bit clean alternatives exist. As a result, > on arm64, __NR_getdents is undefined, resulting in xfsdump failing to > build. > > To avoid that, in the case where __NR_getdents is unset in unistd.h, > avoid building the fallback path entirely, since > __ASSUME_GETDENTS64_SYSCALL will be true, the SYS_getdents64 case will > be the primary (and only) path used. > > Signed-off-by: Kyle McMartin Hey Kyle - were you going to do a V2 of this one? Just checking, thanks - -Eric > --- a/common/getdents.c > +++ b/common/getdents.c > @@ -70,6 +70,9 @@ extern int __have_no_getdents64; > # ifndef SYS_getdents64 > # define SYS_getdents64 __NR_getdents64 > # endif > +# ifndef __NR_getdents > +# define __ONLY_GETDENTS64_SYSCALL 1 > +# endif > #endif > > > @@ -207,6 +210,13 @@ getdents_wrap (int fd, char *buf, size_t nbytes) > # endif > } > #endif > +/* Newer Linux ports are not adding deprecated syscalls, so to avoid compile > + * failures since SYS_getdents will be undefined, we check for that and only > + * build the fall-back case if SYS_getdents is defined. We know that > + * __ASSUME_GETDENTS64_SYSCALL is set, since __NR_getdents64 must exist on > + * those platforms. > + */ > +#ifndef __ONLY_GETDENTS64_SYSCALL > { > size_t red_nbytes; > struct kernel_dirent *skdp, *kdp; > @@ -265,4 +275,5 @@ getdents_wrap (int fd, char *buf, size_t nbytes) > } > > return (char *) dp - buf; > +#endif > } > > _______________________________________________ > xfs mailing list > xfs@oss.sgi.com > http://oss.sgi.com/mailman/listinfo/xfs > _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs