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

mimic: mds: destroy reconnect msg when it is from non-existent session to avoid memory leak #28796

Merged
merged 1 commit into from Jul 17, 2019

Conversation

@shenhang
Copy link
Contributor

shenhang commented Jun 30, 2019

Destroy reconnect msg when it is from non-existent session to avoid memory leak
fixes: http://tracker.ceph.com/issues/40588
Signed-off-by: Shen Hang harryshen18@gmail.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
fixes: http://tracker.ceph.com/issues/40588

This bug is from backporting "#27256"
On version mimic and luminous, the type of msg determines msg
has to be destroyed explictly.
Signed-off-by: Shen Hang <harryshen18@gmail.com>
@shenhang shenhang force-pushed the shenhang:mimic branch from 20b8775 to 43ffd5b Jun 30, 2019
@smithfarm smithfarm changed the title mimic:mds: destroy reconnect msg when it is from non-existent session to avoid memory leak mimic: mds: destroy reconnect msg when it is from non-existent session to avoid memory leak Jun 30, 2019
@smithfarm smithfarm requested a review from batrick Jun 30, 2019
@smithfarm smithfarm added this to the mimic milestone Jun 30, 2019
@smithfarm smithfarm requested a review from ukernel Jun 30, 2019
@yuriw

This comment has been minimized.

Copy link
Contributor

yuriw commented Jul 9, 2019

@yuriw yuriw merged commit 132589e into ceph:mimic Jul 17, 2019
4 checks passed
4 checks passed
Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.