diff options
author | Omar Sandoval <osandov@fb.com> | 2018-05-09 05:08:53 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2018-05-09 10:33:09 -0400 |
commit | 522a777566f5669606a1227bf13f3fb40963780b (patch) | |
tree | 7d37c9d591e5a9abb4fd1209f1c503ae1a0e8475 /block/bfq-iosched.c | |
parent | 4bc6339a583cec650b05d6fdcc83f03f941c1a3b (diff) |
block: consolidate struct request timestamp fields
Currently, struct request has four timestamp fields:
- A start time, set at get_request time, in jiffies, used for iostats
- An I/O start time, set at start_request time, in ktime nanoseconds,
used for blk-stats (i.e., wbt, kyber, hybrid polling)
- Another start time and another I/O start time, used for cfq and bfq
These can all be consolidated into one start time and one I/O start
time, both in ktime nanoseconds, shaving off up to 16 bytes from struct
request depending on the kernel config.
Signed-off-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/bfq-iosched.c')
-rw-r--r-- | block/bfq-iosched.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/bfq-iosched.c b/block/bfq-iosched.c index 771ae9730ac6..ebc264c87a09 100644 --- a/block/bfq-iosched.c +++ b/block/bfq-iosched.c | |||
@@ -4778,8 +4778,8 @@ static void bfq_finish_requeue_request(struct request *rq) | |||
4778 | 4778 | ||
4779 | if (rq->rq_flags & RQF_STARTED) | 4779 | if (rq->rq_flags & RQF_STARTED) |
4780 | bfqg_stats_update_completion(bfqq_group(bfqq), | 4780 | bfqg_stats_update_completion(bfqq_group(bfqq), |
4781 | rq_start_time_ns(rq), | 4781 | rq->start_time_ns, |
4782 | rq_io_start_time_ns(rq), | 4782 | rq->io_start_time_ns, |
4783 | rq->cmd_flags); | 4783 | rq->cmd_flags); |
4784 | 4784 | ||
4785 | if (likely(rq->rq_flags & RQF_STARTED)) { | 4785 | if (likely(rq->rq_flags & RQF_STARTED)) { |