On 3/26/18 3:30 PM, Mathieu Desnoyers wrote:
----- On Mar 26, 2018, at 6:08 PM, Alexei Starovoitov ast@xxxxxxxxxx wrote: [...]#ifdef CONFIG_TRACEPOINTS -void * -for_each_kernel_tracepoint(void *(*fct)(struct tracepoint *tp, void *priv), +void +for_each_kernel_tracepoint(void (*fct)(struct tracepoint *tp, void *priv), void *priv); +struct tracepoint *kernel_tracepoint_find_by_name(const char *name); #else -static inline void * -for_each_kernel_tracepoint(void *(*fct)(struct tracepoint *tp, void *priv), +static inline void +for_each_kernel_tracepoint(void (*fct)(struct tracepoint *tp, void *priv), void *priv) { return NULL; }This patch is not reverting to the old code properly. It introduces a static inline void function that returns NULL. Please compile-test with CONFIG_TRACEPOINTS=n before submitting a patch involving tracepoints.
right. good catch. v2 is coming. -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
