Skip to content
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

WIP: a PoC implementation of a memory profiler #294

Closed
wants to merge 1 commit into from
Closed

Conversation

sj-i
Copy link
Member

@sj-i sj-i commented Oct 7, 2023

  • Absolutely WIP, but can be used to show the basic ideas.
  • Only supports 8.2 NTS target.
  • Unknown type of TMP/VAR has been read on my test. So currently disabling retrieval of TMP/VARs, only reads CVs.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 6442553318

  • 117 of 576 (20.31%) changed or added relevant lines in 23 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-11.7%) to 67.908%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Lib/PhpInternals/Types/Zend/ZendFunction.php 0 1 0.0%
src/Lib/PhpInternals/Types/Zend/ZendValue.php 17 22 77.27%
src/Lib/PhpProcessReader/PhpGlobalsFinder.php 0 5 0.0%
src/Lib/PhpInternals/Types/Zend/ZendType.php 0 6 0.0%
src/Lib/Process/Pointer/Pointer.php 4 10 40.0%
src/Lib/PhpInternals/Types/Zend/Zval.php 3 15 20.0%
src/Lib/PhpInternals/Types/Zend/ZendPropertyInfo.php 0 17 0.0%
src/Lib/PhpInternals/Types/Zend/ZvalU1.php 0 17 0.0%
src/Lib/PhpInternals/Types/Zend/ZendExecutorGlobals.php 20 39 51.28%
src/Lib/PhpProcessReader/PhpZendMemoryManagerChunkFinder.php 0 21 0.0%
Files with Coverage Reduction New Missed Lines %
src/Lib/PhpInternals/Types/Zend/ZendExecuteData.php 1 45.21%
src/Lib/Process/Pointer/Pointer.php 1 73.08%
Totals Coverage Status
Change from base Build 6442466159: -11.7%
Covered Lines: 2006
Relevant Lines: 2954

💛 - Coveralls

@sj-i
Copy link
Member Author

sj-i commented Nov 12, 2023

superseded by #308

@sj-i sj-i closed this Nov 12, 2023
@sj-i sj-i deleted the memory-profiler branch November 21, 2023 23:56
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.

2 participants