Revision 55b6f763d8bcb5546997933105d66d3e6b080e6a authored by Johannes Berg on 05 February 2021, 02:32:28 UTC, committed by Linus Torvalds on 05 February 2021, 19:03:47 UTC
On ARCH=um, loading a module doesn't result in its constructors getting
called, which breaks module gcov since the debugfs files are never
registered.  On the other hand, in-kernel constructors have already been
called by the dynamic linker, so we can't call them again.

Get out of this conundrum by allowing CONFIG_CONSTRUCTORS to be
selected, but avoiding the in-kernel constructor calls.

Also remove the "if !UML" from GCOV selecting CONSTRUCTORS now, since we
really do want CONSTRUCTORS, just not kernel binary ones.

Link: https://lkml.kernel.org/r/20210120172041.c246a2cac2fb.I1358f584b76f1898373adfed77f4462c8705b736@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Reviewed-by: Peter Oberparleiter <oberpar@linux.ibm.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Jessica Yu <jeyu@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
1 parent 4f6ec86
History
File Mode Size
apparmor.h -rw-r--r-- 1.0 KB
apparmorfs.h -rw-r--r-- 3.1 KB
audit.h -rw-r--r-- 4.5 KB
capability.h -rw-r--r-- 1.0 KB
cred.h -rw-r--r-- 4.8 KB
crypto.h -rw-r--r-- 800 bytes
domain.h -rw-r--r-- 912 bytes
file.h -rw-r--r-- 6.6 KB
ipc.h -rw-r--r-- 1.0 KB
label.h -rw-r--r-- 12.9 KB
lib.h -rw-r--r-- 7.6 KB
match.h -rw-r--r-- 5.4 KB
mount.h -rw-r--r-- 1.3 KB
net.h -rw-r--r-- 3.0 KB
path.h -rw-r--r-- 932 bytes
perms.h -rw-r--r-- 4.8 KB
policy.h -rw-r--r-- 8.6 KB
policy_ns.h -rw-r--r-- 4.6 KB
policy_unpack.h -rw-r--r-- 3.4 KB
procattr.h -rw-r--r-- 448 bytes
resource.h -rw-r--r-- 1.1 KB
secid.h -rw-r--r-- 904 bytes
sig_names.h -rw-r--r-- 2.2 KB
task.h -rw-r--r-- 2.0 KB

back to top