[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Irit Katriel]

Change by Irit Katriel :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Xavier de Gaye]

Change by Xavier de Gaye :


--
nosy:  -xdegaye

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 00ada2c1d57c5b8b468bad32ff24fa14113ae5c7 by Victor Stinner 
(xdegaye) in branch 'master':
bpo-38852: Set thread stack size to 8 Mb for debug builds on android platforms 
(GH-17337)
https://github.com/python/cpython/commit/00ada2c1d57c5b8b468bad32ff24fa14113ae5c7


--
nosy: +vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Xavier de Gaye]

Xavier de Gaye  added the comment:

Using test and try with _thread.stack_size(new_size), the pthread stack sizes 
must be set to the following minimums to prevent stack overflow and get a 
RecursionError:
  * debug builds:7 Mb
  * no-debug builds: 1 Mb

The default stack size for the main thread does not need to be changed as test 
test_exceptions.ExceptionTests.testInfiniteRecursion is ok for both build types.

PR 17337 sets the thread stack size to 8 Mb for debug builds on android 
platforms.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Xavier de Gaye]

Change by Xavier de Gaye :


--
keywords: +patch
pull_requests: +16821
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/17337

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Xavier de Gaye]

Xavier de Gaye  added the comment:

The crash occurs only on debug builds.

See the FreeBSD related issue #37906.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38852] test_recursion_limit in test_threading crashes with SIGSEGV on android

[Xavier de Gaye]

New submission from Xavier de Gaye :

Actually it is the script that is spawned by test_recursion_limit that crashes 
with SIGSEGV on android API 24. Lowering the recursion limit in the script from 
1000 to 100 with sys.setrecursionlimit() fixes the problem.

Here is the error:

generic_x86_64:/data/local/tmp/python $ python -m test -v test_threading -m 
test_recursion_limit   
== CPython 3.9.0a0 (heads/abifa-dirty:cf805c25e6, Nov 18 2019, 16:40:26) [Clang 
8.0.2 (https://android.googlesource.com/toolchain/clang 40173bab62ec7462
== Linux-3.10.0+-x86_64-with-libc little-endian
== cwd: /data/local/tmp/python/tmp/test_python_4603
== CPU count: 2
== encodings: locale=UTF-8, FS=utf-8
0:00:00 Run tests sequentially
0:00:00 [1/1] test_threading
test_recursion_limit (test.test_threading.ThreadingExceptionTests) ... FAIL

==
FAIL: test_recursion_limit (test.test_threading.ThreadingExceptionTests)
--
Traceback (most recent call last):
  File "/data/local/tmp/python/lib/python3.9/test/test_threading.py", line 
1086, in test_recursion_limit
self.assertEqual(p.returncode, 0, "Unexpected error: " + stderr.decode())
AssertionError: -11 != 0 : Unexpected error: 

--

Ran 1 test in 0.148s

FAILED (failures=1)
test test_threading failed
test_threading failed

== Tests result: FAILURE ==

1 test failed:
test_threading

Total duration: 354 ms
Tests result: FAILURE

--
components: Tests
messages: 356988
nosy: xdegaye
priority: normal
severity: normal
stage: needs patch
status: open
title: test_recursion_limit in test_threading crashes with SIGSEGV on android
type: crash
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com