All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
@ 2017-09-21 11:25 Rakesh Pandit
  2017-09-22  8:40 ` Javier González
  0 siblings, 1 reply; 7+ messages in thread
From: Rakesh Pandit @ 2017-09-21 11:25 UTC (permalink / raw)
  To: Matias Bjørling, linux-block, linux-kernel; +Cc: Javier González

This is a trivial change which reuses pblk_gc_should_kick instead of
repeating it again in pblk_rl_free_lines_inc.

Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
---
 drivers/lightnvm/pblk-core.c | 1 +
 drivers/lightnvm/pblk-rl.c   | 9 ---------
 2 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
index 64a6a25..a230125 100644
--- a/drivers/lightnvm/pblk-core.c
+++ b/drivers/lightnvm/pblk-core.c
@@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
 	spin_unlock(&l_mg->free_lock);
 
 	pblk_rl_free_lines_inc(&pblk->rl, line);
+	pblk_gc_should_kick(pblk);
 }
 
 static void pblk_line_put_ws(struct work_struct *work)
diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
index 596bdec..e7c162a 100644
--- a/drivers/lightnvm/pblk-rl.c
+++ b/drivers/lightnvm/pblk-rl.c
@@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
 
 void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
 {
-	struct pblk *pblk = container_of(rl, struct pblk, rl);
 	int blk_in_line = atomic_read(&line->blk_in_line);
-	int ret;
 
 	atomic_add(blk_in_line, &rl->free_blocks);
-	/* Rates will not change that often - no need to lock update */
-	ret = pblk_rl_update_rates(rl, rl->rb_budget);
-
-	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
-		pblk_gc_should_start(pblk);
-	else
-		pblk_gc_should_stop(pblk);
 }
 
 void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
  2017-09-21 11:25 [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick Rakesh Pandit
@ 2017-09-22  8:40 ` Javier González
  2017-09-22  9:42     ` Rakesh Pandit
  2017-09-25 10:24     ` Matias Bjørling
  0 siblings, 2 replies; 7+ messages in thread
From: Javier González @ 2017-09-22  8:40 UTC (permalink / raw)
  To: Rakesh Pandit; +Cc: Matias Bjørling, linux-block, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 5675 bytes --]

> On 21 Sep 2017, at 13.25, Rakesh Pandit <rakesh@tuxera.com> wrote:
> 
> This is a trivial change which reuses pblk_gc_should_kick instead of
> repeating it again in pblk_rl_free_lines_inc.
> 
> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> ---
> drivers/lightnvm/pblk-core.c | 1 +
> drivers/lightnvm/pblk-rl.c   | 9 ---------
> 2 files changed, 1 insertion(+), 9 deletions(-)
> 
> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> index 64a6a25..a230125 100644
> --- a/drivers/lightnvm/pblk-core.c
> +++ b/drivers/lightnvm/pblk-core.c
> @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
> 	spin_unlock(&l_mg->free_lock);
> 
> 	pblk_rl_free_lines_inc(&pblk->rl, line);
> +	pblk_gc_should_kick(pblk);
> }
> 
> static void pblk_line_put_ws(struct work_struct *work)
> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> index 596bdec..e7c162a 100644
> --- a/drivers/lightnvm/pblk-rl.c
> +++ b/drivers/lightnvm/pblk-rl.c
> @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> 
> void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
> {
> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
> 	int blk_in_line = atomic_read(&line->blk_in_line);
> -	int ret;
> 
> 	atomic_add(blk_in_line, &rl->free_blocks);
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> }
> 
> void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> --
> 2.5.0

Looking at the patch, I can see that a more general cleanup can be done.
What do you think about this? We can merge it in this patch if you are OK
with it.

From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
From: Rakesh Pandit <rakesh@tuxera.com>
Date: Thu, 21 Sep 2017 14:25:40 +0300
Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick

This is a trivial change which reuses pblk_gc_should_kick instead of
repeating it again in pblk_rl_free_lines_inc.

Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
---
 drivers/lightnvm/pblk-core.c |  2 --
 drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
 drivers/lightnvm/pblk.h      |  1 -
 3 files changed, 9 insertions(+), 27 deletions(-)

diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
index 64a6a255514e..c92113b0a2e7 100644
--- a/drivers/lightnvm/pblk-core.c
+++ b/drivers/lightnvm/pblk-core.c
@@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)

 	spin_unlock(&line->lock);
 	spin_unlock(&l_mg->gc_lock);
-
-	pblk_gc_should_kick(pblk);
 }

 void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
index 596bdec433c3..0896439a91b0 100644
--- a/drivers/lightnvm/pblk-rl.c
+++ b/drivers/lightnvm/pblk-rl.c
@@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
  *
  * Only the total number of free blocks is used to configure the rate limiter.
  */
-static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
+static void pblk_rl_update_rates(struct pblk_rl *rl)
 {
+	struct pblk *pblk = container_of(rl, struct pblk, rl);
 	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
+	int max = rl->rb_budget;

 	if (free_blocks >= rl->high) {
 		rl->rb_user_max = max;
@@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
 		rl->rb_state = PBLK_RL_LOW;
 	}

-	return rl->rb_state;
+	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
+		pblk_gc_should_start(pblk);
+	else
+		pblk_gc_should_stop(pblk);
 }

 void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
 {
-	struct pblk *pblk = container_of(rl, struct pblk, rl);
 	int blk_in_line = atomic_read(&line->blk_in_line);
-	int ret;

 	atomic_add(blk_in_line, &rl->free_blocks);
-	/* Rates will not change that often - no need to lock update */
-	ret = pblk_rl_update_rates(rl, rl->rb_budget);
-
-	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
-		pblk_gc_should_start(pblk);
-	else
-		pblk_gc_should_stop(pblk);
+	pblk_rl_update_rates(rl);
 }

 void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
@@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
 	int blk_in_line = atomic_read(&line->blk_in_line);

 	atomic_sub(blk_in_line, &rl->free_blocks);
-}
-
-void pblk_gc_should_kick(struct pblk *pblk)
-{
-	struct pblk_rl *rl = &pblk->rl;
-	int ret;
-
-	/* Rates will not change that often - no need to lock update */
-	ret = pblk_rl_update_rates(rl, rl->rb_budget);
-	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
-		pblk_gc_should_start(pblk);
-	else
-		pblk_gc_should_stop(pblk);
+	pblk_rl_update_rates(rl);
 }

 int pblk_rl_high_thrs(struct pblk_rl *rl)
diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
index eaf539715d71..d4b7dee0ef50 100644
--- a/drivers/lightnvm/pblk.h
+++ b/drivers/lightnvm/pblk.h
@@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
 void pblk_gc_exit(struct pblk *pblk);
 void pblk_gc_should_start(struct pblk *pblk);
 void pblk_gc_should_stop(struct pblk *pblk);
-void pblk_gc_should_kick(struct pblk *pblk);
 void pblk_gc_kick(struct pblk *pblk);
 void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
 			      int *gc_active);
--
2.7.4


[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
  2017-09-22  8:40 ` Javier González
@ 2017-09-22  9:42     ` Rakesh Pandit
  2017-09-25 10:24     ` Matias Bjørling
  1 sibling, 0 replies; 7+ messages in thread
From: Rakesh Pandit @ 2017-09-22  9:42 UTC (permalink / raw)
  To: Javier González; +Cc: Matias Bjørling, linux-block, linux-kernel

On Fri, Sep 22, 2017 at 10:40:57AM +0200, Javier Gonz�lez wrote:
> > On 21 Sep 2017, at 13.25, Rakesh Pandit wrote:
> > 
> > This is a trivial change which reuses pblk_gc_should_kick instead of
> > repeating it again in pblk_rl_free_lines_inc.
> > 
> > Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> > ---
> > drivers/lightnvm/pblk-core.c | 1 +
> > drivers/lightnvm/pblk-rl.c   | 9 ---------
> > 2 files changed, 1 insertion(+), 9 deletions(-)
> > 
> > diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> > index 64a6a25..a230125 100644
> > --- a/drivers/lightnvm/pblk-core.c
> > +++ b/drivers/lightnvm/pblk-core.c
> > @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
> > 	spin_unlock(&l_mg->free_lock);
> > 
> > 	pblk_rl_free_lines_inc(&pblk->rl, line);
> > +	pblk_gc_should_kick(pblk);
> > }
> > 
> > static void pblk_line_put_ws(struct work_struct *work)
> > diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> > index 596bdec..e7c162a 100644
> > --- a/drivers/lightnvm/pblk-rl.c
> > +++ b/drivers/lightnvm/pblk-rl.c
> > @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> > 
> > void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
> > {
> > -	struct pblk *pblk = container_of(rl, struct pblk, rl);
> > 	int blk_in_line = atomic_read(&line->blk_in_line);
> > -	int ret;
> > 
> > 	atomic_add(blk_in_line, &rl->free_blocks);
> > -	/* Rates will not change that often - no need to lock update */
> > -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> > -
> > -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> > -		pblk_gc_should_start(pblk);
> > -	else
> > -		pblk_gc_should_stop(pblk);
> > }
> > 
> > void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> > --
> > 2.5.0
> 
> Looking at the patch, I can see that a more general cleanup can be done.
> What do you think about this? We can merge it in this patch if you are OK
> with it.

Makes more sense. Feel free to merge. You have my Signed-off-by tag.

> 
> From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
> From: Rakesh Pandit <rakesh@tuxera.com>
> Date: Thu, 21 Sep 2017 14:25:40 +0300
> Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick
> 
> This is a trivial change which reuses pblk_gc_should_kick instead of
> repeating it again in pblk_rl_free_lines_inc.
> 
> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> ---
>  drivers/lightnvm/pblk-core.c |  2 --
>  drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
>  drivers/lightnvm/pblk.h      |  1 -
>  3 files changed, 9 insertions(+), 27 deletions(-)
> 
> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> index 64a6a255514e..c92113b0a2e7 100644
> --- a/drivers/lightnvm/pblk-core.c
> +++ b/drivers/lightnvm/pblk-core.c
> @@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)
> 
>  	spin_unlock(&line->lock);
>  	spin_unlock(&l_mg->gc_lock);
> -
> -	pblk_gc_should_kick(pblk);
>  }
> 
>  void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> index 596bdec433c3..0896439a91b0 100644
> --- a/drivers/lightnvm/pblk-rl.c
> +++ b/drivers/lightnvm/pblk-rl.c
> @@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
>   *
>   * Only the total number of free blocks is used to configure the rate limiter.
>   */
> -static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> +static void pblk_rl_update_rates(struct pblk_rl *rl)
>  {
> +	struct pblk *pblk = container_of(rl, struct pblk, rl);
>  	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
> +	int max = rl->rb_budget;
> 
>  	if (free_blocks >= rl->high) {
>  		rl->rb_user_max = max;
> @@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>  		rl->rb_state = PBLK_RL_LOW;
>  	}
> 
> -	return rl->rb_state;
> +	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
> +		pblk_gc_should_start(pblk);
> +	else
> +		pblk_gc_should_stop(pblk);
>  }
> 
>  void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>  {
> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>  	int blk_in_line = atomic_read(&line->blk_in_line);
> -	int ret;
> 
>  	atomic_add(blk_in_line, &rl->free_blocks);
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>  }
> 
>  void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> @@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>  	int blk_in_line = atomic_read(&line->blk_in_line);
> 
>  	atomic_sub(blk_in_line, &rl->free_blocks);
> -}
> -
> -void pblk_gc_should_kick(struct pblk *pblk)
> -{
> -	struct pblk_rl *rl = &pblk->rl;
> -	int ret;
> -
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>  }
> 
>  int pblk_rl_high_thrs(struct pblk_rl *rl)
> diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
> index eaf539715d71..d4b7dee0ef50 100644
> --- a/drivers/lightnvm/pblk.h
> +++ b/drivers/lightnvm/pblk.h
> @@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
>  void pblk_gc_exit(struct pblk *pblk);
>  void pblk_gc_should_start(struct pblk *pblk);
>  void pblk_gc_should_stop(struct pblk *pblk);
> -void pblk_gc_should_kick(struct pblk *pblk);
>  void pblk_gc_kick(struct pblk *pblk);
>  void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
>  			      int *gc_active);
> --
> 2.7.4
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
@ 2017-09-22  9:42     ` Rakesh Pandit
  0 siblings, 0 replies; 7+ messages in thread
From: Rakesh Pandit @ 2017-09-22  9:42 UTC (permalink / raw)
  To: Javier González; +Cc: Matias Bjørling, linux-block, linux-kernel

On Fri, Sep 22, 2017 at 10:40:57AM +0200, Javier González wrote:
> > On 21 Sep 2017, at 13.25, Rakesh Pandit wrote:
> > 
> > This is a trivial change which reuses pblk_gc_should_kick instead of
> > repeating it again in pblk_rl_free_lines_inc.
> > 
> > Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> > ---
> > drivers/lightnvm/pblk-core.c | 1 +
> > drivers/lightnvm/pblk-rl.c   | 9 ---------
> > 2 files changed, 1 insertion(+), 9 deletions(-)
> > 
> > diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> > index 64a6a25..a230125 100644
> > --- a/drivers/lightnvm/pblk-core.c
> > +++ b/drivers/lightnvm/pblk-core.c
> > @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
> > 	spin_unlock(&l_mg->free_lock);
> > 
> > 	pblk_rl_free_lines_inc(&pblk->rl, line);
> > +	pblk_gc_should_kick(pblk);
> > }
> > 
> > static void pblk_line_put_ws(struct work_struct *work)
> > diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> > index 596bdec..e7c162a 100644
> > --- a/drivers/lightnvm/pblk-rl.c
> > +++ b/drivers/lightnvm/pblk-rl.c
> > @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> > 
> > void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
> > {
> > -	struct pblk *pblk = container_of(rl, struct pblk, rl);
> > 	int blk_in_line = atomic_read(&line->blk_in_line);
> > -	int ret;
> > 
> > 	atomic_add(blk_in_line, &rl->free_blocks);
> > -	/* Rates will not change that often - no need to lock update */
> > -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> > -
> > -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> > -		pblk_gc_should_start(pblk);
> > -	else
> > -		pblk_gc_should_stop(pblk);
> > }
> > 
> > void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> > --
> > 2.5.0
> 
> Looking at the patch, I can see that a more general cleanup can be done.
> What do you think about this? We can merge it in this patch if you are OK
> with it.

Makes more sense. Feel free to merge. You have my Signed-off-by tag.

> 
> From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
> From: Rakesh Pandit <rakesh@tuxera.com>
> Date: Thu, 21 Sep 2017 14:25:40 +0300
> Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick
> 
> This is a trivial change which reuses pblk_gc_should_kick instead of
> repeating it again in pblk_rl_free_lines_inc.
> 
> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> ---
>  drivers/lightnvm/pblk-core.c |  2 --
>  drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
>  drivers/lightnvm/pblk.h      |  1 -
>  3 files changed, 9 insertions(+), 27 deletions(-)
> 
> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> index 64a6a255514e..c92113b0a2e7 100644
> --- a/drivers/lightnvm/pblk-core.c
> +++ b/drivers/lightnvm/pblk-core.c
> @@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)
> 
>  	spin_unlock(&line->lock);
>  	spin_unlock(&l_mg->gc_lock);
> -
> -	pblk_gc_should_kick(pblk);
>  }
> 
>  void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> index 596bdec433c3..0896439a91b0 100644
> --- a/drivers/lightnvm/pblk-rl.c
> +++ b/drivers/lightnvm/pblk-rl.c
> @@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
>   *
>   * Only the total number of free blocks is used to configure the rate limiter.
>   */
> -static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> +static void pblk_rl_update_rates(struct pblk_rl *rl)
>  {
> +	struct pblk *pblk = container_of(rl, struct pblk, rl);
>  	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
> +	int max = rl->rb_budget;
> 
>  	if (free_blocks >= rl->high) {
>  		rl->rb_user_max = max;
> @@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>  		rl->rb_state = PBLK_RL_LOW;
>  	}
> 
> -	return rl->rb_state;
> +	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
> +		pblk_gc_should_start(pblk);
> +	else
> +		pblk_gc_should_stop(pblk);
>  }
> 
>  void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>  {
> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>  	int blk_in_line = atomic_read(&line->blk_in_line);
> -	int ret;
> 
>  	atomic_add(blk_in_line, &rl->free_blocks);
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>  }
> 
>  void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> @@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>  	int blk_in_line = atomic_read(&line->blk_in_line);
> 
>  	atomic_sub(blk_in_line, &rl->free_blocks);
> -}
> -
> -void pblk_gc_should_kick(struct pblk *pblk)
> -{
> -	struct pblk_rl *rl = &pblk->rl;
> -	int ret;
> -
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>  }
> 
>  int pblk_rl_high_thrs(struct pblk_rl *rl)
> diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
> index eaf539715d71..d4b7dee0ef50 100644
> --- a/drivers/lightnvm/pblk.h
> +++ b/drivers/lightnvm/pblk.h
> @@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
>  void pblk_gc_exit(struct pblk *pblk);
>  void pblk_gc_should_start(struct pblk *pblk);
>  void pblk_gc_should_stop(struct pblk *pblk);
> -void pblk_gc_should_kick(struct pblk *pblk);
>  void pblk_gc_kick(struct pblk *pblk);
>  void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
>  			      int *gc_active);
> --
> 2.7.4
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
  2017-09-22  9:42     ` Rakesh Pandit
  (?)
@ 2017-09-22 10:04     ` Javier González
  -1 siblings, 0 replies; 7+ messages in thread
From: Javier González @ 2017-09-22 10:04 UTC (permalink / raw)
  To: Rakesh Pandit; +Cc: Matias Bjørling, linux-block, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 6405 bytes --]

> On 22 Sep 2017, at 11.42, Rakesh Pandit <rakesh@tuxera.com> wrote:
> 
> On Fri, Sep 22, 2017 at 10:40:57AM +0200, Javier González wrote:
>>> On 21 Sep 2017, at 13.25, Rakesh Pandit wrote:
>>> 
>>> This is a trivial change which reuses pblk_gc_should_kick instead of
>>> repeating it again in pblk_rl_free_lines_inc.
>>> 
>>> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
>>> ---
>>> drivers/lightnvm/pblk-core.c | 1 +
>>> drivers/lightnvm/pblk-rl.c   | 9 ---------
>>> 2 files changed, 1 insertion(+), 9 deletions(-)
>>> 
>>> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
>>> index 64a6a25..a230125 100644
>>> --- a/drivers/lightnvm/pblk-core.c
>>> +++ b/drivers/lightnvm/pblk-core.c
>>> @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
>>> 	spin_unlock(&l_mg->free_lock);
>>> 
>>> 	pblk_rl_free_lines_inc(&pblk->rl, line);
>>> +	pblk_gc_should_kick(pblk);
>>> }
>>> 
>>> static void pblk_line_put_ws(struct work_struct *work)
>>> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
>>> index 596bdec..e7c162a 100644
>>> --- a/drivers/lightnvm/pblk-rl.c
>>> +++ b/drivers/lightnvm/pblk-rl.c
>>> @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>>> 
>>> void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>>> {
>>> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>>> 	int blk_in_line = atomic_read(&line->blk_in_line);
>>> -	int ret;
>>> 
>>> 	atomic_add(blk_in_line, &rl->free_blocks);
>>> -	/* Rates will not change that often - no need to lock update */
>>> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
>>> -
>>> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
>>> -		pblk_gc_should_start(pblk);
>>> -	else
>>> -		pblk_gc_should_stop(pblk);
>>> }
>>> 
>>> void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>>> --
>>> 2.5.0
>> 
>> Looking at the patch, I can see that a more general cleanup can be done.
>> What do you think about this? We can merge it in this patch if you are OK
>> with it.
> 
> Makes more sense. Feel free to merge. You have my Signed-off-by tag.

We'll merge it with your patch, which was the original cleanup.

Matias: can you pick the code in the patch below instead?

> 
>> From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
>> From: Rakesh Pandit <rakesh@tuxera.com>
>> Date: Thu, 21 Sep 2017 14:25:40 +0300
>> Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick
>> 
>> This is a trivial change which reuses pblk_gc_should_kick instead of
>> repeating it again in pblk_rl_free_lines_inc.
>> 
>> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
>> ---
>> drivers/lightnvm/pblk-core.c |  2 --
>> drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
>> drivers/lightnvm/pblk.h      |  1 -
>> 3 files changed, 9 insertions(+), 27 deletions(-)
>> 
>> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
>> index 64a6a255514e..c92113b0a2e7 100644
>> --- a/drivers/lightnvm/pblk-core.c
>> +++ b/drivers/lightnvm/pblk-core.c
>> @@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)
>> 
>> 	spin_unlock(&line->lock);
>> 	spin_unlock(&l_mg->gc_lock);
>> -
>> -	pblk_gc_should_kick(pblk);
>> }
>> 
>> void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
>> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
>> index 596bdec433c3..0896439a91b0 100644
>> --- a/drivers/lightnvm/pblk-rl.c
>> +++ b/drivers/lightnvm/pblk-rl.c
>> @@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
>>  *
>>  * Only the total number of free blocks is used to configure the rate limiter.
>>  */
>> -static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>> +static void pblk_rl_update_rates(struct pblk_rl *rl)
>> {
>> +	struct pblk *pblk = container_of(rl, struct pblk, rl);
>> 	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
>> +	int max = rl->rb_budget;
>> 
>> 	if (free_blocks >= rl->high) {
>> 		rl->rb_user_max = max;
>> @@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>> 		rl->rb_state = PBLK_RL_LOW;
>> 	}
>> 
>> -	return rl->rb_state;
>> +	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
>> +		pblk_gc_should_start(pblk);
>> +	else
>> +		pblk_gc_should_stop(pblk);
>> }
>> 
>> void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>> {
>> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>> 	int blk_in_line = atomic_read(&line->blk_in_line);
>> -	int ret;
>> 
>> 	atomic_add(blk_in_line, &rl->free_blocks);
>> -	/* Rates will not change that often - no need to lock update */
>> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
>> -
>> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
>> -		pblk_gc_should_start(pblk);
>> -	else
>> -		pblk_gc_should_stop(pblk);
>> +	pblk_rl_update_rates(rl);
>> }
>> 
>> void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>> @@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>> 	int blk_in_line = atomic_read(&line->blk_in_line);
>> 
>> 	atomic_sub(blk_in_line, &rl->free_blocks);
>> -}
>> -
>> -void pblk_gc_should_kick(struct pblk *pblk)
>> -{
>> -	struct pblk_rl *rl = &pblk->rl;
>> -	int ret;
>> -
>> -	/* Rates will not change that often - no need to lock update */
>> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
>> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
>> -		pblk_gc_should_start(pblk);
>> -	else
>> -		pblk_gc_should_stop(pblk);
>> +	pblk_rl_update_rates(rl);
>> }
>> 
>> int pblk_rl_high_thrs(struct pblk_rl *rl)
>> diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
>> index eaf539715d71..d4b7dee0ef50 100644
>> --- a/drivers/lightnvm/pblk.h
>> +++ b/drivers/lightnvm/pblk.h
>> @@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
>> void pblk_gc_exit(struct pblk *pblk);
>> void pblk_gc_should_start(struct pblk *pblk);
>> void pblk_gc_should_stop(struct pblk *pblk);
>> -void pblk_gc_should_kick(struct pblk *pblk);
>> void pblk_gc_kick(struct pblk *pblk);
>> void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
>> 			      int *gc_active);
>> --
>> 2.7.4

[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
  2017-09-22  8:40 ` Javier González
@ 2017-09-25 10:24     ` Matias Bjørling
  2017-09-25 10:24     ` Matias Bjørling
  1 sibling, 0 replies; 7+ messages in thread
From: Matias Bjørling @ 2017-09-25 10:24 UTC (permalink / raw)
  To: Javier González, Rakesh Pandit; +Cc: linux-block, linux-kernel

On 09/22/2017 10:40 AM, Javier Gonz�lez wrote:
>> On 21 Sep 2017, at 13.25, Rakesh Pandit <rakesh@tuxera.com> wrote:
>>
>> This is a trivial change which reuses pblk_gc_should_kick instead of
>> repeating it again in pblk_rl_free_lines_inc.
>>
>> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
>> ---
>> drivers/lightnvm/pblk-core.c | 1 +
>> drivers/lightnvm/pblk-rl.c   | 9 ---------
>> 2 files changed, 1 insertion(+), 9 deletions(-)
>>
>> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
>> index 64a6a25..a230125 100644
>> --- a/drivers/lightnvm/pblk-core.c
>> +++ b/drivers/lightnvm/pblk-core.c
>> @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
>> 	spin_unlock(&l_mg->free_lock);
>>
>> 	pblk_rl_free_lines_inc(&pblk->rl, line);
>> +	pblk_gc_should_kick(pblk);
>> }
>>
>> static void pblk_line_put_ws(struct work_struct *work)
>> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
>> index 596bdec..e7c162a 100644
>> --- a/drivers/lightnvm/pblk-rl.c
>> +++ b/drivers/lightnvm/pblk-rl.c
>> @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>>
>> void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>> {
>> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>> 	int blk_in_line = atomic_read(&line->blk_in_line);
>> -	int ret;
>>
>> 	atomic_add(blk_in_line, &rl->free_blocks);
>> -	/* Rates will not change that often - no need to lock update */
>> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
>> -
>> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
>> -		pblk_gc_should_start(pblk);
>> -	else
>> -		pblk_gc_should_stop(pblk);
>> }
>>
>> void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>> --
>> 2.5.0
> 
> Looking at the patch, I can see that a more general cleanup can be done.
> What do you think about this? We can merge it in this patch if you are OK
> with it.
> 
>  From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
> From: Rakesh Pandit <rakesh@tuxera.com>
> Date: Thu, 21 Sep 2017 14:25:40 +0300
> Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick
> 
> This is a trivial change which reuses pblk_gc_should_kick instead of
> repeating it again in pblk_rl_free_lines_inc.
> 
> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> ---
>   drivers/lightnvm/pblk-core.c |  2 --
>   drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
>   drivers/lightnvm/pblk.h      |  1 -
>   3 files changed, 9 insertions(+), 27 deletions(-)
> 
> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> index 64a6a255514e..c92113b0a2e7 100644
> --- a/drivers/lightnvm/pblk-core.c
> +++ b/drivers/lightnvm/pblk-core.c
> @@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)
> 
>   	spin_unlock(&line->lock);
>   	spin_unlock(&l_mg->gc_lock);
> -
> -	pblk_gc_should_kick(pblk);
>   }
> 
>   void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> index 596bdec433c3..0896439a91b0 100644
> --- a/drivers/lightnvm/pblk-rl.c
> +++ b/drivers/lightnvm/pblk-rl.c
> @@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
>    *
>    * Only the total number of free blocks is used to configure the rate limiter.
>    */
> -static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> +static void pblk_rl_update_rates(struct pblk_rl *rl)
>   {
> +	struct pblk *pblk = container_of(rl, struct pblk, rl);
>   	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
> +	int max = rl->rb_budget;
> 
>   	if (free_blocks >= rl->high) {
>   		rl->rb_user_max = max;
> @@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>   		rl->rb_state = PBLK_RL_LOW;
>   	}
> 
> -	return rl->rb_state;
> +	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
> +		pblk_gc_should_start(pblk);
> +	else
> +		pblk_gc_should_stop(pblk);
>   }
> 
>   void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>   {
> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>   	int blk_in_line = atomic_read(&line->blk_in_line);
> -	int ret;
> 
>   	atomic_add(blk_in_line, &rl->free_blocks);
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>   }
> 
>   void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> @@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>   	int blk_in_line = atomic_read(&line->blk_in_line);
> 
>   	atomic_sub(blk_in_line, &rl->free_blocks);
> -}
> -
> -void pblk_gc_should_kick(struct pblk *pblk)
> -{
> -	struct pblk_rl *rl = &pblk->rl;
> -	int ret;
> -
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>   }
> 
>   int pblk_rl_high_thrs(struct pblk_rl *rl)
> diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
> index eaf539715d71..d4b7dee0ef50 100644
> --- a/drivers/lightnvm/pblk.h
> +++ b/drivers/lightnvm/pblk.h
> @@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
>   void pblk_gc_exit(struct pblk *pblk);
>   void pblk_gc_should_start(struct pblk *pblk);
>   void pblk_gc_should_stop(struct pblk *pblk);
> -void pblk_gc_should_kick(struct pblk *pblk);
>   void pblk_gc_kick(struct pblk *pblk);
>   void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
>   			      int *gc_active);
> --
> 2.7.4
> 

Thanks, I applied the common version.

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick
@ 2017-09-25 10:24     ` Matias Bjørling
  0 siblings, 0 replies; 7+ messages in thread
From: Matias Bjørling @ 2017-09-25 10:24 UTC (permalink / raw)
  To: Javier González, Rakesh Pandit; +Cc: linux-block, linux-kernel

On 09/22/2017 10:40 AM, Javier González wrote:
>> On 21 Sep 2017, at 13.25, Rakesh Pandit <rakesh@tuxera.com> wrote:
>>
>> This is a trivial change which reuses pblk_gc_should_kick instead of
>> repeating it again in pblk_rl_free_lines_inc.
>>
>> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
>> ---
>> drivers/lightnvm/pblk-core.c | 1 +
>> drivers/lightnvm/pblk-rl.c   | 9 ---------
>> 2 files changed, 1 insertion(+), 9 deletions(-)
>>
>> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
>> index 64a6a25..a230125 100644
>> --- a/drivers/lightnvm/pblk-core.c
>> +++ b/drivers/lightnvm/pblk-core.c
>> @@ -1478,6 +1478,7 @@ static void __pblk_line_put(struct pblk *pblk, struct pblk_line *line)
>> 	spin_unlock(&l_mg->free_lock);
>>
>> 	pblk_rl_free_lines_inc(&pblk->rl, line);
>> +	pblk_gc_should_kick(pblk);
>> }
>>
>> static void pblk_line_put_ws(struct work_struct *work)
>> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
>> index 596bdec..e7c162a 100644
>> --- a/drivers/lightnvm/pblk-rl.c
>> +++ b/drivers/lightnvm/pblk-rl.c
>> @@ -129,18 +129,9 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>>
>> void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>> {
>> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>> 	int blk_in_line = atomic_read(&line->blk_in_line);
>> -	int ret;
>>
>> 	atomic_add(blk_in_line, &rl->free_blocks);
>> -	/* Rates will not change that often - no need to lock update */
>> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
>> -
>> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
>> -		pblk_gc_should_start(pblk);
>> -	else
>> -		pblk_gc_should_stop(pblk);
>> }
>>
>> void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>> --
>> 2.5.0
> 
> Looking at the patch, I can see that a more general cleanup can be done.
> What do you think about this? We can merge it in this patch if you are OK
> with it.
> 
>  From 944936663c7eef7bc32243b89cc554e40ec436c5 Mon Sep 17 00:00:00 2001
> From: Rakesh Pandit <rakesh@tuxera.com>
> Date: Thu, 21 Sep 2017 14:25:40 +0300
> Subject: [PATCH] lightnvm: pblk: reuse pblk_gc_should_kick
> 
> This is a trivial change which reuses pblk_gc_should_kick instead of
> repeating it again in pblk_rl_free_lines_inc.
> 
> Signed-off-by: Rakesh Pandit <rakesh@tuxera.com>
> ---
>   drivers/lightnvm/pblk-core.c |  2 --
>   drivers/lightnvm/pblk-rl.c   | 33 +++++++++------------------------
>   drivers/lightnvm/pblk.h      |  1 -
>   3 files changed, 9 insertions(+), 27 deletions(-)
> 
> diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
> index 64a6a255514e..c92113b0a2e7 100644
> --- a/drivers/lightnvm/pblk-core.c
> +++ b/drivers/lightnvm/pblk-core.c
> @@ -1633,8 +1633,6 @@ void pblk_line_close(struct pblk *pblk, struct pblk_line *line)
> 
>   	spin_unlock(&line->lock);
>   	spin_unlock(&l_mg->gc_lock);
> -
> -	pblk_gc_should_kick(pblk);
>   }
> 
>   void pblk_line_close_meta(struct pblk *pblk, struct pblk_line *line)
> diff --git a/drivers/lightnvm/pblk-rl.c b/drivers/lightnvm/pblk-rl.c
> index 596bdec433c3..0896439a91b0 100644
> --- a/drivers/lightnvm/pblk-rl.c
> +++ b/drivers/lightnvm/pblk-rl.c
> @@ -96,9 +96,11 @@ unsigned long pblk_rl_nr_free_blks(struct pblk_rl *rl)
>    *
>    * Only the total number of free blocks is used to configure the rate limiter.
>    */
> -static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
> +static void pblk_rl_update_rates(struct pblk_rl *rl)
>   {
> +	struct pblk *pblk = container_of(rl, struct pblk, rl);
>   	unsigned long free_blocks = pblk_rl_nr_free_blks(rl);
> +	int max = rl->rb_budget;
> 
>   	if (free_blocks >= rl->high) {
>   		rl->rb_user_max = max;
> @@ -124,23 +126,18 @@ static int pblk_rl_update_rates(struct pblk_rl *rl, unsigned long max)
>   		rl->rb_state = PBLK_RL_LOW;
>   	}
> 
> -	return rl->rb_state;
> +	if (rl->rb_state == (PBLK_RL_MID | PBLK_RL_LOW))
> +		pblk_gc_should_start(pblk);
> +	else
> +		pblk_gc_should_stop(pblk);
>   }
> 
>   void pblk_rl_free_lines_inc(struct pblk_rl *rl, struct pblk_line *line)
>   {
> -	struct pblk *pblk = container_of(rl, struct pblk, rl);
>   	int blk_in_line = atomic_read(&line->blk_in_line);
> -	int ret;
> 
>   	atomic_add(blk_in_line, &rl->free_blocks);
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>   }
> 
>   void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
> @@ -148,19 +145,7 @@ void pblk_rl_free_lines_dec(struct pblk_rl *rl, struct pblk_line *line)
>   	int blk_in_line = atomic_read(&line->blk_in_line);
> 
>   	atomic_sub(blk_in_line, &rl->free_blocks);
> -}
> -
> -void pblk_gc_should_kick(struct pblk *pblk)
> -{
> -	struct pblk_rl *rl = &pblk->rl;
> -	int ret;
> -
> -	/* Rates will not change that often - no need to lock update */
> -	ret = pblk_rl_update_rates(rl, rl->rb_budget);
> -	if (ret == (PBLK_RL_MID | PBLK_RL_LOW))
> -		pblk_gc_should_start(pblk);
> -	else
> -		pblk_gc_should_stop(pblk);
> +	pblk_rl_update_rates(rl);
>   }
> 
>   int pblk_rl_high_thrs(struct pblk_rl *rl)
> diff --git a/drivers/lightnvm/pblk.h b/drivers/lightnvm/pblk.h
> index eaf539715d71..d4b7dee0ef50 100644
> --- a/drivers/lightnvm/pblk.h
> +++ b/drivers/lightnvm/pblk.h
> @@ -829,7 +829,6 @@ int pblk_gc_init(struct pblk *pblk);
>   void pblk_gc_exit(struct pblk *pblk);
>   void pblk_gc_should_start(struct pblk *pblk);
>   void pblk_gc_should_stop(struct pblk *pblk);
> -void pblk_gc_should_kick(struct pblk *pblk);
>   void pblk_gc_kick(struct pblk *pblk);
>   void pblk_gc_sysfs_state_show(struct pblk *pblk, int *gc_enabled,
>   			      int *gc_active);
> --
> 2.7.4
> 

Thanks, I applied the common version.

^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-09-25 10:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-21 11:25 [PATCH 1/6] lightnvm: pblk: reuse pblk_gc_should_kick Rakesh Pandit
2017-09-22  8:40 ` Javier González
2017-09-22  9:42   ` Rakesh Pandit
2017-09-22  9:42     ` Rakesh Pandit
2017-09-22 10:04     ` Javier González
2017-09-25 10:24   ` Matias Bjørling
2017-09-25 10:24     ` Matias Bjørling

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.