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

feat(test): Track memory leak (or) memory corruption issues #334

Open
mittachaitu opened this issue May 21, 2020 · 1 comment
Open

feat(test): Track memory leak (or) memory corruption issues #334

mittachaitu opened this issue May 21, 2020 · 1 comment
Labels
help wanted Extra attention is needed

Comments

@mittachaitu
Copy link

What would you like to be added:
It will help to fix memory issues/corruption in cstor-istgt.

Why is this needed:
Add a test case to track memory leak (or) corruption using Valgrind tool

Describe the solution you'd like:
[A clear and concise description of what you want to happen]

Environment:

  • OpenEBS version:
  • Kubernetes version (use kubectl version):
  • Kubernetes installer & version:
  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
@kmova kmova added this to Pre-commits and Designs - Due: May 31 2020 in 1.11 Release Tracker - Due June 15th. Jun 3, 2020
@kmova kmova moved this from Pre-commits and Designs - Due: May 31 2020 to To do in 1.11 Release Tracker - Due June 15th. Jun 3, 2020
@mittachaitu mittachaitu moved this from To do to Pre-commits and Designs - Due: May 31 2020 in 1.11 Release Tracker - Due June 15th. Jun 16, 2020
@mittachaitu mittachaitu moved this from Pre-commits and Designs - Due: May 31 2020 to To do in 1.11 Release Tracker - Due June 15th. Jun 18, 2020
@mittachaitu mittachaitu added this to Pre-commits and Designs - Due: Jun 30 2020 in 1.12 Release Tracker - Due July 15th. Jun 18, 2020
@mittachaitu mittachaitu moved this from Pre-commits and Designs - Due: Jun 30 2020 to RC1 - Due: Jul 5 2020 in 1.12 Release Tracker - Due July 15th. Jul 6, 2020
@kmova kmova moved this from RC1 - Due: Jul 5 2020 to RC2 - Due Jul 10, 2020 in 1.12 Release Tracker - Due July 15th. Jul 7, 2020
@kmova kmova moved this from RC2 - Due Jul 10, 2020 to Pushed to Next release due to WIP in 1.12 Release Tracker - Due July 15th. Jul 7, 2020
@mittachaitu
Copy link
Author

Explored Valgrind tool to detect memory leak/corruption in istgt

  • It is able to give stats about the memory only once the istgt process is killed/died.
  • It doesn't support attaching the currently running process to it(By its design).
  • If will kill the Valgrind process then it will not give any stats about memory usage.
    Ex: We need to launch istgt process as Valgrind ./istgt to get memory stats but if we kill that process then it is not giving any stats about the memory(since Valgrind itself dies so no one is there to print output).
    ============
    Basically, to overcome the above issue we need to have istgtcontrol command to kill istgt process alone and need to check whether Valgrind is able to give stats.

@mittachaitu mittachaitu added this to Pre-commits and Designs - Due: Aug 31 2020 in 2.1 Release Tracker - Released Sep 15th. Aug 19, 2020
@sonasingh46 sonasingh46 moved this from Pre-commits and Designs - Due: Aug 31 2020 to RC2 - Due: Sep 10 2020 in 2.1 Release Tracker - Released Sep 15th. Sep 1, 2020
@sonasingh46 sonasingh46 moved this from RC2 - Due: Sep 10 2020 to Release Items in 2.1 Release Tracker - Released Sep 15th. Sep 9, 2020
@sonasingh46 sonasingh46 moved this from Release Items to RC1 - Due: Sep 5 2020 in 2.1 Release Tracker - Released Sep 15th. Sep 14, 2020
@sonasingh46 sonasingh46 moved this from RC1 - Due: Sep 5 2020 to To do in 2.1 Release Tracker - Released Sep 15th. Sep 14, 2020
@mittachaitu mittachaitu added the help wanted Extra attention is needed label Sep 17, 2020
@avishnu avishnu removed this from Pushed to Next release due to WIP in 1.12 Release Tracker - Due July 15th. Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant