X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=lttng-statedump-impl.c;h=864637dd457fc1321c9ca09fe23764704e55eb22;hb=72814741fb45c8f7d4c45cb2a4624e20b5a4a76a;hp=dad51ddaa25013f1bd5453aca4352854bd1ec6eb;hpb=d561ecfbc9a17ca47721cd6d7742d2b1835f8ae3;p=lttng-modules.git diff --git a/lttng-statedump-impl.c b/lttng-statedump-impl.c index dad51dda..864637dd 100644 --- a/lttng-statedump-impl.c +++ b/lttng-statedump-impl.c @@ -65,6 +65,7 @@ #define CREATE_TRACE_POINTS #define TRACE_INCLUDE_PATH ../instrumentation/events/lttng-module #define TRACE_INCLUDE_FILE lttng-statedump +#define LTTNG_INSTRUMENTATION #include "instrumentation/events/lttng-module/lttng-statedump.h" DEFINE_TRACE(lttng_statedump_block_device); @@ -378,6 +379,9 @@ int lttng_list_interrupts(struct lttng_session *session) } #endif +/* + * Called with task lock held. + */ static void lttng_statedump_process_ns(struct lttng_session *session, struct task_struct *p, @@ -389,8 +393,18 @@ void lttng_statedump_process_ns(struct lttng_session *session, struct nsproxy *proxy; struct pid_namespace *pid_ns; + /* + * Back and forth on locking strategy within Linux upstream for nsproxy. + * See Linux upstream commit 728dba3a39c66b3d8ac889ddbe38b5b1c264aec3 + * "namespaces: Use task_lock and not rcu to protect nsproxy" + * for details. + */ +#if (LINUX_VERSION_CODE < KERNEL_VERSION(3,17,0)) rcu_read_lock(); proxy = task_nsproxy(p); +#else /* #if (LINUX_VERSION_CODE < KERNEL_VERSION(3,17,0)) */ + proxy = p->nsproxy; +#endif /* #else #if (LINUX_VERSION_CODE < KERNEL_VERSION(3,17,0)) */ if (proxy) { pid_ns = lttng_get_proxy_pid_ns(proxy); do { @@ -402,7 +416,9 @@ void lttng_statedump_process_ns(struct lttng_session *session, trace_lttng_statedump_process_state(session, p, type, mode, submode, status, NULL); } +#if (LINUX_VERSION_CODE < KERNEL_VERSION(3,17,0)) rcu_read_unlock(); +#endif /* #if (LINUX_VERSION_CODE < KERNEL_VERSION(3,17,0)) */ } static