2011-01-14 08:41:55

by Shan Wei

[permalink] [raw]
Subject: [PATCH] nfsd: kill unused ENT_HASHMASK definition

This macro definition has not been using since v2.6.12-rc2.
So, remove it now.


Signed-off-by: Shan Wei <[email protected]>
---
fs/nfsd/nfs4idmap.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c
index f0695e8..ce0f6b9 100644
--- a/fs/nfsd/nfs4idmap.c
+++ b/fs/nfsd/nfs4idmap.c
@@ -62,7 +62,6 @@ struct ent {

#define ENT_HASHBITS 8
#define ENT_HASHMAX (1 << ENT_HASHBITS)
-#define ENT_HASHMASK (ENT_HASHMAX - 1)

static void
ent_init(struct cache_head *cnew, struct cache_head *citm)
--
1.6.3.3


2011-01-14 09:52:45

by Shan Wei

[permalink] [raw]
Subject: [PATCH v2] nfsd: kill unused macro definition

These macros had never been used for several years.
So, remove them.

Signed-off-by: Shan Wei <[email protected]>
---
V2: add two other unused macros, EXPORT_HASHMASK and FILE_HASH_MASK.

---
fs/nfsd/export.c | 1 -
fs/nfsd/nfs4idmap.c | 1 -
fs/nfsd/nfs4state.c | 2 +-
3 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c
index c0fcb7a..98a964a 100644
--- a/fs/nfsd/export.c
+++ b/fs/nfsd/export.c
@@ -300,7 +300,6 @@ svc_expkey_update(struct svc_expkey *new, struct svc_expkey *old)

#define EXPORT_HASHBITS 8
#define EXPORT_HASHMAX (1<< EXPORT_HASHBITS)
-#define EXPORT_HASHMASK (EXPORT_HASHMAX -1)

static struct cache_head *export_table[EXPORT_HASHMAX];

diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c
index f0695e8..ce0f6b9 100644
--- a/fs/nfsd/nfs4idmap.c
+++ b/fs/nfsd/nfs4idmap.c
@@ -62,7 +62,6 @@ struct ent {

#define ENT_HASHBITS 8
#define ENT_HASHMAX (1 << ENT_HASHBITS)
-#define ENT_HASHMASK (ENT_HASHMAX - 1)

static void
ent_init(struct cache_head *cnew, struct cache_head *citm)
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index fbd18c3..547fbc6 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -148,7 +148,7 @@ static struct list_head ownerstr_hashtbl[OWNER_HASH_SIZE];
/* hash table for nfs4_file */
#define FILE_HASH_BITS 8
#define FILE_HASH_SIZE (1 << FILE_HASH_BITS)
-#define FILE_HASH_MASK (FILE_HASH_SIZE - 1)
+
/* hash table for (open)nfs4_stateid */
#define STATEID_HASH_BITS 10
#define STATEID_HASH_SIZE (1 << STATEID_HASH_BITS)
--
1.6.3.3

2011-01-18 21:58:16

by J. Bruce Fields

[permalink] [raw]
Subject: Re: [PATCH v2] nfsd: kill unused macro definition

On Fri, Jan 14, 2011 at 05:35:59PM +0800, Shan Wei wrote:
> These macros had never been used for several years.
> So, remove them.

Thanks, applying.--b.

>
> Signed-off-by: Shan Wei <[email protected]>
> ---
> V2: add two other unused macros, EXPORT_HASHMASK and FILE_HASH_MASK.
>
> ---
> fs/nfsd/export.c | 1 -
> fs/nfsd/nfs4idmap.c | 1 -
> fs/nfsd/nfs4state.c | 2 +-
> 3 files changed, 1 insertions(+), 3 deletions(-)
>
> diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c
> index c0fcb7a..98a964a 100644
> --- a/fs/nfsd/export.c
> +++ b/fs/nfsd/export.c
> @@ -300,7 +300,6 @@ svc_expkey_update(struct svc_expkey *new, struct svc_expkey *old)
>
> #define EXPORT_HASHBITS 8
> #define EXPORT_HASHMAX (1<< EXPORT_HASHBITS)
> -#define EXPORT_HASHMASK (EXPORT_HASHMAX -1)
>
> static struct cache_head *export_table[EXPORT_HASHMAX];
>
> diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c
> index f0695e8..ce0f6b9 100644
> --- a/fs/nfsd/nfs4idmap.c
> +++ b/fs/nfsd/nfs4idmap.c
> @@ -62,7 +62,6 @@ struct ent {
>
> #define ENT_HASHBITS 8
> #define ENT_HASHMAX (1 << ENT_HASHBITS)
> -#define ENT_HASHMASK (ENT_HASHMAX - 1)
>
> static void
> ent_init(struct cache_head *cnew, struct cache_head *citm)
> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> index fbd18c3..547fbc6 100644
> --- a/fs/nfsd/nfs4state.c
> +++ b/fs/nfsd/nfs4state.c
> @@ -148,7 +148,7 @@ static struct list_head ownerstr_hashtbl[OWNER_HASH_SIZE];
> /* hash table for nfs4_file */
> #define FILE_HASH_BITS 8
> #define FILE_HASH_SIZE (1 << FILE_HASH_BITS)
> -#define FILE_HASH_MASK (FILE_HASH_SIZE - 1)
> +
> /* hash table for (open)nfs4_stateid */
> #define STATEID_HASH_BITS 10
> #define STATEID_HASH_SIZE (1 << STATEID_HASH_BITS)
> --
> 1.6.3.3