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

common/TracepointProvider: add assert if dlopen error. #13430

Merged
merged 1 commit into from Feb 18, 2017

Conversation

Projects
None yet
3 participants
@majianpeng
Member

majianpeng commented Feb 15, 2017

Because some reasons, if dlopen error the ceph-osd still work. But we
don't see related tracepoint by 'lttng list -u'. So add this assert to
make easily find the problem.

Signed-off-by: Jianpeng Ma jianpeng.ma@intel.com

@majianpeng

This comment has been minimized.

Member

majianpeng commented Feb 15, 2017

Because the path of lib, when I enable eventtrace but I don't see any tracepoint by lttng list. By review the code I find dlopen error but ceph-osd don't display usefull message. So I add assert to avoid this problem.

common/TracepointProvider: add assert if dlopen error.
Because some reasons, if dlopen error the ceph-osd still work. But we
don't see related tracepoint by 'lttng list -u'. So add this assert to
make easily find the problem.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

@tchaikov tchaikov merged commit f0da8aa into ceph:master Feb 18, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@majianpeng majianpeng deleted the majianpeng:tracepointprovier branch Mar 1, 2017

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