sdcardfs: remove effectless config option
CONFIG_SDCARD_FS_CI_SEARCH only guards a define for LOOKUP_CASE_INSENSITIVE, which is never used in the kernel. Remove both, along with the option matching that supports it. Change-Id: I363a8f31de8ee7a7a934d75300cc9ba8176e2edf Signed-off-by: Daniel Rosenberg <drosen@google.com>
This commit is contained in:
parent
ce15174caf
commit
1a98a1b1e6
5 changed files with 1 additions and 33 deletions
|
@ -11,8 +11,3 @@ config SDCARD_FS_FADV_NOACTIVE
|
||||||
default y
|
default y
|
||||||
help
|
help
|
||||||
Sdcardfs supports fadvise noactive mode.
|
Sdcardfs supports fadvise noactive mode.
|
||||||
|
|
||||||
config SDCARD_FS_CI_SEARCH
|
|
||||||
tristate "sdcardfs case-insensitive search support"
|
|
||||||
depends on SDCARD_FS
|
|
||||||
default y
|
|
||||||
|
|
|
@ -238,13 +238,8 @@ static struct dentry *__sdcardfs_lookup(struct dentry *dentry,
|
||||||
lower_dir_mnt = lower_parent_path->mnt;
|
lower_dir_mnt = lower_parent_path->mnt;
|
||||||
|
|
||||||
/* Use vfs_path_lookup to check if the dentry exists or not */
|
/* Use vfs_path_lookup to check if the dentry exists or not */
|
||||||
if (sbi->options.lower_fs == LOWER_FS_EXT4) {
|
err = vfs_path_lookup(lower_dir_dentry, lower_dir_mnt, name, 0,
|
||||||
err = vfs_path_lookup(lower_dir_dentry, lower_dir_mnt, name,
|
|
||||||
LOOKUP_CASE_INSENSITIVE, &lower_path);
|
|
||||||
} else if (sbi->options.lower_fs == LOWER_FS_FAT) {
|
|
||||||
err = vfs_path_lookup(lower_dir_dentry, lower_dir_mnt, name, 0,
|
|
||||||
&lower_path);
|
&lower_path);
|
||||||
}
|
|
||||||
|
|
||||||
/* no error: handle positive dentries */
|
/* no error: handle positive dentries */
|
||||||
if (!err) {
|
if (!err) {
|
||||||
|
|
|
@ -41,7 +41,6 @@ static const match_table_t sdcardfs_tokens = {
|
||||||
{Opt_fsgid, "fsgid=%u"},
|
{Opt_fsgid, "fsgid=%u"},
|
||||||
{Opt_gid, "gid=%u"},
|
{Opt_gid, "gid=%u"},
|
||||||
{Opt_debug, "debug"},
|
{Opt_debug, "debug"},
|
||||||
{Opt_lower_fs, "lower_fs=%s"},
|
|
||||||
{Opt_mask, "mask=%u"},
|
{Opt_mask, "mask=%u"},
|
||||||
{Opt_userid, "userid=%d"},
|
{Opt_userid, "userid=%d"},
|
||||||
{Opt_multiuser, "multiuser"},
|
{Opt_multiuser, "multiuser"},
|
||||||
|
@ -64,8 +63,6 @@ static int parse_options(struct super_block *sb, char *options, int silent,
|
||||||
opts->multiuser = false;
|
opts->multiuser = false;
|
||||||
opts->fs_user_id = 0;
|
opts->fs_user_id = 0;
|
||||||
opts->gid = 0;
|
opts->gid = 0;
|
||||||
/* by default, we use LOWER_FS_EXT4 as lower fs type */
|
|
||||||
opts->lower_fs = LOWER_FS_EXT4;
|
|
||||||
/* by default, 0MB is reserved */
|
/* by default, 0MB is reserved */
|
||||||
opts->reserved_mb = 0;
|
opts->reserved_mb = 0;
|
||||||
|
|
||||||
|
@ -113,18 +110,6 @@ static int parse_options(struct super_block *sb, char *options, int silent,
|
||||||
case Opt_multiuser:
|
case Opt_multiuser:
|
||||||
opts->multiuser = true;
|
opts->multiuser = true;
|
||||||
break;
|
break;
|
||||||
case Opt_lower_fs:
|
|
||||||
string_option = match_strdup(&args[0]);
|
|
||||||
if (!strcmp("ext4", string_option)) {
|
|
||||||
opts->lower_fs = LOWER_FS_EXT4;
|
|
||||||
} else if (!strcmp("fat", string_option)) {
|
|
||||||
opts->lower_fs = LOWER_FS_FAT;
|
|
||||||
} else {
|
|
||||||
kfree(string_option);
|
|
||||||
goto invalid_option;
|
|
||||||
}
|
|
||||||
kfree(string_option);
|
|
||||||
break;
|
|
||||||
case Opt_reserved_mb:
|
case Opt_reserved_mb:
|
||||||
if (match_int(&args[0], &option))
|
if (match_int(&args[0], &option))
|
||||||
return 0;
|
return 0;
|
||||||
|
|
|
@ -123,11 +123,6 @@ typedef enum {
|
||||||
PERM_ANDROID_MEDIA,
|
PERM_ANDROID_MEDIA,
|
||||||
} perm_t;
|
} perm_t;
|
||||||
|
|
||||||
typedef enum {
|
|
||||||
LOWER_FS_EXT4,
|
|
||||||
LOWER_FS_FAT,
|
|
||||||
} lower_fs_t;
|
|
||||||
|
|
||||||
struct sdcardfs_sb_info;
|
struct sdcardfs_sb_info;
|
||||||
struct sdcardfs_mount_options;
|
struct sdcardfs_mount_options;
|
||||||
|
|
||||||
|
@ -191,7 +186,6 @@ struct sdcardfs_mount_options {
|
||||||
gid_t fs_low_gid;
|
gid_t fs_low_gid;
|
||||||
userid_t fs_user_id;
|
userid_t fs_user_id;
|
||||||
gid_t gid;
|
gid_t gid;
|
||||||
lower_fs_t lower_fs;
|
|
||||||
mode_t mask;
|
mode_t mask;
|
||||||
bool multiuser;
|
bool multiuser;
|
||||||
unsigned int reserved_mb;
|
unsigned int reserved_mb;
|
||||||
|
|
|
@ -43,7 +43,6 @@ enum {LAST_NORM, LAST_ROOT, LAST_DOT, LAST_DOTDOT, LAST_BIND};
|
||||||
#define LOOKUP_JUMPED 0x1000
|
#define LOOKUP_JUMPED 0x1000
|
||||||
#define LOOKUP_ROOT 0x2000
|
#define LOOKUP_ROOT 0x2000
|
||||||
#define LOOKUP_EMPTY 0x4000
|
#define LOOKUP_EMPTY 0x4000
|
||||||
#define LOOKUP_CASE_INSENSITIVE 0x8000
|
|
||||||
|
|
||||||
extern int user_path_at_empty(int, const char __user *, unsigned, struct path *, int *empty);
|
extern int user_path_at_empty(int, const char __user *, unsigned, struct path *, int *empty);
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue