Re: [PATCH 05/11] xfsprogs: missing and dummy calls for OS X support

[Dave Chinner <david@fromorbit.com>]

On Mon, Aug 17, 2015 at 12:32:23PM -0700, Christoph Hellwig wrote:
> On Mon, Aug 17, 2015 at 06:23:20PM +0200, Jan Tulak wrote:
> 
> > +#define XATTR_SIZE_MAX 65536    /* size of an extended attribute value (64k) */
> > +#define XATTR_LIST_MAX 65536    /* size of extended attribute namelist (64k) */
> 
> Eww, looks like we depend on these Linux values in the on disk
> defintion.  I think we need to add new XFS_XATTR_SIZE_MAX and
> XFS_XATTR_LIST_MAX defintions to xfs_format.h and use them where
> we currently use these.

Yeah, that looks like something we need to fix.

> > +#define MREMAP_FIXED 1
> > +#define MREMAP_MAYMOVE 2
> > +static inline void *mremap(void *old_address, size_t old_size,
> > +                    size_t new_size, int flags, ... /* void *new_address */)
> > +{
> > +	return NULL;
> > +}
> > +
> > +/* FSR */
> 
> Please just build the mremap file in xfs_io conditional.

Probably should be done on an autconf rule.

> 
> > +typedef int __fsblkcnt_t;
> > +typedef int __fsfilcnt_t;
> > +typedef long long int __fsblkcnt64_t;
> > +typedef long long int __fsfilcnt64_t;
> > +
> > +struct statvfs64
> > +{
> > +	unsigned long int f_bsize;
> > +	unsigned long int f_frsize;
> > +	__fsblkcnt64_t f_blocks;
> > +	__fsblkcnt64_t f_bfree;
> > +	__fsblkcnt64_t f_bavail;
> > +	__fsfilcnt64_t f_files;
> > +	__fsfilcnt64_t f_ffree;
> > +	__fsfilcnt64_t f_favail;
> > +	unsigned long int f_fsid;
> > +	int __f_unused;
> > +	unsigned long int f_flag;
> > +	unsigned long int f_namemax;
> > +	int __f_spare[6];
> > +};
> 
> Does MacOS support statvfs? or statfs?  We should use the proper API
> here instead of a dummy.  Or maybe just not build fsr on OSX for now,
> as that would solve the fsetxattr issue as well.

Apparently so:

https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man2/statfs.2.html

Note that we'll need to define _DARWIN_USE_64_BIT_INODE for OS X to
work correctly.

> > +struct mntent
> > +{
> 
> Another fsr issue, right?  Seems like it would need similar getmntinfo
> based code.  As a stopgap I'd suggest we stop building fsr on OSX for
> now.

That can be based on the HAVE_MNTENT autoconf detection....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs