lib/glibc
1
0
Fork 0
mirror of https://sourceware.org/git/glibc.git synced 2025-08-08 17:42:12 +03:00
Code Activity

nptl: Fix testcases for new pthread cancellation mechanism

Browse Source 
With upcoming fix for BZ#12683, pthread cancellation does not act for:

  1. If syscall is blocked but with some side effects already having
     taken place (e.g. a partial read or write).
  2. After the syscall has returned.

The main change is due the fact programs need to act in syscalls with
side-effects (for instance, to avoid leak of allocated resources or
handle partial read/write).

This patch changes the NPTL testcase that assumes the old behavior and
also changes the tst-backtrace{5,6} to ignore the cancellable wrappers.

Checked on i686-linux-gnu, x86_64-linux-gnu, x86_64-linux-gnux32,
aarch64-linux-gnu, arm-linux-gnueabihf, powerpc64le-linux-gnu,
powerpc-linux-gnu, sparcv9-linux-gnu, and sparc64-linux-gnu.

	* debug/tst-backtrace5.c (handle_signal): Avoid cancellable wrappers
	in backtrace analysis.
	* nptl/tst-cancel4.c (tf_write): Handle cancelled syscall with
	side-effects.
	(tf_send): Likewise.
This commit is contained in:
Adhemerval Zanella
2015-09-21 15:55:58 -07:00
committed by Adhemerval Zanella
parent 17cc27d5b7
commit d0d7f85f66
3 changed files with 24 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
debug/tst-backtrace5.c
View File

@@ -69,17 +69,18 @@ handle_signal (int signum)
FAIL ();
return;
}
/* Do not check name for signal trampoline. */
i = 2;
if (!match (symbols[i++], "read"))
/* Do not check name for signal trampoline or cancellable syscall
wrappers (__syscall_cancel*). */
for (; i < n - 1; i++)
if (match (symbols[i], "read"))
break;
if (i == n - 1)
{
/* Perhaps symbols[2] is __kernel_vsyscall? */
if (!match (symbols[i++], "read"))
{
FAIL ();
return;
}
FAIL ();
return;
}
for (; i < n - 1; i++)
if (!match (symbols[i], "fn"))
{