Auto merge with /home/aegl/GIT/linus

3 files changed, 50 insertions, 46 deletions

diff --git a/fs/ext2/xip.c b/fs/ext2/xip.c
index d44431d1a338..0aa5ac159c09 100644
--- a/fs/ext2/xip.c
+++ b/fs/ext2/xip.c
@@ -15,66 +15,79 @@
 #include "xip.h"
 
 static inline int
-__inode_direct_access(struct inode *inode, sector_t sector, unsigned long *data) {
+__inode_direct_access(struct inode *inode, sector_t sector,
+                      unsigned long *data)
+{
         BUG_ON(!inode->i_sb->s_bdev->bd_disk->fops->direct_access);
         return inode->i_sb->s_bdev->bd_disk->fops
                 ->direct_access(inode->i_sb->s_bdev,sector,data);
 }
 
+static inline int
+__ext2_get_sector(struct inode *inode, sector_t offset, int create,
+                   sector_t *result)
+{
+        struct buffer_head tmp;
+        int rc;
+
+        memset(&tmp, 0, sizeof(struct buffer_head));
+        rc = ext2_get_block(inode, offset/ (PAGE_SIZE/512), &tmp,
+                            create);
+        *result = tmp.b_blocknr;
+
+        /* did we get a sparse block (hole in the file)? */
+        if (!(*result)) {
+                BUG_ON(create);
+                rc = -ENODATA;
+        }
+
+        return rc;
+}
+
 int
-ext2_clear_xip_target(struct inode *inode, int block) {
-        sector_t sector = block*(PAGE_SIZE/512);
+ext2_clear_xip_target(struct inode *inode, int block)
+{
+        sector_t sector = block * (PAGE_SIZE/512);
         unsigned long data;
         int rc;
 
         rc = __inode_direct_access(inode, sector, &data);
-        if (rc)
-                return rc;
-        clear_page((void*)data);
-        return 0;
+        if (!rc)
+                clear_page((void*)data);
+        return rc;
 }
 
 void ext2_xip_verify_sb(struct super_block *sb)
 {
         struct ext2_sb_info *sbi = EXT2_SB(sb);
 
-        if ((sbi->s_mount_opt & EXT2_MOUNT_XIP)) {
-                if ((sb->s_bdev == NULL) ||
-                        sb->s_bdev->bd_disk == NULL ||
-                        sb->s_bdev->bd_disk->fops == NULL ||
-                        sb->s_bdev->bd_disk->fops->direct_access == NULL) {
-                        sbi->s_mount_opt &= (~EXT2_MOUNT_XIP);
-                        ext2_warning(sb, __FUNCTION__,
-                                "ignoring xip option - not supported by bdev");
-                }
+        if ((sbi->s_mount_opt & EXT2_MOUNT_XIP) &&
+            !sb->s_bdev->bd_disk->fops->direct_access) {
+                sbi->s_mount_opt &= (~EXT2_MOUNT_XIP);
+                ext2_warning(sb, __FUNCTION__,
+                             "ignoring xip option - not supported by bdev");
         }
 }
 
-struct page*
-ext2_get_xip_page(struct address_space *mapping, sector_t blockno,
+struct page *
+ext2_get_xip_page(struct address_space *mapping, sector_t offset,
                    int create)
 {
         int rc;
         unsigned long data;
-        struct buffer_head tmp;
+        sector_t sector;
 
-        tmp.b_state = 0;
-        tmp.b_blocknr = 0;
-        rc = ext2_get_block(mapping->host, blockno/(PAGE_SIZE/512) , &tmp,
-                                create);
+        /* first, retrieve the sector number */
+        rc = __ext2_get_sector(mapping->host, offset, create, &sector);
         if (rc)
-                return ERR_PTR(rc);
-        if (tmp.b_blocknr == 0) {
-                /* SPARSE block */
-                BUG_ON(create);
-                return ERR_PTR(-ENODATA);
-        }
+                goto error;
 
+        /* retrieve address of the target data */
         rc = __inode_direct_access
-                (mapping->host,tmp.b_blocknr*(PAGE_SIZE/512) ,&data);
-        if (rc)
-                return ERR_PTR(rc);
+                (mapping->host, sector * (PAGE_SIZE/512), &data);
+        if (!rc)
+                return virt_to_page(data);
 
-        SetPageUptodate(virt_to_page(data));
-        return virt_to_page(data);
+ error:
+        return ERR_PTR(rc);
 }
diff --git a/fs/hostfs/hostfs_kern.c b/fs/hostfs/hostfs_kern.c
index 4bf43ea87c46..88e68caa3784 100644
--- a/fs/hostfs/hostfs_kern.c
+++ b/fs/hostfs/hostfs_kern.c
@@ -15,7 +15,6 @@
 #include <linux/pagemap.h>
 #include <linux/blkdev.h>
 #include <linux/list.h>
-#include <linux/root_dev.h>
 #include <linux/statfs.h>
 #include <linux/kdev_t.h>
 #include <asm/uaccess.h>
@@ -160,8 +159,6 @@ static int read_name(struct inode *ino, char *name)
         ino->i_size = i_size;
         ino->i_blksize = i_blksize;
         ino->i_blocks = i_blocks;
-        if((ino->i_sb->s_dev == ROOT_DEV) && (ino->i_uid == getuid()))
-                ino->i_uid = 0;
         return(0);
 }
 
@@ -841,16 +838,10 @@ int hostfs_setattr(struct dentry *dentry, struct iattr *attr)
                 attrs.ia_mode = attr->ia_mode;
         }
         if(attr->ia_valid & ATTR_UID){
-                if((dentry->d_inode->i_sb->s_dev == ROOT_DEV) &&
-                   (attr->ia_uid == 0))
-                        attr->ia_uid = getuid();
                 attrs.ia_valid |= HOSTFS_ATTR_UID;
                 attrs.ia_uid = attr->ia_uid;
         }
         if(attr->ia_valid & ATTR_GID){
-                if((dentry->d_inode->i_sb->s_dev == ROOT_DEV) &&
-                   (attr->ia_gid == 0))
-                        attr->ia_gid = getgid();
                 attrs.ia_valid |= HOSTFS_ATTR_GID;
                 attrs.ia_gid = attr->ia_gid;
         }
diff --git a/fs/hppfs/hppfs_kern.c b/fs/hppfs/hppfs_kern.c
index 6f553e17c375..ff150fedb981 100644
--- a/fs/hppfs/hppfs_kern.c
+++ b/fs/hppfs/hppfs_kern.c
@@ -233,7 +233,7 @@ static ssize_t read_proc(struct file *file, char *buf, ssize_t count,
                 set_fs(USER_DS);
 
         if(ppos) *ppos = file->f_pos;
-        return(n);
+        return n;
 }
 
 static ssize_t hppfs_read_file(int fd, char *buf, ssize_t count)
@@ -254,7 +254,7 @@ static ssize_t hppfs_read_file(int fd, char *buf, ssize_t count)
                 err = os_read_file(fd, new_buf, cur);
                 if(err < 0){
                         printk("hppfs_read : read failed, errno = %d\n",
-                               count);
+                               err);
                         n = err;
                         goto out_free;
                 }
@@ -271,7 +271,7 @@ static ssize_t hppfs_read_file(int fd, char *buf, ssize_t count)
  out_free:
         kfree(new_buf);
  out:
-        return(n);
+        return n;
 }
 
 static ssize_t hppfs_read(struct file *file, char *buf, size_t count,