ARM: 6654/1: perf/oprofile: fix off-by-one in stack check
authorRabin Vincent <rabin.vincent@stericsson.com>
Wed, 9 Feb 2011 10:35:12 +0000 (11:35 +0100)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Thu, 10 Feb 2011 15:29:03 +0000 (15:29 +0000)
commitcb06199b1df492fcfbaedd2256b5054f944b664f
tree910e7c88876d5e56be4d4f96dc1ab99868ec047c
parent4a9cb360197684a861bc06f06d33d5fcc8ffcbf5
ARM: 6654/1: perf/oprofile: fix off-by-one in stack check

Since tail is the previous fp - 1, we need to compare the new fp with tail + 1
to ensure that we don't end up passing in the same tail again, in order to
avoid a potential infinite loop in the perf interrupt handler (which has been
observed to occur).  A similar fix seems to be needed in the OProfile code.

Acked-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Rabin Vincent <rabin.vincent@stericsson.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/kernel/perf_event.c
arch/arm/oprofile/common.c