commit f45d32be3a9f4a8b21d4637235ecf5dee9d951e4
author Shu-Yu Fu <shuyufu@gmail.com> Tue Oct 21 07:57:55 2014 -0600
committer Mohamad Ayyash <mkayyash@google.com> Fri Mar 06 17:57:25 2015 -0800
tree 6e7c2bd037bf92e01c78601551da27f726013060
parent 525973851571de29f033758b3dc6528db9e66fc7

Fix typo in alignment check

It checks percentile_list twice and forgets latency_percentile
in the thread_options_pack structure. Add alignment after the
32-bit sync_file_range to make it align properly.

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

libfio.c

diff --git a/libfio.c b/libfio.c
index 523cc36..9ed802a 100644
--- a/libfio.c
+++ b/libfio.c
@@ -307,7 +307,7 @@
 	compiletime_assert((offsetof(struct thread_options_pack, zipf_theta) % 8) == 0, "zipf_theta");
 	compiletime_assert((offsetof(struct thread_options_pack, pareto_h) % 8) == 0, "pareto_h");
 	compiletime_assert((offsetof(struct thread_options_pack, percentile_list) % 8) == 0, "percentile_list");
-	compiletime_assert((offsetof(struct thread_options_pack, percentile_list) % 8) == 0, "latency_percentile");
+	compiletime_assert((offsetof(struct thread_options_pack, latency_percentile) % 8) == 0, "latency_percentile");
 
 	if (endian_check()) {
 		log_err("fio: endianness settings appear wrong.\n");

thread_options.h

diff --git a/thread_options.h b/thread_options.h
index 5c9910a..ecf4e47 100644
--- a/thread_options.h
+++ b/thread_options.h
@@ -477,6 +477,7 @@
 	uint64_t number_ios;
 
 	uint32_t sync_file_range;
+	uint32_t pad2;
 
 	uint64_t latency_target;
 	uint64_t latency_window;