Re: [PATCH v9 1/8] blk-mq: Document the functions that iterate over requests
From: Johannes Thumshirn <hidden>
Date: 2018-09-20 07:35:27
On Wed, Sep 19, 2018 at 03:45:23PM -0700, Bart Van Assche wrote:
Make it easier to understand the purpose of the functions that iterate over requests by documenting their purpose. Fix three minor spelling mistakes in comments in these functions.
[...]
+/* + * Call function @fn(@hctx, rq, @data, @reserved) for each request queued on + * @hctx that has been assigned a driver tag. @reserved indicates whether @bt + * is the breserved_tags member or the bitmap_tags member of struct + * blk_mq_tags. + */ static void bt_for_each(struct blk_mq_hw_ctx *hctx, struct sbitmap_queue *bt, busy_iter_fn *fn, void *data, bool reserved)
One minor nit and only if you have to re-send the series, can you please also add the usual kernel-doc headers to the comments, i.e.: /** * bt_for_each() - run callback on each hctx with a driver tag * @hctx: the hardware context * @bt: the sbitmap_queue * @fn: the callback function to run * @data: the data for the callback function * @reserved: is reserved or not * * Call function @fn(@hctx, rq, @data, @reserved) for each request queued on * @hctx that has been assigned a driver tag. @reserved indicates whether @bt * is the breserved_tags member or the bitmap_tags member of struct * blk_mq_tags. */ static void bt_for_each(struct blk_mq_hw_ctx *hctx, struct sbitmap_queue *bt, busy_iter_fn *fn, void *data, bool reserved) Otherwise: Reviewed-by: Johannes Thumshirn <redacted> -- Johannes Thumshirn Storage jthumshirn@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N�rnberg GF: Felix Imend�rffer, Jane Smithard, Graham Norton HRB 21284 (AG N�rnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850