[PATCH v3 0/3] Optimize wbt and update its comments and doc

[PATCH v3 0/3] Optimize wbt and update its comments and doc

[Tang Yizhou]

From: Tang Yizhou <yizhou.tang@shopee.com>

Some minor optimizations and updates of comments and doc for wbt.

v3:
Fix a typo: change 'cur_win_nsec' to 'curr_win_nsec'.

v2:
Patch #1: Pick up Jan and Kuai's Reviewed-by tag.
Patch #2: Pick up Jan's Reviewed-by tag.
Patch #3: Take Jan and Kuai's advice. Change the name to
'curr_win_nsec'.

Tang Yizhou (3):
  blk-wbt: Optimize wbt_done() for non-throttled writes
  blk-wbt: Eliminate ambiguity in the comments of struct rq_wb
  blk-wbt: doc: Update the doc of the wbt_lat_usec interface

 Documentation/ABI/stable/sysfs-block |  2 +-
 block/blk-wbt.c                      | 15 ++++++++-------
 2 files changed, 9 insertions(+), 8 deletions(-)

-- 
2.25.1

[PATCH v3 1/3] blk-wbt: Optimize wbt_done() for non-throttled writes

[Tang Yizhou]

From: Tang Yizhou <yizhou.tang@shopee.com>

In the current implementation, the sync_cookie and last_cookie members of
struct rq_wb are used only by read requests and not by non-throttled write
requests. Based on this, we can optimize wbt_done() by removing one if
condition check for non-throttled write requests.

Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Reviewed-by: Yu Kuai <yukuai3@huawei.com>
---
 block/blk-wbt.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/block/blk-wbt.c b/block/blk-wbt.c
index a50d4cd55f41..30886d44f6cd 100644
--- a/block/blk-wbt.c
+++ b/block/blk-wbt.c
@@ -248,13 +248,14 @@ static void wbt_done(struct rq_qos *rqos, struct request *rq)
 	struct rq_wb *rwb = RQWB(rqos);
 
 	if (!wbt_is_tracked(rq)) {
-		if (rwb->sync_cookie == rq) {
-			rwb->sync_issue = 0;
-			rwb->sync_cookie = NULL;
-		}
+		if (wbt_is_read(rq)) {
+			if (rwb->sync_cookie == rq) {
+				rwb->sync_issue = 0;
+				rwb->sync_cookie = NULL;
+			}
 
-		if (wbt_is_read(rq))
 			wb_timestamp(rwb, &rwb->last_comp);
+		}
 	} else {
 		WARN_ON_ONCE(rq == rwb->sync_cookie);
 		__wbt_done(rqos, wbt_flags(rq));
-- 
2.25.1

[PATCH v3 2/3] blk-wbt: Eliminate ambiguity in the comments of struct rq_wb

[Tang Yizhou]

From: Tang Yizhou <yizhou.tang@shopee.com>

In the current implementation, the last_issue and last_comp members of
struct rq_wb are used only by read requests and not by non-throttled write
requests. Therefore, eliminate the ambiguity here.

Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
Reviewed-by: Jan Kara <jack@suse.cz>
---
 block/blk-wbt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/block/blk-wbt.c b/block/blk-wbt.c
index 30886d44f6cd..eb8037bae0bd 100644
--- a/block/blk-wbt.c
+++ b/block/blk-wbt.c
@@ -85,8 +85,8 @@ struct rq_wb {
 	u64 sync_issue;
 	void *sync_cookie;
 
-	unsigned long last_issue;		/* last non-throttled issue */
-	unsigned long last_comp;		/* last non-throttled comp */
+	unsigned long last_issue;	/* issue time of last read rq */
+	unsigned long last_comp;	/* completion time of last read rq */
 	unsigned long min_lat_nsec;
 	struct rq_qos rqos;
 	struct rq_wait rq_wait[WBT_NUM_RWQ];
-- 
2.25.1

[PATCH v3 3/3] blk-wbt: doc: Update the doc of the wbt_lat_usec interface

[Tang Yizhou]

From: Tang Yizhou <yizhou.tang@shopee.com>

The symbol wb_window_usec cannot be found. Update the doc to reflect the
latest implementation, in other words, the debugfs interface
'curr_win_nsec'.

Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
---
 Documentation/ABI/stable/sysfs-block | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/ABI/stable/sysfs-block b/Documentation/ABI/stable/sysfs-block
index 4ba771b56b3b..a3cf841ebdff 100644
--- a/Documentation/ABI/stable/sysfs-block
+++ b/Documentation/ABI/stable/sysfs-block
@@ -731,7 +731,7 @@ Contact:	linux-block@vger.kernel.org
 Description:
 		[RW] If the device is registered for writeback throttling, then
 		this file shows the target minimum read latency. If this latency
-		is exceeded in a given window of time (see wb_window_usec), then
+		is exceeded in a given window of time (see curr_win_nsec), then
 		the writeback throttling will start scaling back writes. Writing
 		a value of '0' to this file disables the feature. Writing a
 		value of '-1' to this file resets the value to the default
-- 
2.25.1

Re: [PATCH v3 3/3] blk-wbt: doc: Update the doc of the wbt_lat_usec interface

[Yu Kuai]

在 2025/07/28 1:39, Tang Yizhou 写道:
> From: Tang Yizhou <yizhou.tang@shopee.com>
> 
> The symbol wb_window_usec cannot be found. Update the doc to reflect the
> latest implementation, in other words, the debugfs interface
> 'curr_win_nsec'.
> 
> Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
> ---
>   Documentation/ABI/stable/sysfs-block | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
Reviewed-by: Yu Kuai <yukuai3@huawei.com>
Thanks

> diff --git a/Documentation/ABI/stable/sysfs-block b/Documentation/ABI/stable/sysfs-block
> index 4ba771b56b3b..a3cf841ebdff 100644
> --- a/Documentation/ABI/stable/sysfs-block
> +++ b/Documentation/ABI/stable/sysfs-block
> @@ -731,7 +731,7 @@ Contact:	linux-block@vger.kernel.org
>   Description:
>   		[RW] If the device is registered for writeback throttling, then
>   		this file shows the target minimum read latency. If this latency
> -		is exceeded in a given window of time (see wb_window_usec), then
> +		is exceeded in a given window of time (see curr_win_nsec), then
>   		the writeback throttling will start scaling back writes. Writing
>   		a value of '0' to this file disables the feature. Writing a
>   		value of '-1' to this file resets the value to the default
> 

Re: [PATCH v3 3/3] blk-wbt: doc: Update the doc of the wbt_lat_usec interface

[Jan Kara]

On Mon 28-07-25 01:39:59, Tang Yizhou wrote:
> From: Tang Yizhou <yizhou.tang@shopee.com>
> 
> The symbol wb_window_usec cannot be found. Update the doc to reflect the
> latest implementation, in other words, the debugfs interface
> 'curr_win_nsec'.
> 
> Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>

Looks good! Feel free to add:

Reviewed-by: Jan Kara <jack@suse.cz>

								Honza

> ---
>  Documentation/ABI/stable/sysfs-block | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/ABI/stable/sysfs-block b/Documentation/ABI/stable/sysfs-block
> index 4ba771b56b3b..a3cf841ebdff 100644
> --- a/Documentation/ABI/stable/sysfs-block
> +++ b/Documentation/ABI/stable/sysfs-block
> @@ -731,7 +731,7 @@ Contact:	linux-block@vger.kernel.org
>  Description:
>  		[RW] If the device is registered for writeback throttling, then
>  		this file shows the target minimum read latency. If this latency
> -		is exceeded in a given window of time (see wb_window_usec), then
> +		is exceeded in a given window of time (see curr_win_nsec), then
>  		the writeback throttling will start scaling back writes. Writing
>  		a value of '0' to this file disables the feature. Writing a
>  		value of '-1' to this file resets the value to the default
> -- 
> 2.25.1
> 
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

Re: [PATCH v3 0/3] Optimize wbt and update its comments and doc

[Yizhou Tang]

On Mon, Jul 28, 2025 at 1:40 AM Tang Yizhou <yizhou.tang@shopee.com> wrote:
>
> From: Tang Yizhou <yizhou.tang@shopee.com>
>
> Some minor optimizations and updates of comments and doc for wbt.
>
> v3:
> Fix a typo: change 'cur_win_nsec' to 'curr_win_nsec'.
>
> v2:
> Patch #1: Pick up Jan and Kuai's Reviewed-by tag.
> Patch #2: Pick up Jan's Reviewed-by tag.
> Patch #3: Take Jan and Kuai's advice. Change the name to
> 'curr_win_nsec'.
>
> Tang Yizhou (3):
>   blk-wbt: Optimize wbt_done() for non-throttled writes
>   blk-wbt: Eliminate ambiguity in the comments of struct rq_wb
>   blk-wbt: doc: Update the doc of the wbt_lat_usec interface
>
>  Documentation/ABI/stable/sysfs-block |  2 +-
>  block/blk-wbt.c                      | 15 ++++++++-------
>  2 files changed, 9 insertions(+), 8 deletions(-)
>
> --
> 2.25.1
>

Hi Jens,

If you have no further concerns, please consider merging this patchset.

Thanks,
Yi

Re: [PATCH v3 0/3] Optimize wbt and update its comments and doc

[Jens Axboe]

On Mon, 28 Jul 2025 01:39:56 +0800, Tang Yizhou wrote:
> Some minor optimizations and updates of comments and doc for wbt.
> 
> v3:
> Fix a typo: change 'cur_win_nsec' to 'curr_win_nsec'.
> 
> v2:
> Patch #1: Pick up Jan and Kuai's Reviewed-by tag.
> Patch #2: Pick up Jan's Reviewed-by tag.
> Patch #3: Take Jan and Kuai's advice. Change the name to
> 'curr_win_nsec'.
> 
> [...]

Applied, thanks!

[1/3] blk-wbt: Optimize wbt_done() for non-throttled writes
      commit: d8b96a79622e03813c221450498ca9742704ebf2
[2/3] blk-wbt: Eliminate ambiguity in the comments of struct rq_wb
      commit: bccdfcd56d4b5b78d0d76f46d0e89a51330dfd75
[3/3] blk-wbt: doc: Update the doc of the wbt_lat_usec interface
      commit: 0452f08395f8e7d04fe3744443dad396b3330d0c

Best regards,
-- 
Jens Axboe