From: Eric Biggers <ebiggers@kernel.org>
To: Satya Tangirala <satyat@google.com>
Cc: linux-fscrypt@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org
Subject: Re: [PATCH v3 6/7] fscrypt: document inline encryption support
Date: Mon, 20 Jul 2020 12:34:53 -0700 [thread overview]
Message-ID: <20200720193453.GH1292162@gmail.com> (raw)
In-Reply-To: <20200717014540.71515-7-satyat@google.com>
On Fri, Jul 17, 2020 at 01:45:39AM +0000, Satya Tangirala wrote:
> Update the fscrypt documentation file for inline encryption support.
>
> Signed-off-by: Satya Tangirala <satyat@google.com>
Reviewed-by: Eric Biggers <ebiggers@google.com>
> diff --git a/Documentation/filesystems/fscrypt.rst b/Documentation/filesystems/fscrypt.rst
> index f5d8b0303ddf..f3d87a1a0a7f 100644
> --- a/Documentation/filesystems/fscrypt.rst
> +++ b/Documentation/filesystems/fscrypt.rst
> @@ -1204,6 +1204,18 @@ buffer. Some filesystems, such as UBIFS, already use temporary
> buffers regardless of encryption. Other filesystems, such as ext4 and
> F2FS, have to allocate bounce pages specially for encryption.
>
> +Fscrypt is also able to use inline encryption hardware instead of the
> +kernel crypto API for en/decryption of file contents. When possible, and
> +if directed to do so (by specifying the 'inlinecrypt' mount option for
> +an ext4/F2FS filesystem), it adds encryption contexts to bios and
> +uses blk-crypto to perform the en/decryption instead of making use
> +of the above read/write path changes. Of course, even if directed to make
> +use of inline encryption, fscrypt will only be able to do so if either
> +hardware inline encryption support is available for the selected encryption
> +algorithm or CONFIG_BLK_INLINE_ENCRYPTION_FALLBACK is selected. If neither
> +is the case, fscrypt will fall back to using the above mentioned read/write
> +path changes for en/decryption.
> +
Nit: most of the text in this file is formatted with textwidth=70.
> Filename hashing and encoding
> -----------------------------
>
> @@ -1250,7 +1262,9 @@ Tests
>
> To test fscrypt, use xfstests, which is Linux's de facto standard
> filesystem test suite. First, run all the tests in the "encrypt"
> -group on the relevant filesystem(s). For example, to test ext4 and
> +group on the relevant filesystem(s). One can also run the tests
> +with the 'inlinecrypt' mount option to test the implementation for
> +inline encryption support. For example, to test ext4 and
> f2fs encryption using `kvm-xfstests
> <https://github.com/tytso/xfstests-bld/blob/master/Documentation/kvm-quickstart.md>`_::
>
> --
> 2.28.0.rc0.105.gf9edc3c819-goog
>
WARNING: multiple messages have this Message-ID (diff)
From: Eric Biggers <ebiggers@kernel.org>
To: Satya Tangirala <satyat@google.com>
Cc: linux-fsdevel@vger.kernel.org, linux-xfs@vger.kernel.org,
linux-fscrypt@vger.kernel.org, linux-ext4@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH v3 6/7] fscrypt: document inline encryption support
Date: Mon, 20 Jul 2020 12:34:53 -0700 [thread overview]
Message-ID: <20200720193453.GH1292162@gmail.com> (raw)
In-Reply-To: <20200717014540.71515-7-satyat@google.com>
On Fri, Jul 17, 2020 at 01:45:39AM +0000, Satya Tangirala wrote:
> Update the fscrypt documentation file for inline encryption support.
>
> Signed-off-by: Satya Tangirala <satyat@google.com>
Reviewed-by: Eric Biggers <ebiggers@google.com>
> diff --git a/Documentation/filesystems/fscrypt.rst b/Documentation/filesystems/fscrypt.rst
> index f5d8b0303ddf..f3d87a1a0a7f 100644
> --- a/Documentation/filesystems/fscrypt.rst
> +++ b/Documentation/filesystems/fscrypt.rst
> @@ -1204,6 +1204,18 @@ buffer. Some filesystems, such as UBIFS, already use temporary
> buffers regardless of encryption. Other filesystems, such as ext4 and
> F2FS, have to allocate bounce pages specially for encryption.
>
> +Fscrypt is also able to use inline encryption hardware instead of the
> +kernel crypto API for en/decryption of file contents. When possible, and
> +if directed to do so (by specifying the 'inlinecrypt' mount option for
> +an ext4/F2FS filesystem), it adds encryption contexts to bios and
> +uses blk-crypto to perform the en/decryption instead of making use
> +of the above read/write path changes. Of course, even if directed to make
> +use of inline encryption, fscrypt will only be able to do so if either
> +hardware inline encryption support is available for the selected encryption
> +algorithm or CONFIG_BLK_INLINE_ENCRYPTION_FALLBACK is selected. If neither
> +is the case, fscrypt will fall back to using the above mentioned read/write
> +path changes for en/decryption.
> +
Nit: most of the text in this file is formatted with textwidth=70.
> Filename hashing and encoding
> -----------------------------
>
> @@ -1250,7 +1262,9 @@ Tests
>
> To test fscrypt, use xfstests, which is Linux's de facto standard
> filesystem test suite. First, run all the tests in the "encrypt"
> -group on the relevant filesystem(s). For example, to test ext4 and
> +group on the relevant filesystem(s). One can also run the tests
> +with the 'inlinecrypt' mount option to test the implementation for
> +inline encryption support. For example, to test ext4 and
> f2fs encryption using `kvm-xfstests
> <https://github.com/tytso/xfstests-bld/blob/master/Documentation/kvm-quickstart.md>`_::
>
> --
> 2.28.0.rc0.105.gf9edc3c819-goog
>
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
next prev parent reply other threads:[~2020-07-20 19:34 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-17 1:45 [PATCH v3 0/7] add support for direct I/O with fscrypt using blk-crypto Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-17 1:45 ` [PATCH v3 1/7] fscrypt: Add functions for direct I/O support Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-20 20:14 ` Eric Biggers
2020-07-20 20:14 ` [f2fs-dev] " Eric Biggers
2020-07-17 1:45 ` [PATCH v3 2/7] direct-io: add support for fscrypt using blk-crypto Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-17 1:45 ` [PATCH v3 3/7] iomap: support direct I/O with " Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-20 19:29 ` Eric Biggers
2020-07-20 19:29 ` [f2fs-dev] " Eric Biggers
2020-07-17 1:45 ` [PATCH v3 4/7] ext4: " Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-17 1:45 ` [PATCH v3 5/7] f2fs: " Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-17 1:45 ` [PATCH v3 6/7] fscrypt: document inline encryption support Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-20 19:34 ` Eric Biggers [this message]
2020-07-20 19:34 ` Eric Biggers
2020-07-17 1:45 ` [PATCH v3 7/7] fscrypt: update documentation for direct I/O support Satya Tangirala
2020-07-17 1:45 ` [f2fs-dev] " Satya Tangirala via Linux-f2fs-devel
2020-07-20 19:40 ` Eric Biggers
2020-07-20 19:40 ` [f2fs-dev] " Eric Biggers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200720193453.GH1292162@gmail.com \
--to=ebiggers@kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fscrypt@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=satyat@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.