On 22/04/21 12:24AM, Eric Biggers wrote: > On Thu, Apr 21, 2022 at 10:53:16AM +0530, Ritesh Harjani wrote: > > Hello, > > > > This is 1st in the series to cleanup ext4/super.c, since it has grown quite large. > > This moves out crypto related ops and few definitions to fs/ext4/ext4_crypto.c > > > > Testing > > ========= > > 1. Tested "-g encrypt" with default configs. > > 2. Compiled tested on x86 & Power. > > > > > > Ritesh Harjani (6): > > fscrypt: Provide definition of fscrypt_set_test_dummy_encryption > > ext4: Move ext4 crypto code to its own file ext4_crypto.c > > ext4: Directly opencode ext4_set_test_dummy_encryption > > ext4: Cleanup function defs from ext4.h into ext4_crypto.c > > ext4: Move all encryption related into a common #ifdef > > ext4: Use provided macro for checking dummy_enc_policy > > > > fs/ext4/Makefile | 1 + > > fs/ext4/ext4.h | 81 +++-------------- > > fs/ext4/ext4_crypto.c | 192 ++++++++++++++++++++++++++++++++++++++++ > > fs/ext4/super.c | 158 ++++----------------------------- > > include/linux/fscrypt.h | 7 ++ > > 5 files changed, 227 insertions(+), 212 deletions(-) > > create mode 100644 fs/ext4/ext4_crypto.c > > How about calling it crypto.c instead of ext4_crypto.c? It is already in the > ext4 directory, so ext4 is implied. Sure will do so in next revision which can also cover any other comments from others. > > Otherwise this patchset looks good to me. Sure thanks. May I add your Reviewed-by in next version if I don't see any major changes? > > Did you consider moving any of the other CONFIG_FS_ENCRYPTION code blocks into > the new file as well? The implementation of FS_IOC_GET_ENCRYPTION_PWSALT might > be a good candidate too. Sure, thanks for pointing out. I will take a look at it for next patch series or so. -ritesh
