diff mbox

nfsd: don't calculate mem's size in loop every time

Message ID 5163AF52.2080303@cn.fujitsu.com (mailing list archive)
State New, archived
Headers show

Commit Message

fanchaoting April 9, 2013, 6:04 a.m. UTC
Signed-off-by: fanchaoting <fanchaoting@cn.fujitsu.com>
---
 fs/nfsd/nfs4state.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

J. Bruce Fields April 12, 2013, 2:48 p.m. UTC | #1
On Tue, Apr 09, 2013 at 02:04:02PM +0800, fanchaoting wrote:
> 
>  Signed-off-by: fanchaoting <fanchaoting@cn.fujitsu.com>

Thanks, this makes sense, but meanwhile I've been rewriting that
code--now pushed out to my for-3.10 branch.  Let me know if you see
anything that looks wrong there.

--b.

> ---
>  fs/nfsd/nfs4state.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> index 2e27430..df3e3fd 100644
> --- a/fs/nfsd/nfs4state.c
> +++ b/fs/nfsd/nfs4state.c
> @@ -716,8 +716,8 @@ static struct nfsd4_session *__alloc_session(int slotsize, int numslots)
>  	if (!new)
>  		return NULL;
>  	/* allocate each struct nfsd4_slot and data cache in one piece */
> +	mem = sizeof(struct nfsd4_slot) + slotsize;
>  	for (i = 0; i < numslots; i++) {
> -		mem = sizeof(struct nfsd4_slot) + slotsize;
>  		new->se_slots[i] = kzalloc(mem, GFP_KERNEL);
>  		if (!new->se_slots[i])
>  			goto out_free;
> -- 
> 1.7.1
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 2e27430..df3e3fd 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -716,8 +716,8 @@  static struct nfsd4_session *__alloc_session(int slotsize, int numslots)
 	if (!new)
 		return NULL;
 	/* allocate each struct nfsd4_slot and data cache in one piece */
+	mem = sizeof(struct nfsd4_slot) + slotsize;
 	for (i = 0; i < numslots; i++) {
-		mem = sizeof(struct nfsd4_slot) + slotsize;
 		new->se_slots[i] = kzalloc(mem, GFP_KERNEL);
 		if (!new->se_slots[i])
 			goto out_free;