diff mbox

sunrpc: move address copy/cmp/convert routines and prototypes from clnt.h to addr.h

Message ID 20130204161825.GF815@fieldses.org (mailing list archive)
State New, archived
Headers show

Commit Message

J. Bruce Fields Feb. 4, 2013, 4:18 p.m. UTC
On Mon, Feb 04, 2013 at 04:15:43PM +0000, Myklebust, Trond wrote:
> On Mon, 2013-02-04 at 11:09 -0500, J. Bruce Fields wrote:
> > On Mon, Feb 04, 2013 at 10:23:51AM -0500, Jeff Layton wrote:
> > > These routines are used by server and client code, so having them in a
> > > separate header would be best.
> > 
> > I'll apply this and the scope id patch if it gets Trond's ACK.
> > 
> > (Or they have my ACK if Trond wants to take them.)
> 
> Feel free to take them through the nfsd tree.
> 
> Acked-by: Trond Myklebust <Trond.Myklebust@netapp.com>

Thanks.  I needed to add the following.

We should make sure none of the other files including clnt.h might also
need one of these.

--b.

commit a487a5ec9e2f37472097a3ff3535c87c6af334e9
Author: J. Bruce Fields <bfields@redhat.com>
Date:   Mon Feb 4 11:13:24 2013 -0500

    foldme

--
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

Comments

J. Bruce Fields Feb. 4, 2013, 5:01 p.m. UTC | #1
On Mon, Feb 04, 2013 at 11:18:25AM -0500, J. Bruce Fields wrote:
> On Mon, Feb 04, 2013 at 04:15:43PM +0000, Myklebust, Trond wrote:
> > On Mon, 2013-02-04 at 11:09 -0500, J. Bruce Fields wrote:
> > > On Mon, Feb 04, 2013 at 10:23:51AM -0500, Jeff Layton wrote:
> > > > These routines are used by server and client code, so having them in a
> > > > separate header would be best.
> > > 
> > > I'll apply this and the scope id patch if it gets Trond's ACK.
> > > 
> > > (Or they have my ACK if Trond wants to take them.)
> > 
> > Feel free to take them through the nfsd tree.
> > 
> > Acked-by: Trond Myklebust <Trond.Myklebust@netapp.com>
> 
> Thanks.  I needed to add the following.
> 
> We should make sure none of the other files including clnt.h might also
> need one of these.

Eh, pretty sure that was the only one.  Applied.--b.

> 
> --b.
> 
> commit a487a5ec9e2f37472097a3ff3535c87c6af334e9
> Author: J. Bruce Fields <bfields@redhat.com>
> Date:   Mon Feb 4 11:13:24 2013 -0500
> 
>     foldme
> 
> diff --git a/fs/nfsd/fault_inject.c b/fs/nfsd/fault_inject.c
> index e761ee9..255ae32 100644
> --- a/fs/nfsd/fault_inject.c
> +++ b/fs/nfsd/fault_inject.c
> @@ -10,6 +10,7 @@
>  #include <linux/module.h>
>  #include <linux/nsproxy.h>
>  #include <linux/sunrpc/clnt.h>
> +#include <linux/sunrpc/addr.h>
>  #include <asm/uaccess.h>
>  
>  #include "state.h"
--
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/fault_inject.c b/fs/nfsd/fault_inject.c
index e761ee9..255ae32 100644
--- a/fs/nfsd/fault_inject.c
+++ b/fs/nfsd/fault_inject.c
@@ -10,6 +10,7 @@ 
 #include <linux/module.h>
 #include <linux/nsproxy.h>
 #include <linux/sunrpc/clnt.h>
+#include <linux/sunrpc/addr.h>
 #include <asm/uaccess.h>
 
 #include "state.h"