Skip to content

Conversation

@devin-ai-integration
Copy link

Add jemalloc heap profiling to benchmark workflow

This PR adds support for jemalloc heap profiling to the benchmark workflow, allowing memory usage analysis of the benchmark process.

Changes

  • Add 'heap' option to profiling choices
  • Enable jemalloc profiling feature when heap profiling is selected
  • Configure jemalloc for heap profiling via JEMALLOC_SYS_WITH_MALLOC_CONF
  • Add step to generate and upload heap profiles to S3
  • Include heap profiles in markdown report and step summary

Testing

The changes build on the existing jemalloc integration and follow the same pattern as the existing host/guest profiling options.

Link to Devin run: https://app.devin.ai/sessions/f4dff02e30bd4f388c69529174c98291

- Add 'heap' option to profiling choices
- Enable jemalloc profiling feature when heap profiling is selected
- Configure jemalloc for heap profiling via JEMALLOC_SYS_WITH_MALLOC_CONF
- Add step to generate and upload heap profiles to S3
- Include heap profiles in markdown report and step summary

Co-Authored-By: Jonathan Wang <jpw@intrinsictech.xyz>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

devin-ai-integration bot and others added 2 commits February 21, 2025 00:58
Co-Authored-By: Jonathan Wang <jpw@intrinsictech.xyz>
Co-Authored-By: Jonathan Wang <jpw@intrinsictech.xyz>
@devin-ai-integration
Copy link
Author

Closing due to inactivity.

@jonathanpwang jonathanpwang deleted the devin/1740098503-add-jemalloc-profiling branch March 6, 2025 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant