On Fri, Jun 15, 2007 at 12:16:28AM +0200, Adrian Bunk wrote:
> This patch contains the following cleanups:
> - remove the following no longer used functions:
>   - bitmap.c: reiserfs_claim_blocks_to_be_allocated()
>   - bitmap.c: reiserfs_release_claimed_blocks()
>   - bitmap.c: reiserfs_can_fit_pages()
> - make the following functions static:
>   - inode.c: restart_transaction()
>   - journal.c: reiserfs_async_progress_wait()

If Vladimir acks this one then it's obviously fine by me.

 
> Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
> 
> ---
> 
>  fs/reiserfs/bitmap.c        |   57 ------------------------------------
>  fs/reiserfs/inode.c         |    4 +-
>  fs/reiserfs/journal.c       |    3 +
>  include/linux/reiserfs_fs.h |    7 ----
>  4 files changed, 4 insertions(+), 67 deletions(-)
> 
> --- linux-2.6.22-rc4-mm2/include/linux/reiserfs_fs.h.old      2007-06-14 
> 02:08:17.000000000 +0200
> +++ linux-2.6.22-rc4-mm2/include/linux/reiserfs_fs.h  2007-06-14 
> 02:11:20.000000000 +0200
> @@ -1701,8 +1701,6 @@
>       return th->t_blocks_allocated - th->t_blocks_logged;
>  }
>  
> -int reiserfs_async_progress_wait(struct super_block *s);
> -
>  struct reiserfs_transaction_handle *reiserfs_persistent_transaction(struct
>                                                                   super_block
>                                                                   *,
> @@ -1857,8 +1855,6 @@
>  #define GET_BLOCK_NO_IMUX     8      /* i_mutex is not held, don't 
> preallocate */
>  #define GET_BLOCK_NO_DANGLE   16     /* don't leave any transactions running 
> */
>  
> -int restart_transaction(struct reiserfs_transaction_handle *th,
> -                     struct inode *inode, struct treepath *path);
>  void reiserfs_read_locked_inode(struct inode *inode,
>                               struct reiserfs_iget_args *args);
>  int reiserfs_find_actor(struct inode *inode, void *p);
> @@ -2135,9 +2131,6 @@
>                              struct inode *inode);
>  void reiserfs_discard_all_prealloc(struct reiserfs_transaction_handle *th);
>  #endif
> -void reiserfs_claim_blocks_to_be_allocated(struct super_block *sb, int 
> blocks);
> -void reiserfs_release_claimed_blocks(struct super_block *sb, int blocks);
> -int reiserfs_can_fit_pages(struct super_block *sb);
>  
>  /* hashes.c */
>  __u32 keyed_hash(const signed char *msg, int len);
> --- linux-2.6.22-rc4-mm2/fs/reiserfs/bitmap.c.old     2007-06-14 
> 02:08:45.000000000 +0200
> +++ linux-2.6.22-rc4-mm2/fs/reiserfs/bitmap.c 2007-06-14 02:09:59.000000000 
> +0200
> @@ -1201,63 +1201,6 @@
>       return ret;
>  }
>  
> -/* These 2 functions are here to provide blocks reservation to the rest of 
> kernel */
> -/* Reserve @blocks amount of blocks in fs pointed by @sb. Caller must make 
> sure
> -   there are actually this much blocks on the FS available */
> -void reiserfs_claim_blocks_to_be_allocated(struct super_block *sb,   /* 
> super block of
> -                                                                        
> filesystem where
> -                                                                        
> blocks should be
> -                                                                        
> reserved */
> -                                        int blocks   /* How much to reserve 
> */
> -    )
> -{
> -
> -     /* Fast case, if reservation is zero - exit immediately. */
> -     if (!blocks)
> -             return;
> -
> -     spin_lock(&REISERFS_SB(sb)->bitmap_lock);
> -     REISERFS_SB(sb)->reserved_blocks += blocks;
> -     spin_unlock(&REISERFS_SB(sb)->bitmap_lock);
> -}
> -
> -/* Unreserve @blocks amount of blocks in fs pointed by @sb */
> -void reiserfs_release_claimed_blocks(struct super_block *sb, /* super block 
> of
> -                                                                filesystem 
> where
> -                                                                blocks 
> should be
> -                                                                reserved */
> -                                  int blocks /* How much to unreserve */
> -    )
> -{
> -
> -     /* Fast case, if unreservation is zero - exit immediately. */
> -     if (!blocks)
> -             return;
> -
> -     spin_lock(&REISERFS_SB(sb)->bitmap_lock);
> -     REISERFS_SB(sb)->reserved_blocks -= blocks;
> -     spin_unlock(&REISERFS_SB(sb)->bitmap_lock);
> -     RFALSE(REISERFS_SB(sb)->reserved_blocks < 0,
> -            "amount of blocks reserved became zero?");
> -}
> -
> -/* This function estimates how much pages we will be able to write to FS
> -   used for reiserfs_file_write() purposes for now. */
> -int reiserfs_can_fit_pages(struct super_block *sb    /* superblock of 
> filesystem
> -                                                        to estimate space */ 
> )
> -{
> -     int space;
> -
> -     spin_lock(&REISERFS_SB(sb)->bitmap_lock);
> -     space =
> -         (SB_FREE_BLOCKS(sb) -
> -          REISERFS_SB(sb)->reserved_blocks) >> (PAGE_CACHE_SHIFT -
> -                                                sb->s_blocksize_bits);
> -     spin_unlock(&REISERFS_SB(sb)->bitmap_lock);
> -
> -     return space > 0 ? space : 0;
> -}
> -
>  void reiserfs_cache_bitmap_metadata(struct super_block *sb,
>                                      struct buffer_head *bh,
>                                      struct reiserfs_bitmap_info *info)
> --- linux-2.6.22-rc4-mm2/fs/reiserfs/inode.c.old      2007-06-14 
> 02:10:35.000000000 +0200
> +++ linux-2.6.22-rc4-mm2/fs/reiserfs/inode.c  2007-06-14 02:10:59.000000000 
> +0200
> @@ -208,8 +208,8 @@
>       return 0;
>  }
>  
> -/*static*/ int restart_transaction(struct reiserfs_transaction_handle *th,
> -                                struct inode *inode, struct treepath *path)
> +static int restart_transaction(struct reiserfs_transaction_handle *th,
> +                            struct inode *inode, struct treepath *path)
>  {
>       struct super_block *s = th->t_super;
>       int len = th->t_blocks_allocated;
> --- linux-2.6.22-rc4-mm2/fs/reiserfs/journal.c.old    2007-06-14 
> 02:11:29.000000000 +0200
> +++ linux-2.6.22-rc4-mm2/fs/reiserfs/journal.c        2007-06-14 
> 02:11:40.000000000 +0200
> @@ -966,7 +966,8 @@
>       }
>       return 0;
>  }
> -int reiserfs_async_progress_wait(struct super_block *s)
> +
> +static int reiserfs_async_progress_wait(struct super_block *s)
>  {
>       DEFINE_WAIT(wait);
>       struct reiserfs_journal *j = SB_JOURNAL(s);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to