* [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
@ 2014-07-12 22:01 Trond Myklebust
2014-07-15 17:01 ` Christoph Hellwig
2014-07-16 20:58 ` Bruce Fields
0 siblings, 2 replies; 6+ messages in thread
From: Trond Myklebust @ 2014-07-12 22:01 UTC (permalink / raw)
To: Bruce Fields; +Cc: linux-nfs
Quell another sparse warning.
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
---
net/sunrpc/xdr.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/sunrpc/xdr.c b/net/sunrpc/xdr.c
index 23fb4e75e245..290af97bf6f9 100644
--- a/net/sunrpc/xdr.c
+++ b/net/sunrpc/xdr.c
@@ -509,7 +509,8 @@ void xdr_commit_encode(struct xdr_stream *xdr)
}
EXPORT_SYMBOL_GPL(xdr_commit_encode);
-__be32 *xdr_get_next_encode_buffer(struct xdr_stream *xdr, size_t nbytes)
+static __be32 *xdr_get_next_encode_buffer(struct xdr_stream *xdr,
+ size_t nbytes)
{
static __be32 *p;
int space_left;
--
1.9.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
2014-07-12 22:01 [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static Trond Myklebust
@ 2014-07-15 17:01 ` Christoph Hellwig
2014-07-16 20:58 ` Bruce Fields
1 sibling, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2014-07-15 17:01 UTC (permalink / raw)
To: Trond Myklebust; +Cc: Bruce Fields, linux-nfs
On Sat, Jul 12, 2014 at 06:01:02PM -0400, Trond Myklebust wrote:
> Quell another sparse warning.
>
> Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
2014-07-12 22:01 [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static Trond Myklebust
2014-07-15 17:01 ` Christoph Hellwig
@ 2014-07-16 20:58 ` Bruce Fields
2014-07-16 21:40 ` Trond Myklebust
1 sibling, 1 reply; 6+ messages in thread
From: Bruce Fields @ 2014-07-16 20:58 UTC (permalink / raw)
To: Trond Myklebust; +Cc: linux-nfs
On Sat, Jul 12, 2014 at 06:01:02PM -0400, Trond Myklebust wrote:
> Quell another sparse warning.
I think jlayton sent you the same fix with his set of sunrpc sparse
fixes--could you just take that instead?
(Or I can take this if you still want, I don't really care.)
--b.
>
> Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
> ---
> net/sunrpc/xdr.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/net/sunrpc/xdr.c b/net/sunrpc/xdr.c
> index 23fb4e75e245..290af97bf6f9 100644
> --- a/net/sunrpc/xdr.c
> +++ b/net/sunrpc/xdr.c
> @@ -509,7 +509,8 @@ void xdr_commit_encode(struct xdr_stream *xdr)
> }
> EXPORT_SYMBOL_GPL(xdr_commit_encode);
>
> -__be32 *xdr_get_next_encode_buffer(struct xdr_stream *xdr, size_t nbytes)
> +static __be32 *xdr_get_next_encode_buffer(struct xdr_stream *xdr,
> + size_t nbytes)
> {
> static __be32 *p;
> int space_left;
> --
> 1.9.3
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
2014-07-16 20:58 ` Bruce Fields
@ 2014-07-16 21:40 ` Trond Myklebust
2014-07-16 21:44 ` Bruce Fields
0 siblings, 1 reply; 6+ messages in thread
From: Trond Myklebust @ 2014-07-16 21:40 UTC (permalink / raw)
To: Bruce Fields; +Cc: Linux NFS Mailing List
On Wed, Jul 16, 2014 at 1:58 PM, Bruce Fields <bfields@fieldses.org> wrote:
> On Sat, Jul 12, 2014 at 06:01:02PM -0400, Trond Myklebust wrote:
>> Quell another sparse warning.
>
> I think jlayton sent you the same fix with his set of sunrpc sparse
> fixes--could you just take that instead?
>
> (Or I can take this if you still want, I don't really care.)
I wasn't planning on carrying either patch in my tree. I've offloaded
the merge job to you... :-)
IOW: feel free to take Jeff's patch if that is more convenient.
Cheers,
Trond
--
Trond Myklebust
Linux NFS client maintainer, PrimaryData
trond.myklebust@primarydata.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
2014-07-16 21:40 ` Trond Myklebust
@ 2014-07-16 21:44 ` Bruce Fields
2014-07-17 18:41 ` Bruce Fields
0 siblings, 1 reply; 6+ messages in thread
From: Bruce Fields @ 2014-07-16 21:44 UTC (permalink / raw)
To: Trond Myklebust; +Cc: Linux NFS Mailing List
On Wed, Jul 16, 2014 at 02:40:43PM -0700, Trond Myklebust wrote:
> On Wed, Jul 16, 2014 at 1:58 PM, Bruce Fields <bfields@fieldses.org> wrote:
> > On Sat, Jul 12, 2014 at 06:01:02PM -0400, Trond Myklebust wrote:
> >> Quell another sparse warning.
> >
> > I think jlayton sent you the same fix with his set of sunrpc sparse
> > fixes--could you just take that instead?
> >
> > (Or I can take this if you still want, I don't really care.)
>
> I wasn't planning on carrying either patch in my tree. I've offloaded
> the merge job to you... :-)
> IOW: feel free to take Jeff's patch if that is more convenient.
OK, are you taking the rest of that series or should I get that too?
--b.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static
2014-07-16 21:44 ` Bruce Fields
@ 2014-07-17 18:41 ` Bruce Fields
0 siblings, 0 replies; 6+ messages in thread
From: Bruce Fields @ 2014-07-17 18:41 UTC (permalink / raw)
To: Trond Myklebust; +Cc: Linux NFS Mailing List
On Wed, Jul 16, 2014 at 05:44:55PM -0400, Bruce Fields wrote:
> On Wed, Jul 16, 2014 at 02:40:43PM -0700, Trond Myklebust wrote:
> > On Wed, Jul 16, 2014 at 1:58 PM, Bruce Fields <bfields@fieldses.org> wrote:
> > > On Sat, Jul 12, 2014 at 06:01:02PM -0400, Trond Myklebust wrote:
> > >> Quell another sparse warning.
> > >
> > > I think jlayton sent you the same fix with his set of sunrpc sparse
> > > fixes--could you just take that instead?
> > >
> > > (Or I can take this if you still want, I don't really care.)
> >
> > I wasn't planning on carrying either patch in my tree. I've offloaded
> > the merge job to you... :-)
> > IOW: feel free to take Jeff's patch if that is more convenient.
>
> OK, are you taking the rest of that series or should I get that too?
(Applying the one patch, assuming the resti s yours.)
--b.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-17 18:41 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-12 22:01 [PATCH] SUNRPC: xdr_get_next_encode_buffer should be declared static Trond Myklebust
2014-07-15 17:01 ` Christoph Hellwig
2014-07-16 20:58 ` Bruce Fields
2014-07-16 21:40 ` Trond Myklebust
2014-07-16 21:44 ` Bruce Fields
2014-07-17 18:41 ` Bruce Fields
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.