Hi.

I have changed the interface to mark_buffer_dirty (as per your
suggestion to Arnaldo Carvalho de Melo). This impacts bfs as per 
the following patch.

diff -u --recursive -X misc/dontdiff linux-240test8-pre2/fs/bfs/dir.c 
linux/fs/bfs/dir.c
--- linux-240test8-pre2/fs/bfs/dir.c    Thu Aug 24 09:43:34 2000
+++ linux/fs/bfs/dir.c  Mon Sep  4 20:46:23 2000
@@ -177,7 +177,7 @@
        }
        de->ino = 0;
        dir->i_version = ++event;
-       mark_buffer_dirty(bh, 0);
+       mark_buffer_dirty(bh);
        dir->i_ctime = dir->i_mtime = CURRENT_TIME;
        mark_inode_dirty(dir);
        inode->i_nlink--;
@@ -236,7 +236,7 @@
                new_inode->i_ctime = CURRENT_TIME;
                mark_inode_dirty(new_inode);
        }
-       mark_buffer_dirty(old_bh, 0);
+       mark_buffer_dirty(old_bh);
        error = 0;
 
 end_rename:
@@ -288,7 +288,7 @@
                                de->ino = ino;
                                for (i=0; i<BFS_NAMELEN; i++)
                                        de->name[i] = (i < namelen) ? name[i] : 0;
-                               mark_buffer_dirty(bh, 0);
+                               mark_buffer_dirty(bh);
                                brelse(bh);
                                return 0;
                        }
diff -u --recursive -X misc/dontdiff linux-240test8-pre2/fs/bfs/file.c 
linux/fs/bfs/file.c
--- linux-240test8-pre2/fs/bfs/file.c   Mon May  8 22:51:01 2000
+++ linux/fs/bfs/file.c Mon Sep  4 20:46:32 2000
@@ -33,7 +33,7 @@
                return -EIO;
        new = getblk(dev, to, BFS_BSIZE);
        memcpy(new->b_data, bh->b_data, bh->b_size);
-       mark_buffer_dirty(new, 0);
+       mark_buffer_dirty(new);
        bforget(bh);
        brelse(new);
        return 0;
@@ -98,7 +98,7 @@
                bh_result->b_state |= (1UL << BH_Mapped);
                s->su_lf_eblk = inode->iu_eblock = inode->iu_sblock + block;
                mark_inode_dirty(inode);
-               mark_buffer_dirty(s->su_sbh, 1);
+               mark_buffer_dirty(s->su_sbh);
                err = 0;
                goto out;
        }
@@ -118,7 +118,7 @@
        inode->iu_sblock = next_free_block;
        s->su_lf_eblk = inode->iu_eblock = next_free_block + block;
        mark_inode_dirty(inode);
-       mark_buffer_dirty(s->su_sbh, 1);
+       mark_buffer_dirty(s->su_sbh);
        bh_result->b_dev = inode->i_dev;
        bh_result->b_blocknr = inode->iu_sblock + block;
        bh_result->b_state |= (1UL << BH_Mapped);
diff -u --recursive -X misc/dontdiff linux-240test8-pre2/fs/bfs/inode.c 
linux/fs/bfs/inode.c
--- linux-240test8-pre2/fs/bfs/inode.c  Mon Jul 31 21:04:05 2000
+++ linux/fs/bfs/inode.c        Mon Sep  4 20:46:46 2000
@@ -127,7 +127,7 @@
        di->i_eblock = inode->iu_eblock;
        di->i_eoffset = di->i_sblock * BFS_BSIZE + inode->i_size - 1;
 
-       mark_buffer_dirty(bh, 0);
+       mark_buffer_dirty(bh);
        brelse(bh);
        unlock_kernel();
 }
@@ -169,7 +169,7 @@
        }
        di->i_ino = 0;
        di->i_sblock = 0;
-       mark_buffer_dirty(bh, 0);
+       mark_buffer_dirty(bh);
        brelse(bh);
 
        /* if this was the last file, make the previous 
@@ -177,7 +177,7 @@
           saves us 1 gap */
        if (s->su_lf_eblk == inode->iu_eblock) {
                s->su_lf_eblk = inode->iu_sblock - 1;
-               mark_buffer_dirty(s->su_sbh, 1);
+               mark_buffer_dirty(s->su_sbh);
        }
        unlock_kernel();
        clear_inode(inode);
@@ -205,7 +205,7 @@
 static void bfs_write_super(struct super_block *s)
 {
        if (!(s->s_flags & MS_RDONLY))
-               mark_buffer_dirty(s->su_sbh, 1);
+               mark_buffer_dirty(s->su_sbh);
        s->s_dirt = 0;
 }
 
@@ -314,7 +314,7 @@
                iput(inode);
        }
        if (!(s->s_flags & MS_RDONLY)) {
-               mark_buffer_dirty(bh, 1);
+               mark_buffer_dirty(bh);
                s->s_dirt = 1;
        } 
        dump_imap("read_super", s);


-- 
Regards,
        Rasmus([EMAIL PROTECTED])

That lowdown scoundrel deserves to be kicked to death by a jackass, and
I'm just the one to do it. -A congressional candidate in Texas
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
Please read the FAQ at http://www.tux.org/lkml/

Reply via email to