Joel Becker wrote:
Hi again, Here's version 2 of reflink. Changes since the first version: - One patch, not three. - Documentation/filesystems/reflink.txt is no longer a pseudo-manpage. It also tries to encapsulate all the feedback from the discussion to make the operation clearer.
You certainly did not address: - desire for one single system call to handle both owner preservation and create with current owner. I see no reason to have 2 vfs_xxx and 2 inode functions for those. - please just add the flag to the defined reflink API... there is no reason to keep saying "it is just like link(2)". that not true and you will just cause confusion. - fix the + if (S_ISDIR(inode->i_mode)) + return -EPERM; to be an ISREG check unless you have an argument for special files and symlinks being COWed. jim -- 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
