SQUASHME: pnfsd-lexp: Fix compiler warning when CONFIG_SUNRPC_DEBUG is not set
Subsystems (2)
PATCH 1/1 SQUASHME: pnfsd-lexp: Fix compiler warning when CONFIG_SUNRPC_DEBUG is not set
Fix "Unused variable" warning if CONFIG_SUNRPC_DEBUG is not set
---
git diff --stat -p -M fs/nfsd/nfs4pnfsds.c
fs/nfsd/nfs4pnfsds.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
fs/nfsd/nfs4pnfsds.c +1 -2
@@ -170,10 +170,9 @@ static int stateid_hashval(stateid_t *stidp) hashval =stateid_hashval (stidp);list_for_each_entry (local, &ds_stid_hashtbl[hashval], ds_hash)if (!memcmp (&local->ds_stid.si_opaque, &stidp->si_opaque,sizeof (stidp->si_opaque))) { -stateid_t *sid = &local->ds_stid;dprintk ("NFSD: %s <-- %p ds_flags %lx " STATEID_FMT "\n" , __func__, local, local->ds_flags, -STATEID_VAL (sid)); +STATEID_VAL (&local->ds_stid));return local; }return NULL ;
Benny Halevy
AppliedQuoted 3 lines · Boaz Harrosh
Fix "Unused variable" warning if
CONFIG_SUNRPC_DEBUGis not set
Merged, thanks!
Quoted 26 lines
Signed-off-by Boaz Harrosh <[email protected]>---
git diff --stat -p -Mfs/nfsd/nfs4pnfsds.cfs/nfsd/nfs4pnfsds.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)fs/nfsd/nfs4pnfsds.c +1
-3@@ -170,10 +170,9 @@ static int stateid_hashval(stateid_t *stidp) hashval =stateid_hashval (stidp);list_for_each_entry (local, &ds_stid_hashtbl[hashval], ds_hash)if (!memcmp (&local->ds_stid.si_opaque, &stidp->si_opaque,sizeof (stidp->si_opaque))) { -stateid_t *sid = &local->ds_stid;dprintk ("NFSD: %s <-- %p ds_flags %lx " STATEID_FMT "\n" , __func__, local, local->ds_flags, -STATEID_VAL (sid)); +STATEID_VAL (&local->ds_stid));return local; }return NULL ; --To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Quoted 7 lines · Benny Halevy
Quoted 3 lines · Boaz Harrosh
Fix "Unused variable" warning if
CONFIG_SUNRPC_DEBUGis not setMerged, thanks!
Not that I'm at the stage where I don't configure CONFIG_SUNRPC_DEBUG=y It was recently changed and my oldconfig base removed it.
Actually I got lucky, because I would have run without dprints and would have pulled my hair what's going on with my setup.
I wish it would stay to protect me for a while ;-)
Thanks
Boaz