Patchwork [03/27] fs-tests: integck: srink file_info structure even more

login
register
mail settings
Submitter Artem Bityutskiy
Date April 13, 2011, 3:18 p.m.
Message ID <1302707947-6143-4-git-send-email-dedekind1@gmail.com>
Download mbox | patch
Permalink /patch/91030/
State New
Headers show

Comments

Artem Bityutskiy - April 13, 2011, 3:18 p.m.
From: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

The 'check_run_no' field does not have to be 64-bit, it can be
32-bit instead and we can save 4 bytes per object. Actually, it
could also be 16-bit, and we could make link_count 16-bit as well,
but that would need a bit more work.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
---
 tests/fs-tests/integrity/integck.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Patch

diff --git a/tests/fs-tests/integrity/integck.c b/tests/fs-tests/integrity/integck.c
index f203e3a..0ff94c5 100644
--- a/tests/fs-tests/integrity/integck.c
+++ b/tests/fs-tests/integrity/integck.c
@@ -60,9 +60,9 @@  struct file_info /* Each file has one of these */
 	struct dir_entry_info *links;
 	off_t length;
 	int link_count;
+	unsigned int check_run_no; /* Run number used when checking */
 	unsigned int deleted:1; /* File has been deleted but is still open */
 	unsigned int no_space_error:1; /* File has incurred a ENOSPC error */
-	uint64_t check_run_no; /* Run number used when checking */
 };
 
 struct symlink_info /* Each symlink has one of these */
@@ -134,7 +134,7 @@  static int can_mmap = 0; /* Can write via mmap */
 
 static long mem_page_size; /* Page size for mmap */
 
-static uint64_t check_run_no;
+static unsigned int check_run_no;
 
 /*
  * Allocate a buffer of 'size' bytes and fill it with zeroes.