On Thu, Jan 05, 2017 at 11:26:19AM -0800, Eric Biggers wrote: > Something I'm thinking about is making things easier for filesystems by having > functions like "fscrypt_rename_hook()" which would handle all these needed > checks. It would be easy to do with out-of-line functions in fs/crypto/, but we > don't want to be making ->is_encrypted() calls through the fscrypt_operations > all the time, when an inlined call to ext4_encrypted_inode() (or f2fs or > ubifs_encrypted_inode()) is much faster. I think it could be implemented as > efficiently as now if the hooks were defined in a header and called a macro like > "fs_encrypted_inode()" which filesystems would have to #define first. It would > be a little ugly, but at least it would be less error-prone than having multiple > filesystems replicate these increasingly complex checks. We could make things a lot simpler and a lot more efficient by adding make flags at the VFS level in struct super and struct inode meaning "file systems has fscrypt enabled", and "the inode is encrypted using fscrypt". That way, we wouldn't need to go through through fscrypt_operations to do these tests. We could just set the flags appropriately when the struct inode or struct super is instantiated. - Ted -- 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
