* [PATCH] rpc: remove unneeded function parameter in gss_add_msg()
@ 2009-12-09 17:45 Suresh Jayaraman
2009-12-09 18:57 ` J. Bruce Fields
0 siblings, 1 reply; 3+ messages in thread
From: Suresh Jayaraman @ 2009-12-09 17:45 UTC (permalink / raw)
To: Trond Myklebust; +Cc: linux-nfs, J. Bruce Fields, Suresh Jayaraman
The pointer to struct gss_auth parameter in gss_add_msg is not really needed
after commit 5b7ddd4a. Zap it.
Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
---
net/sunrpc/auth_gss/auth_gss.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/sunrpc/auth_gss/auth_gss.c b/net/sunrpc/auth_gss/auth_gss.c
index fc6a43c..85ee2ca 100644
--- a/net/sunrpc/auth_gss/auth_gss.c
+++ b/net/sunrpc/auth_gss/auth_gss.c
@@ -304,7 +304,7 @@ __gss_find_upcall(struct rpc_inode *rpci, uid_t uid)
* to that upcall instead of adding the new upcall.
*/
static inline struct gss_upcall_msg *
-gss_add_msg(struct gss_auth *gss_auth, struct gss_upcall_msg *gss_msg)
+gss_add_msg(struct gss_upcall_msg *gss_msg)
{
struct rpc_inode *rpci = gss_msg->inode;
struct inode *inode = &rpci->vfs_inode;
@@ -445,7 +445,7 @@ gss_setup_upcall(struct rpc_clnt *clnt, struct gss_auth *gss_auth, struct rpc_cr
gss_new = gss_alloc_msg(gss_auth, uid, clnt, gss_cred->gc_machine_cred);
if (IS_ERR(gss_new))
return gss_new;
- gss_msg = gss_add_msg(gss_auth, gss_new);
+ gss_msg = gss_add_msg(gss_new);
if (gss_msg == gss_new) {
struct inode *inode = &gss_new->inode->vfs_inode;
int res = rpc_queue_upcall(inode, &gss_new->msg);
--
1.6.4.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] rpc: remove unneeded function parameter in gss_add_msg()
2009-12-09 17:45 [PATCH] rpc: remove unneeded function parameter in gss_add_msg() Suresh Jayaraman
@ 2009-12-09 18:57 ` J. Bruce Fields
2009-12-09 20:18 ` Trond Myklebust
0 siblings, 1 reply; 3+ messages in thread
From: J. Bruce Fields @ 2009-12-09 18:57 UTC (permalink / raw)
To: Suresh Jayaraman; +Cc: Trond Myklebust, linux-nfs
On Wed, Dec 09, 2009 at 11:15:22PM +0530, Suresh Jayaraman wrote:
> The pointer to struct gss_auth parameter in gss_add_msg is not really needed
> after commit 5b7ddd4a. Zap it.
Looks straightforward to me, thanks. (Assuming Trond will take it.)
--b.
>
> Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
> ---
> net/sunrpc/auth_gss/auth_gss.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/sunrpc/auth_gss/auth_gss.c b/net/sunrpc/auth_gss/auth_gss.c
> index fc6a43c..85ee2ca 100644
> --- a/net/sunrpc/auth_gss/auth_gss.c
> +++ b/net/sunrpc/auth_gss/auth_gss.c
> @@ -304,7 +304,7 @@ __gss_find_upcall(struct rpc_inode *rpci, uid_t uid)
> * to that upcall instead of adding the new upcall.
> */
> static inline struct gss_upcall_msg *
> -gss_add_msg(struct gss_auth *gss_auth, struct gss_upcall_msg *gss_msg)
> +gss_add_msg(struct gss_upcall_msg *gss_msg)
> {
> struct rpc_inode *rpci = gss_msg->inode;
> struct inode *inode = &rpci->vfs_inode;
> @@ -445,7 +445,7 @@ gss_setup_upcall(struct rpc_clnt *clnt, struct gss_auth *gss_auth, struct rpc_cr
> gss_new = gss_alloc_msg(gss_auth, uid, clnt, gss_cred->gc_machine_cred);
> if (IS_ERR(gss_new))
> return gss_new;
> - gss_msg = gss_add_msg(gss_auth, gss_new);
> + gss_msg = gss_add_msg(gss_new);
> if (gss_msg == gss_new) {
> struct inode *inode = &gss_new->inode->vfs_inode;
> int res = rpc_queue_upcall(inode, &gss_new->msg);
> --
> 1.6.4.2
>
> --
> 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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] rpc: remove unneeded function parameter in gss_add_msg()
2009-12-09 18:57 ` J. Bruce Fields
@ 2009-12-09 20:18 ` Trond Myklebust
0 siblings, 0 replies; 3+ messages in thread
From: Trond Myklebust @ 2009-12-09 20:18 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: Suresh Jayaraman, linux-nfs
On Wed, 2009-12-09 at 13:57 -0500, J. Bruce Fields wrote:
> On Wed, Dec 09, 2009 at 11:15:22PM +0530, Suresh Jayaraman wrote:
> > The pointer to struct gss_auth parameter in gss_add_msg is not really needed
> > after commit 5b7ddd4a. Zap it.
>
> Looks straightforward to me, thanks. (Assuming Trond will take it.)
Yes. I'll pick it up...
Trond
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-12-09 20:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-12-09 17:45 [PATCH] rpc: remove unneeded function parameter in gss_add_msg() Suresh Jayaraman
2009-12-09 18:57 ` J. Bruce Fields
2009-12-09 20:18 ` Trond Myklebust
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox