Hello.
Randy Dunlap wrote:
> On Fri, 05 Nov 2010 17:59:42 +0900 Tetsuo Handa wrote:
>
> >
> > WARNING: modpost: Found 5 section mismatch(es).
> > To see full details build your kernel with:
> > 'make CONFIG_DEBUG_SECTION_MISMATCH=y'
> > fs/built-in.o(.text+0x3e357): In function `alignfile':
> > : undefined reference to `__divdi3'
> > fs/built-in.o(.text+0x3e39b): In function `alignfile':
> > : undefined reference to `__divdi3'
> > fs/built-in.o(.text+0x3f1af): In function `elf_core_dump':
> > : undefined reference to `__divdi3'
> > make: *** [.tmp_vmlinux1] Error 1
>
>
> Hm, all due to usage of roundup() AFAICT. Unfortunate.
Indeed. There is a commit for avoiding 64-bit division.
commit ea7f1b6ee9dc96c5827b06ba21d7769d553efb7d
Author: Bjorn Helgaas <bjorn.helgaas@xxxxxx>
Date: Thu Nov 5 11:17:11 2009 -0600
x86/PCI: remove 64-bit division
The roundup() caused a build error (undefined reference to `__udivdi3').
We're aligning to power-of-two boundaries, so it's simpler to just use
ALIGN() anyway, which avoids the division.
Signed-off-by: Bjorn Helgaas <bjorn.helgaas@xxxxxx>
Acked-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
It seems that the code itself has not changed between 2.6.36 and 2.6.37-rc1
static int elf_core_dump(struct coredump_params *cprm)
{
(...snipped...)
loff_t offset = 0, dataoff, foffset;
(...snipped...)
dataoff = offset = roundup(offset, ELF_EXEC_PAGESIZE);
but the definition of roundup() has changed by
commit b28efd54d9d5c8005a29cd8782335beb9daaa32d
Author: Eric Paris <eparis@xxxxxxxxxx>
Date: Wed Oct 13 17:50:08 2010 -0400
kernel: roundup should only reference arguments once
Currently the roundup macro references it's arguments more than one time.
This patch changes it so it will only use its arguments once.
Suggested-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Signed-off-by: Eric Paris <eparis@xxxxxxxxxx>
Signed-off-by: James Morris <jmorris@xxxxxxxxx>
between 2.6.36 and 2.6.37-rc1.
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
