Skip to content

PEP 703 Deferred reference counts #121459

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
Fidget-Spinner opened this issue Jul 7, 2024 · 3 comments
Closed

PEP 703 Deferred reference counts #121459

Fidget-Spinner opened this issue Jul 7, 2024 · 3 comments
Labels
topic-free-threading type-feature A feature request or enhancement

Comments

@Fidget-Spinner
Copy link
Member

Fidget-Spinner commented Jul 7, 2024 •
edited by bedevere-app bot
Loading

Feature or enhancement

Proposal:

PEP 703 proposes deferred reference counting to reduce contention between threads on shared objects' reference count fields.

This issue is not in conflict with Faster CPython's scheme of fully removing reference counts in the eval loop. In fact, some of the work runs parallel to it and will actually help it.

There are 3 wins I can observe right now:

  • deferring LOAD_GLOBAL
  • deferring LOAD_ATTR
  • deferring frames' strong references on the code object and function (this is the hardest of the three)

I shall proceed in this order.

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs
@Fidget-Spinner Fidget-Spinner added type-feature A feature request or enhancement topic-free-threading labels Jul 7, 2024
@bedevere-app bedevere-app bot mentioned this issue Aug 18, 2024
Merged
@markshannon markshannon mentioned this issue Sep 12, 2024
Closed
Fidget-Spinner added a commit that referenced this issue Sep 13, 2024
@Fidget-Spinner @picnixz @colesbury
gh-121459: Deferred LOAD_GLOBAL (GH-123128)
8810e28 
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Sam Gross <655866+colesbury@users.noreply.github.com>
@bedevere-app bedevere-app bot mentioned this issue Sep 14, 2024
Merged
@bedevere-app bedevere-app bot mentioned this issue Sep 15, 2024
Closed
@bedevere-app bedevere-app bot mentioned this issue Oct 2, 2024
Merged
@bedevere-app bedevere-app bot mentioned this issue Oct 18, 2024
Merged
@Fidget-Spinner
Copy link
Member Author

This is implemented now.

@picnixz
Copy link
Member

picnixz commented Dec 14, 2024

What about #124101? should this one be considered as optional?

@Fidget-Spinner
Copy link
Member Author

@picnixz I'm waiting for the porting over the specializing intepreter to free threaded build to play out. If that works for LOAD_ATTR we wont need the PR anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
topic-free-threading type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@picnixz @Fidget-Spinner