On Mon, May 08, 2006 at 09:16:34PM +1000, Neil Brown wrote: > On Monday May 8, hch@xxxxxxxxxxxxx wrote: > > NACK. As long as we have no HSM support in the tree there's no reason to > > add this. From the kernel's point it's just untested and unused code that > > can break. > > Greg: you seemed to suggest that there was already code in XFS that > could make use of this. If that is so: could you point us to it > please. The code in question is in fs/xfs/linux-2.6/xfs_iops.c::xfs_vn_setattr, fs/xfs/xfs_dmapi.h::AT_DELAY_FLAG, and its users in xfs_setattr and xfs_free_file_space and one or two other spots in XFS. The DMAPI code (driver, outside XFS) can be found in the XFS CVS tree on oss.sgi.com. > Alternately, if this will only be used by future patches, please feel > free to add this patch to that set, and add The DMAPI code was released at the same time as XFS, but it seems to be a spec that is not widely used (unsurprising since not may people have tape robots doing their bidding) and, perhaps more importantly, its not viewed as a particularly clean spec by, erm, certain kernel developers. ;) cheers. -- Nathan - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
