lkml.org 
[lkml]   [2020]   [Nov]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    Subject[PATCH v7 0/8] add support for direct I/O with fscrypt using blk-crypto
    From
    This patch series adds support for direct I/O with fscrypt using
    blk-crypto. It has been rebased on fscrypt/master (i.e. the "master"
    branch of the fscrypt tree at
    https://git.kernel.org/pub/scm/fs/fscrypt/fscrypt.git)

    Patch 1 ensures that bios are not split in the middle of a crypto data
    unit.

    Till now, blk-crypto expected the offset and length of each bvec in a bio
    to be aligned to the crypto data unit size. Patch 2 enables
    blk-crypto-fallback to work without this requirement. It also relaxes the
    alignment requirement that blk-crypto checks for - now, blk-crypto only
    requires that the length of the I/O is aligned to the crypto data unit
    size.

    Patch 3 adds two functions to fscrypt that need to be called to determine
    if direct I/O is supported for a request.

    Patches 4 and 5 modify direct-io and iomap respectively to set bio crypt
    contexts on bios when appropriate by calling into fscrypt.

    Patches 6 and 7 allow ext4 and f2fs direct I/O to support fscrypt without
    falling back to buffered I/O.

    Patch 8 updates the fscrypt documentation for direct I/O support.
    The documentation now notes the required conditions for inline encryption
    and direct I/O on encrypted files.

    This patch series was tested by running xfstests with test_dummy_encryption
    with and without the 'inlinecrypt' mount option, and there were no
    meaningful regressions. One regression was for generic/587 on ext4,
    but that test isn't compatible with test_dummy_encryption in the first
    place, and the test "incorrectly" passes without the 'inlinecrypt' mount
    option - a patch will be sent out to exclude that test when
    test_dummy_encryption is turned on with ext4 (like the other quota related
    tests that use user visible quota files).

    Changes v6 => v7:
    - add patches 1 and 2 to allow blk-crypto to work with user buffers not
    aligned to crypto data unit size, so that direct I/O doesn't require
    that alignment either.
    - some cleanups

    Changes v5 => v6:
    - fix bug with fscrypt_limit_io_blocks() and make it ready for 64 bit
    block numbers.
    - remove Reviewed-by for Patch 1 due to significant changes from when
    the Reviewed-by was given.

    Changes v4 => v5:
    - replace fscrypt_limit_io_pages() with fscrypt_limit_io_block(), which
    is now called by individual filesystems (currently only ext4) instead
    of the iomap code. This new function serves the same end purpose as
    the one it replaces (ensuring that DUNs within a bio are contiguous)
    but operates purely with blocks instead of with pages.
    - make iomap_dio_zero() set bio_crypt_ctx's again, instead of just a
    WARN_ON() since some folks prefer that instead.
    - add Reviewed-by's

    Changes v3 => v4:
    - Fix bug in iomap_dio_bio_actor() where fscrypt_limit_io_pages() was
    being called too early (thanks Eric!)
    - Improve comments and fix formatting in documentation
    - iomap_dio_zero() is only called to zero out partial blocks, but
    direct I/O is only supported on encrypted files when I/O is
    blocksize aligned, so it doesn't need to set encryption contexts on
    bios. Replace setting the encryption context with a WARN_ON(). (Eric)

    Changes v2 => v3:
    - add changelog to coverletter

    Changes v1 => v2:
    - Fix bug in f2fs caused by replacing f2fs_post_read_required() with
    !fscrypt_dio_supported() since the latter doesn't check for
    compressed inodes unlike the former.
    - Add patches 6 and 7 for fscrypt documentation
    - cleanups and comments

    Eric Biggers (5):
    fscrypt: add functions for direct I/O support
    direct-io: add support for fscrypt using blk-crypto
    iomap: support direct I/O with fscrypt using blk-crypto
    ext4: support direct I/O with fscrypt using blk-crypto
    f2fs: support direct I/O with fscrypt using blk-crypto

    Satya Tangirala (3):
    block: ensure bios are not split in middle of crypto data unit
    blk-crypto: don't require user buffer alignment
    fscrypt: update documentation for direct I/O support

    Documentation/filesystems/fscrypt.rst | 21 ++-
    block/bio.c | 1 +
    block/blk-crypto-fallback.c | 212 +++++++++++++++++++-------
    block/blk-crypto-internal.h | 18 +++
    block/blk-crypto.c | 19 +--
    block/blk-merge.c | 96 ++++++++++--
    block/blk-mq.c | 3 +
    block/bounce.c | 4 +
    fs/crypto/crypto.c | 8 +
    fs/crypto/inline_crypt.c | 74 +++++++++
    fs/direct-io.c | 15 +-
    fs/ext4/file.c | 10 +-
    fs/ext4/inode.c | 7 +
    fs/f2fs/f2fs.h | 6 +-
    fs/iomap/direct-io.c | 6 +
    include/linux/fscrypt.h | 18 +++
    16 files changed, 431 insertions(+), 87 deletions(-)

    --
    2.29.2.299.gdc1121823c-goog

    \
     
     \ /
      Last update: 2020-11-17 15:13    [W:4.577 / U:0.116 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site