mapping already created during EFI intiialization.
Since the EFI code already maintains a memory map for that memory, add a
function efi_lookup_mapped_addr to look up mappings in that memory map.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/efi.c | 28
driver. Rework the existing ACPI BGRT driver to
act as a simple wrapper exposing that image (and the properties from the
BGRT) via sysfs.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/Makefile |1 +
arch/x86/platform/efi/efi-bgrt.c | 76 ++
arch
On Thu, Aug 30, 2012 at 02:46:03PM -0700, Paul E. McKenney wrote:
> On Thu, Aug 30, 2012 at 11:56:09AM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 11:44:48AM -0700, Paul E. McKenney wrote:
> > > Hello!
> > >
> > > This series covers changes to
On Thu, Aug 30, 2012 at 01:38:42PM -0700, Paul E. McKenney wrote:
> On Thu, Aug 30, 2012 at 12:00:18PM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 11:45:09AM -0700, Paul E. McKenney wrote:
> > > From: "Paul E. McKenney"
> > >
> > > Many
On Thu, Aug 30, 2012 at 02:35:36PM -0700, Paul E. McKenney wrote:
> On Thu, Aug 30, 2012 at 11:57:05AM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 11:45:08AM -0700, Paul E. McKenney wrote:
> > > From: "Paul E. McKenney"
> > >
> > &g
mit simplifies the code accordingly.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Impressive simplification!
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree.c | 83 ++-
> kernel/rcutree.h |
_rcu() to determine exactly when posting callbacks
> is illegal.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
One suggestion below; with or without that change:
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree.c | 10 ++
> 1 files chang
to increase the probability of those sorts of
> bugs occurring.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
If you end up adding more such conditional race-provoking delays
elsewhere in the code, consider creating a prove_r
ed by
> Michael Wang . ]
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> kernel/rcutiny.c | 31 ---
> 1 files changed, 16 insertions(+), 15 deletions(-)
>
> diff --git a/ker
es this issue.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree_plugin.h |7 ---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/rcutree_plugin.h b/kernel/rcutre
handlers might live in modules. In what
situation might a module interrupt handler get called from the idle
loop, before rcu_irq_enter or after rcu_irq_exit, and need to know that
when using RCU?
- Josh Triplett
> ---
> include/linux/rcupdate.h |6 ++
> kernel/rcutiny.c |
- but in this latter
> case, the root node's tasks needed to be boosted in any case.
>
> This commit therefore adds a check of the ->boost_tasks pointer against
> the ->exp_tasks pointer to the list that prevents updating ->boost_tasks.
Seems like some hint of this explan
It also inlines rcu_implicit_offline_qs() into
> rcu_implicit_dynticks_qs().
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree.c | 53 +
> 1 files
e's ->lock, but failed to update the header comment
> accordingly. This commit therefore brings the header comment up to date.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree_plugin.h |
On Thu, Aug 30, 2012 at 11:56:21AM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney"
>
> The increment_cpu_stall_ticks() function listed each RCU flavor
> explicitly, with an ifdef to handle preemptible RCU. This commit
> therefore applies for_each_rcu_flavor() to save a line of code.
A
raw_spin_lock_irqsave(&rnp->lock, flags);
if (rcu_preempt_blocked_readers_cgp(rnp)) {
...
}
raw_spin_unlock_irqrestore(&rnp->lock, flags);
- Josh Triplett
--
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 Thu, Aug 30, 2012 at 11:56:23AM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney"
>
> The print_other_cpu_stall() function accesses a number of rcu_node
> fields without protection from the ->lock. In theory, this is not
> a problem because the fields accessed are all integers, but in
es this issue by adding a check for
> ongoing grace period to avoid these spurious stall warnings.
>
> Reported-by: Becky Bruce
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> kernel/rcutree.c |3 ++-
> 1 files c
tream yet? If so, could you note that patch dependency explicitly
in the commit message?
- Josh Triplett
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
> ---
> kernel/rcutree.c |2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff
ore, this commit moves to local_irq_save().
It looks like the non-raw versions also include tracing, which typically
has recursive dependency problems with RCU. Can all of these call sites
safely call into tracing without recursing back into RCU?
- Josh Triplett
> Reported-by: Fengguang Wu
>
On Fri, Aug 31, 2012 at 11:30:35AM -0700, Josh Triplett wrote:
> On Thu, Aug 30, 2012 at 11:56:25AM -0700, Paul E. McKenney wrote:
> > From: "Paul E. McKenney"
> >
> > The first memory barrier in __call_rcu() is supposed to order any
> > updates done before
aul E. McKenney
> Signed-off-by: Paul E. McKenney
> Tested-by: Sedat Dilek
Reviewed-by: Josh Triplett
> ---
> include/linux/interrupt.h |2 ++
> kernel/time/tick-sched.c |3 ++-
> 2 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/include/linux/inter
-off-by: Michael Wang
> Acked-by: Catalin Marinas
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> ---
> mm/kmemleak.c |6 ++
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/mm/kmemleak.c b/mm/kmemleak.c
> index 45eb621..0d
c: Stephen Hemminger
> Cc: Steven Rostedt
> Cc: Sven-Thorsten Dietrich
> Cc: Thomas Gleixner
A few suggestions below: an optional microoptimization and some bugfixes.
With the bugfixes, and with or without the microoptimization:
Reviewed-by: Josh Triplett
> --- a/kernel/rcutree.c
t;enter" and "exit" names seem confusing. This seems
more like a flag you can set and clear, rather than a delimited region
as suggested by an enter/exit pair.
How about something vaguely like rcu_user_irq_set_eqs and
rcu_user_irq_clear_eqs?
- Josh Triplett
--
To unsubscrib
On Fri, Aug 31, 2012 at 09:54:39PM +0200, Frederic Weisbecker wrote:
> 2012/8/31 Josh Triplett :
> > Given that, the "enter" and "exit" names seem confusing. This seems
> > more like a flag you can set and clear, rather than a delimited region
> > as sug
m your other patch series, I don't know that it
makes as much sense in the idle case, either; perhaps it would make more
sense to wait and batch up more callbacks as long as you have memory,
and then run them in one big burst.
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E.
f
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
> Cc: Max Krasnyansky
> Cc: Peter Zijlstra
> Cc: Stephen Hemminger
> Cc: Steven Rostedt
> Cc: Sven-Thorsten Dietrich
> Cc: Thomas Gleixner
> Signed-off-by:
y
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
> Cc: Max Krasnyansky
> Cc: Peter Zijlstra
> Cc: Stephen Hemminger
sef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
> Cc: Max Krasnyansky
> Cc: Peter Zijlstra
> Cc: Stephen Hemminger
> Cc: Steven Rostedt
> Cc: Sven-Thorsten Dietrich
> Cc: Thomas Gleixner
> Sig
; Also set the syscalls hook on the next task if needed.
>
> Signed-off-by: Frederic Weisbecker
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
-by: Frederic Weisbecker
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> C
off-by: Frederic Weisbecker
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
&
Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
> Cc: Max Krasnyansky
> Cc: Peter Zijlstra
> Cc: Stephen Hemminger
> Cc: Steven Rostedt
> Cc: Sven-Thorsten Dietrich
> Cc: Thomas Gl
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
> Cc: Max Krasnyansky
&g
extended quiescent state
> in x86-64.
>
> Signed-off-by: Frederic Weisbecker
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter
eisbecker
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
ysky
> Cc: Matt Turner
> Cc: alpha
> Cc: Paul E. McKenney
> Cc: Michael Cree
Reviewed-by: Josh Triplett
> arch/alpha/kernel/process.c |3 ++-
> arch/alpha/kernel/smp.c |1 +
> 2 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/arch/
r
> Cc: Alessio Igor Bogani
> Cc: Andrew Morton
> Cc: Avi Kivity
> Cc: Chris Metcalf
> Cc: Christoph Lameter
> Cc: Geoff Levand
> Cc: Gilad Ben Yossef
> Cc: Hakan Akkan
> Cc: H. Peter Anvin
> Cc: Ingo Molnar
> Cc: Josh Triplett
> Cc: Kevin Hilman
&
y
> Signed-off-by: Frederic Weisbecker
> Cc: Richard Henderson
> Cc: Ivan Kokshaysky
> Cc: Matt Turner
> Cc: alpha
> Cc: Paul E. McKenney
> Cc: Michael Cree
> Cc: 3.2.x..
Reviewed-by: Josh Triplett
> arch/alpha/kernel/process.c |3 +++
> 1 files change
nney
> Signed-off-by: Frederic Weisbecker
> Cc: Mikael Starvik
> Cc: Jesper Nilsson
> Cc: Cris
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/cris/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --g
> Signed-off-by: Frederic Weisbecker
> Cc: David Howells
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/frv/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/frv/kernel/process.c b/arch/frv/k
> Signed-off-by: Frederic Weisbecker
> Cc: Yoshinori Sato
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/h8300/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/h8300/kernel/process.c b/a
> Signed-off-by: Frederic Weisbecker
> Cc: Hirokazu Takata
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/m32r/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/m32r/kernel/process.c b
> Signed-off-by: Frederic Weisbecker
> Acked-by: Geert Uytterhoeven
> Cc: m68k
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/m68k/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/m68
enney
> Signed-off-by: Frederic Weisbecker
> Cc: David Howells
> Cc: Koichi Yasutake
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/mn10300/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch
nney
> Signed-off-by: Frederic Weisbecker
> Cc: James E.J. Bottomley
> Cc: Helge Deller
> Cc: Parisc
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/parisc/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
&g
e scores's/score's/
> Reported-by: Paul E. McKenney
> Signed-off-by: Frederic Weisbecker
> Cc: Chen Liqin
> Cc: Lennox Wu
> Cc: 3.2.x..
> Cc: Paul E. McKenney
With the fix above,
Reviewed-by: Josh Triplett
> arch/score/kernel/process.c |4 +++-
> 1 files cha
y
> Signed-off-by: Frederic Weisbecker
> Cc: Chris Zankel
> Cc: 3.2.x..
> Cc: Paul E. McKenney
Reviewed-by: Josh Triplett
> arch/xtensa/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/xtensa/kernel/process.c b/arc
ia64, a shortcoming remedied
> by this commit.
>
> Reported by: Tony Luck
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
> Tested by: Tony Luck
Reviewed-by: Josh Triplett
> arch/ia64/kernel/process.c |3 +++
> 1 files changed, 3 insertions(+
eded to keep large-system scheduling
> latency at reasonable levels.
>
> Reported-by: Mike Galbraith
> Reported-by: Dimitri Sivanich
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 191
>
ent already
signalled?
In any case:
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 17 ++---
> 1 files changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/kernel/rcutree.c b/kernel/rcutree.c
> index e1c5868..ef56aa3 100644
> --- a/kernel/rc
lize grace periods. This is needed to keep
> scheduling latency down to a dull roar.
>
> Reported-by: Mike Galbraith
> Reported-by: Dimitri Sivanich
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 112
> ++-
: Mike Galbraith
> Reported-by: Dimitri Sivanich
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 11 +--
> 1 files changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/kernel/rcutree.c b/kernel/rcutree.c
> index 9fad21c..300
line CPUs. This commit therefore enforces this restriction.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c |2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/kernel/rcutree
() into rcu_gp_init() and rcu_gp_cleanup().
>
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 260
> +-
> 1 files changed, 138 insertions(+), 122 deletions(-)
>
> diff --git a/kerne
imely advancement for these callbacks.
Did you mean "every CPU with lazy callbacks" here?
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
> Tested-by: Sasha Levin
Reviewed-by: Josh Triplett
> kernel/rc
is can result in excessive
> memory contention on large systems, so this commit applies
> cacheline_internodealigned_in_smp to the ->onofflock field in
> order to segregate them.
>
> Signed-off-by: Dimitri Sivanich
> Signed-off-by: Paul E. McKenney
> Tested-by: Dimitri
nds of CPUs). This patch therefore inserts
> a voluntary preemption point into force_qs_rnp(), which should greatly
> reduce the magnitude of these spikes.
>
> Reported-by: Mike Galbraith
> Reported-by: Dimitri Sivanich
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplet
On Thu, Aug 30, 2012 at 11:18:26AM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney"
>
> Moving quiescent-state forcing into a kthread dispenses with the need
> for the ->n_rp_need_fqs field, so this commit removes it.
>
> Signed-off-by: Paul E. McKenn
quot;,
> -"rcu_node_level_3" }; /* Match MAX_RCU_LVLS */
> + static char *buf[] = { "rcu_node_0",
> + "rcu_node_1",
> +"rcu_node_2",
> +"rcu_node_3" }; /
general use) has a dependency on trace_clock. So,
only compile in trace_clock with CONFIG_RING_BUFFER or CONFIG_TRACING
enabled.
Signed-off-by: Josh Triplett
---
kernel/Makefile |2 +-
kernel/trace/Kconfig |5 +
kernel/trace/Makefile |6 +-
3 files changed, 7 insertions
SNDRV_MAIN_OBJECT_FILE hasn't done anything since the pre-git days, and
the only remaining reference occurs as a #define in sound/last.c. Drop
that last mention of it.
Signed-off-by: Josh Triplett
---
sound/last.c |1 -
1 file changed, 1 deletion(-)
diff --git a/sound/last.c b/
n treating them as one device), and the hardware works
just fine with the default USB HID support, drop the "default m".
Signed-off-by: Josh Triplett
---
drivers/hid/Kconfig |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
index fbf4950..
that represents the one and only module
built by default. These seem reasonable to me, but if anyone finds one of
these three changes objectionable, please feel free to drop that change. I
primarily care about getting patches 1-2 merged, to avoid a very common
annoyance.
Josh Triplett (5):
kernel builds don't need it;
disable it by default.
Signed-off-by: Josh Triplett
---
arch/x86/configs/i386_defconfig |1 -
arch/x86/configs/x86_64_defconfig |1 -
2 files changed, 2 deletions(-)
diff --git a/arch/x86/configs/i386_defconfig b/arch/x86/configs/i386_defconfig
index a6
The x86 defconfigs have become somewhat out of date compared to the
current result of "make savedefconfig". Update them to the current output, as
a prelude to further defconfig changes, to avoid unrelated noise in
those further changes.
Signed-off-by: Josh Triplett
---
arch/x
The current x86 and x86-64 defconfigs do not enable ext4, which most
current distributions default to. Switch the defconfigs to ext4, so
they will boot on current systems without additional configuration.
Signed-off-by: Josh Triplett
---
arch/x86/configs/i386_defconfig |7 +++
arch
CONFIG_CRC_T10DIF explicitly states that it exists only for use by
out-of-tree modules; anything in-kernel that needs it selects it. Thus,
compile it out by default.
Signed-off-by: Josh Triplett
---
arch/x86/configs/i386_defconfig |1 -
arch/x86/configs/x86_64_defconfig |1 -
2 files
Drop initrd support (and specifically RAM
block device support) from the defconfigs.
Signed-off-by: Josh Triplett
---
arch/x86/configs/i386_defconfig |2 --
arch/x86/configs/x86_64_defconfig |2 --
2 files changed, 4 deletions(-)
diff --git a/arch/x86/configs/i386_defconfig b/arc
ffies_till_next_fqs" that specifies the delay (again in jiffies)
> between subsequent attempts to force quiescent states. They both default
> to three jiffies, which is compatible with the old hard-coded behavior.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: P
On Mon, Sep 03, 2012 at 02:30:16AM -0700, Josh Triplett wrote:
> On Thu, Aug 30, 2012 at 11:18:28AM -0700, Paul E. McKenney wrote:
> > From: "Paul E. McKenney"
> >
> > Some uses of RCU benefit from shorter grace periods, while others benefit
> > more fr
removes them.
>
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.h |7 ---
> 1 files changed, 0 insertions(+), 7 deletions(-)
>
> diff --git a/kernel/rcutree.h b/kernel/rcutree.h
> index 2d4cc18..8f0293c 100644
> --- a/kernel/
This is unnecessary
> and inconvenient to administrators who might like an easy way to
> see what these values are on a running system. This commit therefore
> sets their permission masks to 0444, allowing them to be read but
> not written.
>
> Reported-by: Rusty Russell
> Rep
us avoiding this race.
I don't think it makes sense to introduce and subsequently fix a race in
the same patch series. :)
Could you squash this patch into the one moving grace-period
initialization into a kthread?
- Josh Triplett
> Signed-off-by: Paul E. McKenney
> ---
> kern
that this callback is now considered ready to be invoked.
>
> 12. CPU 16 invokes the callback, freeing data item A while CPU 1
> is still referencing it.
>
> This scenario represents a day-zero bug for TREE_RCU. This commit
> therefore ensures that the old grac
EE_RCU, however,
> the recent changes that permit RCU grace-period initialization to
> be preempted made it much more probable. Still, it is sufficiently
> improbable to make validation lengthy and inconvenient, so this commit
> adds an anti-heisenbug to greatly increase the coll
new grace period at grace-period initialization
> time. This commit therefore inserts a WARN_ON_ONCE() to verify this
> invariant.
>
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c |4 +++-
> 1 files changed, 3 insertions(+), 1 delet
f reasoning about
> correctness, this commit removes it.
>
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c | 19 ---
> 1 files changed, 0 insertions(+), 19 deletions(-)
>
> diff --git a/kernel/rcutree.c b/kernel/rcutree.
; rights to optimize to a simple "if (1)", which might not be what you want.
>
> This commit therefore converts synchronize_rcu_expedited()'s work-done
> detection counter from signed to unsigned.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. M
arge systems (hundreds or especially thousands
> of CPUs). This commit therefore holds off CPU-hotplug operations using
> get_online_cpus(), and removes the prior acquisiion of the ->onofflock
> (which required disabling interrupts).
>
> Signed-off-by: Paul E. McKenney
Reviewed-by:
t; Therefore, this commit makes this change.
>
> Signed-off-by: Paul E. McKenney
> Signed-off-by: Paul E. McKenney
Reviewed-by: Josh Triplett
> kernel/rcutree.c| 27 +++
> kernel/rcutree.h|2 --
> kernel/rcutree_plugin.h |2 --
> ke
On Tue, Sep 04, 2012 at 03:27:20PM +0100, Matt Fleming wrote:
> On Thu, 2012-08-30 at 14:28 -0700, Josh Triplett wrote:
> > The ACPI BGRT lets the OS access the BIOS logo image and its position on the
> > screen at boot time, allowing it to maintain that image on the screen unt
On Tue, Sep 04, 2012 at 11:10:54AM -0700, H. Peter Anvin wrote:
> On 09/04/2012 10:59 AM, Josh Triplett wrote:
> >
> >Unfortunately not. We need enough of ACPI available to go read the
> >BGRT to know what to copy, so we need to defer freeing boot services
> >code unt
On Tue, Sep 04, 2012 at 01:24:03PM -0700, H. Peter Anvin wrote:
> On 09/04/2012 12:45 PM, Josh Triplett wrote:
> >>
> >>There are some platforms which have bugs in this area, so there are
> >>other reasons to defer freeing up boot memory until as late in the
> >
tion of whether the sense is correct - you'll
> only have this capability if you don't boot with any form of
> authentication. CAP_KERNEL_ACCESS?
I'm fine with whatever name we come up with, but I'd like to avoid
bikeshedding it in every patch. Maybe we could work o
. Incorporated
feedback from H. Peter Anvin and Matt Fleming: added stubs for
x86-specific EFI functions called from init/main.c to eliminate the
corresponding ifdefs in start_kernel; deferred
efi_free_boot_services even later, to just before free_initmem.
Josh Triplett (4):
efi: Add a stub
This eliminates an ifdef in init/main.c.
Signed-off-by: Josh Triplett
---
include/linux/efi.h |4
init/main.c |2 --
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/include/linux/efi.h b/include/linux/efi.h
index ec45ccd..52fbedf 100644
--- a/include/linux
Some new ACPI 5.0 tables reference resources stored in boot services
memory, so keep that memory around until we have ACPI and can extract
data from it.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/efi.c | 31 ++-
include/linux/efi.h |2
mapping already created during EFI intiialization.
Since the EFI code already maintains a memory map for that memory, add a
function efi_lookup_mapped_addr to look up mappings in that memory map.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/efi.c | 28
driver. Rework the existing ACPI BGRT driver to
act as a simple wrapper exposing that image (and the properties from the
BGRT) via sysfs.
Signed-off-by: Josh Triplett
---
arch/x86/platform/efi/Makefile |1 +
arch/x86/platform/efi/efi-bgrt.c | 76 ++
arch
On Tue, Sep 04, 2012 at 03:33:50PM -0700, Paul E. McKenney wrote:
> On Fri, Aug 31, 2012 at 11:00:52AM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 11:56:17AM -0700, Paul E. McKenney wrote:
> > > From: "Paul E. McKenney"
> > >
> > > The
On Tue, Sep 04, 2012 at 03:46:59PM -0700, Paul E. McKenney wrote:
> On Fri, Aug 31, 2012 at 11:19:17AM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 11:56:22AM -0700, Paul E. McKenney wrote:
> > > From: "Paul E. McKenney"
> > >
> > >
On Tue, Sep 04, 2012 at 06:51:22PM -0400, Steven Rostedt wrote:
> On Tue, 2012-09-04 at 15:33 -0700, Paul E. McKenney wrote:
> > On Fri, Aug 31, 2012 at 11:00:52AM -0700, Josh Triplett wrote:
> > > On Thu, Aug 30, 2012 at 11:56:17AM -0700, Paul E. McKenney wrote:
> > >
On Tue, Sep 04, 2012 at 02:21:19PM -0700, Josh Triplett wrote:
> v2: Made the new internal function efi_unmap_memmap static. Incorporated
> feedback from H. Peter Anvin and Matt Fleming: added stubs for
> x86-specific EFI functions called from init/main.c to elim
On Tue, Sep 04, 2012 at 07:23:51PM -0400, Steven Rostedt wrote:
> On Tue, 2012-09-04 at 16:08 -0700, Josh Triplett wrote:
> > On Tue, Sep 04, 2012 at 06:51:22PM -0400, Steven Rostedt wrote:
> > > On Tue, 2012-09-04 at 15:33 -0700, Paul E. McKenney wrote:
> > > > On F
On Tue, Sep 04, 2012 at 07:46:42PM -0400, Steven Rostedt wrote:
> On Tue, 2012-09-04 at 16:33 -0700, Josh Triplett wrote:
> >
> > > > +#ifdef MODULE
> > > > +#define __DECLARE_TRACE_RCU(name, proto, args, cond, data_proto,
> > > > data_args) \
&g
On Tue, Sep 04, 2012 at 05:34:59PM -0700, Paul E. McKenney wrote:
> On Fri, Aug 31, 2012 at 04:44:01PM -0700, Josh Triplett wrote:
> > On Thu, Aug 30, 2012 at 02:05:21PM -0700, Paul E. McKenney wrote:
> > > From: Frederic Weisbecker
> > >
> > > Create a new c
, the
declaration of those otherwise unused functions causes the module to
reference rcu_idle_exit and rcu_idle_enter, which RCU does not export to
modules.
To avoid this, don't generate trace_*_rcuidle functions for tracepoints
declared in module code.
Reported-by: Steven Rostedt
Signed-off-by: Jos
t
>> CR4.OSXSAVE.
>>
>> Matt
>
> Hey Matt,
>
> Thank you for testing. CC-ing some of the Fedora folks so they are aware that
> they
> can ditch the: fix_xen_guest_on_old_EC2.patch
Ditched. Thanks Matt and Konrad.
(BTW, kernel-t...@fedoraproject.org is the easiest way to get things to
the fedora kernel team.)
josh
--
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/
301 - 400 of 7660 matches
Mail list logo
