options: fix bs_cmp()
authorJens Axboe <axboe@fb.com>
Tue, 12 Jan 2016 17:18:50 +0000 (10:18 -0700)
committerJens Axboe <axboe@fb.com>
Tue, 12 Jan 2016 17:18:50 +0000 (10:18 -0700)
Return the int difference between the two, not a bool.

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

index 871552b..0bf1390 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;
 
-       return bsp1->perc < bsp2->perc;
+       return (int) bsp1->perc - (int) bsp2->perc;
 }
 
 static int bssplit_ddir(struct thread_options *o, int ddir, char *str)