Re: [PATCH rcu] Remove unused code originally used for context tracking
On Mon, Jan 07, 2013 at 06:46:07PM +0100, Frederic Weisbecker wrote: > 2013/1/7 Paul E. McKenney : > > On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: > >> As new context tracking subsystem added, it seems ignore_user_qs and > >> in_user defined in struct rcu_dynticks are no longer needed, so remove > >> them. > >> > >> Signed-off-by: Li Zhong > > > > Hearing no objections from Frederic, I have queued this patch for 3.9 > > Thanks Paul! > > And feel free to add my ack. Done, thank you! Thanx, Paul -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH rcu] Remove unused code originally used for context tracking
2013/1/7 Paul E. McKenney : > On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: >> As new context tracking subsystem added, it seems ignore_user_qs and >> in_user defined in struct rcu_dynticks are no longer needed, so remove >> them. >> >> Signed-off-by: Li Zhong > > Hearing no objections from Frederic, I have queued this patch for 3.9 Thanks Paul! And feel free to add my ack. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH rcu] Remove unused code originally used for context tracking
On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: > As new context tracking subsystem added, it seems ignore_user_qs and > in_user defined in struct rcu_dynticks are no longer needed, so remove > them. > > Signed-off-by: Li Zhong Hearing no objections from Frederic, I have queued this patch for 3.9 Thanx, Paul > --- > kernel/rcutree.c | 3 --- > kernel/rcutree.h | 4 > 2 files changed, 7 deletions(-) > > diff --git a/kernel/rcutree.c b/kernel/rcutree.c > index e441b77..b8fae5d 100644 > --- a/kernel/rcutree.c > +++ b/kernel/rcutree.c > @@ -2719,9 +2719,6 @@ rcu_boot_init_percpu_data(int cpu, struct rcu_state > *rsp) > rdp->dynticks = _cpu(rcu_dynticks, cpu); > WARN_ON_ONCE(rdp->dynticks->dynticks_nesting != DYNTICK_TASK_EXIT_IDLE); > WARN_ON_ONCE(atomic_read(>dynticks->dynticks) != 1); > -#ifdef CONFIG_RCU_USER_QS > - WARN_ON_ONCE(rdp->dynticks->in_user); > -#endif > rdp->cpu = cpu; > rdp->rsp = rsp; > rcu_boot_init_nocb_percpu_data(rdp); > diff --git a/kernel/rcutree.h b/kernel/rcutree.h > index 4b69291..6f21f2e 100644 > --- a/kernel/rcutree.h > +++ b/kernel/rcutree.h > @@ -102,10 +102,6 @@ struct rcu_dynticks { > /* idle-period nonlazy_posted snapshot. */ > int tick_nohz_enabled_snap; /* Previously seen value from sysfs. */ > #endif /* #ifdef CONFIG_RCU_FAST_NO_HZ */ > -#ifdef CONFIG_RCU_USER_QS > - bool ignore_user_qs;/* Treat userspace as extended QS or not */ > - bool in_user; /* Is the CPU in userland from RCU POV? */ > -#endif > }; > > /* RCU's kthread states for tracing. */ > -- > 1.7.11.4 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH rcu] Remove unused code originally used for context tracking
On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: As new context tracking subsystem added, it seems ignore_user_qs and in_user defined in struct rcu_dynticks are no longer needed, so remove them. Signed-off-by: Li Zhong zh...@linux.vnet.ibm.com Hearing no objections from Frederic, I have queued this patch for 3.9 Thanx, Paul --- kernel/rcutree.c | 3 --- kernel/rcutree.h | 4 2 files changed, 7 deletions(-) diff --git a/kernel/rcutree.c b/kernel/rcutree.c index e441b77..b8fae5d 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c @@ -2719,9 +2719,6 @@ rcu_boot_init_percpu_data(int cpu, struct rcu_state *rsp) rdp-dynticks = per_cpu(rcu_dynticks, cpu); WARN_ON_ONCE(rdp-dynticks-dynticks_nesting != DYNTICK_TASK_EXIT_IDLE); WARN_ON_ONCE(atomic_read(rdp-dynticks-dynticks) != 1); -#ifdef CONFIG_RCU_USER_QS - WARN_ON_ONCE(rdp-dynticks-in_user); -#endif rdp-cpu = cpu; rdp-rsp = rsp; rcu_boot_init_nocb_percpu_data(rdp); diff --git a/kernel/rcutree.h b/kernel/rcutree.h index 4b69291..6f21f2e 100644 --- a/kernel/rcutree.h +++ b/kernel/rcutree.h @@ -102,10 +102,6 @@ struct rcu_dynticks { /* idle-period nonlazy_posted snapshot. */ int tick_nohz_enabled_snap; /* Previously seen value from sysfs. */ #endif /* #ifdef CONFIG_RCU_FAST_NO_HZ */ -#ifdef CONFIG_RCU_USER_QS - bool ignore_user_qs;/* Treat userspace as extended QS or not */ - bool in_user; /* Is the CPU in userland from RCU POV? */ -#endif }; /* RCU's kthread states for tracing. */ -- 1.7.11.4 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH rcu] Remove unused code originally used for context tracking
2013/1/7 Paul E. McKenney paul...@linux.vnet.ibm.com: On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: As new context tracking subsystem added, it seems ignore_user_qs and in_user defined in struct rcu_dynticks are no longer needed, so remove them. Signed-off-by: Li Zhong zh...@linux.vnet.ibm.com Hearing no objections from Frederic, I have queued this patch for 3.9 Thanks Paul! And feel free to add my ack. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH rcu] Remove unused code originally used for context tracking
On Mon, Jan 07, 2013 at 06:46:07PM +0100, Frederic Weisbecker wrote: 2013/1/7 Paul E. McKenney paul...@linux.vnet.ibm.com: On Fri, Nov 30, 2012 at 02:19:22PM +0800, Li Zhong wrote: As new context tracking subsystem added, it seems ignore_user_qs and in_user defined in struct rcu_dynticks are no longer needed, so remove them. Signed-off-by: Li Zhong zh...@linux.vnet.ibm.com Hearing no objections from Frederic, I have queued this patch for 3.9 Thanks Paul! And feel free to add my ack. Done, thank you! Thanx, Paul -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH rcu] Remove unused code originally used for context tracking
As new context tracking subsystem added, it seems ignore_user_qs and in_user defined in struct rcu_dynticks are no longer needed, so remove them. Signed-off-by: Li Zhong --- kernel/rcutree.c | 3 --- kernel/rcutree.h | 4 2 files changed, 7 deletions(-) diff --git a/kernel/rcutree.c b/kernel/rcutree.c index e441b77..b8fae5d 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c @@ -2719,9 +2719,6 @@ rcu_boot_init_percpu_data(int cpu, struct rcu_state *rsp) rdp->dynticks = _cpu(rcu_dynticks, cpu); WARN_ON_ONCE(rdp->dynticks->dynticks_nesting != DYNTICK_TASK_EXIT_IDLE); WARN_ON_ONCE(atomic_read(>dynticks->dynticks) != 1); -#ifdef CONFIG_RCU_USER_QS - WARN_ON_ONCE(rdp->dynticks->in_user); -#endif rdp->cpu = cpu; rdp->rsp = rsp; rcu_boot_init_nocb_percpu_data(rdp); diff --git a/kernel/rcutree.h b/kernel/rcutree.h index 4b69291..6f21f2e 100644 --- a/kernel/rcutree.h +++ b/kernel/rcutree.h @@ -102,10 +102,6 @@ struct rcu_dynticks { /* idle-period nonlazy_posted snapshot. */ int tick_nohz_enabled_snap; /* Previously seen value from sysfs. */ #endif /* #ifdef CONFIG_RCU_FAST_NO_HZ */ -#ifdef CONFIG_RCU_USER_QS - bool ignore_user_qs;/* Treat userspace as extended QS or not */ - bool in_user; /* Is the CPU in userland from RCU POV? */ -#endif }; /* RCU's kthread states for tracing. */ -- 1.7.11.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH rcu] Remove unused code originally used for context tracking
As new context tracking subsystem added, it seems ignore_user_qs and in_user defined in struct rcu_dynticks are no longer needed, so remove them. Signed-off-by: Li Zhong zh...@linux.vnet.ibm.com --- kernel/rcutree.c | 3 --- kernel/rcutree.h | 4 2 files changed, 7 deletions(-) diff --git a/kernel/rcutree.c b/kernel/rcutree.c index e441b77..b8fae5d 100644 --- a/kernel/rcutree.c +++ b/kernel/rcutree.c @@ -2719,9 +2719,6 @@ rcu_boot_init_percpu_data(int cpu, struct rcu_state *rsp) rdp-dynticks = per_cpu(rcu_dynticks, cpu); WARN_ON_ONCE(rdp-dynticks-dynticks_nesting != DYNTICK_TASK_EXIT_IDLE); WARN_ON_ONCE(atomic_read(rdp-dynticks-dynticks) != 1); -#ifdef CONFIG_RCU_USER_QS - WARN_ON_ONCE(rdp-dynticks-in_user); -#endif rdp-cpu = cpu; rdp-rsp = rsp; rcu_boot_init_nocb_percpu_data(rdp); diff --git a/kernel/rcutree.h b/kernel/rcutree.h index 4b69291..6f21f2e 100644 --- a/kernel/rcutree.h +++ b/kernel/rcutree.h @@ -102,10 +102,6 @@ struct rcu_dynticks { /* idle-period nonlazy_posted snapshot. */ int tick_nohz_enabled_snap; /* Previously seen value from sysfs. */ #endif /* #ifdef CONFIG_RCU_FAST_NO_HZ */ -#ifdef CONFIG_RCU_USER_QS - bool ignore_user_qs;/* Treat userspace as extended QS or not */ - bool in_user; /* Is the CPU in userland from RCU POV? */ -#endif }; /* RCU's kthread states for tracing. */ -- 1.7.11.4 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/