diff options
author | Omar Sandoval <osandov@fb.com> | 2018-05-09 05:08:50 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2018-05-09 10:33:05 -0400 |
commit | 544ccc8dc904db55d4576c27a1eb66a888ffacea (patch) | |
tree | 5fa92e3cf1a5d33eddc61dfd65d088dd5cdb6f84 /block/blk-wbt.h | |
parent | 5238dcf4136fd7287be8e7d38752645bfa5782ec (diff) |
block: get rid of struct blk_issue_stat
struct blk_issue_stat squashes three things into one u64:
- The time the driver started working on a request
- The original size of the request (for the io.low controller)
- Flags for writeback throttling
It turns out that on x86_64, we have a 4 byte hole in struct request
which we can fill with the non-timestamp fields from blk_issue_stat,
simplifying things quite a bit.
Signed-off-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/blk-wbt.h')
-rw-r--r-- | block/blk-wbt.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/blk-wbt.h b/block/blk-wbt.h index 85fbcccf12a5..300df531d0a6 100644 --- a/block/blk-wbt.h +++ b/block/blk-wbt.h | |||
@@ -63,7 +63,7 @@ struct rq_wb { | |||
63 | 63 | ||
64 | struct blk_stat_callback *cb; | 64 | struct blk_stat_callback *cb; |
65 | 65 | ||
66 | s64 sync_issue; | 66 | u64 sync_issue; |
67 | void *sync_cookie; | 67 | void *sync_cookie; |
68 | 68 | ||
69 | unsigned int wc; | 69 | unsigned int wc; |
@@ -90,7 +90,7 @@ static inline unsigned int wbt_inflight(struct rq_wb *rwb) | |||
90 | 90 | ||
91 | static inline void wbt_track(struct request *rq, enum wbt_flags flags) | 91 | static inline void wbt_track(struct request *rq, enum wbt_flags flags) |
92 | { | 92 | { |
93 | rq->issue_stat.stat |= ((u64)flags) << BLK_STAT_RES_SHIFT; | 93 | rq->wbt_flags |= flags; |
94 | } | 94 | } |
95 | 95 | ||
96 | void __wbt_done(struct rq_wb *, enum wbt_flags); | 96 | void __wbt_done(struct rq_wb *, enum wbt_flags); |