UBIFS: do not select KALLSYMS_ALL
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Wed, 30 Mar 2011 08:18:54 +0000 (11:18 +0300)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Tue, 5 Apr 2011 07:45:45 +0000 (10:45 +0300)
All UBIFS needs is to make sure we stacktraces when UBIFS debugging
is enabled. It is enough to select KALLSYMS for this, KALLSYMS_ALL
is not necessary. Moreover, Randy Dunlap reported that UBIFS causes
the following Kconfig dependency warning:

warning: (UBIFS_FS_DEBUG && LOCKDEP && LATENCYTOP) selects KALLSYMS_ALL
which has unmet direct dependencies (DEBUG_KERNEL && KALLSYMS)

The reason is that KALLSYMS_ALL requires DEBUG_KERNEL and KALLSYMS, so
ideally, to select KALLSYMS_ALL we'd need to select DEBUG_KERNEL and
KALLSYMS first.

This seems to be too much to select. The easiest way to go is to forget
about KALLSYMS_ALL and just select KALLSYMS when UBIFS debugging is
enabled - that should be enough for stackdumps.

Reported-by: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
fs/ubifs/Kconfig

index d7440904be170871c473cb09229de0e11b48fa16..f8b0160da2da8be993d8fe93c246c9bdd1dd35aa 100644 (file)
@@ -47,7 +47,7 @@ config UBIFS_FS_DEBUG
        bool "Enable debugging support"
        depends on UBIFS_FS
        select DEBUG_FS
-       select KALLSYMS_ALL
+       select KALLSYMS
        help
          This option enables UBIFS debugging support. It makes sure various
          assertions, self-checks, debugging messages and test modes are compiled