Skip to content

test_basic_multiple_interpreters_reset_each: _PyRefchain_Remove: Assertion `value == REFCHAIN_VALUE' failed #121110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
furkanonder opened this issue Jun 27, 2024 · 2 comments
Closed

test_basic_multiple_interpreters_reset_each: _PyRefchain_Remove: Assertion `value == REFCHAIN_VALUE' failed #121110

furkanonder opened this issue Jun 27, 2024 · 2 comments
Labels
3.13 bugs and security fixes 3.14 new features, bugs and security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) topic-subinterpreters type-crash A hard crash of the interpreter, possibly with a core dump

Comments

@furkanonder
Copy link
Contributor

furkanonder commented Jun 27, 2024 •
edited by bedevere-app bot
Loading

Crash report

What happened?

Configuration

./configure  --with-trace-refs --with-pydebug

Test:

./python -m test -v -m test_basic_multiple_interpreters_reset_each test_import

Output:

== CPython 3.14.0a0 (heads/main:1a2e7a7475, Jun 28 2024, 02:37:08) [GCC 14.1.1 20240522]
== Linux-6.9.6-arch1-1-x86_64-with-glibc2.39 little-endian
== Python build: debug TraceRefs
== cwd: /home/arf/cpython/build/test_python_worker_246671æ
== CPU count: 16
== encodings: locale=UTF-8 FS=utf-8
== resources: all test resources are disabled, use -u option to unskip tests

Using random seed: 4201083195
0:00:00 load avg: 2.50 Run 1 test sequentially in a single process
0:00:00 load avg: 2.50 [1/1] test_import
test_basic_multiple_interpreters_reset_each (test.test_import.SinglephaseInitTests.test_basic_multiple_interpreters_reset_each) ... python: Objects/object.c:195: _PyRefchain_Remove: Assertion `value == REFCHAIN_VALUE' failed.
Fatal Python error: Aborted

Current thread 0x00007cde0f816740 (most recent call first):
  File "<string>", line 54 in <module>

Extension modules: _testinternalcapi, _testmultiphase, _testcapi (total: 3)
[1]    246671 IOT instruction (core dumped)  ./python -m test -v -m test_basic_multiple_interpreters_reset_each test_impor

CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Output from running 'python -VV' on the command line:

Python 3.14.0a0 (heads/main:1a2e7a7475, Jun 28 2024, 02:37:08) [GCC 14.1.1 20240522]

Linked PRs
@furkanonder furkanonder added tests Tests in the Lib/test dir type-crash A hard crash of the interpreter, possibly with a core dump labels Jun 27, 2024
@furkanonder
Copy link
Contributor Author

Bisected to b2cd54a
cc @ericsnowcurrently

@ericsnowcurrently ericsnowcurrently mentioned this issue Jul 1, 2024
Closed
@ericsnowcurrently
Copy link
Member

ericsnowcurrently commented Jul 1, 2024 •
edited
Loading

FYI, I plan on taking care of this, but probably won't be available to do so until the end of the week at the soonest. In the meantime I'm disabling the test.

@bedevere-app bedevere-app bot mentioned this issue Jul 1, 2024
Merged
@ericsnowcurrently ericsnowcurrently added 3.13 bugs and security fixes 3.14 new features, bugs and security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) topic-subinterpreters and removed tests Tests in the Lib/test dir labels Jul 1, 2024
ericsnowcurrently added a commit that referenced this issue Jul 1, 2024
@ericsnowcurrently
gh-121110: Temporarily Skip test_basic_multiple_interpreters_reset_ea…
9bcb7d8 
…ch (gh-121236)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 1, 2024
@ericsnowcurrently @miss-islington
pythongh-121110: Temporarily Skip test_basic_multiple_interpreters_re…
1af2509 
…set_each (pythongh-121236)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.
(cherry picked from commit 9bcb7d8)

Co-authored-by: Eric Snow <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Jul 1, 2024
Merged
ericsnowcurrently added a commit that referenced this issue Jul 1, 2024
@miss-islington @ericsnowcurrently
[3.13] gh-121110: Temporarily Skip test_basic_multiple_interpreters_r…
fc0b1cb 
…eset_each (gh-121238)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.

(cherry picked from commit 9bcb7d8, AKA gh-121236)

Co-authored-by: Eric Snow <[email protected]>
Akasurde pushed a commit to Akasurde/cpython that referenced this issue Jul 3, 2024
@ericsnowcurrently @Akasurde
pythongh-121110: Temporarily Skip test_basic_multiple_interpreters_re…
954a1c1 
…set_each (pythongh-121236)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.
@bedevere-app bedevere-app bot mentioned this issue Jul 8, 2024
Merged
ericsnowcurrently added a commit that referenced this issue Jul 8, 2024
@ericsnowcurrently
gh-121110: Fix Extension Module Tests Under Py_TRACE_REFS Builds (gh-…
15d48ae 
…121503)

The change in gh-118157 (b2cd54a) should have also updated clear_singlephase_extension() but didn't.  We fix that here.  Note that clear_singlephase_extension() (AKA _PyImport_ClearExtension()) is only used in tests.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jul 8, 2024
@ericsnowcurrently @miss-islington
pythongh-121110: Fix Extension Module Tests Under Py_TRACE_REFS Builds (
b52e053 
pythongh-121503)

The change in pythongh-118157 (b2cd54a) should have also updated clear_singlephase_extension() but didn't.  We fix that here.  Note that clear_singlephase_extension() (AKA _PyImport_ClearExtension()) is only used in tests.
(cherry picked from commit 15d48ae)

Co-authored-by: Eric Snow <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Jul 8, 2024
Merged
@github-project-automation github-project-automation bot moved this from Todo to Done in Subinterpreters Jul 8, 2024
ericsnowcurrently added a commit that referenced this issue Jul 8, 2024
@miss-islington @ericsnowcurrently
[3.13] gh-121110: Fix Extension Module Tests Under Py_TRACE_REFS Buil…
c1891e2 
…ds (gh-121517)

The change in gh-118157 (b2cd54a) should have also updated clear_singlephase_extension() but didn't.  We fix that here.  Note that clear_singlephase_extension() (AKA _PyImport_ClearExtension()) is only used in tests.

(cherry picked from commit 15d48ae, AKA gh-121503)

Co-authored-by: Eric Snow <[email protected]>
noahbkim pushed a commit to hudson-trading/cpython that referenced this issue Jul 11, 2024
@ericsnowcurrently @noahbkim
pythongh-121110: Temporarily Skip test_basic_multiple_interpreters_re…
ae608af 
…set_each (pythongh-121236)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.
noahbkim pushed a commit to hudson-trading/cpython that referenced this issue Jul 11, 2024
@ericsnowcurrently @noahbkim
pythongh-121110: Fix Extension Module Tests Under Py_TRACE_REFS Builds (
ba47add 
pythongh-121503)

The change in pythongh-118157 (b2cd54a) should have also updated clear_singlephase_extension() but didn't.  We fix that here.  Note that clear_singlephase_extension() (AKA _PyImport_ClearExtension()) is only used in tests.
estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
@ericsnowcurrently @estyxx
pythongh-121110: Temporarily Skip test_basic_multiple_interpreters_re…
1276b40 
…set_each (pythongh-121236)

This will allow Py_TRACE_REFS builds to pass the test suite, until the underlying issue can be resolved.
estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
@ericsnowcurrently @estyxx
pythongh-121110: Fix Extension Module Tests Under Py_TRACE_REFS Builds (
2b6968f 
pythongh-121503)

The change in pythongh-118157 (b2cd54a) should have also updated clear_singlephase_extension() but didn't.  We fix that here.  Note that clear_singlephase_extension() (AKA _PyImport_ClearExtension()) is only used in tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.13 bugs and security fixes 3.14 new features, bugs and security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs) topic-subinterpreters type-crash A hard crash of the interpreter, possibly with a core dump
Projects
Subinterpreters
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ericsnowcurrently @furkanonder