From: Joel Rosdahl <joel@rosdahl.net>
Date: Mon, 25 Jul 2011 19:47:53 +0000 (+0200)
Subject: Use real size instead of multiples of 1024 for sizes
X-Git-Tag: v3.2~193
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=cadd1e1793d744a2964f3a39decd05172435e834;p=thirdparty%2Fccache.git

Use real size instead of multiples of 1024 for sizes

Exception: Size values in stats files are still in units of 1024 for
backward compatibility reasons.
---

diff --git a/ccache.c b/ccache.c
index 978da4084..ec618ac26 100644
--- a/ccache.c
+++ b/ccache.c
@@ -69,10 +69,6 @@ static const char USAGE_TEXT[] =
 "\n"
 "See also <http://ccache.samba.org>.\n";
 
-#ifndef DEFAULT_MAXSIZE
-#define DEFAULT_MAXSIZE (1024*1024)
-#endif
-
 /* Global configuration data. */
 struct conf *conf = NULL;
 
@@ -685,7 +681,7 @@ to_cache(struct args *args)
 		failed();
 	}
 
-	stats_update_size(STATS_TOCACHE, added_bytes / 1024, added_files);
+	stats_update_size(STATS_TOCACHE, added_bytes, added_files);
 
 	/* Make sure we have a CACHEDIR.TAG
 	 * This can be almost anywhere, but might as well do it near the end
@@ -1131,7 +1127,7 @@ from_cache(enum fromcache_call_mode mode, bool put_object_in_manifest)
 		} else {
 			cc_log("Stored in cache: %s", cached_dep);
 			stat(cached_dep, &st);
-			stats_update_size(STATS_NONE, file_size(&st) / 1024, 1);
+			stats_update_size(STATS_NONE, file_size(&st), 1);
 		}
 	}
 
@@ -1157,7 +1153,7 @@ from_cache(enum fromcache_call_mode mode, bool put_object_in_manifest)
 			update_mtime(manifest_path);
 			stat(manifest_path, &st);
 			stats_update_size(STATS_NONE,
-			                  (file_size(&st) - old_size) / 1024,
+			                  (file_size(&st) - old_size),
 			                  old_size == 0 ? 1 : 0);
 		} else {
 			cc_log("Failed to add object file hash to %s", manifest_path);
@@ -1776,7 +1772,8 @@ out:
 static void
 create_initial_config_file(struct conf *conf, const char *path)
 {
-	unsigned max_files, max_size;
+	unsigned max_files;
+	uint64_t max_size;
 	char *stats_dir;
 	FILE *f;
 	struct stat st;
@@ -1793,7 +1790,7 @@ create_initial_config_file(struct conf *conf, const char *path)
 		max_size *= 16;
 	} else {
 		max_files = 0;
-		max_size = DEFAULT_MAXSIZE;
+		max_size = conf->max_size;
 	}
 	free(stats_dir);
 
@@ -2088,7 +2085,6 @@ static int
 ccache_main_options(int argc, char *argv[])
 {
 	int c;
-	size_t v;
 	char *errmsg;
 
 	enum longopts {
@@ -2130,34 +2126,42 @@ ccache_main_options(int argc, char *argv[])
 			exit(0);
 
 		case 'F': /* --max-files */
-			initialize();
-			v = atoi(optarg);
-			if (conf_set_value_in_file(primary_config_path, "max_files", optarg,
-			                           &errmsg)) {
-				if (v == 0) {
-					printf("Unset cache file limit\n");
+			{
+				unsigned files;
+				initialize();
+				files = atoi(optarg);
+				if (conf_set_value_in_file(primary_config_path, "max_files", optarg,
+				                           &errmsg)) {
+					if (files == 0) {
+						printf("Unset cache file limit\n");
+					} else {
+						printf("Set cache file limit to %u\n", files);
+					}
 				} else {
-					printf("Set cache file limit to %u\n", (unsigned)v);
+					fatal("could not set cache file limit: %s", errmsg);
 				}
-			} else {
-				fatal("could not set cache file limit: %s", errmsg);
 			}
 			break;
 
 		case 'M': /* --max-size */
-			initialize();
-			parse_size_with_suffix(optarg, &v);
-			if (conf_set_value_in_file(primary_config_path, "max_size", optarg,
-			                           &errmsg)) {
-				if (v == 0) {
-					printf("Unset cache size limit\n");
+			{
+				uint64_t size;
+				initialize();
+				if (!parse_size_with_suffix(optarg, &size)) {
+					fatal("invalid size: %s", optarg);
+				}
+				if (conf_set_value_in_file(primary_config_path, "max_size", optarg,
+				                           &errmsg)) {
+					if (size == 0) {
+						printf("Unset cache size limit\n");
+					} else {
+						char *s = format_human_readable_size(size);
+						printf("Set cache size limit to %s\n", s);
+						free(s);
+					}
 				} else {
-					char *s = format_human_readable_size(v);
-					printf("Set cache size limit to %s\n", s);
-					free(s);
+					fatal("could not set cache size limit: %s", errmsg);
 				}
-			} else {
-				fatal("could not set cache size limit: %s", errmsg);
 			}
 			break;
 
diff --git a/ccache.h b/ccache.h
index 3522533d6..2b55aca69 100644
--- a/ccache.h
+++ b/ccache.h
@@ -129,9 +129,9 @@ const char *get_extension(const char *path);
 char *remove_extension(const char *path);
 size_t file_size(struct stat *st);
 int safe_create_wronly(const char *fname);
-char *format_human_readable_size(size_t size);
-char *format_parsable_size_with_suffix(size_t size);
-bool parse_size_with_suffix(const char *str, size_t *size);
+char *format_human_readable_size(uint64_t size);
+char *format_parsable_size_with_suffix(uint64_t size);
+bool parse_size_with_suffix(const char *str, uint64_t *size);
 char *x_realpath(const char *path);
 char *gnu_getcwd(void);
 #ifndef HAVE_STRTOK_R
@@ -164,9 +164,9 @@ void stats_flush(void);
 unsigned stats_get_pending(enum stats stat);
 void stats_zero(void);
 void stats_summary(struct conf *conf);
-void stats_update_size(enum stats stat, size_t size, unsigned files);
+void stats_update_size(enum stats stat, uint64_t size, unsigned files);
 void stats_get_obsolete_limits(const char *dir, unsigned *maxfiles,
-                               unsigned *maxsize);
+                               uint64_t *maxsize);
 void stats_set_sizes(const char *dir, size_t num_files, size_t total_size);
 void stats_read(const char *path, struct counters *counters);
 void stats_write(const char *path, struct counters *counters);
diff --git a/cleanup.c b/cleanup.c
index 6b65b3bd2..a33bd2b28 100644
--- a/cleanup.c
+++ b/cleanup.c
@@ -29,14 +29,14 @@
 static struct files {
 	char *fname;
 	time_t mtime;
-	size_t size; /* In KiB. */
+	uint64_t size;
 } **files;
 static unsigned allocated; /* Size of the files array. */
 static unsigned num_files; /* Number of used entries in the files array. */
 
-static size_t cache_size; /* In KiB. */
+static uint64_t cache_size;
 static size_t files_in_cache;
-static size_t cache_size_threshold;
+static uint64_t cache_size_threshold;
 static size_t files_in_cache_threshold;
 
 /* File comparison function that orders files in mtime order, oldest first. */
@@ -86,7 +86,7 @@ traverse_fn(const char *fname, struct stat *st)
 	files[num_files] = (struct files *)x_malloc(sizeof(struct files));
 	files[num_files]->fname = x_strdup(fname);
 	files[num_files]->mtime = st->st_mtime;
-	files[num_files]->size = file_size(st) / 1024;
+	files[num_files]->size = file_size(st);
 	cache_size += files[num_files]->size;
 	files_in_cache++;
 	num_files++;
@@ -114,7 +114,7 @@ delete_sibling_file(const char *base, const char *extension)
 
 	path = format("%s%s", base, extension);
 	if (lstat(path, &st) == 0) {
-		delete_file(path, file_size(&st) / 1024);
+		delete_file(path, file_size(&st));
 	} else if (errno != ENOENT) {
 		cc_log("Failed to stat %s (%s)", path, strerror(errno));
 	}
diff --git a/conf.c b/conf.c
index 0e80a6360..d1ac50297 100644
--- a/conf.c
+++ b/conf.c
@@ -78,8 +78,8 @@ parse_octal(const char *str, void *result, char **errmsg)
 static bool
 parse_size(const char *str, void *result, char **errmsg)
 {
-	unsigned *value = (unsigned *)result;
-	size_t size;
+	uint64_t *value = (uint64_t *)result;
+	uint64_t size;
 	*errmsg = NULL;
 	if (parse_size_with_suffix(str, &size)) {
 		*value = size;
@@ -426,7 +426,7 @@ conf_create(void)
 	conf->hash_dir = false;
 	conf->log_file = x_strdup("");
 	conf->max_files = 0;
-	conf->max_size = 1024 * 1024; /* kilobyte */
+	conf->max_size = 1024 * 1024 * 1024;
 	conf->path = x_strdup("");
 	conf->prefix_command = x_strdup("");
 	conf->read_only = false;
diff --git a/conf.h b/conf.h
index 257faa966..79c831444 100644
--- a/conf.h
+++ b/conf.h
@@ -19,7 +19,7 @@ struct conf {
 	bool hash_dir;
 	char *log_file;
 	unsigned max_files;
-	unsigned max_size;
+	uint64_t max_size;
 	char *path;
 	char *prefix_command;
 	bool read_only;
diff --git a/stats.c b/stats.c
index da45cd46b..a7b239ff0 100644
--- a/stats.c
+++ b/stats.c
@@ -43,7 +43,7 @@ static struct counters *counter_updates;
 #define FLAG_ALWAYS 2 /* always show, even if zero */
 #define FLAG_NEVER 4 /* never show */
 
-static void display_size(size_t v);
+static void display_size_times_1024(uint64_t size);
 
 /* statistics fields in display order */
 static struct {
@@ -77,20 +77,26 @@ static struct {
 	{ STATS_NOINPUT,      "no input file                  ", NULL, 0 },
 	{ STATS_BADEXTRAFILE, "error hashing extra file       ", NULL, 0 },
 	{ STATS_NUMFILES,     "files in cache                 ", NULL, FLAG_NOZERO|FLAG_ALWAYS },
-	{ STATS_TOTALSIZE,    "cache size                     ", display_size , FLAG_NOZERO|FLAG_ALWAYS },
+	{ STATS_TOTALSIZE,    "cache size                     ", display_size_times_1024 , FLAG_NOZERO|FLAG_ALWAYS },
 	{ STATS_OBSOLETE_MAXFILES, "OBSOLETE",                   NULL, FLAG_NOZERO|FLAG_NEVER},
 	{ STATS_OBSOLETE_MAXSIZE, "OBSOLETE",                    NULL, FLAG_NOZERO|FLAG_NEVER},
 	{ STATS_NONE, NULL, NULL, 0 }
 };
 
 static void
-display_size(size_t v)
+display_size(uint64_t size)
 {
-	char *s = format_human_readable_size(v);
+	char *s = format_human_readable_size(size);
 	printf("%15s", s);
 	free(s);
 }
 
+static void
+display_size_times_1024(uint64_t size)
+{
+	display_size(size * 1024);
+}
+
 /* parse a stats file from a buffer - adding to the counters */
 static void
 parse_stats(struct counters *counters, const char *buf)
@@ -159,14 +165,14 @@ init_counter_updates(void)
  * number of bytes and files have been added to the cache. Size is in KiB.
  */
 void
-stats_update_size(enum stats stat, size_t size, unsigned files)
+stats_update_size(enum stats stat, uint64_t size, unsigned files)
 {
 	init_counter_updates();
 	if (stat != STATS_NONE) {
 		counter_updates->data[stat]++;
 	}
 	counter_updates->data[STATS_NUMFILES] += files;
-	counter_updates->data[STATS_TOTALSIZE] += size;
+	counter_updates->data[STATS_TOTALSIZE] += size / 1024;
 }
 
 /* Read in the stats from one directory and add to the counters. */
@@ -244,7 +250,7 @@ stats_flush(void)
 		need_cleanup = true;
 	}
 	if (conf->max_size != 0
-	    && counters->data[STATS_TOTALSIZE] > conf->max_size / 16) {
+	    && counters->data[STATS_TOTALSIZE] * 1024 > conf->max_size / 16) {
 		need_cleanup = true;
 	}
 
@@ -361,13 +367,13 @@ stats_zero(void)
 
 /* Get the per directory limits */
 void
-stats_get_obsolete_limits(const char *dir, unsigned *maxfiles, unsigned *maxsize)
+stats_get_obsolete_limits(const char *dir, unsigned *maxfiles, uint64_t *maxsize)
 {
 	struct counters *counters = counters_init(STATS_END);
 	char *sname = format("%s/stats", dir);
 	stats_read(sname, counters);
 	*maxfiles = counters->data[STATS_OBSOLETE_MAXFILES];
-	*maxsize = counters->data[STATS_OBSOLETE_MAXSIZE];
+	*maxsize = counters->data[STATS_OBSOLETE_MAXSIZE] * 1024;
 	free(sname);
 	counters_free(counters);
 }
@@ -384,7 +390,7 @@ stats_set_sizes(const char *dir, size_t num_files, size_t total_size)
 	if (lockfile_acquire(statsfile, lock_staleness_limit)) {
 		stats_read(statsfile, counters);
 		counters->data[STATS_NUMFILES] = num_files;
-		counters->data[STATS_TOTALSIZE] = total_size;
+		counters->data[STATS_TOTALSIZE] = total_size / 1024;
 		stats_write(statsfile, counters);
 		lockfile_release(statsfile);
 	}
diff --git a/test/test_conf.c b/test/test_conf.c
index 2ee60012d..83c1b696b 100644
--- a/test/test_conf.c
+++ b/test/test_conf.c
@@ -53,7 +53,7 @@ TEST(conf_create)
 	CHECK(!conf->hash_dir);
 	CHECK_STR_EQ("", conf->log_file);
 	CHECK_INT_EQ(0, conf->max_files);
-	CHECK_INT_EQ(1024*1024, conf->max_size);
+	CHECK_INT_EQ(1024 * 1024 * 1024, conf->max_size);
 	CHECK_STR_EQ("", conf->path);
 	CHECK_STR_EQ("", conf->prefix_command);
 	CHECK(!conf->read_only);
@@ -123,7 +123,7 @@ TEST(conf_read_valid_config)
 	CHECK(conf->hash_dir);
 	CHECK_STR_EQ_FREE1(format("%s%s", user, user), conf->log_file);
 	CHECK_INT_EQ(17, conf->max_files);
-	CHECK_INT_EQ(123 * 1024, conf->max_size);
+	CHECK_INT_EQ(123 * 1024 * 1024, conf->max_size);
 	CHECK_STR_EQ_FREE1(format("%s.x", user), conf->path);
 	CHECK_STR_EQ_FREE1(format("x%s", user), conf->prefix_command);
 	CHECK(conf->read_only);
diff --git a/test/test_util.c b/test/test_util.c
index f00c03664..0d9319170 100644
--- a/test/test_util.c
+++ b/test/test_util.c
@@ -96,47 +96,50 @@ TEST(subst_env_in_string)
 
 TEST(format_human_readable_size)
 {
-	CHECK_STR_EQ_FREE2("0 Kbytes", format_human_readable_size(0));
-	CHECK_STR_EQ_FREE2("42 Kbytes", format_human_readable_size(42));
-	CHECK_STR_EQ_FREE2("1.0 Mbytes", format_human_readable_size(1024));
-	CHECK_STR_EQ_FREE2("1.2 Mbytes", format_human_readable_size(1234));
-	CHECK_STR_EQ_FREE2("438.5 Mbytes", format_human_readable_size(438.5 * 1024));
-	CHECK_STR_EQ_FREE2("1.0 Gbytes", format_human_readable_size(1024 * 1024));
+	CHECK_STR_EQ_FREE2("0 bytes", format_human_readable_size(0));
+	CHECK_STR_EQ_FREE2("42.0 Kbytes", format_human_readable_size(42 * 1024));
+	CHECK_STR_EQ_FREE2("1.0 Mbytes", format_human_readable_size(1024 * 1024));
+	CHECK_STR_EQ_FREE2("1.2 Mbytes", format_human_readable_size(1234 * 1024));
+	CHECK_STR_EQ_FREE2("438.5 Mbytes",
+	                   format_human_readable_size(438.5 * 1024 * 1024));
+	CHECK_STR_EQ_FREE2("1.0 Gbytes",
+	                   format_human_readable_size(1024 * 1024 * 1024));
 	CHECK_STR_EQ_FREE2("17.1 Gbytes",
-	                   format_human_readable_size(17.11 * 1024 * 1024));
+	                   format_human_readable_size(17.11 * 1024 * 1024 * 1024));
 }
 
 TEST(format_parsable_size_with_suffix)
 {
 	CHECK_STR_EQ_FREE2("0", format_parsable_size_with_suffix(0));
-	CHECK_STR_EQ_FREE2("42K", format_parsable_size_with_suffix(42));
-	CHECK_STR_EQ_FREE2("1.0M", format_parsable_size_with_suffix(1024));
-	CHECK_STR_EQ_FREE2("1.2M", format_parsable_size_with_suffix(1234));
+	CHECK_STR_EQ_FREE2("42.0K", format_parsable_size_with_suffix(42 * 1024));
+	CHECK_STR_EQ_FREE2("1.0M", format_parsable_size_with_suffix(1024 * 1024));
+	CHECK_STR_EQ_FREE2("1.2M", format_parsable_size_with_suffix(1234 * 1024));
 	CHECK_STR_EQ_FREE2("438.5M",
-	                   format_parsable_size_with_suffix(438.5 * 1024));
+	                   format_parsable_size_with_suffix(438.5 * 1024 * 1024));
 	CHECK_STR_EQ_FREE2("1.0G",
-	                   format_parsable_size_with_suffix(1024 * 1024));
-	CHECK_STR_EQ_FREE2("17.1G",
-	                   format_parsable_size_with_suffix(17.11 * 1024 * 1024));
+	                   format_parsable_size_with_suffix(1024 * 1024 * 1024));
+	CHECK_STR_EQ_FREE2(
+		"17.1G",
+		format_parsable_size_with_suffix(17.11 * 1024 * 1024 * 1024));
 }
 
 TEST(parse_size_with_suffix)
 {
-	size_t size;
+	uint64_t size;
 	CHECK(parse_size_with_suffix("0", &size));
 	CHECK_INT_EQ(0, size);
 	CHECK(parse_size_with_suffix("42K", &size));
-	CHECK_INT_EQ(42, size);
+	CHECK_INT_EQ(42 * 1024, size);
 	CHECK(parse_size_with_suffix("1.0M", &size));
-	CHECK_INT_EQ(1024, size);
+	CHECK_INT_EQ(1024 * 1024, size);
 	CHECK(parse_size_with_suffix("1.1M", &size));
-	CHECK_INT_EQ(1.1 * 1024, size);
+	CHECK_INT_EQ(1.1 * 1024 * 1024, size);
 	CHECK(parse_size_with_suffix("438.5M", &size));
-	CHECK_INT_EQ(438.5 * 1024, size);
+	CHECK_INT_EQ(438.5 * 1024 * 1024, size);
 	CHECK(parse_size_with_suffix("1.0G", &size));
-	CHECK_INT_EQ(1024 * 1024, size);
+	CHECK_INT_EQ(1024 * 1024 * 1024, size);
 	CHECK(parse_size_with_suffix("17.1G", &size));
-	CHECK_INT_EQ(17.1 * 1024 * 1024, size);
+	CHECK_INT_EQ(17.1 * 1024 * 1024 * 1024, size);
 }
 
 TEST_SUITE_END
diff --git a/util.c b/util.c
index 860cd71d9..74e9646e3 100644
--- a/util.c
+++ b/util.c
@@ -814,44 +814,45 @@ safe_create_wronly(const char *fname)
 	return fd;
 }
 
-/* Format a size (in KiB) as a human-readable string. Caller frees. */
+/* Format a size as a human-readable string. Caller frees. */
 char *
-format_human_readable_size(size_t v)
+format_human_readable_size(uint64_t v)
 {
 	char *s;
-	if (v >= 1024*1024) {
-		s = format("%.1f Gbytes", v/((double)(1024*1024)));
+	if (v >= 1024*1024*1024) {
+		s = format("%.1f Gbytes", v/((double)(1024*1024*1024)));
+	} else if (v >= 1024*1024) {
+		s = format("%.1f Mbytes", v/((double)(1024*1024)));
 	} else if (v >= 1024) {
-		s = format("%.1f Mbytes", v/((double)(1024)));
+		s = format("%.1f Kbytes", v/((double)(1024)));
 	} else {
-		s = format("%.0f Kbytes", (double)v);
+		s = format("%u bytes", (unsigned)v);
 	}
 	return s;
 }
 
 /* Format a size (in KiB) as a human-readable string. Caller frees. */
 char *
-format_parsable_size_with_suffix(size_t size)
+format_parsable_size_with_suffix(uint64_t size)
 {
 	char *s;
-	if (size >= 1024*1024) {
-		s = format("%.1fG", size / ((double)(1024*1024)));
+	if (size >= 1024*1024*1024) {
+		s = format("%.1fG", size / ((double)(1024*1024*1024)));
+	} else if (size >= 1024*1024) {
+		s = format("%.1fM", size / ((double)(1024*1024)));
 	} else if (size >= 1024) {
-		s = format("%.1fM", size / ((double)(1024)));
-	} else if (size > 0) {
-		s = format("%.0fK", (double)size);
+		s = format("%.1fK", size / ((double)(1024)));
 	} else {
-		s = x_strdup("0");
+		s = format("%u", (unsigned)size);
 	}
 	return s;
 }
 
 /*
- * Parse a value in multiples of 1024 given a string that can end in K, M or G.
- * Default suffix: G.
+ * Parse a value given a string that can end in K, M or G. Default suffix: G.
  */
 bool
-parse_size_with_suffix(const char *str, size_t *size)
+parse_size_with_suffix(const char *str, uint64_t *size)
 {
 	char *endptr;
 	double x;
@@ -877,7 +878,7 @@ parse_size_with_suffix(const char *str, size_t *size)
 	default:
 		return false;
 	}
-	*size = x;
+	*size = x * 1024;
 	return true;
 }