From: Christoph Hellwig <hch@lst.de> To: Jens Axboe <axboe@kernel.dk> Cc: "Martin K. Petersen" <martin.petersen@oracle.com>, Miquel Raynal <miquel.raynal@bootlin.com>, Richard Weinberger <richard@nod.at>, Vignesh Raghavendra <vigneshr@ti.com>, linux-block@vger.kernel.org, linux-scsi@vger.kernel.org, linux-mtd@lists.infradead.org Subject: [PATCH 11/12] block: move blk_print_req_error to blk-mq.c Date: Mon, 25 Oct 2021 09:05:16 +0200 [thread overview] Message-ID: <20211025070517.1548584-12-hch@lst.de> (raw) In-Reply-To: <20211025070517.1548584-1-hch@lst.de> This function is only used by the request completion path. Factor out a blk_status_to_str to keep blk_errors private in blk-core.c. Signed-off-by: Christoph Hellwig <hch@lst.de> --- block/blk-core.c | 15 +++------------ block/blk-mq.c | 13 +++++++++++++ block/blk.h | 2 +- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/block/blk-core.c b/block/blk-core.c index 9547a75e7a7aa..ebcea0dac973d 100644 --- a/block/blk-core.c +++ b/block/blk-core.c @@ -199,22 +199,13 @@ int blk_status_to_errno(blk_status_t status) } EXPORT_SYMBOL_GPL(blk_status_to_errno); -void blk_print_req_error(struct request *req, blk_status_t status) +const char *blk_status_to_str(blk_status_t status) { int idx = (__force int)status; if (WARN_ON_ONCE(idx >= ARRAY_SIZE(blk_errors))) - return; - - printk_ratelimited(KERN_ERR - "%s error, dev %s, sector %llu op 0x%x:(%s) flags 0x%x " - "phys_seg %u prio class %u\n", - blk_errors[idx].name, - req->rq_disk ? req->rq_disk->disk_name : "?", - blk_rq_pos(req), req_op(req), blk_op_str(req_op(req)), - req->cmd_flags & ~REQ_OP_MASK, - req->nr_phys_segments, - IOPRIO_PRIO_CLASS(req->ioprio)); + return "<null>"; + return blk_errors[idx].name; } /** diff --git a/block/blk-mq.c b/block/blk-mq.c index 425cd134ac358..3edd4ee2007b0 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -698,6 +698,19 @@ static void blk_account_io_completion(struct request *req, unsigned int bytes) } } +static void blk_print_req_error(struct request *req, blk_status_t status) +{ + printk_ratelimited(KERN_ERR + "%s error, dev %s, sector %llu op 0x%x:(%s) flags 0x%x " + "phys_seg %u prio class %u\n", + blk_status_to_str(status), + req->rq_disk ? req->rq_disk->disk_name : "?", + blk_rq_pos(req), req_op(req), blk_op_str(req_op(req)), + req->cmd_flags & ~REQ_OP_MASK, + req->nr_phys_segments, + IOPRIO_PRIO_CLASS(req->ioprio)); +} + /** * blk_update_request - Complete multiple bytes without completing the request * @req: the request being processed diff --git a/block/blk.h b/block/blk.h index 8bcca0538faac..573e308851d27 100644 --- a/block/blk.h +++ b/block/blk.h @@ -215,7 +215,7 @@ static inline void blk_integrity_del(struct gendisk *disk) unsigned long blk_rq_timeout(unsigned long timeout); void blk_add_timer(struct request *req); -void blk_print_req_error(struct request *req, blk_status_t status); +const char *blk_status_to_str(blk_status_t status); bool blk_attempt_plug_merge(struct request_queue *q, struct bio *bio, unsigned int nr_segs, bool *same_queue_rq); -- 2.30.2
WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de> To: Jens Axboe <axboe@kernel.dk> Cc: "Martin K. Petersen" <martin.petersen@oracle.com>, Miquel Raynal <miquel.raynal@bootlin.com>, Richard Weinberger <richard@nod.at>, Vignesh Raghavendra <vigneshr@ti.com>, linux-block@vger.kernel.org, linux-scsi@vger.kernel.org, linux-mtd@lists.infradead.org Subject: [PATCH 11/12] block: move blk_print_req_error to blk-mq.c Date: Mon, 25 Oct 2021 09:05:16 +0200 [thread overview] Message-ID: <20211025070517.1548584-12-hch@lst.de> (raw) In-Reply-To: <20211025070517.1548584-1-hch@lst.de> This function is only used by the request completion path. Factor out a blk_status_to_str to keep blk_errors private in blk-core.c. Signed-off-by: Christoph Hellwig <hch@lst.de> --- block/blk-core.c | 15 +++------------ block/blk-mq.c | 13 +++++++++++++ block/blk.h | 2 +- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/block/blk-core.c b/block/blk-core.c index 9547a75e7a7aa..ebcea0dac973d 100644 --- a/block/blk-core.c +++ b/block/blk-core.c @@ -199,22 +199,13 @@ int blk_status_to_errno(blk_status_t status) } EXPORT_SYMBOL_GPL(blk_status_to_errno); -void blk_print_req_error(struct request *req, blk_status_t status) +const char *blk_status_to_str(blk_status_t status) { int idx = (__force int)status; if (WARN_ON_ONCE(idx >= ARRAY_SIZE(blk_errors))) - return; - - printk_ratelimited(KERN_ERR - "%s error, dev %s, sector %llu op 0x%x:(%s) flags 0x%x " - "phys_seg %u prio class %u\n", - blk_errors[idx].name, - req->rq_disk ? req->rq_disk->disk_name : "?", - blk_rq_pos(req), req_op(req), blk_op_str(req_op(req)), - req->cmd_flags & ~REQ_OP_MASK, - req->nr_phys_segments, - IOPRIO_PRIO_CLASS(req->ioprio)); + return "<null>"; + return blk_errors[idx].name; } /** diff --git a/block/blk-mq.c b/block/blk-mq.c index 425cd134ac358..3edd4ee2007b0 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -698,6 +698,19 @@ static void blk_account_io_completion(struct request *req, unsigned int bytes) } } +static void blk_print_req_error(struct request *req, blk_status_t status) +{ + printk_ratelimited(KERN_ERR + "%s error, dev %s, sector %llu op 0x%x:(%s) flags 0x%x " + "phys_seg %u prio class %u\n", + blk_status_to_str(status), + req->rq_disk ? req->rq_disk->disk_name : "?", + blk_rq_pos(req), req_op(req), blk_op_str(req_op(req)), + req->cmd_flags & ~REQ_OP_MASK, + req->nr_phys_segments, + IOPRIO_PRIO_CLASS(req->ioprio)); +} + /** * blk_update_request - Complete multiple bytes without completing the request * @req: the request being processed diff --git a/block/blk.h b/block/blk.h index 8bcca0538faac..573e308851d27 100644 --- a/block/blk.h +++ b/block/blk.h @@ -215,7 +215,7 @@ static inline void blk_integrity_del(struct gendisk *disk) unsigned long blk_rq_timeout(unsigned long timeout); void blk_add_timer(struct request *req); -void blk_print_req_error(struct request *req, blk_status_t status); +const char *blk_status_to_str(blk_status_t status); bool blk_attempt_plug_merge(struct request_queue *q, struct bio *bio, unsigned int nr_segs, bool *same_queue_rq); -- 2.30.2 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/
next prev parent reply other threads:[~2021-10-25 7:05 UTC|newest] Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-10-25 7:05 move all struct request releated code out of blk-core.c Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-25 7:05 ` [PATCH 01/12] block: move blk_rq_err_bytes to scsi Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-26 23:55 ` Chaitanya Kulkarni 2021-10-26 23:55 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 02/12] block: remove blk_{get,put}_request Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-26 23:56 ` Chaitanya Kulkarni 2021-10-26 23:56 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 03/12] block: remove rq_flush_dcache_pages Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-25 7:05 ` [PATCH 04/12] block: remove blk-exec.c Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-26 23:58 ` Chaitanya Kulkarni 2021-10-26 23:58 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 05/12] blk-mq: move blk_mq_flush_plug_list Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:00 ` Chaitanya Kulkarni 2021-10-27 0:00 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 06/12] block: move request based cloning helpers to blk-mq.c Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:01 ` Chaitanya Kulkarni 2021-10-27 0:01 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 07/12] block: move blk_rq_init " Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:01 ` Chaitanya Kulkarni 2021-10-27 0:01 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 08/12] block: move blk_steal_bios " Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:02 ` Chaitanya Kulkarni 2021-10-27 0:02 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 09/12] block: move blk_account_io_{start,done} " Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:04 ` Chaitanya Kulkarni 2021-10-27 0:04 ` [PATCH 09/12] block: move blk_account_io_{start, done} " Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 10/12] block: move blk_dump_rq_flags " Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:04 ` Chaitanya Kulkarni 2021-10-27 0:04 ` Chaitanya Kulkarni 2021-10-25 7:05 ` Christoph Hellwig [this message] 2021-10-25 7:05 ` [PATCH 11/12] block: move blk_print_req_error " Christoph Hellwig 2021-10-27 0:06 ` Chaitanya Kulkarni 2021-10-27 0:06 ` Chaitanya Kulkarni 2021-10-25 7:05 ` [PATCH 12/12] block: don't include blk-mq headers in blk-core.c Christoph Hellwig 2021-10-25 7:05 ` Christoph Hellwig 2021-10-27 0:07 ` Chaitanya Kulkarni 2021-10-27 0:07 ` Chaitanya Kulkarni 2021-10-29 12:52 ` (subset) move all struct request releated code out of blk-core.c Jens Axboe 2021-10-29 12:52 ` Jens Axboe
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20211025070517.1548584-12-hch@lst.de \ --to=hch@lst.de \ --cc=axboe@kernel.dk \ --cc=linux-block@vger.kernel.org \ --cc=linux-mtd@lists.infradead.org \ --cc=linux-scsi@vger.kernel.org \ --cc=martin.petersen@oracle.com \ --cc=miquel.raynal@bootlin.com \ --cc=richard@nod.at \ --cc=vigneshr@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.