xfs: simplify xfs_map_at_offset

Move the buffer locking into the callers as they need to do it
wether they call xfs_map_at_offset or not.  Remove the b_bdev
assignment, which is already done by get_blocks.  Remove the
duplicate extent type asserts in xfs_convert_page just before
calling xfs_map_at_offset.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>

1 files changed, 4 insertions, 11 deletions

diff --git a/fs/xfs/linux-2.6/xfs_aops.c b/fs/xfs/linux-2.6/xfs_aops.c
index 4d982dc8b862..ec7bbb5645b6 100644
--- a/fs/xfs/linux-2.6/xfs_aops.c
+++ b/fs/xfs/linux-2.6/xfs_aops.c
@@ -626,9 +626,7 @@ xfs_map_at_offset(
         ASSERT(imap->br_startblock != HOLESTARTBLOCK);
         ASSERT(imap->br_startblock != DELAYSTARTBLOCK);
 
-        lock_buffer(bh);
         xfs_map_buffer(inode, bh, imap, offset);
-        bh->b_bdev = xfs_find_bdev_for_inode(inode);
         set_buffer_mapped(bh);
         clear_buffer_delay(bh);
         clear_buffer_unwritten(bh);
@@ -751,12 +749,8 @@ xfs_convert_page(
                                 continue;
                         }
 
-                        ASSERT(imap->br_startblock != HOLESTARTBLOCK);
-                        ASSERT(imap->br_startblock != DELAYSTARTBLOCK);
-
-                        if (type == IO_OVERWRITE)
-                                lock_buffer(bh);
-                        else
+                        lock_buffer(bh);
+                        if (type != IO_OVERWRITE)
                                 xfs_map_at_offset(inode, bh, imap, offset);
                         xfs_add_to_ioend(inode, bh, offset, type,
                                          ioendp, done);
@@ -1041,9 +1035,8 @@ xfs_vm_writepage(
                         imap_valid = xfs_imap_valid(inode, &imap, offset);
                 }
                 if (imap_valid) {
-                        if (type == IO_OVERWRITE)
-                                lock_buffer(bh);
-                        else
+                        lock_buffer(bh);
+                        if (type != IO_OVERWRITE)
                                 xfs_map_at_offset(inode, bh, &imap, offset);
                         xfs_add_to_ioend(inode, bh, offset, type, &ioend,
                                          new_ioend);