* [PATCH] NFSD: Use correct reply size calculating function
@ 2015-03-15 15:35 Kinglong Mee
2015-03-16 15:03 ` Anna Schumaker
2015-03-20 20:24 ` J. Bruce Fields
0 siblings, 2 replies; 3+ messages in thread
From: Kinglong Mee @ 2015-03-15 15:35 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: Anna.Schumaker, Linux NFS Mailing List
ALLOCATE/DEALLOCATE only reply one status value to client,
so, using nfsd4_only_status_rsize for reply size calculating.
Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
---
fs/nfsd/nfs4proc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index 92b9d97..4449359 100644
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -2282,13 +2282,13 @@ static struct nfsd4_operation nfsd4_ops[] = {
.op_func = (nfsd4op_func)nfsd4_allocate,
.op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
.op_name = "OP_ALLOCATE",
- .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
+ .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
},
[OP_DEALLOCATE] = {
.op_func = (nfsd4op_func)nfsd4_deallocate,
.op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
.op_name = "OP_DEALLOCATE",
- .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
+ .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
},
[OP_SEEK] = {
.op_func = (nfsd4op_func)nfsd4_seek,
--
2.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] NFSD: Use correct reply size calculating function
2015-03-15 15:35 [PATCH] NFSD: Use correct reply size calculating function Kinglong Mee
@ 2015-03-16 15:03 ` Anna Schumaker
2015-03-20 20:24 ` J. Bruce Fields
1 sibling, 0 replies; 3+ messages in thread
From: Anna Schumaker @ 2015-03-16 15:03 UTC (permalink / raw)
To: Kinglong Mee, J. Bruce Fields; +Cc: Linux NFS Mailing List
On 03/15/2015 11:35 AM, Kinglong Mee wrote:
> ALLOCATE/DEALLOCATE only reply one status value to client,
> so, using nfsd4_only_status_rsize for reply size calculating.
Makes sense to me. Thanks for catching this!
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
> ---
> fs/nfsd/nfs4proc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> index 92b9d97..4449359 100644
> --- a/fs/nfsd/nfs4proc.c
> +++ b/fs/nfsd/nfs4proc.c
> @@ -2282,13 +2282,13 @@ static struct nfsd4_operation nfsd4_ops[] = {
> .op_func = (nfsd4op_func)nfsd4_allocate,
> .op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
> .op_name = "OP_ALLOCATE",
> - .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
> + .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
> },
> [OP_DEALLOCATE] = {
> .op_func = (nfsd4op_func)nfsd4_deallocate,
> .op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
> .op_name = "OP_DEALLOCATE",
> - .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
> + .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
> },
> [OP_SEEK] = {
> .op_func = (nfsd4op_func)nfsd4_seek,
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] NFSD: Use correct reply size calculating function
2015-03-15 15:35 [PATCH] NFSD: Use correct reply size calculating function Kinglong Mee
2015-03-16 15:03 ` Anna Schumaker
@ 2015-03-20 20:24 ` J. Bruce Fields
1 sibling, 0 replies; 3+ messages in thread
From: J. Bruce Fields @ 2015-03-20 20:24 UTC (permalink / raw)
To: Kinglong Mee; +Cc: Anna.Schumaker, Linux NFS Mailing List
On Sun, Mar 15, 2015 at 11:35:11PM +0800, Kinglong Mee wrote:
> ALLOCATE/DEALLOCATE only reply one status value to client,
> so, using nfsd4_only_status_rsize for reply size calculating.
Thanks! Applying for 4.1. (It's a bug, but I don't see an obvious
serious impact, so I'm assuming it doesn't need to go to 4.0 and stable.
Tell me if I've overlooked something.)
--b.
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
> ---
> fs/nfsd/nfs4proc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> index 92b9d97..4449359 100644
> --- a/fs/nfsd/nfs4proc.c
> +++ b/fs/nfsd/nfs4proc.c
> @@ -2282,13 +2282,13 @@ static struct nfsd4_operation nfsd4_ops[] = {
> .op_func = (nfsd4op_func)nfsd4_allocate,
> .op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
> .op_name = "OP_ALLOCATE",
> - .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
> + .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
> },
> [OP_DEALLOCATE] = {
> .op_func = (nfsd4op_func)nfsd4_deallocate,
> .op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME,
> .op_name = "OP_DEALLOCATE",
> - .op_rsize_bop = (nfsd4op_rsize)nfsd4_write_rsize,
> + .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize,
> },
> [OP_SEEK] = {
> .op_func = (nfsd4op_func)nfsd4_seek,
> --
> 2.3.1
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-03-20 20:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-15 15:35 [PATCH] NFSD: Use correct reply size calculating function Kinglong Mee
2015-03-16 15:03 ` Anna Schumaker
2015-03-20 20:24 ` J. Bruce Fields
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).