block: Introduce get_current_ioprio()
authorDamien Le Moal <damien.lemoal@wdc.com>
Tue, 20 Nov 2018 01:52:35 +0000 (10:52 +0900)
committerJens Axboe <axboe@kernel.dk>
Tue, 20 Nov 2018 02:03:46 +0000 (19:03 -0700)
Define get_current_ioprio() as an inline helper to obtain the caller
I/O priority from its task I/O context. Use this helper in
blk_init_request_from_bio() to set a request ioprio.

Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-core.c
include/linux/ioprio.h

index 492648c969925b263be61dd3757353befdce2087..4450d3c08f255170c432a176c3f4ae05e0fbe234 100644 (file)
@@ -813,18 +813,14 @@ out:
 
 void blk_init_request_from_bio(struct request *req, struct bio *bio)
 {
-       struct io_context *ioc = current->io_context;
-
        if (bio->bi_opf & REQ_RAHEAD)
                req->cmd_flags |= REQ_FAILFAST_MASK;
 
        req->__sector = bio->bi_iter.bi_sector;
        if (ioprio_valid(bio_prio(bio)))
                req->ioprio = bio_prio(bio);
-       else if (ioc)
-               req->ioprio = ioc->ioprio;
        else
-               req->ioprio = IOPRIO_PRIO_VALUE(IOPRIO_CLASS_NONE, 0);
+               req->ioprio = get_current_ioprio();
        req->write_hint = bio->bi_write_hint;
        blk_rq_bio_prep(req->q, req, bio);
 }
index 9e30ed6443dbc20b8db1afd42f86a39860a6a13d..e9bfe6972aed9f326cd42de03db6f47312732cd3 100644 (file)
@@ -70,6 +70,19 @@ static inline int task_nice_ioclass(struct task_struct *task)
                return IOPRIO_CLASS_BE;
 }
 
+/*
+ * If the calling process has set an I/O priority, use that. Otherwise, return
+ * the default I/O priority.
+ */
+static inline int get_current_ioprio(void)
+{
+       struct io_context *ioc = current->io_context;
+
+       if (ioc)
+               return ioc->ioprio;
+       return IOPRIO_PRIO_VALUE(IOPRIO_CLASS_NONE, 0);
+}
+
 /*
  * For inheritance, return the highest of the two given priorities
  */