> + /* > + * Some xattrs must be resistent to allocation failure at > + * ENOSPC. e.g. creating an inode with ACLs or security > + * attributes requires the allocation of the xattr holding > + * that information to succeed. Hence we allow xattrs in the If you repin this anyway because of the spelling nit maybe use up all 80 characters to make the comment a bit easier to read. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>