Re: [PATCH 1/2] blk-mq-tag: change busy_iter_fn to return whether to continue or not

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2018-11-08 at 10:50 -0700, Jens Axboe wrote:
+AD4 How about this incremental?
+AD4 
+AD4 diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c
+AD4 index 097e9a67d5f5..87bc5df72d48 100644
+AD4 --- a/block/blk-mq-tag.c
+AD4 +-+-+- b/block/blk-mq-tag.c
+AD4 +AEAAQA -248,7 +-248,8 +AEAAQA static bool bt+AF8-iter(struct sbitmap +ACo-bitmap, unsigned int bitnr, void +ACo-data)
+AD4   +ACo +AEA-fn:		Pointer to the function that will be called for each request
+AD4   +ACo		associated with +AEA-hctx that has been assigned a driver tag.
+AD4   +ACo		+AEA-fn will be called as follows: +AEA-fn(+AEA-hctx, rq, +AEA-data, +AEA-reserved)
+AD4 - +ACo		where rq is a pointer to a request.
+AD4 +- +ACo		where rq is a pointer to a request. Return true to continue
+AD4 +- +ACo		iterating tags, false to stop.
+AD4   +ACo +AEA-data:	Will be passed as third argument to +AEA-fn.
+AD4   +ACo +AEA-reserved:	Indicates whether +AEA-bt is the breserved+AF8-tags member or the
+AD4   +ACo		bitmap+AF8-tags member of struct blk+AF8-mq+AF8-tags.
+AD4 +AEAAQA -301,7 +-302,8 +AEAAQA static bool bt+AF8-tags+AF8-iter(struct sbitmap +ACo-bitmap, unsigned int bitnr, void +ACo-data)
+AD4   +ACo		or the bitmap+AF8-tags member of struct blk+AF8-mq+AF8-tags.
+AD4   +ACo +AEA-fn:		Pointer to the function that will be called for each started
+AD4   +ACo		request. +AEA-fn will be called as follows: +AEA-fn(rq, +AEA-data,
+AD4 - +ACo		+AEA-reserved) where rq is a pointer to a request.
+AD4 +- +ACo		+AEA-reserved) where rq is a pointer to a request. Return true
+AD4 +- +ACo		to continue iterating tags, false to stop.
+AD4   +ACo +AEA-data:	Will be passed as second argument to +AEA-fn.
+AD4   +ACo +AEA-reserved:	Indicates whether +AEA-bt is the breserved+AF8-tags member or the
+AD4   +ACo		bitmap+AF8-tags member of struct blk+AF8-mq+AF8-tags.
+AD4 +AEAAQA -326,7 +-328,8 +AEAAQA static void bt+AF8-tags+AF8-for+AF8-each(struct blk+AF8-mq+AF8-tags +ACo-tags, struct sbitmap+AF8-queue +ACo-bt,
+AD4   +ACo +AEA-fn:		Pointer to the function that will be called for each started
+AD4   +ACo		request. +AEA-fn will be called as follows: +AEA-fn(rq, +AEA-priv,
+AD4   +ACo		reserved) where rq is a pointer to a request. 'reserved'
+AD4 - +ACo		indicates whether or not +AEA-rq is a reserved request.
+AD4 +- +ACo		indicates whether or not +AEA-rq is a reserved request. Return
+AD4 +- +ACo		true to continue iterating tags, false to stop.
+AD4   +ACo +AEA-priv:	Will be passed as second argument to +AEA-fn.
+AD4   +ACo-/
+AD4  static void blk+AF8-mq+AF8-all+AF8-tag+AF8-busy+AF8-iter(struct blk+AF8-mq+AF8-tags +ACo-tags,
+AD4 +AEAAQA -343,7 +-346,8 +AEAAQA static void blk+AF8-mq+AF8-all+AF8-tag+AF8-busy+AF8-iter(struct blk+AF8-mq+AF8-tags +ACo-tags,
+AD4   +ACo +AEA-fn:		Pointer to the function that will be called for each started
+AD4   +ACo		request. +AEA-fn will be called as follows: +AEA-fn(rq, +AEA-priv,
+AD4   +ACo		reserved) where rq is a pointer to a request. 'reserved'
+AD4 - +ACo		indicates whether or not +AEA-rq is a reserved request.
+AD4 +- +ACo		indicates whether or not +AEA-rq is a reserved request. Return
+AD4 +- +ACo		true to continue iterating tags, false to stop.
+AD4   +ACo +AEA-priv:	Will be passed as second argument to +AEA-fn.
+AD4   +ACo-/
+AD4  void blk+AF8-mq+AF8-tagset+AF8-busy+AF8-iter(struct blk+AF8-mq+AF8-tag+AF8-set +ACo-tagset,

This looks fine to me. Thanks+ACE

Bart.



[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux