2009-01-27 01:59:26

by Jeff Chua

[permalink] [raw]
Subject: multiple definition of `debugfs_create_size_t' - please revert 8adb711f3668b034e7b956fac951ed08b53e0d55

Linus,

The below commit cause this problem. Reverting it solved the problem.

arch/x86/mm/pat.o: In function `debugfs_create_size_t':
pat.c:(.text+0x0): multiple definition of `debugfs_create_size_t'
arch/x86/mm/pageattr.o:(.text+0x0): first defined here
make[1]: *** [arch/x86/mm/built-in.o] Error 1
make: *** [arch/x86/mm] Error 2


Thanks,
Jeff.


commit 8adb711f3668b034e7b956fac951ed08b53e0d55
Author: Inaky Perez-Gonzalez <[email protected]>
Date: Tue Jan 20 12:17:28 2009 -0800

debugfs: introduce stub for debugfs_create_size_t() when DEBUG_FS=n

Toralf Förster <[email protected]> reported a build failure in
the WiMAX stack when CONFIG_DEBUG_FS=n

http://linuxwimax.org/pipermail/wimax/2009-January/000449.html

This is due to debugfs_create_size_t() missing an stub that returns
-ENODEV when the DEBUGFS subsystem is not configured in (like the rest
of the debugfs API).

This patch adds said stub.

Reported-by: Toralf Förster <[email protected]>
Signed-off-by: Inaky Perez-Gonzalez <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>