| Message ID | 1508256866-12798-1-git-send-email-bhumirks@gmail.com |
|---|---|
| Headers | show |
| Series | make function arg and structures as const | expand |
On Tue, 2017-10-17 at 18:14 +0200, Bhumika Goyal wrote: > Make the function argument as const. After thing change, make > the cache_detail structures as const. > > Bhumika Goyal (4): > sunrpc: make the function arg as const > NFS: make cache_detail structures const > NFSD: make cache_detail structures const > SUNRPC: make cache_detail structures const > > fs/nfs/dns_resolve.c | 2 +- > fs/nfsd/export.c | 4 ++-- > fs/nfsd/nfs4idmap.c | 4 ++-- > include/linux/sunrpc/cache.h | 2 +- > net/sunrpc/auth_gss/svcauth_gss.c | 4 ++-- > net/sunrpc/cache.c | 2 +- > net/sunrpc/svcauth_unix.c | 4 ++-- > 7 files changed, 11 insertions(+), 11 deletions(-) > Looks pretty straightforward. You can add this to the set: Reviewed-by: Jeff Layton <jlayton@redhat.com>
On Tue, Oct 17, 2017 at 12:40:27PM -0400, Jeff Layton wrote: > On Tue, 2017-10-17 at 18:14 +0200, Bhumika Goyal wrote: > > Make the function argument as const. After thing change, make > > the cache_detail structures as const. > > > > Bhumika Goyal (4): > > sunrpc: make the function arg as const > > NFS: make cache_detail structures const > > NFSD: make cache_detail structures const > > SUNRPC: make cache_detail structures const > > > > fs/nfs/dns_resolve.c | 2 +- > > fs/nfsd/export.c | 4 ++-- > > fs/nfsd/nfs4idmap.c | 4 ++-- > > include/linux/sunrpc/cache.h | 2 +- > > net/sunrpc/auth_gss/svcauth_gss.c | 4 ++-- > > net/sunrpc/cache.c | 2 +- > > net/sunrpc/svcauth_unix.c | 4 ++-- > > 7 files changed, 11 insertions(+), 11 deletions(-) > > > > Looks pretty straightforward. You can add this to the set: > > Reviewed-by: Jeff Layton <jlayton@redhat.com> Thanks, I've applied 1, 3, and 4 and could take #2 as well if it's OK with Trond/Anna. --b.
On 11/09/2017 09:21 PM, J. Bruce Fields wrote: > On Tue, Oct 17, 2017 at 12:40:27PM -0400, Jeff Layton wrote: >> On Tue, 2017-10-17 at 18:14 +0200, Bhumika Goyal wrote: >>> Make the function argument as const. After thing change, make >>> the cache_detail structures as const. >>> >>> Bhumika Goyal (4): >>> sunrpc: make the function arg as const >>> NFS: make cache_detail structures const >>> NFSD: make cache_detail structures const >>> SUNRPC: make cache_detail structures const >>> >>> fs/nfs/dns_resolve.c | 2 +- >>> fs/nfsd/export.c | 4 ++-- >>> fs/nfsd/nfs4idmap.c | 4 ++-- >>> include/linux/sunrpc/cache.h | 2 +- >>> net/sunrpc/auth_gss/svcauth_gss.c | 4 ++-- >>> net/sunrpc/cache.c | 2 +- >>> net/sunrpc/svcauth_unix.c | 4 ++-- >>> 7 files changed, 11 insertions(+), 11 deletions(-) >>> >> >> Looks pretty straightforward. You can add this to the set: >> >> Reviewed-by: Jeff Layton <jlayton@redhat.com> > > Thanks, I've applied 1, 3, and 4 and could take #2 as well if it's OK > with Trond/Anna. I don't mind taking #2, it's already in my branch :) Anna > > --b. >
On Fri, Nov 10, 2017 at 10:09:46AM -0500, Anna Schumaker wrote: > > > On 11/09/2017 09:21 PM, J. Bruce Fields wrote: > > On Tue, Oct 17, 2017 at 12:40:27PM -0400, Jeff Layton wrote: > >> On Tue, 2017-10-17 at 18:14 +0200, Bhumika Goyal wrote: > >>> Make the function argument as const. After thing change, make > >>> the cache_detail structures as const. > >>> > >>> Bhumika Goyal (4): > >>> sunrpc: make the function arg as const > >>> NFS: make cache_detail structures const > >>> NFSD: make cache_detail structures const > >>> SUNRPC: make cache_detail structures const > >>> > >>> fs/nfs/dns_resolve.c | 2 +- > >>> fs/nfsd/export.c | 4 ++-- > >>> fs/nfsd/nfs4idmap.c | 4 ++-- > >>> include/linux/sunrpc/cache.h | 2 +- > >>> net/sunrpc/auth_gss/svcauth_gss.c | 4 ++-- > >>> net/sunrpc/cache.c | 2 +- > >>> net/sunrpc/svcauth_unix.c | 4 ++-- > >>> 7 files changed, 11 insertions(+), 11 deletions(-) > >>> > >> > >> Looks pretty straightforward. You can add this to the set: > >> > >> Reviewed-by: Jeff Layton <jlayton@redhat.com> > > > > Thanks, I've applied 1, 3, and 4 and could take #2 as well if it's OK > > with Trond/Anna. > > I don't mind taking #2, it's already in my branch :) OK, thanks.--b.