From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754476AbeCRU4j (ORCPT ); Sun, 18 Mar 2018 16:56:39 -0400 Received: from mail-wm0-f66.google.com ([74.125.82.66]:33262 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754121AbeCRU4i (ORCPT ); Sun, 18 Mar 2018 16:56:38 -0400 X-Google-Smtp-Source: AG47ELui5cR+aIpbvjBabm5v7bo6+dJlII8DfLrLNVIm/0zVSsa50T8ALY4eFPo6L5rmZ3XLe1P0JA== Date: Sun, 18 Mar 2018 20:56:35 +0000 From: Phillip Potter To: dushistov@mail.ru Cc: linux-kernel@vger.kernel.org Subject: [PATCH] fs: ufs: Convert ufs_set_de_type to use lookup table Message-ID: <20180318205635.GA31421@pathfinder> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Modify ufs_set_de_type function in fs/ufs/util.h to use a lookup table rather than a switch statement, as per the TODO comment. Signed-off-by: Phillip Potter --- fs/ufs/util.h | 50 ++++++++++++++++++++++---------------------------- 1 file changed, 22 insertions(+), 28 deletions(-) diff --git a/fs/ufs/util.h b/fs/ufs/util.h index 1907be6d5808..1edf4c6454e3 100644 --- a/fs/ufs/util.h +++ b/fs/ufs/util.h @@ -155,37 +155,31 @@ ufs_set_de_namlen(struct super_block *sb, struct ufs_dir_entry *de, u16 value) static inline void ufs_set_de_type(struct super_block *sb, struct ufs_dir_entry *de, int mode) { + /* type lookup table, DT_UNKNOWN is default type if no case holds */ + const int mode_table[] = { + DT_UNKNOWN, + DT_FIFO, /* mode & S_IFMT == S_IFIFO case */ + DT_CHR, /* mode & S_IFMT == S_IFCHR case */ + DT_UNKNOWN, + DT_DIR, /* mode & S_IFMT == S_IFDIR case */ + DT_UNKNOWN, + DT_BLK, /* mode & S_IFMT == S_IFBLK case */ + DT_UNKNOWN, + DT_REG, /* mode & S_IFMT == S_IFREG case */ + DT_UNKNOWN, + DT_LNK, /* mode & S_IFMT == S_IFLNK case */ + DT_UNKNOWN, + DT_SOCK, /* mode & S_IFMT == S_IFSOCK case */ + DT_UNKNOWN, + DT_UNKNOWN, + DT_UNKNOWN + }; + if ((UFS_SB(sb)->s_flags & UFS_DE_MASK) != UFS_DE_44BSD) return; - /* - * TODO turn this into a table lookup - */ - switch (mode & S_IFMT) { - case S_IFSOCK: - de->d_u.d_44.d_type = DT_SOCK; - break; - case S_IFLNK: - de->d_u.d_44.d_type = DT_LNK; - break; - case S_IFREG: - de->d_u.d_44.d_type = DT_REG; - break; - case S_IFBLK: - de->d_u.d_44.d_type = DT_BLK; - break; - case S_IFDIR: - de->d_u.d_44.d_type = DT_DIR; - break; - case S_IFCHR: - de->d_u.d_44.d_type = DT_CHR; - break; - case S_IFIFO: - de->d_u.d_44.d_type = DT_FIFO; - break; - default: - de->d_u.d_44.d_type = DT_UNKNOWN; - } + /* shift (mode & S_IFMT) right 12 bits to index into table */ + de->d_u.d_44.d_type = mode_table[(mode & S_IFMT) >> 12]; } static inline u32 -- 2.14.3