diff mbox

rcu: remove rcu_user_hooks_switch

Message ID 1445291102-13176-1-git-send-email-yang.shi@linaro.org
State Superseded
Headers show

Commit Message

Yang Shi Oct. 19, 2015, 9:45 p.m. UTC
It is not invoked by anyone now, just remove it.

Signed-off-by: Yang Shi <yang.shi@linaro.org>
---
 include/linux/rcupdate.h | 2 --
 1 file changed, 2 deletions(-)

Comments

Paul E. McKenney Oct. 19, 2015, 10:59 p.m. UTC | #1
On Mon, Oct 19, 2015 at 02:45:02PM -0700, Yang Shi wrote:
> It is not invoked by anyone now, just remove it.
> 
> Signed-off-by: Yang Shi <yang.shi@linaro.org>

We should check with Frederic -- any plans to use this, Frederic?
It was last touched by commit 4d9a5d4319e2 (rcu: Remove rcu_switch())
back in 2012.

							Thanx, Paul

> ---
>  include/linux/rcupdate.h | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/include/linux/rcupdate.h b/include/linux/rcupdate.h
> index 581abf8..56355bc 100644
> --- a/include/linux/rcupdate.h
> +++ b/include/linux/rcupdate.h
> @@ -345,8 +345,6 @@ void rcu_user_exit(void);
>  #else
>  static inline void rcu_user_enter(void) { }
>  static inline void rcu_user_exit(void) { }
> -static inline void rcu_user_hooks_switch(struct task_struct *prev,
> -					 struct task_struct *next) { }
>  #endif /* CONFIG_NO_HZ_FULL */
> 
>  #ifdef CONFIG_RCU_NOCB_CPU
> -- 
> 2.0.2
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
Frederic Weisbecker Oct. 19, 2015, 11:46 p.m. UTC | #2
On Mon, Oct 19, 2015 at 03:59:07PM -0700, Paul E. McKenney wrote:
> On Mon, Oct 19, 2015 at 02:45:02PM -0700, Yang Shi wrote:
> > It is not invoked by anyone now, just remove it.
> > 
> > Signed-off-by: Yang Shi <yang.shi@linaro.org>
> 
> We should check with Frederic -- any plans to use this, Frederic?
> It was last touched by commit 4d9a5d4319e2 (rcu: Remove rcu_switch())
> back in 2012.

Not at all, feel free to remove it with my ACK!

Thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
Paul E. McKenney Oct. 20, 2015, 8:35 p.m. UTC | #3
On Tue, Oct 20, 2015 at 01:46:03AM +0200, Frederic Weisbecker wrote:
> On Mon, Oct 19, 2015 at 03:59:07PM -0700, Paul E. McKenney wrote:
> > On Mon, Oct 19, 2015 at 02:45:02PM -0700, Yang Shi wrote:
> > > It is not invoked by anyone now, just remove it.
> > > 
> > > Signed-off-by: Yang Shi <yang.shi@linaro.org>
> > 
> > We should check with Frederic -- any plans to use this, Frederic?
> > It was last touched by commit 4d9a5d4319e2 (rcu: Remove rcu_switch())
> > back in 2012.
> 
> Not at all, feel free to remove it with my ACK!

Very good, queued for testing and review with your ack.

							Thanx, Paul

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
diff mbox

Patch

diff --git a/include/linux/rcupdate.h b/include/linux/rcupdate.h
index 581abf8..56355bc 100644
--- a/include/linux/rcupdate.h
+++ b/include/linux/rcupdate.h
@@ -345,8 +345,6 @@  void rcu_user_exit(void);
 #else
 static inline void rcu_user_enter(void) { }
 static inline void rcu_user_exit(void) { }
-static inline void rcu_user_hooks_switch(struct task_struct *prev,
-					 struct task_struct *next) { }
 #endif /* CONFIG_NO_HZ_FULL */
 
 #ifdef CONFIG_RCU_NOCB_CPU