Re: [PATCH 5/5] xfs: fold xfs_change_file_space into xfs_ioc_space

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 14, 2013 at 04:08:07PM +1100, Dave Chinner wrote:
> One question, though:
> 
> > +	case XFS_IOC_ALLOCSP:
> > +	case XFS_IOC_ALLOCSP64:
> > +	case XFS_IOC_FREESP:
> > +	case XFS_IOC_FREESP64:
> 
> Should we, at this point, mark these ioctls as deprecated and
> schedule then for removal given that we've recommended against using
> them for the past 10 years and we have fallocate() now?

I don't see any reason to remove them given that it's only about 15
extra lines of code.  But if you care enough to get rid of them we
probably need multiple years of actuall warnings emmited when used
before actually removing them.  I would be very surprised if there
aren't same users that wouldn't argue very vocally against their
removal.

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




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux