blktrace: kill zero sized write test
authorJens Axboe <axboe@kernel.dk>
Thu, 26 Apr 2018 21:10:42 +0000 (15:10 -0600)
committerJens Axboe <axboe@kernel.dk>
Thu, 26 Apr 2018 21:10:42 +0000 (15:10 -0600)
Turns out they were flushes. Get rid of the check and turn it into
an assert.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
blktrace.c

index b37a54e9e1cd36ec75dffb7e9a03bfbec8e3f886..e256d633d929cd5f7b60936bdb39bb84cddc9b48 100644 (file)
@@ -312,13 +312,9 @@ static void handle_trace_fs(struct thread_data *td, struct blk_io_trace *t,
 
        rw = (t->action & BLK_TC_ACT(BLK_TC_WRITE)) != 0;
 
 
        rw = (t->action & BLK_TC_ACT(BLK_TC_WRITE)) != 0;
 
-       /*
-        * Need to figure out why 0 byte writes end up here sometimes, for
-        * now just ignore them.
-        */
-       if (!t->bytes)
-               return;
-       else if (t->bytes > rw_bs[rw])
+       assert(t->bytes);
+
+       if (t->bytes > rw_bs[rw])
                rw_bs[rw] = t->bytes;
 
        ios[rw]++;
                rw_bs[rw] = t->bytes;
 
        ios[rw]++;