Cleanup symbols that should be static

Run analysis on symbols not used outside of their current file,
turn them into statics.

Signed-off-by: Jens Axboe <axboe@fb.com>

diff --git a/cconv.c b/cconv.c
index 2843a71ca555c664961dfa88826c46ae5f46ed68..5b9c3be2590c79b3b6ec50f664c3ed49bec3d88a 100644 (file)
--- a/cconv.c
+++ b/cconv.c
@@ -18,7 +18,7 @@ static void string_to_net(uint8_t *dst, const char *src)
                dst[0] = '\0';
 }
 
                dst[0] = '\0';
 }
 

-void free_thread_options_to_cpu(struct thread_options *o)
+static void free_thread_options_to_cpu(struct thread_options *o)

 {
        free(o->description);
        free(o->name);
 {
        free(o->description);
        free(o->name);

diff --git a/crc/crc32c.c b/crc/crc32c.c
index b830b9fe026ba77e5727e4a485121fa6a1f76d2e..f6fc688748bfccae5edf0b31bd136a35585978d5 100644 (file)
--- a/crc/crc32c.c
+++ b/crc/crc32c.c
@@ -32,6 +32,8 @@
  */
 #include <inttypes.h>
 
  */
 #include <inttypes.h>
 

+#include "crc32c.h"
+

 /*
  * This is the CRC-32C table
  * Generated with:
 /*
  * This is the CRC-32C table
  * Generated with:

diff --git a/crc/xxhash.c b/crc/xxhash.c
index db7890b027826938e8aa61a8c49420c39332a6b2..eedaecb4912db91625af09d5f5a617e0dc9bb2ca 100644 (file)
--- a/crc/xxhash.c
+++ b/crc/xxhash.c
@@ -148,7 +148,7 @@ typedef enum { XXH_bigEndian=0, XXH_littleEndian=1 } XXH_endianess;
 //****************************
 typedef enum { XXH_aligned, XXH_unaligned } XXH_alignment;
 
 //****************************
 typedef enum { XXH_aligned, XXH_unaligned } XXH_alignment;
 

-uint32_t XXH_readLE32_align(const uint32_t* ptr, XXH_endianess endian, XXH_alignment align)
+static uint32_t XXH_readLE32_align(const uint32_t* ptr, XXH_endianess endian, XXH_alignment align)

 {
     if (align==XXH_unaligned)
         return endian==XXH_littleEndian ? A32(ptr) : XXH_swap32(A32(ptr));
 {
     if (align==XXH_unaligned)
         return endian==XXH_littleEndian ? A32(ptr) : XXH_swap32(A32(ptr));


@@ -156,13 +156,13 @@ uint32_t XXH_readLE32_align(const uint32_t* ptr, XXH_endianess endian, XXH_align
         return endian==XXH_littleEndian ? *ptr : XXH_swap32(*ptr);
 }
 
         return endian==XXH_littleEndian ? *ptr : XXH_swap32(*ptr);
 }
 

-uint32_t XXH_readLE32(const uint32_t* ptr, XXH_endianess endian) { return XXH_readLE32_align(ptr, endian, XXH_unaligned); }
+static uint32_t XXH_readLE32(const uint32_t* ptr, XXH_endianess endian) { return XXH_readLE32_align(ptr, endian, XXH_unaligned); }

 
 
 //****************************
 // Simple Hash Functions
 //****************************
 
 
 //****************************
 // Simple Hash Functions
 //****************************

-uint32_t XXH32_endian_align(const void* input, int len, uint32_t seed, XXH_endianess endian, XXH_alignment align)
+static uint32_t XXH32_endian_align(const void* input, int len, uint32_t seed, XXH_endianess endian, XXH_alignment align)

 {
     const uint8_t *p = (const uint8_t *)input;
     const uint8_t * const bEnd = p + len;
 {
     const uint8_t *p = (const uint8_t *)input;
     const uint8_t * const bEnd = p + len;


@@ -282,7 +282,7 @@ void* XXH32_init (uint32_t seed)
 }
 
 
 }
 
 

-XXH_errorcode XXH32_update_endian (void* state_in, const void* input, int len, XXH_endianess endian)
+static XXH_errorcode XXH32_update_endian (void* state_in, const void* input, int len, XXH_endianess endian)

 {
     struct XXH_state32_t * state = (struct XXH_state32_t *) state_in;
     const uint8_t *p = (const uint8_t *)input;
 {
     struct XXH_state32_t * state = (struct XXH_state32_t *) state_in;
     const uint8_t *p = (const uint8_t *)input;


@@ -358,7 +358,7 @@ XXH_errorcode XXH32_update (void* state_in, const void* input, int len)
 
 
 
 
 
 

-uint32_t XXH32_intermediateDigest_endian (void* state_in, XXH_endianess endian)
+static uint32_t XXH32_intermediateDigest_endian (void* state_in, XXH_endianess endian)

 {
     struct XXH_state32_t * state = (struct XXH_state32_t *) state_in;
     const uint8_t *p = (const uint8_t *)state->memory;
 {
     struct XXH_state32_t * state = (struct XXH_state32_t *) state_in;
     const uint8_t *p = (const uint8_t *)state->memory;

diff --git a/diskutil.c b/diskutil.c
index bc12b02e6e979345ec66aa3d6d86e34458cdc90a..9aa1fa15acab9ed27d0925e2926816bd8478d90e 100644 (file)
--- a/diskutil.c
+++ b/diskutil.c
@@ -652,8 +652,8 @@ void json_array_add_disk_util(struct disk_util_stat *dus,
        json_object_add_value_float(obj, "aggr_util", agg->max_util.u.f);
 }
 
        json_object_add_value_float(obj, "aggr_util", agg->max_util.u.f);
 }
 

-void json_object_add_disk_utils(struct json_object *obj,
-               struct flist_head *head)
+static void json_object_add_disk_utils(struct json_object *obj,
+                                      struct flist_head *head)

 {
        struct json_array *array = json_create_array();
        struct flist_head *entry;
 {
        struct json_array *array = json_create_array();
        struct flist_head *entry;

diff --git a/engines/falloc.c b/engines/falloc.c
index 17a901ad93f54e08802d88260d0c48a6bde9070f..2b00d525d648e071bb17a7e97c0b63405f192ec7 100644 (file)
--- a/engines/falloc.c
+++ b/engines/falloc.c
@@ -23,7 +23,7 @@
  * generic_open_file is not appropriate because does not allow to perform
  * TRIM in to file
  */
  * generic_open_file is not appropriate because does not allow to perform
  * TRIM in to file
  */

-int open_file(struct thread_data *td, struct fio_file *f)
+static int open_file(struct thread_data *td, struct fio_file *f)

 {
        int from_hash = 0;
 
 {
        int from_hash = 0;
 

diff --git a/engines/net.c b/engines/net.c
index 1df8d0635cf1b746ae7d597d1b4b4398f0019d2d..110e158f114cbe210656476ff81af305d7d80248 100644 (file)
--- a/engines/net.c
+++ b/engines/net.c
@@ -392,7 +392,7 @@ static int fio_netio_send(struct thread_data *td, struct io_u *io_u)
 
        do {
                if (is_udp(o)) {
 
        do {
                if (is_udp(o)) {

-                       struct sockaddr *to;
+                       const struct sockaddr *to;

                        socklen_t len;
 
                        if (is_ipv6(o)) {
                        socklen_t len;
 
                        if (is_ipv6(o)) {

diff --git a/engines/rbd.c b/engines/rbd.c
index 5a4a3e271f6c32ce7ee073df139e47e490f6daec..9d64efd233cab1dfd4188f8ada16351d649c8f5e 100644 (file)
--- a/engines/rbd.c
+++ b/engines/rbd.c
@@ -422,7 +422,7 @@ static int fio_rbd_io_u_init(struct thread_data *td, struct io_u *io_u)
        return 0;
 }
 
        return 0;
 }
 

-struct ioengine_ops ioengine = {
+static struct ioengine_ops ioengine = {

        .name               = "rbd",
        .version            = FIO_IOOPS_VERSION,
        .setup              = fio_rbd_setup,
        .name               = "rbd",
        .version            = FIO_IOOPS_VERSION,
        .setup              = fio_rbd_setup,

diff --git a/filehash.c b/filehash.c
index 2d9659c5039a0e46cb79db758dcd694bc9d9971c..c6ebe761282edd5ef645042c2bbed7f8ff98792c 100644 (file)
--- a/filehash.c
+++ b/filehash.c
@@ -4,6 +4,7 @@
 #include "fio.h"
 #include "flist.h"
 #include "hash.h"
 #include "fio.h"
 #include "flist.h"
 #include "hash.h"

+#include "filehash.h"

 
 #define HASH_BUCKETS   512
 #define HASH_MASK      (HASH_BUCKETS - 1)
 
 #define HASH_BUCKETS   512
 #define HASH_MASK      (HASH_BUCKETS - 1)

diff --git a/gettime.c b/gettime.c
index 8991703fd0662fad4616ed8ab66f154f12c0ade9..b89cd46fdd14e7cbe6290a424416b6268a6543d8 100644 (file)
--- a/gettime.c
+++ b/gettime.c
@@ -32,7 +32,7 @@ static pthread_key_t tv_tls_key;
 
 enum fio_cs fio_clock_source = FIO_PREFERRED_CLOCK_SOURCE;
 int fio_clock_source_set = 0;
 
 enum fio_cs fio_clock_source = FIO_PREFERRED_CLOCK_SOURCE;
 int fio_clock_source_set = 0;

-enum fio_cs fio_clock_source_inited = CS_INVAL;
+static enum fio_cs fio_clock_source_inited = CS_INVAL;

 
 #ifdef FIO_DEBUG_TIME
 
 
 #ifdef FIO_DEBUG_TIME
 

diff --git a/idletime.c b/idletime.c
index 9e6960702ca0face97c1602534bf79616f2afbaf..c0bc0bfce19675e311fcbd7394540b02a7010eba 100644 (file)
--- a/idletime.c
+++ b/idletime.c
@@ -379,7 +379,7 @@ static double fio_idle_prof_cpu_stat(int cpu)
        return p * 100.0;
 }
 
        return p * 100.0;
 }
 

-void fio_idle_prof_cleanup(void)
+static void fio_idle_prof_cleanup(void)

 {
        if (ipc.ipts) {
                free(ipc.ipts);
 {
        if (ipc.ipts) {
                free(ipc.ipts);

diff --git a/init.c b/init.c
index c7e056fa60e399d20297c003615506887f1b4875..c3996a76c12840e3ab3cf6514e5d5dce611a1077 100644 (file)
--- a/init.c
+++ b/init.c
@@ -28,6 +28,8 @@
 #include "lib/getopt.h"
 #include "lib/strcasestr.h"
 
 #include "lib/getopt.h"
 #include "lib/strcasestr.h"
 

+#include "crc/test.h"
+

 const char fio_version_string[] = FIO_VERSION;
 
 #define FIO_RANDSEED           (0xb1899bedUL)
 const char fio_version_string[] = FIO_VERSION;
 
 #define FIO_RANDSEED           (0xb1899bedUL)


@@ -40,6 +42,8 @@ static int parse_only;
 
 static struct thread_data def_thread;
 struct thread_data *threads = NULL;
 
 static struct thread_data def_thread;
 struct thread_data *threads = NULL;

+static char **job_sections;
+static int nr_job_sections;

 
 int exitall_on_terminate = 0;
 int output_format = FIO_OUTPUT_NORMAL;
 
 int exitall_on_terminate = 0;
 int output_format = FIO_OUTPUT_NORMAL;


@@ -48,8 +52,6 @@ int eta_print = FIO_ETA_AUTO;
 int eta_new_line = 0;
 FILE *f_out = NULL;
 FILE *f_err = NULL;
 int eta_new_line = 0;
 FILE *f_out = NULL;
 FILE *f_err = NULL;

-char **job_sections = NULL;
-int nr_job_sections = 0;

 char *exec_profile = NULL;
 int warnings_fatal = 0;
 int terse_version = 3;
 char *exec_profile = NULL;
 int warnings_fatal = 0;
 int terse_version = 3;


@@ -247,7 +249,7 @@ void free_threads_shm(void)
        }
 }
 
        }
 }
 

-void free_shm(void)
+static void free_shm(void)

 {
        if (threads) {
                file_hash_exit();
 {
        if (threads) {
                file_hash_exit();


@@ -1658,13 +1660,11 @@ static int client_flag_set(char c)
        return 0;
 }
 
        return 0;
 }
 

-void parse_cmd_client(void *client, char *opt)
+static void parse_cmd_client(void *client, char *opt)

 {
        fio_client_add_cmd_option(client, opt);
 }
 
 {
        fio_client_add_cmd_option(client, opt);
 }
 

-extern int fio_crctest(const char *);
-

 int parse_cmd_line(int argc, char *argv[], int client_type)
 {
        struct thread_data *td = NULL;
 int parse_cmd_line(int argc, char *argv[], int client_type)
 {
        struct thread_data *td = NULL;

diff --git a/lib/lfsr.c b/lib/lfsr.c
index 9771318ab00163b8b6a9f68dc076cd896c00fda6..329ef85a8bf7a4e9529838c201482ee0eab78641 100644 (file)
--- a/lib/lfsr.c
+++ b/lib/lfsr.c
@@ -186,7 +186,7 @@ static uint8_t *find_lfsr(uint64_t size)
  * Thus, [1] is equivalent to (y * i) % (spin + 1) == 0;
  * Also, the cycle's length will be (x * i) + (y * i) / (spin + 1)
  */
  * Thus, [1] is equivalent to (y * i) % (spin + 1) == 0;
  * Also, the cycle's length will be (x * i) + (y * i) / (spin + 1)
  */

-int prepare_spin(struct fio_lfsr *fl, unsigned int spin)
+static int prepare_spin(struct fio_lfsr *fl, unsigned int spin)

 {
        uint64_t max = (fl->cached_bit << 1) - 1;
        uint64_t x, y;
 {
        uint64_t max = (fl->cached_bit << 1) - 1;
        uint64_t x, y;

diff --git a/lib/num2str.c b/lib/num2str.c
index a1041926c5e5aff50dc540227a89d9d76d17dfc6..12d6f39aa5dbca892e9d03af9150aa28a3a47861 100644 (file)
--- a/lib/num2str.c
+++ b/lib/num2str.c
@@ -2,6 +2,8 @@
 #include <stdio.h>
 #include <string.h>
 
 #include <stdio.h>
 #include <string.h>
 

+#include "../fio.h"
+

 /*
  * Cheesy number->string conversion, complete with carry rounding error.
  */
 /*
  * Cheesy number->string conversion, complete with carry rounding error.
  */

diff --git a/lib/prio_tree.c b/lib/prio_tree.c
index b0e935cdace5f051af293dd2ea9a2aef8997794e..e18ae324bac4200ee85b5ddda738f6fca006938a 100644 (file)
--- a/lib/prio_tree.c
+++ b/lib/prio_tree.c
@@ -49,7 +49,7 @@ static void get_index(const struct prio_tree_node *node,
 
 static unsigned long index_bits_to_maxindex[BITS_PER_LONG];
 
 
 static unsigned long index_bits_to_maxindex[BITS_PER_LONG];
 

-void fio_init prio_tree_init(void)
+static void fio_init prio_tree_init(void)

 {
        unsigned int i;
 
 {
        unsigned int i;
 

diff --git a/log.c b/log.c
index a05a7ec5d3f28f2bc2fd67b08fa8afc132585da4..c4a3b52075367919b191219f84d14a9b46936338 100644 (file)
--- a/log.c
+++ b/log.c
@@ -34,25 +34,6 @@ int log_local_buf(const char *buf, size_t len)
        return len;
 }
 
        return len;
 }
 

-int log_local(const char *format, ...)
-{
-       char buffer[1024];
-       va_list args;
-       size_t len;
-
-       va_start(args, format);
-       len = vsnprintf(buffer, sizeof(buffer), format, args);
-       va_end(args);
-       len = min(len, sizeof(buffer) - 1);
-
-       if (log_syslog)
-               syslog(LOG_INFO, "%s", buffer);
-       else
-               len = fwrite(buffer, len, 1, f_out);
-
-       return len;
-}
-

 int log_info(const char *format, ...)
 {
        char buffer[1024];
 int log_info(const char *format, ...)
 {
        char buffer[1024];

diff --git a/profiles/tiobench.c b/profiles/tiobench.c
index 7a7030afac0697c871179cb53e0188e138876737..b4331d75ae52c00437c9183d1ef4e76dc7cdf7ca 100644 (file)
--- a/profiles/tiobench.c
+++ b/profiles/tiobench.c
@@ -8,7 +8,7 @@ static unsigned int bs = 4096;
 static unsigned int nthreads = 1;
 static char *dir;
 
 static unsigned int nthreads = 1;
 static char *dir;
 

-char sz_idx[80], bs_idx[80], loop_idx[80], dir_idx[80], t_idx[80];
+static char sz_idx[80], bs_idx[80], loop_idx[80], dir_idx[80], t_idx[80];

 
 static const char *tb_opts[] = {
        "buffered=0", sz_idx, bs_idx, loop_idx, dir_idx, t_idx,
 
 static const char *tb_opts[] = {
        "buffered=0", sz_idx, bs_idx, loop_idx, dir_idx, t_idx,

diff --git a/smalloc.c b/smalloc.c
index 5dae7e7c9b79928c2ef69aa792ac71277536fea4..c8f1642eb54cfc7863182286be03d03599dd4dae 100644 (file)
--- a/smalloc.c
+++ b/smalloc.c
@@ -16,6 +16,7 @@
 #include "mutex.h"
 #include "arch/arch.h"
 #include "os/os.h"
 #include "mutex.h"
 #include "arch/arch.h"
 #include "os/os.h"

+#include "smalloc.h"

 
 #define SMALLOC_REDZONE                /* define to detect memory corruption */
 
 
 #define SMALLOC_REDZONE                /* define to detect memory corruption */
 


@@ -30,7 +31,7 @@
 #define SMALLOC_POST_RED       0x5aa55aa5U
 
 unsigned int smalloc_pool_size = INITIAL_SIZE;
 #define SMALLOC_POST_RED       0x5aa55aa5U
 
 unsigned int smalloc_pool_size = INITIAL_SIZE;

-const int int_mask = sizeof(int) - 1;
+static const int int_mask = sizeof(int) - 1;

 
 struct pool {
        struct fio_mutex *lock;                 /* protects this pool */
 
 struct pool {
        struct fio_mutex *lock;                 /* protects this pool */

diff --git a/stat.c b/stat.c
index f84ce5300f605736b5ef8b9556286792b2b53cd8..4529f6948bcb5bb64a938fdfe66a7a1b57992a21 100644 (file)
--- a/stat.c
+++ b/stat.c
@@ -497,7 +497,8 @@ static void show_latencies(struct thread_stat *ts)
        show_lat_m(io_u_lat_m);
 }
 
        show_lat_m(io_u_lat_m);
 }
 

-void show_thread_status_normal(struct thread_stat *ts, struct group_run_stats *rs)
+static void show_thread_status_normal(struct thread_stat *ts,
+                                     struct group_run_stats *rs)

 {
        double usr_cpu, sys_cpu;
        unsigned long runtime;
 {
        double usr_cpu, sys_cpu;
        unsigned long runtime;


@@ -1012,7 +1013,7 @@ struct json_object *show_thread_status(struct thread_stat *ts,
        if (output_format == FIO_OUTPUT_TERSE)
                show_thread_status_terse(ts, rs);
        else if (output_format == FIO_OUTPUT_JSON)
        if (output_format == FIO_OUTPUT_TERSE)
                show_thread_status_terse(ts, rs);
        else if (output_format == FIO_OUTPUT_JSON)

-               return(show_thread_status_json(ts, rs));
+               return show_thread_status_json(ts, rs);

        else
                show_thread_status_normal(ts, rs);
        return NULL;
        else
                show_thread_status_normal(ts, rs);
        return NULL;