From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-f67.google.com ([209.85.128.67]:38434 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725859AbeJXEow (ORCPT ); Wed, 24 Oct 2018 00:44:52 -0400 Received: by mail-wm1-f67.google.com with SMTP id b14-v6so487925wmj.3 for ; Tue, 23 Oct 2018 13:19:54 -0700 (PDT) Date: Tue, 23 Oct 2018 21:19:52 +0100 From: Phillip Potter To: viro@zeniv.linux.org.uk Cc: linux-kernel@vger.kernel.org, amir73il@gmail.com, linux-fsdevel@vger.kernel.org Subject: [RFC][PATCH 00/11] common implementation of dirent file types Message-ID: <20181023201952.GA15676@pathfinder> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Sender: linux-fsdevel-owner@vger.kernel.org List-ID: This cleanup series is a respin of Amir Goldstein's work, created in late 2016. It removes several instances of duplicated code. Most of the duplication dates back to git pre-historic era. The controversial aspect of this cleanup is that it uses common code to handle file system specific on-disk format bits. All 7 file systems use a single byte to store dirent file type on-disk and all of them use the same conversion routines from i_mode to 4bits DT_* constants to 3bits on-disk FT_* constants. Patch 1 places a common implementation in file_type.h and add some useful conversion helpers. Patches 2-3 make use of some helpers in ufs and hfsplus without any on-disk implications. Patches 4-10 replace the specific implementations in ext2, exofs, ext4, ocfs2, f2fs, nilfs and btrfs with the common implementation. These patches also now include compile-time checks to ensure that the file system specific on-disk bits are equivalent to the common implementation FT_* bits. These compile-time checks are only included once per file system, as my reasoning is that regardless of their location, the build will fail/succeed anyway. In addition, where needed (for patches which no longer apply), I've rebased them to apply to the newest 4.19 kernel sources. Each patch is independent of the others, except for the common implementation itself which they all depend on. I would love feedback as newish kernel contributor, particularly from the original author Amir who suggested this task for me. I hope the various subsystem/fs maintainers will see fit to accept this work also. Phillip Potter (10): fs: common implementation of file type conversions ufs: use fs_umode_to_dtype() helper hfsplus: use fs_umode_to_dtype() helper ext2: use common file type conversion exofs: use common file type conversion ext4: use common file type conversion ocfs2: use common file type conversion f2fs: use common file type conversion nilfs2: use common file type conversion btrfs: use common file type conversion --- MAINTAINERS | 1 + fs/btrfs/btrfs_inode.h | 2 - fs/btrfs/delayed-inode.c | 2 +- fs/btrfs/inode.c | 35 +++++----- fs/exofs/dir.c | 49 +++++-------- fs/ext2/dir.c | 51 ++++++-------- fs/ext4/ext4.h | 35 +++++----- fs/f2fs/dir.c | 43 +++++------- fs/f2fs/inline.c | 2 +- fs/hfsplus/dir.c | 16 +---- fs/nilfs2/dir.c | 54 +++++---------- fs/ocfs2/dir.c | 32 +++++---- fs/ocfs2/ocfs2_fs.h | 13 +--- fs/ufs/util.h | 29 +------- include/linux/f2fs_fs.h | 8 ++- include/linux/file_type.h | 108 +++++++++++++++++++++++++++++ include/linux/fs.h | 17 +---- include/uapi/linux/btrfs_tree.h | 2 + include/uapi/linux/nilfs2_ondisk.h | 1 + 19 files changed, 256 insertions(+), 244 deletions(-) create mode 100644 include/linux/file_type.h -- 2.17.2