On Mon, Oct 09, 2017 at 12:15:33PM -0700, Eric Biggers wrote:
> From: Eric Biggers <ebiggers@xxxxxxxxxx>
>
> This series prepares to reduce code duplication among ext4, f2fs, and
> ubifs by introducing a S_ENCRYPTED inode flag (so we don't have to call
> back into the filesystem to test the filesystem-specific inode flag),
> then introducing new helper functions that are called at the beginning
> of the open, link, rename, lookup, and setattr operations.
>
> In the future we maybe should even call these new helpers from the VFS
> so that each individual filesystem doesn't have to do it. But that's
> not possible currently because fs/crypto/ can be built as a module.
>
> The patches to switch the filesystems over to use the helper functions
> were included in v1 of this patchset. They are not included now since
> I'm planning to get them picked up by the individual filesystem
> maintainers after this goes in.
Thanks, I've applied these patches plus the ones for ext4 on the
fscrypt.git tree.
The only concern I have is that in fscrypt_file_name() the warning
prints just the inode numbers, but not the block device (it replaced
an ext4_warning() call which automatically printed the block device).
I'll add a patch to fix that up in the next day or two.
- Ted
