On Fri, Dec 13, 2013 at 03:56:44AM -0800, Christoph Hellwig wrote: > On the nfsv4 list it was recently discussed how atomic / transaction > xattr updates are. It turns out none of that seems documented on the > syscall level, but for XFS we have an odd inconsistancy in that attr > updates generally are atomic and logged, except when we go out to > remote attributes in xfs_attr_rmtval_set, in which case attr updates > are no logged, and we do synchronous writes instead. Yes, but they are still atomic from a user and crash recovery point of view.... I'd been wondering a while back if we could make remote xattrs use an ordered buffer so we don't need to log it back can leave it for async write but still have it pin the log tail. However, I don't think we can do that as we can't recover the attr data that is lost if we crash. Hence I think our only option is to log it if we want ot make it an async write. > Besides the weird semantic difference that is impossible to explain to > users performance will also generally be bad with a synchronous buffer > write. Is there any good reason to not log the buffer for the remote > attributes? Given that attribute are limited to 64kB it's not like > the value is larger than large directory blocks that we already > support. Well, I think it's a bit different to the directory block case - the directory blocks are filesystem metadata, while xattrs contain user data. Hence if we log user xattrs a user can consume all of the log bandwidth writing xattrs and degrade the metadata modification performance of the rest of the filesystem. One issue that we'll need to deal with is that it may change the minimum log size calculations if we add 64k of data to the attribute transaction reservations. We currently calculate the remote attr reservation in xfs_log_calc_max_attrsetm_res() and that will need to change. If it pushes the remote attr reservation to be the largest transaction reservation, we could have log size issues on existing filesystems and that would lead to only enabling logging of remote xattrs if the log is physically big enough. The thing is, it's really only user data that ever ends up in a remote attr block - system xattrs like ACLs and selinux contexts, application xattrs from DMF, gluster, swift, Samba, etc rarely consume enough bytes to push the xattr out of line. So, IMO, the first question we need to answer is whether the current behaviour is actually a problem for anyone.... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
