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.

Sign up for GitHub

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

Jump to bottom

In gfapi fix memory leak during graph switch #61

Closed
rajeshjoseph opened this issue Dec 9, 2016 · 4 comments
Closed

In gfapi fix memory leak during graph switch #61

rajeshjoseph opened this issue Dec 9, 2016 · 4 comments
Assignees
@nixpanic @rajeshjoseph
Labels
CB: gfapi Prio: High Type:Bug
Milestone
Release 3.11 (STM)

Comments

@rajeshjoseph
Copy link

During graph switch we do not perform much cleanup operation on the old graph, leading to memory leak.

  • Inode table of old graph needs cleanup.
    Fix inode leaks
    Fix forget of each xl to free inode ctx properly
  • The xl objects itself (xlator_t)
  • The mem_accnt structure in every xl object.
    Fix all the leaks so that the ref count of mem_accnt structure is 0
  • Implement fini() in every xlator
  • Some of the above items cannot be fully completed, e.g. fini in every xlator because it needs effort from all the xlator owners.
@rajeshjoseph rajeshjoseph added this to the Release 3.10 (LTM) milestone Dec 9, 2016
@rajeshjoseph rajeshjoseph self-assigned this Dec 9, 2016
@rajeshjoseph
Copy link
Author

https://bugzilla.redhat.com/show_bug.cgi?id=1403156

@ShyamsundarR
Copy link
Contributor

Marking this for 3.11 as this is not making into the 3.10 release

@ShyamsundarR ShyamsundarR added this to Release 3.11 in Releases Jan 26, 2017
@nixpanic nixpanic self-assigned this Mar 7, 2017
@gluster-ant
Copy link
Collaborator

A patch https://review.gluster.org/17390 has been posted that references this issue.
Commit message: doc: Updated release notes for several features/changes in 3.11.0

mscherer pushed a commit that referenced this issue May 26, 2017
@ShyamsundarR
doc: Updated release notes for several features/changes in 3.11.0
3efadd8 
Updates #61, Updates #156
Fixes #166, Fixes #167

Change-Id: I031bf944493b959d44c97fb0ddf7c1b80e53bdda
Signed-off-by: Shyam <srangana@redhat.com>
Reviewed-on: https://review.gluster.org/17390
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
@prashanthpai
Copy link
Contributor

Inadvertently closed (and published in release notes ) ?

References:
https://bugzilla.redhat.com/show_bug.cgi?id=1403156
https://bugzilla.redhat.com/show_bug.cgi?id=1397406

@gluster-ant gluster-ant mentioned this issue Mar 12, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nixpanic nixpanic

@rajeshjoseph rajeshjoseph

Labels
CB: gfapi Prio: High Type:Bug
Projects
None yet
Milestone
Release 3.11 (STM)
Development

No branches or pull requests
5 participants
@nixpanic @prashanthpai @ShyamsundarR @rajeshjoseph @gluster-ant