Return-Path: Received: from mx141.netapp.com ([216.240.21.12]:51455 "EHLO mx141.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751455AbdKJPKf (ORCPT ); Fri, 10 Nov 2017 10:10:35 -0500 Subject: Re: [PATCH 0/4] make function arg and structures as const To: "J. Bruce Fields" , Jeff Layton CC: Bhumika Goyal , , , , , , References: <1508256866-12798-1-git-send-email-bhumirks@gmail.com> <1508258427.4747.7.camel@redhat.com> <20171110022125.GA13874@fieldses.org> From: Anna Schumaker Message-ID: <7c61a3c8-4042-7b05-1298-e3abbef916c2@Netapp.com> Date: Fri, 10 Nov 2017 10:09:46 -0500 MIME-Version: 1.0 In-Reply-To: <20171110022125.GA13874@fieldses.org> Content-Type: text/plain; charset=utf-8 Sender: linux-nfs-owner@vger.kernel.org List-ID: 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 > > 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. >