options: fix bs_cmp()

Return the int difference between the two, not a bool.

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

diff --git a/options.c b/options.c
index 871552b5b4eefa1f355f03532981e566d3d6d15b..0bf1390a0d9f19c4f99d9f3e44582aca446e1c67 100644 (file)
--- a/options.c
+++ b/options.c
@@ -41,7 +41,7 @@ static int bs_cmp(const void *p1, const void *p2)
        const struct bssplit *bsp1 = p1;
        const struct bssplit *bsp2 = p2;
 
        const struct bssplit *bsp1 = p1;
        const struct bssplit *bsp2 = p2;
 

-       return bsp1->perc < bsp2->perc;
+       return (int) bsp1->perc - (int) bsp2->perc;

 }
 
 static int bssplit_ddir(struct thread_options *o, int ddir, char *str)
 }
 
 static int bssplit_ddir(struct thread_options *o, int ddir, char *str)