xfsprogs: more static function tidyups
diff mbox series

Message ID d0ba0699-12c8-8c42-3e69-c0e7976d579d@redhat.com
State New
Headers show
Series
  • xfsprogs: more static function tidyups
Related show

Commit Message

Eric Sandeen April 18, 2019, 8:01 p.m. UTC
Make many more functions static if they can be.  This is not all
of them; some are omitted for symmetry reasons, i.e. even if
btree_lookup_prev() is not used, I've left it available.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---

Comments

Darrick J. Wong April 22, 2019, 6:34 p.m. UTC | #1
On Thu, Apr 18, 2019 at 03:01:33PM -0500, Eric Sandeen wrote:
> Make many more functions static if they can be.  This is not all
> of them; some are omitted for symmetry reasons, i.e. even if
> btree_lookup_prev() is not used, I've left it available.
> 
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> ---
> 
> diff --git a/copy/xfs_copy.c b/copy/xfs_copy.c
> index a6d6703..54679d7 100644
> --- a/copy/xfs_copy.c
> +++ b/copy/xfs_copy.c
> @@ -48,9 +48,9 @@ static pthread_mutex_t	mainwait;
>  #define ACTIVE		1
>  #define INACTIVE	2
>  
> -xfs_off_t	write_log_trailer(int fd, wbuf *w, xfs_mount_t *mp);
> -xfs_off_t	write_log_header(int fd, wbuf *w, xfs_mount_t *mp);
> -static int	format_logs(struct xfs_mount *);
> +static xfs_off_t	write_log_trailer(int fd, wbuf *w, xfs_mount_t *mp);
> +static xfs_off_t	write_log_header(int fd, wbuf *w, xfs_mount_t *mp);
> +static int		format_logs(struct xfs_mount *);
>  
>  /* general purpose message reporting routine */
>  
> diff --git a/db/io.c b/db/io.c
> index 6b66472..bb16618 100644
> --- a/db/io.c
> +++ b/db/io.c
> @@ -51,10 +51,10 @@ static const cmdinfo_t  ring_cmd =
>  	{ "ring", NULL, ring_f, 0, 1, 0, NULL,
>  	  N_("show position ring or move to a specific entry"), ring_help };
>  
> -iocur_t	*iocur_base;
> -iocur_t	*iocur_top;
> -int	iocur_sp = -1;
> -int	iocur_len;
> +static iocur_t	*iocur_base;
> +iocur_t		*iocur_top;

iocur_t -> struct iocur?

> +int		iocur_sp = -1;
> +static int	iocur_len;
>  
>  #define RING_ENTRIES 20
>  static iocur_t iocur_ring[RING_ENTRIES];
> diff --git a/db/io.h b/db/io.h
> index c29a748..0e4f74a 100644
> --- a/db/io.h
> +++ b/db/io.h
> @@ -35,10 +35,8 @@ typedef struct iocur {
>  #define DB_RING_ADD 1                   /* add to ring on set_cur */
>  #define DB_RING_IGN 0                   /* do not add to ring on set_cur */
>  
> -extern iocur_t	*iocur_base;		/* base of stack */
>  extern iocur_t	*iocur_top;		/* top element of stack */
>  extern int	iocur_sp;		/* current top of stack */
> -extern int	iocur_len;		/* length of stack array */
>  
>  extern void	io_init(void);
>  extern void	off_cur(int off, int len);
> diff --git a/estimate/xfs_estimate.c b/estimate/xfs_estimate.c
> index 9e01cce..277b371 100644
> --- a/estimate/xfs_estimate.c
> +++ b/estimate/xfs_estimate.c
> @@ -33,7 +33,7 @@ cvtnum(char *s)
>  	return 0LL;
>  }
>  
> -int ffn(const char *, const struct stat *, int, struct FTW *);
> +static int ffn(const char *, const struct stat *, int, struct FTW *);
>  
>  #define BLOCKSIZE	4096
>  #define INODESIZE	256
> diff --git a/fsr/xfs_fsr.c b/fsr/xfs_fsr.c
> index fef6262..55711bf 100644
> --- a/fsr/xfs_fsr.c
> +++ b/fsr/xfs_fsr.c
> @@ -61,7 +61,7 @@ static time_t starttime;
>  static xfs_ino_t	leftoffino = 0;
>  static int	pagesize;
>  
> -void usage(int ret);
> +static void usage(int ret);
>  static int  fsrfile(char *fname, xfs_ino_t ino);
>  static int  fsrfile_common( char *fname, char *tname, char *mnt,
>                              int fd, xfs_bstat_t *statp);
> @@ -73,17 +73,16 @@ static void initallfs(char *mtab);
>  static void fsrallfs(char *mtab, int howlong, char *leftofffile);
>  static void fsrall_cleanup(int timeout);
>  static int  getnextents(int);
> -int xfsrtextsize(int fd);
> -int xfs_getrt(int fd, struct statvfs *sfbp);
> -char * gettmpname(char *fname);
> -char * getparent(char *fname);
> -int fsrprintf(const char *fmt, ...);
> -int read_fd_bmap(int, xfs_bstat_t *, int *);
> -int cmp(const void *, const void *);
> +static int xfs_getrt(int fd, struct statvfs *sfbp);
> +static char * gettmpname(char *fname);
> +static char * getparent(char *fname);
> +static int fsrprintf(const char *fmt, ...);
> +static int read_fd_bmap(int, xfs_bstat_t *, int *);
> +static int cmp(const void *, const void *);
>  static void tmp_init(char *mnt);
>  static char * tmp_next(char *mnt);
>  static void tmp_close(char *mnt);
> -int xfs_getgeom(int , xfs_fsop_geom_v1_t * );
> +static int xfs_getgeom(int , xfs_fsop_geom_v1_t * );

FYI I'm going to send you a patch for 5.1 that cleans all of these "get
geometry" helpers into a single libfrog function.

>  
>  static xfs_fsop_geom_v1_t fsgeom;	/* geometry of active mounted system */
>  
> diff --git a/include/libxlog.h b/include/libxlog.h
> index 92c54f4..14544db 100644
> --- a/include/libxlog.h
> +++ b/include/libxlog.h
> @@ -76,8 +76,6 @@ extern int xlog_is_dirty(struct xfs_mount *, struct xlog *, libxfs_init_t *,
>  			 int);
>  extern struct xfs_buf *xlog_get_bp(struct xlog *, int);
>  extern void	xlog_put_bp(struct xfs_buf *);
> -extern int	xlog_bread(struct xlog *log, xfs_daddr_t blk_no, int nbblks,
> -				xfs_buf_t *bp, char **offset);
>  extern int	xlog_bread_noalign(struct xlog *log, xfs_daddr_t blk_no,
>  				int nbblks, xfs_buf_t *bp);
>  
> @@ -90,7 +88,6 @@ extern int	xlog_find_tail(struct xlog *log, xfs_daddr_t *head_blk,
>  
>  extern int	xlog_recover(struct xlog *log, int readonly);
>  extern void	xlog_recover_print_data(char *p, int len);
> -extern void	xlog_recover_print_logitem(xlog_recover_item_t *item);
>  extern void	xlog_recover_print_trans_head(xlog_recover_t *tr);
>  extern int	xlog_print_find_oldest(struct xlog *log, xfs_daddr_t *last_blk);
>  
> diff --git a/io/io.h b/io/io.h
> index 6469179..0b5b4dc 100644
> --- a/io/io.h
> +++ b/io/io.h
> @@ -59,8 +59,6 @@ typedef struct mmap_region {
>  	char		*name;		/* name of backing file */
>  } mmap_region_t;
>  
> -extern mmap_region_t	*maptable;	/* mmap'd region array */
> -extern int		mapcount;	/* #entries in the mapping table */
>  extern mmap_region_t	*mapping;	/* active mapping table entry */
>  extern int maplist_f(void);
>  extern void *check_mapping_range(mmap_region_t *, off64_t, size_t, int);
> diff --git a/io/mmap.c b/io/mmap.c
> index f9383e5..db57c3e 100644
> --- a/io/mmap.c
> +++ b/io/mmap.c
> @@ -20,9 +20,9 @@ static cmdinfo_t mwrite_cmd;
>  static cmdinfo_t mremap_cmd;
>  #endif /* HAVE_MREMAP */
>  
> -mmap_region_t	*maptable;
> -int		mapcount;
> -mmap_region_t	*mapping;
> +static mmap_region_t	*maptable;
> +static int		mapcount;
> +mmap_region_t		*mapping;
>  
>  static void
>  print_mapping(
> diff --git a/io/pread.c b/io/pread.c
> index 1b4352b..b14c51b 100644
> --- a/io/pread.c
> +++ b/io/pread.c
> @@ -47,11 +47,11 @@ pread_help(void)
>  "\n"));
>  }
>  
> -void	*io_buffer;
> -size_t	highwater;
> -size_t	io_buffersize;
> -int	vectors;
> -struct iovec *iov;
> +void		*io_buffer;
> +static size_t	highwater;
> +size_t		io_buffersize;
> +int		vectors;
> +struct iovec 	*iov;
>  
>  static int
>  alloc_iovec(
> @@ -112,7 +112,7 @@ alloc_buffer(
>  	return 0;
>  }
>  
> -void
> +static void
>  __dump_buffer(
>  	void		*buf,
>  	off64_t		offset,
> diff --git a/libxlog/xfs_log_recover.c b/libxlog/xfs_log_recover.c
> index ab56fcd..2b3683a 100644
> --- a/libxlog/xfs_log_recover.c
> +++ b/libxlog/xfs_log_recover.c
> @@ -125,7 +125,7 @@ xlog_bread_noalign(
>  	return libxfs_readbufr(log->l_dev, XFS_BUF_ADDR(bp), bp, nbblks, 0);
>  }
>  
> -int
> +static int
>  xlog_bread(
>  	struct xlog	*log,
>  	xfs_daddr_t	blk_no,
> diff --git a/logprint/log_misc.c b/logprint/log_misc.c
> index c325f04..2368e9e 100644
> --- a/logprint/log_misc.c
> +++ b/logprint/log_misc.c
> @@ -26,7 +26,7 @@ typedef struct xlog_split_item {
>  
>  static xlog_split_item_t *split_list = NULL;
>  
> -void
> +static void
>  print_xlog_op_line(void)
>  {
>      printf("--------------------------------------"
> @@ -47,7 +47,7 @@ print_xlog_record_line(void)
>  	   "======================================\n");
>  }	/* print_xlog_record_line */
>  
> -void
> +static void
>  print_stars(void)
>  {
>      printf("***********************************"
> diff --git a/logprint/log_print_all.c b/logprint/log_print_all.c
> index eafffe2..082d11c 100644
> --- a/logprint/log_print_all.c
> +++ b/logprint/log_print_all.c
> @@ -380,7 +380,7 @@ xlog_recover_print_icreate(
>  		be32_to_cpu(icl->icl_isize), be32_to_cpu(icl->icl_gen));
>  }
>  
> -void
> +static void
>  xlog_recover_print_logitem(
>  	xlog_recover_item_t	*item)
>  {
> diff --git a/logprint/logprint.h b/logprint/logprint.h
> index 98ac0d4..0515628 100644
> --- a/logprint/logprint.h
> +++ b/logprint/logprint.h
> @@ -26,8 +26,6 @@ extern void xfs_log_print(struct xlog *, int, int);
>  extern void xfs_log_print_trans(struct xlog *, int);
>  
>  extern void print_xlog_record_line(void);
> -extern void print_xlog_op_line(void);
> -extern void print_stars(void);
>  
>  extern struct xfs_inode_log_format *
>  	xfs_inode_item_format_convert(char *, uint, struct xfs_inode_log_format *);
> diff --git a/repair/btree.c b/repair/btree.c
> index 292fa51..f101233 100644
> --- a/repair/btree.c
> +++ b/repair/btree.c
> @@ -306,7 +306,7 @@ btree_find(
>  	return root->cursor->node->ptrs[root->cursor->index];
>  }
>  
> -void *
> +static void *
>  btree_lookup(
>  	struct btree_root	*root,
>  	unsigned long		key)
> diff --git a/repair/btree.h b/repair/btree.h
> index a9ea61a..0e57db8 100644
> --- a/repair/btree.h
> +++ b/repair/btree.h
> @@ -23,11 +23,6 @@ btree_is_empty(
>  	struct btree_root	*root);
>  
>  void *
> -btree_lookup(
> -	struct btree_root	*root,
> -	unsigned long		key);
> -
> -void *
>  btree_find(
>  	struct btree_root	*root,
>  	unsigned long		key,
> diff --git a/repair/globals.c b/repair/globals.c
> index ae9d55b..706303a 100644
> --- a/repair/globals.c
> +++ b/repair/globals.c
> @@ -13,25 +13,6 @@
>  char	*fs_name;		/* name of filesystem */
>  int	verbose;		/* verbose flag, mostly for debugging */
>  
> -
> -/* for reading stuff in manually (bypassing libsim) */
> -
> -char	*iobuf;			/* large buffer */
> -int	iobuf_size;
> -char	*smallbuf;		/* small (1-4 page) buffer */
> -int	smallbuf_size;
> -int	sbbuf_size;
> -
> -/* direct I/O info */
> -
> -int	minio_align;		/* min I/O size and alignment */
> -int	mem_align;		/* memory alignment */
> -int	max_iosize;		/* max I/O size */
> -
> -/* file descriptors */
> -
> -int	fs_fd;			/* filesystem fd */
> -

This is making them static? :)

Maybe change commit message to something like:

"Withdraw externally visible symbols, whether by removing them entirely
or making them static."

--D

>  /* command-line flags */
>  
>  int	verbose;
> diff --git a/repair/globals.h b/repair/globals.h
> index 05121d4..a85e96a 100644
> --- a/repair/globals.h
> +++ b/repair/globals.h
> @@ -54,25 +54,6 @@
>  extern char	*fs_name;		/* name of filesystem */
>  extern int	verbose;		/* verbose flag, mostly for debugging */
>  
> -
> -/* for reading stuff in manually (bypassing libsim) */
> -
> -extern char	*iobuf;			/* large buffer */
> -extern int	iobuf_size;
> -extern char	*smallbuf;		/* small (1-4 page) buffer */
> -extern int	smallbuf_size;
> -extern int	sbbuf_size;
> -
> -/* direct I/O info */
> -
> -extern int	minio_align;		/* min I/O size and alignment */
> -extern int	mem_align;		/* memory alignment */
> -extern int	max_iosize;		/* max I/O size */
> -
> -/* file descriptors */
> -
> -extern int	fs_fd;			/* filesystem fd */
> -
>  /* command-line flags */
>  
>  extern int	verbose;
> diff --git a/repair/prefetch.c b/repair/prefetch.c
> index 1de0e2f..a9af8de 100644
> --- a/repair/prefetch.c
> +++ b/repair/prefetch.c
> @@ -870,7 +870,7 @@ init_prefetch(
>  	pf_batch_fsbs = DEF_BATCH_BYTES >> (mp->m_sb.sb_blocklog + 1);
>  }
>  
> -prefetch_args_t *
> +static prefetch_args_t *
>  start_inode_prefetch(
>  	xfs_agnumber_t		agno,
>  	int			dirs_only,
> diff --git a/repair/prefetch.h b/repair/prefetch.h
> index 54ece48..504e617 100644
> --- a/repair/prefetch.h
> +++ b/repair/prefetch.h
> @@ -37,12 +37,6 @@ void
>  init_prefetch(
>  	xfs_mount_t		*pmp);
>  
> -prefetch_args_t *
> -start_inode_prefetch(
> -	xfs_agnumber_t		agno,
> -	int			dirs_only,
> -	prefetch_args_t		*prev_args);
> -
>  void
>  do_inode_prefetch(
>  	struct xfs_mount	*mp,
> diff --git a/repair/versions.c b/repair/versions.c
> index 4c44b4e..16805b3 100644
> --- a/repair/versions.c
> +++ b/repair/versions.c
> @@ -15,13 +15,13 @@
>   * is on, 0 otherwise
>   */
>  
> -int fs_attributes;
> -int fs_attributes2;
> -int fs_inode_nlink;
> +static int fs_attributes;
> +static int fs_attributes2;
> +static int fs_inode_nlink;
> +static int fs_sb_feature_bits;
> +static int fs_has_extflgbit;
>  int fs_quotas;
>  int fs_aligned_inodes;
> -int fs_sb_feature_bits;
> -int fs_has_extflgbit;
>  
>  /*
>   * inode chunk alignment, fsblocks
> diff --git a/repair/versions.h b/repair/versions.h
> index e1e2521..512dbec 100644
> --- a/repair/versions.h
> +++ b/repair/versions.h
> @@ -16,13 +16,8 @@
>   * is on, 0 otherwise
>   */
>  
> -extern int		fs_attributes;
> -extern int		fs_attributes2;
> -extern int		fs_inode_nlink;
>  extern int		fs_quotas;
>  extern int		fs_aligned_inodes;
> -extern int		fs_sb_feature_bits;
> -extern int		fs_has_extflgbit;
>  
>  /*
>   * inode chunk alignment, fsblocks
> diff --git a/rtcp/xfs_rtcp.c b/rtcp/xfs_rtcp.c
> index f928a86..d9dd6be 100644
> --- a/rtcp/xfs_rtcp.c
> +++ b/rtcp/xfs_rtcp.c
> @@ -6,8 +6,8 @@
>  
>  #include "libxfs.h"
>  
> -int rtcp(char *, char *, int);
> -int xfsrtextsize(char *path);
> +static int rtcp(char *, char *, int);
> +static int xfsrtextsize(char *path);
>  
>  static int pflag;
>  char *progname;
> @@ -96,7 +96,7 @@ main(int argc, char **argv)
>  	exit(r?2:0);
>  }
>  
> -int
> +static int
>  rtcp( char *source, char *target, int fextsize)
>  {
>  	int		fromfd, tofd, readct, writect, iosz, reopen;
> @@ -365,7 +365,7 @@ rtcp( char *source, char *target, int fextsize)
>  /*
>   * Determine the realtime extent size of the XFS file system
>   */
> -int
> +static int
>  xfsrtextsize( char *path)
>  {
>  	int fd, rval, rtextsize;
> diff --git a/scrub/repair.c b/scrub/repair.c
> index 4ed3c09..f33c1f1 100644
> --- a/scrub/repair.c
> +++ b/scrub/repair.c
> @@ -202,7 +202,7 @@ xfs_action_list_add(
>  }
>  
>  /* Splice two repair lists. */
> -void
> +static void
>  xfs_action_list_splice(
>  	struct xfs_action_list		*dest,
>  	struct xfs_action_list		*src)
> diff --git a/scrub/repair.h b/scrub/repair.h
> index c8693cc..ab39795 100644
> --- a/scrub/repair.h
> +++ b/scrub/repair.h
> @@ -19,8 +19,6 @@ void xfs_action_list_init(struct xfs_action_list *alist);
>  size_t xfs_action_list_length(struct xfs_action_list *alist);
>  void xfs_action_list_add(struct xfs_action_list *dest,
>  		struct action_item *item);
> -void xfs_action_list_splice(struct xfs_action_list *dest,
> -		struct xfs_action_list *src);
>  
>  void xfs_action_list_find_mustfix(struct xfs_action_list *actions,
>  		struct xfs_action_list *immediate_alist,
>

Patch
diff mbox series

diff --git a/copy/xfs_copy.c b/copy/xfs_copy.c
index a6d6703..54679d7 100644
--- a/copy/xfs_copy.c
+++ b/copy/xfs_copy.c
@@ -48,9 +48,9 @@  static pthread_mutex_t	mainwait;
 #define ACTIVE		1
 #define INACTIVE	2
 
-xfs_off_t	write_log_trailer(int fd, wbuf *w, xfs_mount_t *mp);
-xfs_off_t	write_log_header(int fd, wbuf *w, xfs_mount_t *mp);
-static int	format_logs(struct xfs_mount *);
+static xfs_off_t	write_log_trailer(int fd, wbuf *w, xfs_mount_t *mp);
+static xfs_off_t	write_log_header(int fd, wbuf *w, xfs_mount_t *mp);
+static int		format_logs(struct xfs_mount *);
 
 /* general purpose message reporting routine */
 
diff --git a/db/io.c b/db/io.c
index 6b66472..bb16618 100644
--- a/db/io.c
+++ b/db/io.c
@@ -51,10 +51,10 @@  static const cmdinfo_t  ring_cmd =
 	{ "ring", NULL, ring_f, 0, 1, 0, NULL,
 	  N_("show position ring or move to a specific entry"), ring_help };
 
-iocur_t	*iocur_base;
-iocur_t	*iocur_top;
-int	iocur_sp = -1;
-int	iocur_len;
+static iocur_t	*iocur_base;
+iocur_t		*iocur_top;
+int		iocur_sp = -1;
+static int	iocur_len;
 
 #define RING_ENTRIES 20
 static iocur_t iocur_ring[RING_ENTRIES];
diff --git a/db/io.h b/db/io.h
index c29a748..0e4f74a 100644
--- a/db/io.h
+++ b/db/io.h
@@ -35,10 +35,8 @@  typedef struct iocur {
 #define DB_RING_ADD 1                   /* add to ring on set_cur */
 #define DB_RING_IGN 0                   /* do not add to ring on set_cur */
 
-extern iocur_t	*iocur_base;		/* base of stack */
 extern iocur_t	*iocur_top;		/* top element of stack */
 extern int	iocur_sp;		/* current top of stack */
-extern int	iocur_len;		/* length of stack array */
 
 extern void	io_init(void);
 extern void	off_cur(int off, int len);
diff --git a/estimate/xfs_estimate.c b/estimate/xfs_estimate.c
index 9e01cce..277b371 100644
--- a/estimate/xfs_estimate.c
+++ b/estimate/xfs_estimate.c
@@ -33,7 +33,7 @@  cvtnum(char *s)
 	return 0LL;
 }
 
-int ffn(const char *, const struct stat *, int, struct FTW *);
+static int ffn(const char *, const struct stat *, int, struct FTW *);
 
 #define BLOCKSIZE	4096
 #define INODESIZE	256
diff --git a/fsr/xfs_fsr.c b/fsr/xfs_fsr.c
index fef6262..55711bf 100644
--- a/fsr/xfs_fsr.c
+++ b/fsr/xfs_fsr.c
@@ -61,7 +61,7 @@  static time_t starttime;
 static xfs_ino_t	leftoffino = 0;
 static int	pagesize;
 
-void usage(int ret);
+static void usage(int ret);
 static int  fsrfile(char *fname, xfs_ino_t ino);
 static int  fsrfile_common( char *fname, char *tname, char *mnt,
                             int fd, xfs_bstat_t *statp);
@@ -73,17 +73,16 @@  static void initallfs(char *mtab);
 static void fsrallfs(char *mtab, int howlong, char *leftofffile);
 static void fsrall_cleanup(int timeout);
 static int  getnextents(int);
-int xfsrtextsize(int fd);
-int xfs_getrt(int fd, struct statvfs *sfbp);
-char * gettmpname(char *fname);
-char * getparent(char *fname);
-int fsrprintf(const char *fmt, ...);
-int read_fd_bmap(int, xfs_bstat_t *, int *);
-int cmp(const void *, const void *);
+static int xfs_getrt(int fd, struct statvfs *sfbp);
+static char * gettmpname(char *fname);
+static char * getparent(char *fname);
+static int fsrprintf(const char *fmt, ...);
+static int read_fd_bmap(int, xfs_bstat_t *, int *);
+static int cmp(const void *, const void *);
 static void tmp_init(char *mnt);
 static char * tmp_next(char *mnt);
 static void tmp_close(char *mnt);
-int xfs_getgeom(int , xfs_fsop_geom_v1_t * );
+static int xfs_getgeom(int , xfs_fsop_geom_v1_t * );
 
 static xfs_fsop_geom_v1_t fsgeom;	/* geometry of active mounted system */
 
diff --git a/include/libxlog.h b/include/libxlog.h
index 92c54f4..14544db 100644
--- a/include/libxlog.h
+++ b/include/libxlog.h
@@ -76,8 +76,6 @@  extern int xlog_is_dirty(struct xfs_mount *, struct xlog *, libxfs_init_t *,
 			 int);
 extern struct xfs_buf *xlog_get_bp(struct xlog *, int);
 extern void	xlog_put_bp(struct xfs_buf *);
-extern int	xlog_bread(struct xlog *log, xfs_daddr_t blk_no, int nbblks,
-				xfs_buf_t *bp, char **offset);
 extern int	xlog_bread_noalign(struct xlog *log, xfs_daddr_t blk_no,
 				int nbblks, xfs_buf_t *bp);
 
@@ -90,7 +88,6 @@  extern int	xlog_find_tail(struct xlog *log, xfs_daddr_t *head_blk,
 
 extern int	xlog_recover(struct xlog *log, int readonly);
 extern void	xlog_recover_print_data(char *p, int len);
-extern void	xlog_recover_print_logitem(xlog_recover_item_t *item);
 extern void	xlog_recover_print_trans_head(xlog_recover_t *tr);
 extern int	xlog_print_find_oldest(struct xlog *log, xfs_daddr_t *last_blk);
 
diff --git a/io/io.h b/io/io.h
index 6469179..0b5b4dc 100644
--- a/io/io.h
+++ b/io/io.h
@@ -59,8 +59,6 @@  typedef struct mmap_region {
 	char		*name;		/* name of backing file */
 } mmap_region_t;
 
-extern mmap_region_t	*maptable;	/* mmap'd region array */
-extern int		mapcount;	/* #entries in the mapping table */
 extern mmap_region_t	*mapping;	/* active mapping table entry */
 extern int maplist_f(void);
 extern void *check_mapping_range(mmap_region_t *, off64_t, size_t, int);
diff --git a/io/mmap.c b/io/mmap.c
index f9383e5..db57c3e 100644
--- a/io/mmap.c
+++ b/io/mmap.c
@@ -20,9 +20,9 @@  static cmdinfo_t mwrite_cmd;
 static cmdinfo_t mremap_cmd;
 #endif /* HAVE_MREMAP */
 
-mmap_region_t	*maptable;
-int		mapcount;
-mmap_region_t	*mapping;
+static mmap_region_t	*maptable;
+static int		mapcount;
+mmap_region_t		*mapping;
 
 static void
 print_mapping(
diff --git a/io/pread.c b/io/pread.c
index 1b4352b..b14c51b 100644
--- a/io/pread.c
+++ b/io/pread.c
@@ -47,11 +47,11 @@  pread_help(void)
 "\n"));
 }
 
-void	*io_buffer;
-size_t	highwater;
-size_t	io_buffersize;
-int	vectors;
-struct iovec *iov;
+void		*io_buffer;
+static size_t	highwater;
+size_t		io_buffersize;
+int		vectors;
+struct iovec 	*iov;
 
 static int
 alloc_iovec(
@@ -112,7 +112,7 @@  alloc_buffer(
 	return 0;
 }
 
-void
+static void
 __dump_buffer(
 	void		*buf,
 	off64_t		offset,
diff --git a/libxlog/xfs_log_recover.c b/libxlog/xfs_log_recover.c
index ab56fcd..2b3683a 100644
--- a/libxlog/xfs_log_recover.c
+++ b/libxlog/xfs_log_recover.c
@@ -125,7 +125,7 @@  xlog_bread_noalign(
 	return libxfs_readbufr(log->l_dev, XFS_BUF_ADDR(bp), bp, nbblks, 0);
 }
 
-int
+static int
 xlog_bread(
 	struct xlog	*log,
 	xfs_daddr_t	blk_no,
diff --git a/logprint/log_misc.c b/logprint/log_misc.c
index c325f04..2368e9e 100644
--- a/logprint/log_misc.c
+++ b/logprint/log_misc.c
@@ -26,7 +26,7 @@  typedef struct xlog_split_item {
 
 static xlog_split_item_t *split_list = NULL;
 
-void
+static void
 print_xlog_op_line(void)
 {
     printf("--------------------------------------"
@@ -47,7 +47,7 @@  print_xlog_record_line(void)
 	   "======================================\n");
 }	/* print_xlog_record_line */
 
-void
+static void
 print_stars(void)
 {
     printf("***********************************"
diff --git a/logprint/log_print_all.c b/logprint/log_print_all.c
index eafffe2..082d11c 100644
--- a/logprint/log_print_all.c
+++ b/logprint/log_print_all.c
@@ -380,7 +380,7 @@  xlog_recover_print_icreate(
 		be32_to_cpu(icl->icl_isize), be32_to_cpu(icl->icl_gen));
 }
 
-void
+static void
 xlog_recover_print_logitem(
 	xlog_recover_item_t	*item)
 {
diff --git a/logprint/logprint.h b/logprint/logprint.h
index 98ac0d4..0515628 100644
--- a/logprint/logprint.h
+++ b/logprint/logprint.h
@@ -26,8 +26,6 @@  extern void xfs_log_print(struct xlog *, int, int);
 extern void xfs_log_print_trans(struct xlog *, int);
 
 extern void print_xlog_record_line(void);
-extern void print_xlog_op_line(void);
-extern void print_stars(void);
 
 extern struct xfs_inode_log_format *
 	xfs_inode_item_format_convert(char *, uint, struct xfs_inode_log_format *);
diff --git a/repair/btree.c b/repair/btree.c
index 292fa51..f101233 100644
--- a/repair/btree.c
+++ b/repair/btree.c
@@ -306,7 +306,7 @@  btree_find(
 	return root->cursor->node->ptrs[root->cursor->index];
 }
 
-void *
+static void *
 btree_lookup(
 	struct btree_root	*root,
 	unsigned long		key)
diff --git a/repair/btree.h b/repair/btree.h
index a9ea61a..0e57db8 100644
--- a/repair/btree.h
+++ b/repair/btree.h
@@ -23,11 +23,6 @@  btree_is_empty(
 	struct btree_root	*root);
 
 void *
-btree_lookup(
-	struct btree_root	*root,
-	unsigned long		key);
-
-void *
 btree_find(
 	struct btree_root	*root,
 	unsigned long		key,
diff --git a/repair/globals.c b/repair/globals.c
index ae9d55b..706303a 100644
--- a/repair/globals.c
+++ b/repair/globals.c
@@ -13,25 +13,6 @@ 
 char	*fs_name;		/* name of filesystem */
 int	verbose;		/* verbose flag, mostly for debugging */
 
-
-/* for reading stuff in manually (bypassing libsim) */
-
-char	*iobuf;			/* large buffer */
-int	iobuf_size;
-char	*smallbuf;		/* small (1-4 page) buffer */
-int	smallbuf_size;
-int	sbbuf_size;
-
-/* direct I/O info */
-
-int	minio_align;		/* min I/O size and alignment */
-int	mem_align;		/* memory alignment */
-int	max_iosize;		/* max I/O size */
-
-/* file descriptors */
-
-int	fs_fd;			/* filesystem fd */
-
 /* command-line flags */
 
 int	verbose;
diff --git a/repair/globals.h b/repair/globals.h
index 05121d4..a85e96a 100644
--- a/repair/globals.h
+++ b/repair/globals.h
@@ -54,25 +54,6 @@ 
 extern char	*fs_name;		/* name of filesystem */
 extern int	verbose;		/* verbose flag, mostly for debugging */
 
-
-/* for reading stuff in manually (bypassing libsim) */
-
-extern char	*iobuf;			/* large buffer */
-extern int	iobuf_size;
-extern char	*smallbuf;		/* small (1-4 page) buffer */
-extern int	smallbuf_size;
-extern int	sbbuf_size;
-
-/* direct I/O info */
-
-extern int	minio_align;		/* min I/O size and alignment */
-extern int	mem_align;		/* memory alignment */
-extern int	max_iosize;		/* max I/O size */
-
-/* file descriptors */
-
-extern int	fs_fd;			/* filesystem fd */
-
 /* command-line flags */
 
 extern int	verbose;
diff --git a/repair/prefetch.c b/repair/prefetch.c
index 1de0e2f..a9af8de 100644
--- a/repair/prefetch.c
+++ b/repair/prefetch.c
@@ -870,7 +870,7 @@  init_prefetch(
 	pf_batch_fsbs = DEF_BATCH_BYTES >> (mp->m_sb.sb_blocklog + 1);
 }
 
-prefetch_args_t *
+static prefetch_args_t *
 start_inode_prefetch(
 	xfs_agnumber_t		agno,
 	int			dirs_only,
diff --git a/repair/prefetch.h b/repair/prefetch.h
index 54ece48..504e617 100644
--- a/repair/prefetch.h
+++ b/repair/prefetch.h
@@ -37,12 +37,6 @@  void
 init_prefetch(
 	xfs_mount_t		*pmp);
 
-prefetch_args_t *
-start_inode_prefetch(
-	xfs_agnumber_t		agno,
-	int			dirs_only,
-	prefetch_args_t		*prev_args);
-
 void
 do_inode_prefetch(
 	struct xfs_mount	*mp,
diff --git a/repair/versions.c b/repair/versions.c
index 4c44b4e..16805b3 100644
--- a/repair/versions.c
+++ b/repair/versions.c
@@ -15,13 +15,13 @@ 
  * is on, 0 otherwise
  */
 
-int fs_attributes;
-int fs_attributes2;
-int fs_inode_nlink;
+static int fs_attributes;
+static int fs_attributes2;
+static int fs_inode_nlink;
+static int fs_sb_feature_bits;
+static int fs_has_extflgbit;
 int fs_quotas;
 int fs_aligned_inodes;
-int fs_sb_feature_bits;
-int fs_has_extflgbit;
 
 /*
  * inode chunk alignment, fsblocks
diff --git a/repair/versions.h b/repair/versions.h
index e1e2521..512dbec 100644
--- a/repair/versions.h
+++ b/repair/versions.h
@@ -16,13 +16,8 @@ 
  * is on, 0 otherwise
  */
 
-extern int		fs_attributes;
-extern int		fs_attributes2;
-extern int		fs_inode_nlink;
 extern int		fs_quotas;
 extern int		fs_aligned_inodes;
-extern int		fs_sb_feature_bits;
-extern int		fs_has_extflgbit;
 
 /*
  * inode chunk alignment, fsblocks
diff --git a/rtcp/xfs_rtcp.c b/rtcp/xfs_rtcp.c
index f928a86..d9dd6be 100644
--- a/rtcp/xfs_rtcp.c
+++ b/rtcp/xfs_rtcp.c
@@ -6,8 +6,8 @@ 
 
 #include "libxfs.h"
 
-int rtcp(char *, char *, int);
-int xfsrtextsize(char *path);
+static int rtcp(char *, char *, int);
+static int xfsrtextsize(char *path);
 
 static int pflag;
 char *progname;
@@ -96,7 +96,7 @@  main(int argc, char **argv)
 	exit(r?2:0);
 }
 
-int
+static int
 rtcp( char *source, char *target, int fextsize)
 {
 	int		fromfd, tofd, readct, writect, iosz, reopen;
@@ -365,7 +365,7 @@  rtcp( char *source, char *target, int fextsize)
 /*
  * Determine the realtime extent size of the XFS file system
  */
-int
+static int
 xfsrtextsize( char *path)
 {
 	int fd, rval, rtextsize;
diff --git a/scrub/repair.c b/scrub/repair.c
index 4ed3c09..f33c1f1 100644
--- a/scrub/repair.c
+++ b/scrub/repair.c
@@ -202,7 +202,7 @@  xfs_action_list_add(
 }
 
 /* Splice two repair lists. */
-void
+static void
 xfs_action_list_splice(
 	struct xfs_action_list		*dest,
 	struct xfs_action_list		*src)
diff --git a/scrub/repair.h b/scrub/repair.h
index c8693cc..ab39795 100644
--- a/scrub/repair.h
+++ b/scrub/repair.h
@@ -19,8 +19,6 @@  void xfs_action_list_init(struct xfs_action_list *alist);
 size_t xfs_action_list_length(struct xfs_action_list *alist);
 void xfs_action_list_add(struct xfs_action_list *dest,
 		struct action_item *item);
-void xfs_action_list_splice(struct xfs_action_list *dest,
-		struct xfs_action_list *src);
 
 void xfs_action_list_find_mustfix(struct xfs_action_list *actions,
 		struct xfs_action_list *immediate_alist,