diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2010-05-07 13:34:08 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-05-14 15:09:31 -0400 |
| commit | dfe52c0419b8324bacd69bd28aae77e2d6ee0379 (patch) | |
| tree | f9e099617bd44fd7f63d56bbc86adf30d422158b | |
| parent | a6d5ff64bae02438d914f088672cab1916153954 (diff) | |
NFS: Squelch compiler warning in nfs_add_server_stats()
Clean up:
fs/nfs/iostat.h: In function ‘nfs_add_server_stats’:
fs/nfs/iostat.h:41: warning: comparison between signed and unsigned integer expressions
fs/nfs/iostat.h:41: warning: comparison between signed and unsigned integer expressions
fs/nfs/iostat.h:41: warning: comparison between signed and unsigned integer expressions
fs/nfs/iostat.h:41: warning: comparison between signed and unsigned integer expressions
Commit fce22848 replaced the open-coded per-cpu logic in several
functions in fs/nfs/iostat.h with a single invocation of
this_cpu_ptr(). This macro assumes its second argument is signed,
not unsigned.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
| -rw-r--r-- | fs/nfs/iostat.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/nfs/iostat.h b/fs/nfs/iostat.h index 1d8d5c813b01..c5832487c456 100644 --- a/fs/nfs/iostat.h +++ b/fs/nfs/iostat.h | |||
| @@ -36,14 +36,14 @@ static inline void nfs_inc_stats(const struct inode *inode, | |||
| 36 | 36 | ||
| 37 | static inline void nfs_add_server_stats(const struct nfs_server *server, | 37 | static inline void nfs_add_server_stats(const struct nfs_server *server, |
| 38 | enum nfs_stat_bytecounters stat, | 38 | enum nfs_stat_bytecounters stat, |
| 39 | unsigned long addend) | 39 | long addend) |
| 40 | { | 40 | { |
| 41 | this_cpu_add(server->io_stats->bytes[stat], addend); | 41 | this_cpu_add(server->io_stats->bytes[stat], addend); |
| 42 | } | 42 | } |
| 43 | 43 | ||
| 44 | static inline void nfs_add_stats(const struct inode *inode, | 44 | static inline void nfs_add_stats(const struct inode *inode, |
| 45 | enum nfs_stat_bytecounters stat, | 45 | enum nfs_stat_bytecounters stat, |
| 46 | unsigned long addend) | 46 | long addend) |
| 47 | { | 47 | { |
| 48 | nfs_add_server_stats(NFS_SERVER(inode), stat, addend); | 48 | nfs_add_server_stats(NFS_SERVER(inode), stat, addend); |
| 49 | } | 49 | } |
| @@ -51,7 +51,7 @@ static inline void nfs_add_stats(const struct inode *inode, | |||
| 51 | #ifdef CONFIG_NFS_FSCACHE | 51 | #ifdef CONFIG_NFS_FSCACHE |
| 52 | static inline void nfs_add_fscache_stats(struct inode *inode, | 52 | static inline void nfs_add_fscache_stats(struct inode *inode, |
| 53 | enum nfs_stat_fscachecounters stat, | 53 | enum nfs_stat_fscachecounters stat, |
| 54 | unsigned long addend) | 54 | long addend) |
| 55 | { | 55 | { |
| 56 | this_cpu_add(NFS_SERVER(inode)->io_stats->fscache[stat], addend); | 56 | this_cpu_add(NFS_SERVER(inode)->io_stats->fscache[stat], addend); |
| 57 | } | 57 | } |
