[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: I added PyInterpreterState.runtime in bpo-36710: commit 01b1cc12e7c6a3d6a3d27ba7c731687d57aae92a Author: Victor Stinner Date: Wed Nov 20 02:27:56 2019 +0100 bpo-36710: Add PyInterpreterState.runtime field (GH-17270) Add

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: See https://bugs.python.org/issue37135#msg344509 to reproduce https://bugs.python.org/issue37135 crash without multiprocessing: with this procedure, I reproduce the crash immediately on Linux. No need for a magic super slow buildbot. --

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: By the way, I have questions about the rationale of this change. > Currently we use the _PyRuntime static global to access the runtime state in > various places. At the same time, in thread contexts we get access to the > thread state from Thread-Local

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: This change introduced a regression: https://bugs.python.org/issue37135#msg344511 I had to revert the change to fix Python for 3.8 beta1 release. -- resolution: fixed -> status: closed -> open ___ Python tracker

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: New changeset 0fd2c300c2a85b3b227a907b2a39ef79fa86d850 by Victor Stinner in branch 'master': Revert "bpo-36818: Add PyInterpreterState.runtime field. (gh-13129)" (GH-13795) https://github.com/python/cpython/commit/0fd2c300c2a85b3b227a907b2a39ef79fa86d850

[issue36818] Add PyInterpreterState.runtime.

Change by STINNER Victor : -- pull_requests: +13680 pull_request: https://github.com/python/cpython/pull/13795 ___ Python tracker ___

[issue36818] Add PyInterpreterState.runtime.

Change by Eric Snow : -- resolution: rejected -> fixed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue36818] Add PyInterpreterState.runtime.

Eric Snow added the comment: New changeset 396e0a8d9dc65453cb9d53500d0a620602656cfe by Eric Snow in branch 'master': bpo-36818: Add PyInterpreterState.runtime field. (gh-13129) https://github.com/python/cpython/commit/396e0a8d9dc65453cb9d53500d0a620602656cfe --

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: Eric is working on a different approach: https://bugs.python.org/issue36854 -- resolution: -> rejected stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue36818] Add PyInterpreterState.runtime.

STINNER Victor added the comment: I don't see the point of PyInterpreterState.runtime: * I understood that we should have and only one _PyRuntimeState instance: _PyRuntime. Why not accessing it directly? * I understood that most _PyRuntimeState fields should be moved to PyInterpreterState.

[issue36818] Add PyInterpreterState.runtime.

New submission from Eric Snow : Currently we use the _PyRuntime static global to access the runtime state in various places. At the same time, in thread contexts we get access to the thread state from Thread-Local Storage and the interpreter state by indirection from there. We should do

[issue36818] Add PyInterpreterState.runtime.

Change by Eric Snow : -- keywords: +patch pull_requests: +13042 ___ Python tracker ___ ___ Python-bugs-list mailing list