On Mon, Apr 06, 2015 at 02:16:44PM -0400, cmetc...@ezchip.com wrote:
> From: Chris Metcalf
>
> The "clear" API is useful, for example, to modify a cpumask to avoid
> the nohz cores so that interrupts aren't sent to them.
>
> Likewise the "set" API is useful to modify a cpumask indicating some
>
On Tue, Apr 21, 2015 at 05:26:00PM +0200, Peter Zijlstra wrote:
> On Tue, Apr 21, 2015 at 02:23:07PM +0200, Frederic Weisbecker wrote:
> > +void __init context_tracking_cpu_set(int cpu)
> > {
> > + static __initdata bool initialized = false;
> > + struct task_stru
On Tue, Apr 21, 2015 at 10:52:18PM +0200, Peter Zijlstra wrote:
> On Tue, Apr 21, 2015 at 06:51:54PM +0200, Frederic Weisbecker wrote:
> > On Tue, Apr 21, 2015 at 05:26:00PM +0200, Peter Zijlstra wrote:
> > > On Tue, Apr 21, 2015 at 02:23:07PM +0200, Frederic Weisbecker wrote:
&g
t; to raise the softirq which only serves the purpose of getting the
> machine out of the inner idle loop.
>
> Signed-off-by: Thomas Gleixner
> Cc: John Stultz
> Cc: Preeti U Murthy
> Cc: Marcelo Tosatti
> Cc: Frederic Weisbecker
>
&
On Wed, Apr 22, 2015 at 05:35:53PM +0200, Peter Zijlstra wrote:
> On Tue, Apr 21, 2015 at 11:06:04PM +0200, Frederic Weisbecker wrote:
> >
> > Because I need the tick nohz intialization to be after irq initialization
> > (init_IRQ()),
> > which it depends on.
>
On Wed, Apr 22, 2015 at 03:39:35PM -0400, Tejun Heo wrote:
> Hello,
>
> Generally looks good to me. Some minor things below.
>
> On Tue, Apr 07, 2015 at 07:26:37PM +0800, Lai Jiangshan wrote:
> > diff --git a/kernel/workqueue.c b/kernel/workqueue.c
> > index cbccf5d..557612e 100644
> > --- a/ker
On Wed, Apr 22, 2015 at 04:32:11PM +0200, Thomas Gleixner wrote:
> On Wed, 22 Apr 2015, Frederic Weisbecker wrote:
> > But the reprogramming happens only under "if ((long)delta_jiffies >= 1)".
> > Probably this condition should go away as well.
>
> Errm
On Thu, Apr 23, 2015 at 03:07:59PM +0200, Thomas Gleixner wrote:
> On Thu, 23 Apr 2015, Frederic Weisbecker wrote:
> > On Wed, Apr 22, 2015 at 04:32:11PM +0200, Thomas Gleixner wrote:
> > > On Wed, 22 Apr 2015, Frederic Weisbecker wrote:
> > > > But the reprogramming
nticks.git
nohz/core
HEAD: 3c2b3b78a1cca2865a932d0265690997db9706b1
Thanks,
Frederic
---
Chris Metcalf (2):
nohz: Add tick_nohz_full_add_cpus_to() API
nohz: Set isolcpus when nohz_full is set
Frederic Weisbecker (2):
context_tracking: Protect against recursion
contex
defsky
Cc: Mike Galbraith
Cc: Ingo Molnar
Cc: Rik van Riel
Signed-off-by: Frederic Weisbecker
---
kernel/sched/core.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 6f149f8..e95b4d8 100644
--- a/kernel/sched/core.c
+++ b/kernel/sc
. Wysocki
Cc: Martin Schwidefsky
Cc: Ingo Molnar
Link:
http://lkml.kernel.org/r/1429024675-18938-1-git-send-email-cmetc...@ezchip.com
Signed-off-by: Frederic Weisbecker
---
include/linux/tick.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/linux/tick.h b/include/linux/tick.h
Oleg Nesterov
Reviewed-by: Rik van Riel
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking.h | 10 -
include/linux/sched.h
Riel
Acked-by: Peter Zijlstra (Intel)
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking_state.h | 1 +
kernel/context_tracking.c
On Fri, Apr 10, 2015 at 04:53:51PM -0400, Chris Metcalf wrote:
> The "removes_cpus_from" API is useful, for example, to modify a cpumask
> to avoid the nohz cores so that interrupts aren't sent to them.
>
> Likewise the "add_cpus_to" API is useful to modify a cpumask indicating
> some special nohz
On Fri, Apr 10, 2015 at 04:53:52PM -0400, Chris Metcalf wrote:
> nohz_full is only useful with isolcpus also set, since otherwise the
> scheduler has to run periodically to try to determine whether to steal
> work from other cores.
>
> Accordingly, when booting with nohz_full=xxx on the command li
On Mon, Apr 13, 2015 at 12:06:50PM -0400, Chris Metcalf wrote:
> Also, I want to ask Linus to pull the tile-specific changes for nohz_full
> for the tile architecture. This includes a copy of the change to add the
> tick_nohz_full_add_cpus_to() and tick_nohz_full_remove_cpus_from()
> routines here
on the command line, we
> should act as if isolcpus=xxx was also set, and set (or extend) the
> isolcpus set to include the nohz_full cpus.
>
> Acked-by: Frederic Weisbecker
No need to put my ack, since I'll add my Signed-off-by when I apply the patches.
Note this won't go t
On Mon, Apr 13, 2015 at 08:49:20PM -0400, Chris Metcalf wrote:
> On 4/13/2015 8:33 PM, Frederic Weisbecker wrote:
> >On Fri, Apr 10, 2015 at 04:53:51PM -0400, Chris Metcalf wrote:
> >>The "removes_cpus_from" API is useful, for example, to modify a cpumask
> &g
On Tue, Apr 14, 2015 at 04:09:45PM -0700, Jason Low wrote:
> While running a database workload, we found a scalability issue with itimers.
>
> Much of the problem was caused by the thread_group_cputimer spinlock.
> Each time we account for group system/user time, we need to obtain a
> thread_group
; Cc: Marcelo Tosatti
Reviewed-by: Frederic Weisbecker
--
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/
On Tue, Apr 14, 2015 at 03:37:31PM -0400, Chris Metcalf wrote:
> diff --git a/kernel/smpboot.c b/kernel/smpboot.c
> index c697f73d82d6..c5d53a335387 100644
> --- a/kernel/smpboot.c
> +++ b/kernel/smpboot.c
> @@ -92,6 +92,9 @@ enum {
> HP_THREAD_PARKED,
> };
>
> +/* Statically allocated and
On Thu, Apr 16, 2015 at 11:50:06AM -0400, Chris Metcalf wrote:
> On 4/16/2015 11:28 AM, Frederic Weisbecker wrote:
> >>+ /* Unpark any threads that were voluntarily parked. */
> >>>+ if (ht->cpumask) {
> >>>+ cpumask_andnot(
On Mon, Mar 09, 2015 at 03:09:17PM +0100, Sebastian Andrzej Siewior wrote:
> I just stumbled over this in -RT. The code:
>
> 87 bool irq_work_queue(struct irq_work *work)
> 88 {
> …
> 96 /* If the work is "lazy", handle it from next tick if any */
> 97 if (work->flags & IRQ_WOR
. McKenney
Signed-off-by: Rik van Riel
Cc: Paul E. McKenney
Cc: Andy Lutomirski
Cc: Will deacon
Cc: Marcelo Tosatti
Cc: Christian Borntraeger
Cc: Luiz Capitulino
Cc: Paolo Bonzini
Signed-off-by: Frederic Weisbecker
---
include/linux/context_tracking.h | 6 ++
include/linux
guest
context_tracking: Add stub context_tracking_is_enabled
context_tracking: Run vtime_user_enter/exit only when state ==
CONTEXT_USER
context_tracking: Export context_tracking_user_enter/exit
kvm,rcu,nohz: use RCU extended quiescent state when running KVM guest
Fre
lo Bonzini
Signed-off-by: Frederic Weisbecker
---
arch/x86/kernel/traps.c| 2 +-
include/linux/context_tracking.h | 2 +-
include/linux/context_tracking_state.h | 6 +++---
kernel/context_tracking.c | 8
kernel/sched/core.c| 2 +-
5 fil
c: Christian Borntraeger
Cc: Luiz Capitulino
Cc: Paolo Bonzini
Signed-off-by: Frederic Weisbecker
---
kernel/context_tracking.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/kernel/context_tracking.c b/kernel/context_tracking.c
index 17715d8..a2c
er code, and cannot easily use C enum
parameters.
Reviewed-by: Paul E. McKenney
Signed-off-by: Rik van Riel
Cc: Paul E. McKenney
Cc: Andy Lutomirski
Cc: Will deacon
Cc: Marcelo Tosatti
Cc: Christian Borntraeger
Cc: Luiz Capitulino
Cc: Paolo Bonzini
Signed-off-by: Frederic Weisbecker
---
i
Signed-off-by: Frederic Weisbecker
---
kernel/context_tracking.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/kernel/context_tracking.c b/kernel/context_tracking.c
index a2c0866..72d59a1 100644
--- a/kernel/context_tracking.c
+++ b/kernel/context_tracking.c
@@ -108,6 +108,7 @@ void
E. McKenney
Cc: Andy Lutomirski
Cc: Will deacon
Cc: Marcelo Tosatti
Cc: Christian Borntraeger
Cc: Luiz Capitulino
Cc: Paolo Bonzini
Signed-off-by: Frederic Weisbecker
---
include/linux/context_tracking_state.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/linux
These don't seem to be used anywhere.
Acked-by: Rik van Riel
Cc: Benjamin Herrenschmidt
Cc: Alexander Graf
Cc: Paul E. McKenney
Cc: Andy Lutomirski
Cc: Will deacon
Cc: Marcelo Tosatti
Cc: Christian Borntraeger
Cc: Luiz Capitulino
Cc: Paolo Bonzini
Signed-off-by: Frederic Weisb
On Fri, Apr 24, 2015 at 03:24:27PM -0400, r...@redhat.com wrote:
> From: Rik van Riel
>
> After system bootup, there is no totally reliable way to see
> which CPUs are isolated, because the kernel may modify the
> CPUs specified on the isolcpus= kernel command line option.
>
> Export the CPU lis
On Fri, Apr 24, 2015 at 04:07:52PM -0400, Gene Heskett wrote:
> On Friday 24 April 2015 11:58:31 Frederic Weisbecker wrote:
> > From: Chris Metcalf
> >
> > nohz_full is only useful with isolcpus also set, since otherwise the
> > scheduler has to run periodically to
On Thu, Apr 02, 2015 at 06:08:09PM -0300, Marcelo Tosatti wrote:
> On Thu, Apr 02, 2015 at 04:59:40PM +0200, Frederic Weisbecker wrote:
> > On Wed, Apr 01, 2015 at 10:44:55PM -0300, Marcelo Tosatti wrote:
> > >
> > > It is only necessary to raise timer softirq
>
On Mon, Apr 06, 2015 at 08:51:26PM -0300, Marcelo Tosatti wrote:
> On Tue, Apr 07, 2015 at 01:34:15AM +0200, Frederic Weisbecker wrote:
> > Yeah, it would be nice to make sure that the cause of these softirqs isn't
> > mistakenly ignored.
> > And also I want to be sure
On Tue, Apr 07, 2015 at 11:10:49PM +0200, Thomas Gleixner wrote:
> On Mon, 6 Apr 2015, Marcelo Tosatti wrote:
> > It is only necessary to raise timer softirq
> > in case there are active timers.
>
> Depends. See below.
>
> > Limit the ksoftirqd wakeup to that case.
> >
> > Fixes a latency spike
On Mon, Apr 06, 2015 at 02:16:45PM -0400, cmetc...@ezchip.com wrote:
> From: Chris Metcalf
>
> It's not clear that nohz_full is useful without isolcpus also
> set, since otherwise the scheduler has to run periodically to
> try to determine whether to steal work from other cores.
>
> Signed-off-b
On Mon, Apr 06, 2015 at 03:45:55PM -0400, cmetc...@ezchip.com wrote:
> From: Chris Metcalf
>
> This change allows some cores to be excluded from running the
> smp_hotplug_thread tasks. The motivating example for this is
> the watchdog threads, which by default we don't want to run
> on any enabl
On Mon, Apr 06, 2015 at 03:45:56PM -0400, cmetc...@ezchip.com wrote:
> From: Chris Metcalf
>
> Change the default behavior of watchdog so it only runs on the
> housekeeping cores when nohz_full is enabled at build and boot time.
>
> Allow modifying the set of cores the watchdog is currently runn
On Wed, Apr 08, 2015 at 11:21:56AM -0400, Chris Metcalf wrote:
> On 04/08/2015 10:26 AM, Peter Zijlstra wrote:
> >On Wed, Apr 08, 2015 at 10:04:43AM -0400, Chris Metcalf wrote:
> >>On 04/08/2015 05:41 AM, Peter Zijlstra wrote:
> >>>On Mon, Apr 06, 2015 at 02:16:45PM -0400, cmetc...@ezchip.com wrote
On Wed, Apr 08, 2015 at 10:06:44AM -0400, Chris Metcalf wrote:
> On 04/08/2015 09:28 AM, Frederic Weisbecker wrote:
> >On Mon, Apr 06, 2015 at 03:45:55PM -0400, cmetc...@ezchip.com wrote:
> >>From: Chris Metcalf
> >>
> >>This change allows some c
On Thu, Apr 30, 2015 at 03:39:24PM -0400, Chris Metcalf wrote:
> This change allows some cores to be excluded from running the
> smp_hotplug_thread tasks. The following commit to update
> kernel/watchdog.c to use this functionality is the motivating
> example, and more information on the motivatio
On Fri, May 01, 2015 at 03:57:51PM -0400, Chris Metcalf wrote:
> On 05/01/2015 04:53 AM, Frederic Weisbecker wrote:
> >>+ /* Unpark any threads that were voluntarily parked. */
> >>+ for_each_cpu_not(cpu, &ht->cpumask) {
> >>+ if (cpu_online(c
. Wysocki
Cc: Martin Schwidefsky
Cc: Ingo Molnar
Link:
http://lkml.kernel.org/r/1429024675-18938-1-git-send-email-cmetc...@ezchip.com
Signed-off-by: Frederic Weisbecker
---
include/linux/tick.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/linux/tick.h b/include/linux/tick.h
Metcalf (2):
nohz: Add tick_nohz_full_add_cpus_to() API
nohz: Set isolcpus when nohz_full is set
Frederic Weisbecker (2):
context_tracking: Protect against recursion
context_tracking: Inherit TIF_NOHZ through forks instead of context
switches
include/linux/context_
, and set (or extend) the
isolcpus set to include the nohz_full cpus.
Acked-by: Frederic Weisbecker
Acked-by: Mike Galbraith
Acked-by: Rik van Riel
Acked-by: Peter Zijlstra (Intel)
Signed-off-by: Chris Metcalf
Cc: Peter Zijlstra (Intel)
Cc: Paul E. McKenney
Cc: Rafael J. Wysocki
Cc: Martin
Oleg Nesterov
Reviewed-by: Rik van Riel
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking.h | 10
kernel/context_tracking.c
Riel
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking_state.h | 1 +
kernel/context_tracking.c | 30
On Thu, May 21, 2015 at 05:57:59AM -0700, Paul E. McKenney wrote:
> On Thu, May 21, 2015 at 05:42:46PM +0530, Afzal Mohammed wrote:
> > Hi,
> >
> > On Wed, May 20, 2015 at 02:00:26PM -0700, Paul E. McKenney wrote:
> >
> > > > > Given that kernel initiated association to isolcpus, a user turning
>
On Thu, May 21, 2015 at 07:14:09AM -0700, Paul E. McKenney wrote:
> On Thu, May 21, 2015 at 06:59:05PM +0530, Afzal Mohammed wrote:
> > Hi,
> >
> > On Thu, May 21, 2015 at 03:06:23PM +0200, Frederic Weisbecker wrote:
> > > On Thu, May 21, 2015 at 05:57:59AM
On Thu, May 21, 2015 at 08:59:38PM +0200, Mike Galbraith wrote:
> I think it's a mistake to make any rash assumption and/or mandate that
> the user WILL use nohz_full CPUs immediately, or even at all for that
> matter. nohz_full currently is nothing but a CPU attribute, period,
> nothing more, not
. Wysocki
Cc: Martin Schwidefsky
Cc: Ingo Molnar
Link:
http://lkml.kernel.org/r/1429024675-18938-1-git-send-email-cmetc...@ezchip.com
Signed-off-by: Frederic Weisbecker
---
include/linux/tick.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/linux/tick.h b/include/linux/tick.h
: Add tick_nohz_full_add_cpus_to() API
nohz: Set isolcpus when nohz_full is set
Frederic Weisbecker (2):
context_tracking: Protect against recursion
context_tracking: Inherit TIF_NOHZ through forks instead of context
switches
include/linux/context_tracking.h | 10 -
include/
Riel
Acked-by: Peter Zijlstra (Intel)
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking_state.h | 1 +
kernel/context_tracking.c
Oleg Nesterov
Reviewed-by: Rik van Riel
Signed-off-by: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Paul E. McKenney
Cc: Ingo Molnar
Cc: Dave Jones
Cc: Oleg Nesterov
---
include/linux/context_tracking.h | 10 -
include/linux/sched.h
defsky
Cc: Mike Galbraith
Cc: Ingo Molnar
Cc: Rik van Riel
Signed-off-by: Frederic Weisbecker
---
kernel/sched/core.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 6f149f8..e95b4d8 100644
--- a/kernel/sched/core.c
+++ b/kernel/sc
On Mon, May 04, 2015 at 04:53:16PM -0400, Rik van Riel wrote:
> On 05/04/2015 04:38 PM, Paul E. McKenney wrote:
> > On Mon, May 04, 2015 at 04:13:50PM -0400, Rik van Riel wrote:
> >> On 05/04/2015 04:02 PM, Paul E. McKenney wrote:
>
> >>> Hmmm... But didn't earlier performance measurements show t
"CHECK" suggests it's only used as a comparison mask. But now it's used
further as a config-conditional preempt disabler offset. Lets
disambiguate this name.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/preempt.h b/include/linux/preempt.h
index 45da394..4057696 100644
--- a
-dynticks.git
sched/preempt
HEAD: c8bda0e6afc163e7e2804d30b9e4301652dc679c
Thanks,
Frederic
---
Frederic Weisbecker (7):
preempt: Merge preempt_mask.h into preempt.h
preempt: Rearrange a few symbols after headers merge
preempt: Rename PREEMPT_CHECK_OFFSET to
Adjust a few comments, and further integrate a few definitions after
the dumb headers copy.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 34 +++---
1 file changed, 15
: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 12
kernel/sched/core.c | 20 ++--
2 files changed, 18 insertions(+), 14 deletions(-)
diff --git a/include/linux/preempt.h b/include/linux/preempt.h
index 4689ef2
Now that PREEMPT_ACTIVE implies PREEMPT_DISABLE_OFFSET, ignoring
PREEMPT_ACTIVE from in_atomic() check isn't useful anymore.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 2 +-
1 file chang
gments are
performed in a subsequent patch to ease review.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
arch/m68k/include/asm/irqflags.h | 3 -
include/linux/bottom_half.h | 1 -
include/linux/hardirq.h | 2 +-
inc
: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
kernel/sched/core.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 182127a..b23def2 100644
--- a/kernel/sched/core.c
+++ b
On Mon, May 11, 2015 at 08:57:59AM -0400, Steven Rostedt wrote:
>
> NO_HZ_LEAVE_ME_THE_FSCK_ALONE!
>
>
> On Sat, 9 May 2015 09:05:38 +0200
> Ingo Molnar wrote:
>
> > So I think we should either rename NO_HZ_FULL to NO_HZ_PURE, or keep
> > it at NO_HZ_FULL: because the intention of NO_HZ_FULL
On Mon, May 11, 2015 at 05:52:13PM +0200, Peter Zijlstra wrote:
> On Mon, May 11, 2015 at 05:08:22PM +0200, Frederic Weisbecker wrote:
> > +++ b/kernel/sched/core.c
> > @@ -2723,9 +2723,7 @@ again:
> > * - return from syscall or exception to user-space
> >
On Mon, May 11, 2015 at 10:27:44AM -0700, Andrew Morton wrote:
> On Mon, 11 May 2015 10:19:16 -0700 "Paul E. McKenney"
> wrote:
>
> > On Mon, May 11, 2015 at 08:57:59AM -0400, Steven Rostedt wrote:
> > >
> > > NO_HZ_LEAVE_ME_THE_FSCK_ALONE!
> >
> > NO_HZ_OVERFLOWING?
>
> Actually, "NO_HZ" sho
,
Frederic
---
Frederic Weisbecker (6):
preempt: Merge preempt_mask.h into preempt.h
preempt: Rearrange a few symbols after headers merge
preempt: Rename PREEMPT_CHECK_OFFSET to PREEMPT_DISABLE_OFFSET
sched: Optimize preemption operations on __schedule() callers
Now that PREEMPT_ACTIVE implies PREEMPT_DISABLE_OFFSET, ignoring
PREEMPT_ACTIVE from in_atomic() check isn't useful anymore.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 2 +-
1 file chang
order to optimize preempt_count()
operations in a single place.
Suggested-by: Linus Torvalds
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 12
kernel/sched/core.c | 29
"CHECK" suggests it's only used as a comparison mask. But now it's used
further as a config-conditional preempt disabler offset. Lets
disambiguate this name.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
Adjust a few comments, and further integrate a few definitions after
the dumb headers copy.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 34 +++---
1 file changed, 15
gments are
performed in a subsequent patch to ease review.
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
arch/m68k/include/asm/irqflags.h | 3 -
include/linux/bottom_half.h | 1 -
include/linux/hardirq.h | 2 +-
inc
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Thomas Gleixner
Cc: Linus Torvalds
Signed-off-by: Frederic Weisbecker
---
include/linux/preempt.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/preempt.h b/include/linux/preempt.h
index 45da394..4057696 100644
--- a
On Tue, May 12, 2015 at 02:34:40PM +0200, Ingo Molnar wrote:
>
> * Peter Zijlstra wrote:
>
> > On Tue, May 12, 2015 at 11:10:32AM +0200, Ingo Molnar wrote:
> > >
> > > So I'd vote for Frederic's CONFIG_ISOLATION=y, mostly because this
> > > is a high level kernel feature, so it won't conflict
r. This commit therefore directly drives
> the value of RCU_USER_QS via NO_HZ_FULL's "select" statement.
>
> Reported-by: Ingo Molnar
> Signed-off-by: Paul E. McKenney
> Cc: Frederic Weisbecker
ACK. And we should remove it completely and use NO_HZ_FULL instead.
The
On Wed, May 13, 2015 at 10:45:05AM -0700, Paul E. McKenney wrote:
> On Wed, May 13, 2015 at 02:37:52AM +0200, Frederic Weisbecker wrote:
> > On Tue, May 12, 2015 at 03:30:45PM -0700, Paul E. McKenney wrote:
> > > From: "Paul E. McKenney"
> > >
> > >
On Thu, May 07, 2015 at 11:58:32AM +0200, Ingo Molnar wrote:
>
> * Frederic Weisbecker wrote:
>
> > @@ -75,6 +94,11 @@ void context_tracking_enter(enum ctx_state state)
> > WARN_ON_ONCE(!current->mm);
> >
> > local_irq_save(flags);
> >
On Thu, May 07, 2015 at 12:48:45PM +0200, Ingo Molnar wrote:
>
> * Rik van Riel wrote:
>
> > > If, on the other hand, you're just going to remotely sample the
> > > in-memory context, that sounds good.
> >
> > It's the latter.
> >
> > If you look at /proc//{stack,syscall,wchan} and other file
On Wed, Apr 08, 2015 at 03:21:00PM -0400, Chris Metcalf wrote:
> On 04/08/2015 10:01 AM, Frederic Weisbecker wrote:
> >How about creating smpboot_update_mask_percpu_thread() and handle it from
> >smpboot,
> >this way future evolutions of smpboot won't overlook th
On Thu, Apr 09, 2015 at 08:02:28AM -0400, Chris Metcalf wrote:
> On 4/9/2015 4:29 AM, Peter Zijlstra wrote:
> >On Wed, Apr 08, 2015 at 02:12:34PM -0400, Chris Metcalf wrote:
> >
> >>>But you're doing the reverse! You're setting nohz_full for isolcpus, not
> >>>limiting the nohz_full mask to isolcpu
On Fri, Apr 10, 2015 at 12:33:38PM -0400, Chris Metcalf wrote:
> On 04/09/2015 09:58 PM, Frederic Weisbecker wrote:
> >On Thu, Apr 09, 2015 at 04:29:01PM -0400, Chris Metcalf wrote:
> >>--- a/include/linux/smpboot.h
> >>+++ b/include/linux/smpboot.h
> >>@@ -27,6
On Mon, Apr 13, 2015 at 12:06:50PM -0400, Chris Metcalf wrote:
> >>The problem with the code you provided, as I see it, is that the cpumask
> >>field being kept in the struct smp_hotplug_thread is awkward to
> >>initialize while keeping the default that it doesn't have to be mentioned
> >>in the in
On Tue, Mar 10, 2015 at 03:37:11PM +0100, Frederic Weisbecker wrote:
> Ingo,
>
> Please pull the nohz/guest branch that can be found at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
> nohz/guest
>
> This is
On Mon, Mar 16, 2015 at 10:12:12AM -0700, Kevin Hilman wrote:
> Lai Jiangshan writes:
>
> > On 03/14/2015 07:49 AM, Kevin Hilman wrote:
> >> Lai Jiangshan writes:
> >>
> >>> From: Frederic Weisbecker
> >>>
> >>> Create a cpu
: Frederic Weisbecker
---
arch/s390/include/asm/cputime.h | 47 +
1 file changed, 24 insertions(+), 23 deletions(-)
diff --git a/arch/s390/include/asm/cputime.h b/arch/s390/include/asm/cputime.h
index f8c1969..820b38a 100644
--- a/arch/s390/include/asm
: Frederic Weisbecker
---
include/linux/jiffies.h | 2 ++
kernel/time/time.c | 10 ++
kernel/time/timeconst.bc | 6 ++
3 files changed, 18 insertions(+)
diff --git a/include/linux/jiffies.h b/include/linux/jiffies.h
index c367cbd..1161e5c 100644
--- a/include/linux/jiffies.h
+++ b
: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
kernel/sched/cputime.c | 24 +---
kernel/sched/sched.h | 47 +--
2 files changed, 22 insertions(+), 49 deletions(-)
diff --git a/kernel/sched/cputime.c b/kernel
display the nanosec
value and stick with that.
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
kernel/delayacct.c | 10 +-
1
: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
kernel/sched/cputime.c | 48
kernel/sched/sched.h | 1 +
2 files changed, 21 insertions(+), 28
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
kernel/signal.c | 16
: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
arch/alpha/kernel/osf_sys.c| 2 +-
arch/x86/kernel/apm_32.c | 2 +-
drivers/isdn
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
kernel/sched/cputime.c | 24
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
arch/alpha/kernel/osf_sys.c | 8
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
fs/binfmt_elf.c| 26
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
arch/ia64/kernel/time.c | 5
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
arch/powerpc/kernel/time.c | 2
Cc: Benjamin Herrenschmidt
Cc: Heiko Carstens
Cc: Ingo Molnar
Cc: Martin Schwidefsky
Cc: Oleg Nesterov
Cc: Paul Mackerras
Cc: Peter Zijlstra
Cc: Rik van Riel
Cc: Thomas Gleixner
Cc: Tony Luck
Cc: Wu Fengguang
Signed-off-by: Frederic Weisbecker
---
arch/ia64/kernel/time.c | 2
2101 - 2200 of 4554 matches
Mail list logo