On Tue, 2020-02-11 at 03:38:31 UTC, Gustavo Luiz Duarte wrote: > The test case tm-signal-context-force-tm expects a segfault to happen on > returning from signal handler, and then does a setcontext() to run the test > again. However, the test doesn't always segfault, causing the test to run a > single time. > > This patch fixes the test by putting it within a loop and jumping, via > setcontext, just prior to the loop in case it segfaults. This way we get the > desired behavior (run the test COUNT_MAX times) regardless if it segfaults or > not. This also reduces the use of setcontext for control flow logic, keeping > it > only in the segfault handler. > > Also, since 'count' is changed within the signal handler, it is declared as > volatile to prevent any compiler optimization getting confused with > asynchronous changes. > > Signed-off-by: Gustavo Luiz Duarte <gustav...@linux.ibm.com>
Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/0f8f554e5244f56f496b4ce30ada1126fe290345 cheers