Subject: Markers Support for Proprierary Modules Date: Wed, 19 Mar 2008 20:27:41 -0400 From: Mathieu Desnoyers There seems to be good arguments for markers to support proprierary modules. So I am throwing this one-liner in and let's see how people react. It only makes sure that a module that has been "forced" to be loaded won't have its markers used. It is important to leave this check to make sure the kernel does not crash by expecting the markers part of the struct module by mistake in the case there is an incorrect checksum. Signed-off-by: Mathieu Desnoyers Acked-by: Jon Masters Signed-off-by: Rusty Russell --- kernel/module.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6-lttng/kernel/module.c =================================================================== --- linux-2.6-lttng.orig/kernel/module.c 2008-03-14 08:52:12.000000000 -0400 +++ linux-2.6-lttng/kernel/module.c 2008-03-14 08:54:10.000000000 -0400 @@ -2040,7 +2040,7 @@ static struct module *load_module(void _ add_kallsyms(mod, sechdrs, symindex, strindex, secstrings); #ifdef CONFIG_MARKERS - if (!mod->taints) + if (!(mod->taints & TAINT_FORCED_MODULE)) marker_update_probe_range(mod->markers, mod->markers + mod->num_markers); #endif