[PATCH] D28857: [ELF] - Allow relative relocations to a absolute value defined in linker script.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 06:45:45 PST 2017


>That is a bug in bfd. Its handling of what is absolute is broken. On ELF
>absolute really means absolute, like a size. Anything that is a position
>in the file is not absolute.
>
>Cheers,
>Rafael

This script (https://github.com/torvalds/linux/blob/d33d5a6c88fcd53fec329a1521010f1bc55fa191/arch/x86/entry/vdso/vdso-layout.lds.S)
starts from:

SECTIONS
{
  vvar_start = . - 2 * (1 << 12);
  vvar_page = vvar_start;
  vvar_vsyscall_gtod_data = vvar_page + 128;

I think vvar_start is always constant, do not know why they wrote ". - 2 * (1 << 12)" when it looks
could be just "- 2 * (1 << 12)".
At least I think original comment ("These variables live in a page of kernel data that has an extra RO* mapping for userspace. Each variable needs a unique offset within that page; specify that offset with the DECLARE_VVAR macro.")
saying that vvar_vsyscall_gtod_data value is a constant + offset == constant. So it not relative to position in file and behaves as absolute.

How do you think we should fix that ?

George.





More information about the llvm-commits mailing list