diff options
author | James Hogan <james.hogan@imgtec.com> | 2013-04-28 22:06:59 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2013-04-28 22:11:42 -0400 |
commit | a53a11f35762ff1d5e268adedf2ab9ce313f871d (patch) | |
tree | 40bc55804d50143f5807c0551278dd2058032abf /scripts/mod/modpost.c | |
parent | 04b00bdb41d0fd8d9cf3b146e334369cc2b0acdc (diff) |
modpost: fix unwanted VMLINUX_SYMBOL_STR expansion
Commit a4b6a77b77ba4f526392612c2365797fab956014 ("module: fix symbol
versioning with symbol prefixes") broke the MODVERSIONS loading of any
module using memcmp (e.g. ipv6) on x86_32, as it's defined to
__builtin_memcmp which is expanded by VMLINUX_SYMBOL_STR. Use
__VMLINUX_SYMBOL_STR instead which doesn't expand the argument.
Reported-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Reported-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: James Hogan <james.hogan@imgtec.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Tested-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Tested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Cc: <stable@vger.kernel.org> # 3.9
Diffstat (limited to 'scripts/mod/modpost.c')
-rw-r--r-- | scripts/mod/modpost.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 1f90961ada77..a4be8e112bb6 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c | |||
@@ -1957,7 +1957,7 @@ static int add_versions(struct buffer *b, struct module *mod) | |||
1957 | s->name, mod->name); | 1957 | s->name, mod->name); |
1958 | continue; | 1958 | continue; |
1959 | } | 1959 | } |
1960 | buf_printf(b, "\t{ %#8x, VMLINUX_SYMBOL_STR(%s) },\n", | 1960 | buf_printf(b, "\t{ %#8x, __VMLINUX_SYMBOL_STR(%s) },\n", |
1961 | s->crc, s->name); | 1961 | s->crc, s->name); |
1962 | } | 1962 | } |
1963 | 1963 | ||