Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-129223: Do not allow the compiler to optimise away symbols for debug sections #129225

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.

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

Merged
merged 4 commits into from
Jan 24, 2025

Conversation

pablogsal
Copy link
Member

@pablogsal pablogsal commented Jan 23, 2025

@@ -414,7 +414,11 @@ get_py_runtime(pid_t pid)
static uintptr_t
get_async_debug(pid_t pid)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is to avoid returning NULL without an error set as things under search_map_for_section may return without setting an error.

@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@python python deleted a comment from bedevere-bot Jan 23, 2025
@pablogsal pablogsal added 🔨 test-with-buildbots Test PR w/ buildbots; report in status section and removed 🔨 test-with-buildbots Test PR w/ buildbots; report in status section labels Jan 23, 2025
@pablogsal pablogsal force-pushed the gh-129223 branch 3 times, most recently from 3bfb5f7 to 170075a Compare January 23, 2025 14:42
@pablogsal pablogsal added 🔨 test-with-buildbots Test PR w/ buildbots; report in status section and removed 🔨 test-with-buildbots Test PR w/ buildbots; report in status section labels Jan 23, 2025
@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@python python deleted a comment from bedevere-bot Jan 23, 2025
@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@python python deleted a comment from bedevere-bot Jan 23, 2025
@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@JacobCoffee JacobCoffee added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @JacobCoffee for commit bd41fda 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F30617%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@ambv ambv added 🔨 test-with-buildbots Test PR w/ buildbots; report in status section and removed 🔨 test-with-buildbots Test PR w/ buildbots; report in status section labels Jan 23, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ambv for commit bd41fda 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F30617%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 23, 2025
@ambv
Copy link
Contributor

ambv commented Jan 23, 2025

!buildbot aarch64 Fedora Stable Clang

@python python deleted a comment from bedevere-bot Jan 23, 2025
@python python deleted a comment from bedevere-bot Jan 23, 2025
@python python deleted a comment from bedevere-bot Jan 24, 2025
@python python deleted a comment from ambv Jan 24, 2025
@python python deleted a comment from bedevere-bot Jan 24, 2025
@pablogsal pablogsal added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 24, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @pablogsal for commit 596a58f 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F30617%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jan 24, 2025
@ambv
Copy link
Contributor

ambv commented Jan 24, 2025

The test_interpreters.test_create_many_threaded failure looks unrelated to me, it's a failure to import encodings in one of the sub-interpreters on a thread:
https://buildbot.python.org/#/builders/1255/builds/370/steps/6/logs/stdio

@ambv
Copy link
Contributor

ambv commented Jan 24, 2025

!buildbot PPC64LE .* Refleaks PR

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ambv for commit 596a58f 🤖

The command will test the builders whose names match following regular expression: PPC64LE .* Refleaks PR

The builders matched are:

  • PPC64LE Fedora Rawhide Refleaks PR
  • PPC64LE CentOS9 Refleaks PR
  • PPC64LE Fedora Stable Refleaks PR
  • PPC64LE RHEL8 Refleaks PR
  • PPC64LE Fedora Rawhide NoGIL refleaks PR

@ambv
Copy link
Contributor

ambv commented Jan 24, 2025

It's weird that Windows 11 and PPC64LE refleaks buildbots are consistently failing with failures that don't look related to external inspection infrastructure:

8 tests failed:
    test_capi test_embed test_import test_interpreters test_json
    test_logging test_sysconfig test_zoneinfo

Refleaks on all other platforms, including s390x, are clean. At the moment we're treating this as unrelated and we'll be looking at a solution to those failures separately.

@ambv ambv merged commit 3a3a6b8 into python:main Jan 24, 2025
113 of 120 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants