[patch] Fix vdso32 relocation for paddr

mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed

* [patch] Fix vdso32 relocation for paddr
@ 2009-08-26 22:31 Jan Kratochvil
  2009-08-28  9:13 ` Américo Wang
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Kratochvil @ 2009-08-26 22:31 UTC (permalink / raw)
  To: linux-kernel

Hi,

`readelf -Wl file-dumped-from-/proc/self/maps-[vdso]' for x86_64 process prints:
Program Headers:
  Type           Offset   VirtAddr           PhysAddr           FileSiz  MemSiz   Flg Align
  LOAD           0x000000 0xffffffffff700000 0xffffffffff700000 0x000890 0x000890 R E 0x1000
  DYNAMIC        0x000410 0xffffffffff700410 0xffffffffff700410 0x0000f0 0x0000f0 R   0x8
  NOTE           0x0002d0 0xffffffffff7002d0 0xffffffffff7002d0 0x00003c 0x00003c R   0x4
  GNU_EH_FRAME   0x00030c 0xffffffffff70030c 0xffffffffff70030c 0x000034 0x000034 R   0x4

while such dump for i386 process prints:
Program Headers:
  Type           Offset   VirtAddr   PhysAddr   FileSiz MemSiz  Flg Align
  LOAD           0x000000 0xffffe000 0x00000000 0x00434 0x00434 R E 0x1000
  DYNAMIC        0x000350 0xffffe350 0x00000350 0x00078 0x00078 R   0x4
  NOTE           0x0001c4 0xffffe1c4 0x000001c4 0x00060 0x00060 R   0x4
  GNU_EH_FRAME   0x000224 0xffffe224 0x00000224 0x00024 0x00024 R   0x4

The mapped location can be arbirary (according to the rules of PIC - Position
Independent Code) but currently the i386 vdso (vdso32) looks as an overlayed
code as it has LMA != VMA (rule contained in GDB function section_is_overlay,
as suggested by Daniel Jacobowitz).

Fixed vdso32 prints:
Program Headers:
  Type           Offset   VirtAddr   PhysAddr   FileSiz MemSiz  Flg Align
  LOAD           0x000000 0xffffe000 0xffffe000 0x00434 0x00434 R E 0x1000
  DYNAMIC        0x000350 0xffffe350 0xffffe350 0x00078 0x00078 R   0x4
  NOTE           0x0001c4 0xffffe1c4 0xffffe1c4 0x00060 0x00060 R   0x4
  GNU_EH_FRAME   0x000224 0xffffe224 0xffffe224 0x00024 0x00024 R   0x4

It required now a workaround in GDB patch so at least to fix it in Linux kernel:
	http://sourceware.org/ml/gdb-patches/2009-08/msg00437.html


Thanks,
Jan


Signed-off-by: Jan Kratochvil <jan.kratochvil@redhat.com>

--- linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c-orig	2009-08-26 22:23:08.000000000 +0200
+++ linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c	2009-08-26 22:38:10.000000000 +0200
@@ -172,6 +172,7 @@ static __init void relocate_vdso(Elf32_E
 	phdr = (void *)ehdr + ehdr->e_phoff;
 	for (i = 0; i < ehdr->e_phnum; i++) {
 		phdr[i].p_vaddr += VDSO_ADDR_ADJUST;
+		phdr[i].p_paddr += VDSO_ADDR_ADJUST;
 
 		/* relocate dynamic stuff */
 		if (phdr[i].p_type == PT_DYNAMIC)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [patch] Fix vdso32 relocation for paddr
  2009-08-26 22:31 [patch] Fix vdso32 relocation for paddr Jan Kratochvil
@ 2009-08-28  9:13 ` Américo Wang
       [not found]   ` <20090828181534.0823745B02@magilla.sf.frob.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Américo Wang @ 2009-08-28  9:13 UTC (permalink / raw)
  To: Jan Kratochvil; +Cc: linux-kernel, Roland McGrath

On Thu, Aug 27, 2009 at 6:31 AM, Jan
Kratochvil<jan.kratochvil@redhat.com> wrote:
> Hi,
>
> `readelf -Wl file-dumped-from-/proc/self/maps-[vdso]' for x86_64 process prints:
> Program Headers:
>  Type           Offset   VirtAddr           PhysAddr           FileSiz  MemSiz   Flg Align
>  LOAD           0x000000 0xffffffffff700000 0xffffffffff700000 0x000890 0x000890 R E 0x1000
>  DYNAMIC        0x000410 0xffffffffff700410 0xffffffffff700410 0x0000f0 0x0000f0 R   0x8
>  NOTE           0x0002d0 0xffffffffff7002d0 0xffffffffff7002d0 0x00003c 0x00003c R   0x4
>  GNU_EH_FRAME   0x00030c 0xffffffffff70030c 0xffffffffff70030c 0x000034 0x000034 R   0x4
>
> while such dump for i386 process prints:
> Program Headers:
>  Type           Offset   VirtAddr   PhysAddr   FileSiz MemSiz  Flg Align
>  LOAD           0x000000 0xffffe000 0x00000000 0x00434 0x00434 R E 0x1000
>  DYNAMIC        0x000350 0xffffe350 0x00000350 0x00078 0x00078 R   0x4
>  NOTE           0x0001c4 0xffffe1c4 0x000001c4 0x00060 0x00060 R   0x4
>  GNU_EH_FRAME   0x000224 0xffffe224 0x00000224 0x00024 0x00024 R   0x4
>
> The mapped location can be arbirary (according to the rules of PIC - Position
> Independent Code) but currently the i386 vdso (vdso32) looks as an overlayed
> code as it has LMA != VMA (rule contained in GDB function section_is_overlay,
> as suggested by Daniel Jacobowitz).
>
> Fixed vdso32 prints:
> Program Headers:
>  Type           Offset   VirtAddr   PhysAddr   FileSiz MemSiz  Flg Align
>  LOAD           0x000000 0xffffe000 0xffffe000 0x00434 0x00434 R E 0x1000
>  DYNAMIC        0x000350 0xffffe350 0xffffe350 0x00078 0x00078 R   0x4
>  NOTE           0x0001c4 0xffffe1c4 0xffffe1c4 0x00060 0x00060 R   0x4
>  GNU_EH_FRAME   0x000224 0xffffe224 0xffffe224 0x00024 0x00024 R   0x4
>
> It required now a workaround in GDB patch so at least to fix it in Linux kernel:
>        http://sourceware.org/ml/gdb-patches/2009-08/msg00437.html
>
>

Reviewed-by: WANG Cong <xiyou.wangcong@gmail.com>

This patch looks fine for me. But I think we should hear from Roland.
Roland?


> Thanks,
> Jan
>
>
> Signed-off-by: Jan Kratochvil <jan.kratochvil@redhat.com>
>
> --- linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c-orig       2009-08-26 22:23:08.000000000 +0200
> +++ linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c    2009-08-26 22:38:10.000000000 +0200
> @@ -172,6 +172,7 @@ static __init void relocate_vdso(Elf32_E
>        phdr = (void *)ehdr + ehdr->e_phoff;
>        for (i = 0; i < ehdr->e_phnum; i++) {
>                phdr[i].p_vaddr += VDSO_ADDR_ADJUST;
> +               phdr[i].p_paddr += VDSO_ADDR_ADJUST;
>
>                /* relocate dynamic stuff */
>                if (phdr[i].p_type == PT_DYNAMIC)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

[parent not found: <20090828181534.0823745B02@magilla.sf.frob.com>]

* Re: [patch] Fix vdso32 relocation for paddr
       [not found]   ` <20090828181534.0823745B02@magilla.sf.frob.com>
@ 2009-08-28 21:07     ` Jan Kratochvil
  2009-08-28 21:33       ` Roland McGrath
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Kratochvil @ 2009-08-28 21:07 UTC (permalink / raw)
  To: Roland McGrath; +Cc: Américo Wang, linux-kernel

On Fri, 28 Aug 2009 20:15:34 +0200, Roland McGrath wrote:
> Thanks for heads-up.  It's definitely right to CC me on patches that change
> the build/setup vDSO details.

OK, I thought more about {p,u}trace.


> That change seems fine.  Or just explicit p_addr = p_vaddr would be fine
> too (and not assume that what the linker produced is consistent, in case
> some old linkers produce stranger results from the vdso linker scripts).

Updated.  Verified on x86_64 host it works right for an i386 binary.


Thanks,
Jan


Signed-off-by: Jan Kratochvil <jan.kratochvil@redhat.com>

--- linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c-orig	2009-08-26 22:23:08.000000000 +0200
+++ linux-2.6.30.x86_64/arch/x86/vdso/vdso32-setup.c	2009-08-26 22:38:10.000000000 +0200
@@ -172,6 +172,7 @@ static __init void relocate_vdso(Elf32_E
 	phdr = (void *)ehdr + ehdr->e_phoff;
 	for (i = 0; i < ehdr->e_phnum; i++) {
 		phdr[i].p_vaddr += VDSO_ADDR_ADJUST;
+		phdr[i].p_paddr = phdr[i].p_vaddr;
 
 		/* relocate dynamic stuff */
 		if (phdr[i].p_type == PT_DYNAMIC)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [patch] Fix vdso32 relocation for paddr
  2009-08-28 21:07     ` Jan Kratochvil
@ 2009-08-28 21:33       ` Roland McGrath
  0 siblings, 0 replies; 4+ messages in thread
From: Roland McGrath @ 2009-08-28 21:33 UTC (permalink / raw)
  To: Jan Kratochvil; +Cc: xiyou.wangcong, linux-kernel

> On Fri, 28 Aug 2009 20:15:34 +0200, Roland McGrath wrote:
> > Thanks for heads-up.  It's definitely right to CC me on patches that change
> > the build/setup vDSO details.
> 
> OK, I thought more about {p,u}trace.

It so happens I'm also the vDSO and ELF-magic guru for the kernel.
It's an unrelated hat.

> Updated.  Verified on x86_64 host it works right for an i386 binary.

Looks good to me!


Thanks,
Roland

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-08-28 21:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-26 22:31 [patch] Fix vdso32 relocation for paddr Jan Kratochvil
2009-08-28  9:13 ` Américo Wang
     [not found]   ` <20090828181534.0823745B02@magilla.sf.frob.com>
2009-08-28 21:07     ` Jan Kratochvil
2009-08-28 21:33       ` Roland McGrath

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®