From: Frederic Weisbecker <fweisbec@gmail.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: LKML <linux-kernel@vger.kernel.org>,
Dave Jones <davej@redhat.com>,
Thomas Gleixner <tglx@linutronix.de>,
Russell King <linux@arm.linux.org.uk>,
Catalin Iacob <iacobcatalin@gmail.com>,
"Paul E . McKenney" <paulmck@linux.vnet.ibm.com>,
Will Deacon <will.deacon@arm.com>, Ingo Molnar <mingo@kernel.org>,
Catalin Marinas <catalin.marinas@arm.com>
Subject: Re: [PATCH 0/8] nohz: Fix nohz kick irq work on tick v2
Date: Fri, 12 Sep 2014 20:05:49 +0200 [thread overview]
Message-ID: <20140912180547.GB5824@lerouge> (raw)
In-Reply-To: <20140911082812.GU6758@twins.programming.kicks-ass.net>
On Thu, Sep 11, 2014 at 10:28:12AM +0200, Peter Zijlstra wrote:
> On Wed, Sep 10, 2014 at 11:33:20PM +0200, Frederic Weisbecker wrote:
>
> > Frederic Weisbecker (7):
> > nohz: Move nohz full init call to tick init
> > irq_work: Force raised irq work to run on irq work interrupt
> > x86: Tell irq work about self IPI support
> > arm: Tell irq work about self IPI support
> > arm64: Tell irq work about self IPI support
> > nohz: Consolidate nohz full init code
> > nohz: nohz full depends on irq work self IPI support
> >
> > Peter Zijlstra (1):
> > irq_work: Introduce arch_irq_work_has_interrupt()
>
> Looks ok to me.
>
> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Intel? :-) Wow.
Thanks a lot!
Note, I fear I'll have to turn the static_cpu_has into cpu_has. The
alternatives code which initialize all static_cpu_has() things
is called from check_bugs() in start_kernel() which is way after
IRQs are enabled for the 1st time. irq_work_tick() has many opportunities
to run in between.
next prev parent reply other threads:[~2014-09-12 18:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-10 21:33 [PATCH 0/8] nohz: Fix nohz kick irq work on tick v2 Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 1/8] nohz: Move nohz full init call to tick init Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 2/8] irq_work: Introduce arch_irq_work_has_interrupt() Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 3/8] irq_work: Force raised irq work to run on irq work interrupt Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 4/8] x86: Tell irq work about self IPI support Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 5/8] arm: " Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 6/8] arm64: " Frederic Weisbecker
2014-09-11 9:00 ` Catalin Marinas
2014-09-13 16:47 ` Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 7/8] nohz: Consolidate nohz full init code Frederic Weisbecker
2014-09-10 21:33 ` [PATCH 8/8] nohz: nohz full depends on irq work self IPI support Frederic Weisbecker
2014-09-11 8:28 ` [PATCH 0/8] nohz: Fix nohz kick irq work on tick v2 Peter Zijlstra
2014-09-12 18:05 ` Frederic Weisbecker [this message]
2014-09-12 19:00 ` Peter Zijlstra
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140912180547.GB5824@lerouge \
--to=fweisbec@gmail.com \
--cc=catalin.marinas@arm.com \
--cc=davej@redhat.com \
--cc=iacobcatalin@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=mingo@kernel.org \
--cc=paulmck@linux.vnet.ibm.com \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=will.deacon@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox