From 766ae96eaf63c874a8eedf2e201f716c92070a0d Mon Sep 17 00:00:00 2001 From: Konrad Sztyber Date: Tue, 21 Feb 2023 11:04:25 +0100 Subject: [PATCH] bdev: adjust comment on spdk_bdev_io.internal.link This field hasn't been used for linking child IOs for a long time, so adjust the comment to reflect its current uses. Signed-off-by: Konrad Sztyber Change-Id: I5775434e466ee1e3ad6e66cc94a16a4800da6b27 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16976 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Aleksey Marchuk --- include/spdk/bdev_module.h | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/include/spdk/bdev_module.h b/include/spdk/bdev_module.h index a32bdd496..97d069606 100644 --- a/include/spdk/bdev_module.h +++ b/include/spdk/bdev_module.h @@ -972,7 +972,12 @@ struct spdk_bdev_io { /** Callback for when buf is allocated */ spdk_bdev_io_get_buf_cb get_buf_cb; - /** Member used for linking child I/Os together. */ + /** + * Queue entry used in several cases: + * 1. IOs awaiting retry due to NOMEM status, + * 2. IOs awaiting submission due to QoS, + * 3. queued reset requests. + */ TAILQ_ENTRY(spdk_bdev_io) link; /** Entry to the list need_buf of struct spdk_bdev. */