Skip to content

common/ceph_context: leak some memory fail to show in valgrind#47802

Merged
tchaikov merged 1 commit intoceph:mainfrom
NitzanMordhai:wip-nitzan-leak-some-memory-with-memory-optimization
Aug 31, 2022
Merged

common/ceph_context: leak some memory fail to show in valgrind#47802
tchaikov merged 1 commit intoceph:mainfrom
NitzanMordhai:wip-nitzan-leak-some-memory-with-memory-optimization

Conversation

@NitzanMordhai
Copy link
Contributor

@NitzanMordhai NitzanMordhai commented Aug 25, 2022

GCC 20 memory optimization broke the memory leak for testing.

Fixes:https://tracker.ceph.com/issues/57165
Signed-off-by: Nitzan Mordechai nmordec@redhat.com

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@NitzanMordhai NitzanMordhai force-pushed the wip-nitzan-leak-some-memory-with-memory-optimization branch from ace0f25 to 325907c Compare August 25, 2022 14:31
@NitzanMordhai NitzanMordhai requested a review from tchaikov August 25, 2022 14:31
Copy link
Member

@ljflores ljflores left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the concept; +1 to the change @tchaikov requested

@NitzanMordhai NitzanMordhai force-pushed the wip-nitzan-leak-some-memory-with-memory-optimization branch from 325907c to 0664d29 Compare August 25, 2022 15:38
GCC 20 memory optimization broke the memory leak for testing.

Fixes:https://tracker.ceph.com/issues/57165
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
@NitzanMordhai NitzanMordhai force-pushed the wip-nitzan-leak-some-memory-with-memory-optimization branch from 0664d29 to fc21669 Compare August 25, 2022 15:40
@tchaikov
Copy link
Contributor

jenkins test make check

@NitzanMordhai
Copy link
Contributor Author

jenkins test make check

@rzarzynski
Copy link
Contributor

The following tests FAILED:
	 33 - run-rbd-unit-tests-109.sh (Failed)
	 34 - run-rbd-unit-tests-127.sh (Failed)

@rzarzynski
Copy link
Contributor

jenkins test make check

@tchaikov
Copy link
Contributor

@tchaikov
Copy link
Contributor

jenkins test make check

3 similar comments
@tchaikov
Copy link
Contributor

jenkins test make check

@tchaikov
Copy link
Contributor

jenkins test make check

@tchaikov
Copy link
Contributor

jenkins test make check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants