Merge pull request ceph#65181 from qiuxinyidian/lttng

SrinivasaBharath · web-flow · commit b45f2fb83d2f · 2025-09-14T16:19:14.000+05:30
rados/lttng: add lttng open *tp.so error message
diff --git a/src/common/TracepointProvider.cc b/src/common/TracepointProvider.cc
@@ -3,6 +3,7 @@
 
 #include "common/TracepointProvider.h"
 #include "common/config.h"
+#include "common/dout.h"
 
 TracepointProvider::TracepointProvider(CephContext *cct, const char *library,
                                        const char *config_key)
@@ -40,6 +41,10 @@ void TracepointProvider::verify_config(const ConfigProxy& conf) {
   }
 
   m_handle = dlopen(m_library.c_str(), RTLD_NOW | RTLD_NODELETE);
+  if (!m_handle) {
+    lderr(m_cct) << __func__ << " failed dlopen(): "<< m_library.c_str() <<", "
+    << dlerror()  << dendl;
+   } 
   ceph_assert(m_handle);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`
`4`	`4`	`#include "common/TracepointProvider.h"`
`5`	`5`	`#include "common/config.h"`
	`6`	`+#include "common/dout.h"`
`6`	`7`
`7`	`8`	`TracepointProvider::TracepointProvider(CephContext cct, const char library,`
`8`	`9`	`const char *config_key)`
`@@ -40,6 +41,10 @@ void TracepointProvider::verify_config(const ConfigProxy& conf) {`
`40`	`41`	`}`
`41`	`42`
`42`	`43`	`m_handle = dlopen(m_library.c_str(), RTLD_NOW \| RTLD_NODELETE);`
	`44`	`+ if (!m_handle) {`
	`45`	`+ lderr(m_cct) << __func__ << " failed dlopen(): "<< m_library.c_str() <<", "`
	`46`	`+ << dlerror() << dendl;`
	`47`	`+ }`
`43`	`48`	`ceph_assert(m_handle);`
`44`	`49`	`}`
`45`	`50`