GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — DO NOT MERGE: Always use -Xuops by gvanrossum · Pull Request #107820 · python/cpython

Conversation

@gvanrossum

@gvanrossum gvanrossum commented Aug 9, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

This exists so we can occasionally assess whether uops work everywhere, and run benchmarks. I will occasionally rebase (not merge!) to the latest main.

Failure of tests bedevere/issue-number and bedever/news, as well as Check labels / DO-NOT-MERGE / unresolved review, is expected and intentional (so I don't accidentally merge this). It is expected to be slower than main.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@gvanrossum gvanrossum added the DO-NOT-MERGE label Aug 9, 2023
@gvanrossum
gvanrossum force-pushed the uops-forever branch 2 times, most recently from 2771e69 to 3c00eae Compare August 21, 2023 21:35
@gvanrossum gvanrossum mentioned this pull request Aug 21, 2023
Decoding instructions should handle ENTER_EXECUTOR #107265
Closed
8 of 10 tasks
@gvanrossum
gvanrossum force-pushed the uops-forever branch 4 times, most recently from 074ed54 to 929ca3e Compare August 26, 2023 15:47
@gvanrossum
gvanrossum force-pushed the uops-forever branch 3 times, most recently from 0b6e4cb to 75c669a Compare September 1, 2023 15:44
@gvanrossum gvanrossum mentioned this pull request Sep 1, 2023
Fix test_regrtest when run with uops always on #108778
Merged
@gvanrossum
gvanrossum force-pushed the uops-forever branch 3 times, most recently from ad83bdb to 825ff87 Compare September 8, 2023 03:56
@gvanrossum
gvanrossum force-pushed the uops-forever branch from 825ff87 to 932fee0 Compare September 14, 2023 00:27
@gvanrossum
gvanrossum force-pushed the uops-forever branch 4 times, most recently from 27d7b8d to a01cff1 Compare October 3, 2023 23:07
@gvanrossum
gvanrossum force-pushed the uops-forever branch from a01cff1 to 0cc6bca Compare October 9, 2023 07:42
@gvanrossum

gvanrossum commented Oct 9, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

There seems to be a real problem here. I can repro this on the main branch.

./python.exe -Xuops -m test test_asyncio.test_ssl

Gives lots of warnings about dangling threads, extra weakrefs, and eventually "1 test altered the execution environment (env changed)". Without -Xuops it passes silently.

Brandt found the root cause, and fixed it in GH-110593.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@gvanrossum
gvanrossum force-pushed the uops-forever branch from 0cc6bca to ab9b54e Compare October 30, 2023 18:11
@gvanrossum
DO NOT MERGE: Always use -Xuops
03100e5
@gvanrossum
gvanrossum force-pushed the uops-forever branch from ab9b54e to 03100e5 Compare January 12, 2024 19:17
@gvanrossum gvanrossum closed this May 1, 2024
@gvanrossum
gvanrossum deleted the uops-forever branch May 1, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently will be requested when the pull request is marked ready for review ericsnowcurrently is a code owner
@ZeroIntensity ZeroIntensity Awaiting requested review from ZeroIntensity ZeroIntensity will be requested when the pull request is marked ready for review ZeroIntensity is a code owner
@FFY00 FFY00 Awaiting requested review from FFY00 FFY00 will be requested when the pull request is marked ready for review FFY00 is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

DO-NOT-MERGE

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@gvanrossum
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.