diff mbox series

[v2,2/2] blk-wbt: Cleanup a comment in wb_timer_fn

Message ID 20250213100611.209997-3-yizhou.tang@shopee.com (mailing list archive)
State New
Headers show
Series Fix and cleanup some comments in blk-wbt | expand

Commit Message

Tang Yizhou Feb. 13, 2025, 10:06 a.m. UTC
From: Tang Yizhou <yizhou.tang@shopee.com>

The original comment contains a grammatical error. Rewrite it into a more
easily understandable sentence.

Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
---
 block/blk-wbt.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Yu Kuai Feb. 13, 2025, 11:16 a.m. UTC | #1
Hi,

在 2025/02/13 18:06, Tang Yizhou 写道:
> From: Tang Yizhou <yizhou.tang@shopee.com>
> 
> The original comment contains a grammatical error. Rewrite it into a more
> easily understandable sentence.
> 
> Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
> ---
>   block/blk-wbt.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 

Personally, I don't think this patch is necessary, people can still
understand with this error. I'm not expecting a seperate patch :(

Thanks,
Kuai

> diff --git a/block/blk-wbt.c b/block/blk-wbt.c
> index 8b73c0c11aec..f1754d07f7e0 100644
> --- a/block/blk-wbt.c
> +++ b/block/blk-wbt.c
> @@ -447,9 +447,9 @@ static void wb_timer_fn(struct blk_stat_callback *cb)
>   		break;
>   	case LAT_UNKNOWN_WRITES:
>   		/*
> -		 * We started a the center step, but don't have a valid
> -		 * read/write sample, but we do have writes going on.
> -		 * Allow step to go negative, to increase write perf.
> +		 * We don't have a valid read/write sample, but we do have
> +		 * writes going on. Allow step to go negative, to increase
> +		 * write performance.
>   		 */
>   		scale_up(rwb);
>   		break;
>
Tang Yizhou Feb. 13, 2025, 2:28 p.m. UTC | #2
On Thu, Feb 13, 2025 at 7:17 PM Yu Kuai <yukuai1@huaweicloud.com> wrote:
>
> Hi,
>
> 在 2025/02/13 18:06, Tang Yizhou 写道:
> > From: Tang Yizhou <yizhou.tang@shopee.com>
> >
> > The original comment contains a grammatical error. Rewrite it into a more
> > easily understandable sentence.
> >
> > Signed-off-by: Tang Yizhou <yizhou.tang@shopee.com>
> > ---
> >   block/blk-wbt.c | 6 +++---
> >   1 file changed, 3 insertions(+), 3 deletions(-)
> >
>
> Personally, I don't think this patch is necessary, people can still
> understand with this error. I'm not expecting a seperate patch :(
>
> Thanks,
> Kuai
>

Got it. Let's see if others have any suggestions.

Thanks,
Yi

> > diff --git a/block/blk-wbt.c b/block/blk-wbt.c
> > index 8b73c0c11aec..f1754d07f7e0 100644
> > --- a/block/blk-wbt.c
> > +++ b/block/blk-wbt.c
> > @@ -447,9 +447,9 @@ static void wb_timer_fn(struct blk_stat_callback *cb)
> >               break;
> >       case LAT_UNKNOWN_WRITES:
> >               /*
> > -              * We started a the center step, but don't have a valid
> > -              * read/write sample, but we do have writes going on.
> > -              * Allow step to go negative, to increase write perf.
> > +              * We don't have a valid read/write sample, but we do have
> > +              * writes going on. Allow step to go negative, to increase
> > +              * write performance.
> >                */
> >               scale_up(rwb);
> >               break;
> >
>
diff mbox series

Patch

diff --git a/block/blk-wbt.c b/block/blk-wbt.c
index 8b73c0c11aec..f1754d07f7e0 100644
--- a/block/blk-wbt.c
+++ b/block/blk-wbt.c
@@ -447,9 +447,9 @@  static void wb_timer_fn(struct blk_stat_callback *cb)
 		break;
 	case LAT_UNKNOWN_WRITES:
 		/*
-		 * We started a the center step, but don't have a valid
-		 * read/write sample, but we do have writes going on.
-		 * Allow step to go negative, to increase write perf.
+		 * We don't have a valid read/write sample, but we do have
+		 * writes going on. Allow step to go negative, to increase
+		 * write performance.
 		 */
 		scale_up(rwb);
 		break;