summaryrefslogtreecommitdiffstats
path: root/block
diff options
context:
space:
mode:
authorJens Axboe <axboe@fb.com>2017-04-21 07:55:42 -0600
committerJens Axboe <axboe@fb.com>2017-04-21 07:56:23 -0600
commit99c749a4c4f4ea2c9eee01f81f79bdbf26c3914e (patch)
tree71cdbf2a4d9bcaeb26487403f88a66011f3ecfe9 /block
parent1cc1f17aab7e596d0a7373fc1ed11dbddfa82bc9 (diff)
downloadlinux-99c749a4c4f4ea2c9eee01f81f79bdbf26c3914e.tar.bz2
blk-stat: kill blk_stat_rq_ddir()
No point in providing and exporting this helper. There's just one (real) user of it, just use rq_data_dir(). Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'block')
-rw-r--r--block/blk-mq.c2
-rw-r--r--block/blk-stat.c6
-rw-r--r--block/blk-stat.h11
-rw-r--r--block/blk-wbt.c7
4 files changed, 7 insertions, 19 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c
index fa7d86f59687..0b2f60407748 100644
--- a/block/blk-mq.c
+++ b/block/blk-mq.c
@@ -46,7 +46,7 @@ static int blk_mq_poll_stats_bkt(const struct request *rq)
{
int ddir, bytes, bucket;
- ddir = blk_stat_rq_ddir(rq);
+ ddir = rq_data_dir(rq);
bytes = blk_rq_bytes(rq);
bucket = ddir + 2*(ilog2(bytes) - 9);
diff --git a/block/blk-stat.c b/block/blk-stat.c
index dde9d399f707..6c2f40940439 100644
--- a/block/blk-stat.c
+++ b/block/blk-stat.c
@@ -19,12 +19,6 @@ struct blk_queue_stats {
bool enable_accounting;
};
-int blk_stat_rq_ddir(const struct request *rq)
-{
- return rq_data_dir(rq);
-}
-EXPORT_SYMBOL_GPL(blk_stat_rq_ddir);
-
static void blk_stat_init(struct blk_rq_stat *stat)
{
stat->min = -1ULL;
diff --git a/block/blk-stat.h b/block/blk-stat.h
index 622a62ce6213..2fb20d1a341a 100644
--- a/block/blk-stat.h
+++ b/block/blk-stat.h
@@ -112,17 +112,6 @@ static inline void blk_stat_set_issue(struct blk_issue_stat *stat,
/* record time/size info in request but not add a callback */
void blk_stat_enable_accounting(struct request_queue *q);
-/*
- * blk_stat_rq_ddir() - Bucket callback function for the request data direction.
- * @rq: Request.
- *
- * This is the same as rq_data_dir() but as a function so it can be used as
- * @bucket_fn for blk_stat_alloc_callback().
- *
- * Return: Data direction of the request, either READ or WRITE.
- */
-int blk_stat_rq_ddir(const struct request *rq);
-
/**
* blk_stat_alloc_callback() - Allocate a block statistics callback.
* @timer_fn: Timer callback function.
diff --git a/block/blk-wbt.c b/block/blk-wbt.c
index 26e1bb617877..17676f4d7fd1 100644
--- a/block/blk-wbt.c
+++ b/block/blk-wbt.c
@@ -696,6 +696,11 @@ u64 wbt_default_latency_nsec(struct request_queue *q)
return 75000000ULL;
}
+static int wbt_data_dir(const struct request *rq)
+{
+ return rq_data_dir(rq);
+}
+
int wbt_init(struct request_queue *q)
{
struct rq_wb *rwb;
@@ -707,7 +712,7 @@ int wbt_init(struct request_queue *q)
if (!rwb)
return -ENOMEM;
- rwb->cb = blk_stat_alloc_callback(wb_timer_fn, blk_stat_rq_ddir, 2, rwb);
+ rwb->cb = blk_stat_alloc_callback(wb_timer_fn, wbt_data_dir, 2, rwb);
if (!rwb->cb) {
kfree(rwb);
return -ENOMEM;