NFS: Fix O_DIRECT compile warnings
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 4 May 2012 17:47:16 +0000 (13:47 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 4 May 2012 18:08:09 +0000 (14:08 -0400)
Fix the following compile warnings:
fs/nfs/direct.c: In function 'nfs_direct_read_schedule_segment':
fs/nfs/direct.c:325:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:325:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:325:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:352:27: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c: In function 'nfs_direct_write_schedule_segment':
fs/nfs/direct.c:622:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:622:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:622:11: warning: comparison of distinct pointer types
lacks a cast [enabled by default]
fs/nfs/direct.c:650:27: warning: comparison of distinct pointer types
lacks a cast [enabled by default]

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Fred Isaman <iisaman@netapp.com>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
fs/nfs/direct.c

index aab3016c437ce23e151353ade5493cdd00f2218e..dca9c81430095c6c6ceec588417c4829991cb968 100644 (file)
@@ -322,7 +322,7 @@ static ssize_t nfs_direct_read_schedule_segment(struct nfs_pageio_descriptor *de
                int i;
 
                pgbase = user_addr & ~PAGE_MASK;
-               bytes = min(max(rsize, PAGE_SIZE), count);
+               bytes = min(max_t(size_t, rsize, PAGE_SIZE), count);
 
                result = -ENOMEM;
                npages = nfs_page_array_len(pgbase, bytes);
@@ -349,7 +349,7 @@ static ssize_t nfs_direct_read_schedule_segment(struct nfs_pageio_descriptor *de
 
                for (i = 0; i < npages; i++) {
                        struct nfs_page *req;
-                       unsigned int req_len = min(bytes, PAGE_SIZE - pgbase);
+                       unsigned int req_len = min_t(size_t, bytes, PAGE_SIZE - pgbase);
                        /* XXX do we need to do the eof zeroing found in async_filler? */
                        req = nfs_create_request(dreq->ctx, dreq->inode,
                                                 pagevec[i],
@@ -619,7 +619,7 @@ static ssize_t nfs_direct_write_schedule_segment(struct nfs_pageio_descriptor *d
                int i;
 
                pgbase = user_addr & ~PAGE_MASK;
-               bytes = min(max(wsize, PAGE_SIZE), count);
+               bytes = min(max_t(size_t, wsize, PAGE_SIZE), count);
 
                result = -ENOMEM;
                npages = nfs_page_array_len(pgbase, bytes);
@@ -647,7 +647,7 @@ static ssize_t nfs_direct_write_schedule_segment(struct nfs_pageio_descriptor *d
 
                for (i = 0; i < npages; i++) {
                        struct nfs_page *req;
-                       unsigned int req_len = min(bytes, PAGE_SIZE - pgbase);
+                       unsigned int req_len = min_t(size_t, bytes, PAGE_SIZE - pgbase);
 
                        req = nfs_create_request(dreq->ctx, dreq->inode,
                                                 pagevec[i],