[Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Aurelien Jarno]

Fix .rel.plt sections in the output to not only include .rel.plt
sections from the input but also the .rel.iplt sections and to define
the hidden symbols __rel_iplt_start and __rel_iplt_end around
.rel.iplt as otherwise we get undefined references to these when
linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).

Blue Swirl <blauwirbel@gmail.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
---
 sparc.ld |   16 ++++++++++++++--
 1 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/sparc.ld b/sparc.ld
index 56efe34..e52c3d2 100644
--- a/sparc.ld
+++ b/sparc.ld
@@ -37,8 +37,20 @@ SECTIONS
   .rela.fini     : { *(.rela.fini)      }
   .rel.bss       : { *(.rel.bss)                }
   .rela.bss      : { *(.rela.bss)               }
-  .rel.plt       : { *(.rel.plt)                }
-  .rela.plt      : { *(.rela.plt)               }
+  .rel.plt      :
+  {
+    *(.rel.plt)
+    PROVIDE_HIDDEN (__rel_iplt_start = .);
+    *(.rel.iplt)
+    PROVIDE_HIDDEN (__rel_iplt_end = .);
+  }
+  .rela.plt       :
+  {
+    *(.rela.plt)
+    PROVIDE_HIDDEN (__rela_iplt_start = .);
+    *(.rela.iplt)
+    PROVIDE_HIDDEN (__rela_iplt_end = .);
+  }
   .init          : { *(.init)   } =0x47ff041f
   .text      :
   {
-- 
1.7.7.3

Re: [Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Blue Swirl]

On Sat, Jan 7, 2012 at 20:16, Aurelien Jarno <aurelien@aurel32.net> wrote:
> Fix .rel.plt sections in the output to not only include .rel.plt
> sections from the input but also the .rel.iplt sections and to define
> the hidden symbols __rel_iplt_start and __rel_iplt_end around
> .rel.iplt as otherwise we get undefined references to these when
> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>
> Blue Swirl <blauwirbel@gmail.com>
> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> ---
>  sparc.ld |   16 ++++++++++++++--
>  1 files changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/sparc.ld b/sparc.ld
> index 56efe34..e52c3d2 100644
> --- a/sparc.ld
> +++ b/sparc.ld
> @@ -37,8 +37,20 @@ SECTIONS
>   .rela.fini     : { *(.rela.fini)      }
>   .rel.bss       : { *(.rel.bss)                }
>   .rela.bss      : { *(.rela.bss)               }
> -  .rel.plt       : { *(.rel.plt)                }
> -  .rela.plt      : { *(.rela.plt)               }
> +  .rel.plt      :
> +  {
> +    *(.rel.plt)
> +    PROVIDE_HIDDEN (__rel_iplt_start = .);
> +    *(.rel.iplt)
> +    PROVIDE_HIDDEN (__rel_iplt_end = .);
> +  }
> +  .rela.plt       :
> +  {
> +    *(.rela.plt)
> +    PROVIDE_HIDDEN (__rela_iplt_start = .);
> +    *(.rela.iplt)
> +    PROVIDE_HIDDEN (__rela_iplt_end = .);
> +  }

PROVIDE_HIDDEN etc. were removed by
8733f6093c2b77502e7228503fc22024e51599b8 in order to support BSDs with
older binutils (2.15). Maybe the ld scripts should be generated from a
source file for example with CPP, so these lines could be tweaked.

Doesn't guest base make the whole ld script thing useless? It's not
implemented for Sparc though.

>   .init          : { *(.init)   } =0x47ff041f
>   .text      :
>   {
> --
> 1.7.7.3
>
>

Re: [Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Aurelien Jarno]

On Sat, Jan 07, 2012 at 08:36:12PM +0000, Blue Swirl wrote:
> On Sat, Jan 7, 2012 at 20:16, Aurelien Jarno <aurelien@aurel32.net> wrote:
> > Fix .rel.plt sections in the output to not only include .rel.plt
> > sections from the input but also the .rel.iplt sections and to define
> > the hidden symbols __rel_iplt_start and __rel_iplt_end around
> > .rel.iplt as otherwise we get undefined references to these when
> > linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
> >
> > Blue Swirl <blauwirbel@gmail.com>
> > Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> > ---
> >  sparc.ld |   16 ++++++++++++++--
> >  1 files changed, 14 insertions(+), 2 deletions(-)
> >
> > diff --git a/sparc.ld b/sparc.ld
> > index 56efe34..e52c3d2 100644
> > --- a/sparc.ld
> > +++ b/sparc.ld
> > @@ -37,8 +37,20 @@ SECTIONS
> >   .rela.fini     : { *(.rela.fini)      }
> >   .rel.bss       : { *(.rel.bss)                }
> >   .rela.bss      : { *(.rela.bss)               }
> > -  .rel.plt       : { *(.rel.plt)                }
> > -  .rela.plt      : { *(.rela.plt)               }
> > +  .rel.plt      :
> > +  {
> > +    *(.rel.plt)
> > +    PROVIDE_HIDDEN (__rel_iplt_start = .);
> > +    *(.rel.iplt)
> > +    PROVIDE_HIDDEN (__rel_iplt_end = .);
> > +  }
> > +  .rela.plt       :
> > +  {
> > +    *(.rela.plt)
> > +    PROVIDE_HIDDEN (__rela_iplt_start = .);
> > +    *(.rela.iplt)
> > +    PROVIDE_HIDDEN (__rela_iplt_end = .);
> > +  }
> 
> PROVIDE_HIDDEN etc. were removed by
> 8733f6093c2b77502e7228503fc22024e51599b8 in order to support BSDs with
> older binutils (2.15). Maybe the ld scripts should be generated from a
> source file for example with CPP, so these lines could be tweaked.
> 

It seems this patch was in my local tree for too long, and it seems I
should have updated it before sending it...

Other architectures are now using PROVIDE instead of PROVIDE_HIDDEN for 
__rel{a,}_iplt_{start,end}, so I guess the same can be done for sparc and
ppc. I am going to do a test build to check that, and I'll come with a
new patch if it works.

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

Re: [Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Aurelien Jarno]

On Sat, Jan 07, 2012 at 10:01:09PM +0100, Aurelien Jarno wrote:
> On Sat, Jan 07, 2012 at 08:36:12PM +0000, Blue Swirl wrote:
> > On Sat, Jan 7, 2012 at 20:16, Aurelien Jarno <aurelien@aurel32.net> wrote:
> > > Fix .rel.plt sections in the output to not only include .rel.plt
> > > sections from the input but also the .rel.iplt sections and to define
> > > the hidden symbols __rel_iplt_start and __rel_iplt_end around
> > > .rel.iplt as otherwise we get undefined references to these when
> > > linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
> > >
> > > Blue Swirl <blauwirbel@gmail.com>
> > > Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> > > ---
> > >  sparc.ld |   16 ++++++++++++++--
> > >  1 files changed, 14 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/sparc.ld b/sparc.ld
> > > index 56efe34..e52c3d2 100644
> > > --- a/sparc.ld
> > > +++ b/sparc.ld
> > > @@ -37,8 +37,20 @@ SECTIONS
> > >   .rela.fini     : { *(.rela.fini)      }
> > >   .rel.bss       : { *(.rel.bss)                }
> > >   .rela.bss      : { *(.rela.bss)               }
> > > -  .rel.plt       : { *(.rel.plt)                }
> > > -  .rela.plt      : { *(.rela.plt)               }
> > > +  .rel.plt      :
> > > +  {
> > > +    *(.rel.plt)
> > > +    PROVIDE_HIDDEN (__rel_iplt_start = .);
> > > +    *(.rel.iplt)
> > > +    PROVIDE_HIDDEN (__rel_iplt_end = .);
> > > +  }
> > > +  .rela.plt       :
> > > +  {
> > > +    *(.rela.plt)
> > > +    PROVIDE_HIDDEN (__rela_iplt_start = .);
> > > +    *(.rela.iplt)
> > > +    PROVIDE_HIDDEN (__rela_iplt_end = .);
> > > +  }
> > 
> > PROVIDE_HIDDEN etc. were removed by
> > 8733f6093c2b77502e7228503fc22024e51599b8 in order to support BSDs with
> > older binutils (2.15). Maybe the ld scripts should be generated from a
> > source file for example with CPP, so these lines could be tweaked.
> > 
> 
> It seems this patch was in my local tree for too long, and it seems I
> should have updated it before sending it...
> 
> Other architectures are now using PROVIDE instead of PROVIDE_HIDDEN for 
> __rel{a,}_iplt_{start,end}, so I guess the same can be done for sparc and
> ppc. I am going to do a test build to check that, and I'll come with a
> new patch if it works.

The new patch below also fixes the issue, and uses PROVIDE instead of
PROVIDE_HIDDEN.

sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

Fix .rel.plt sections in the output to not only include .rel.plt
sections from the input but also the .rel.iplt sections and to define
the hidden symbols __rel_iplt_start and __rel_iplt_end around
.rel.iplt as otherwise we get undefined references to these when
linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).

Cc: Blue Swirl <blauwirbel@gmail.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
---
 sparc.ld |   16 ++++++++++++++--
 1 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/sparc.ld b/sparc.ld
index 56efe34..cec17c9 100644
--- a/sparc.ld
+++ b/sparc.ld
@@ -37,8 +37,20 @@ SECTIONS
   .rela.fini     : { *(.rela.fini)      }
   .rel.bss       : { *(.rel.bss)                }
   .rela.bss      : { *(.rela.bss)               }
-  .rel.plt       : { *(.rel.plt)                }
-  .rela.plt      : { *(.rela.plt)               }
+  .rel.plt      :
+  {
+    *(.rel.plt)
+    PROVIDE (__rel_iplt_start = .);
+    *(.rel.iplt)
+    PROVIDE (__rel_iplt_end = .);
+  }
+  .rela.plt       :
+  {
+    *(.rela.plt)
+    PROVIDE (__rela_iplt_start = .);
+    *(.rela.iplt)
+    PROVIDE (__rela_iplt_end = .);
+  }
   .init          : { *(.init)   } =0x47ff041f
   .text      :
   {
-- 
1.7.7.3


-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

Re: [Qemu-devel] [PATCH] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Blue Swirl]

On Tue, Jan 10, 2012 at 17:38, Aurelien Jarno <aurelien@aurel32.net> wrote:
> On Sat, Jan 07, 2012 at 10:01:09PM +0100, Aurelien Jarno wrote:
>> On Sat, Jan 07, 2012 at 08:36:12PM +0000, Blue Swirl wrote:
>> > On Sat, Jan 7, 2012 at 20:16, Aurelien Jarno <aurelien@aurel32.net> wrote:
>> > > Fix .rel.plt sections in the output to not only include .rel.plt
>> > > sections from the input but also the .rel.iplt sections and to define
>> > > the hidden symbols __rel_iplt_start and __rel_iplt_end around
>> > > .rel.iplt as otherwise we get undefined references to these when
>> > > linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>> > >
>> > > Blue Swirl <blauwirbel@gmail.com>
>> > > Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
>> > > ---
>> > >  sparc.ld |   16 ++++++++++++++--
>> > >  1 files changed, 14 insertions(+), 2 deletions(-)
>> > >
>> > > diff --git a/sparc.ld b/sparc.ld
>> > > index 56efe34..e52c3d2 100644
>> > > --- a/sparc.ld
>> > > +++ b/sparc.ld
>> > > @@ -37,8 +37,20 @@ SECTIONS
>> > >   .rela.fini     : { *(.rela.fini)      }
>> > >   .rel.bss       : { *(.rel.bss)                }
>> > >   .rela.bss      : { *(.rela.bss)               }
>> > > -  .rel.plt       : { *(.rel.plt)                }
>> > > -  .rela.plt      : { *(.rela.plt)               }
>> > > +  .rel.plt      :
>> > > +  {
>> > > +    *(.rel.plt)
>> > > +    PROVIDE_HIDDEN (__rel_iplt_start = .);
>> > > +    *(.rel.iplt)
>> > > +    PROVIDE_HIDDEN (__rel_iplt_end = .);
>> > > +  }
>> > > +  .rela.plt       :
>> > > +  {
>> > > +    *(.rela.plt)
>> > > +    PROVIDE_HIDDEN (__rela_iplt_start = .);
>> > > +    *(.rela.iplt)
>> > > +    PROVIDE_HIDDEN (__rela_iplt_end = .);
>> > > +  }
>> >
>> > PROVIDE_HIDDEN etc. were removed by
>> > 8733f6093c2b77502e7228503fc22024e51599b8 in order to support BSDs with
>> > older binutils (2.15). Maybe the ld scripts should be generated from a
>> > source file for example with CPP, so these lines could be tweaked.
>> >
>>
>> It seems this patch was in my local tree for too long, and it seems I
>> should have updated it before sending it...
>>
>> Other architectures are now using PROVIDE instead of PROVIDE_HIDDEN for
>> __rel{a,}_iplt_{start,end}, so I guess the same can be done for sparc and
>> ppc. I am going to do a test build to check that, and I'll come with a
>> new patch if it works.
>
> The new patch below also fixes the issue, and uses PROVIDE instead of
> PROVIDE_HIDDEN.

This should work, the info page for OpenBSD ld talks about PROVIDE
(but not PROVIDE_HIDDEN).

> sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections
>
> Fix .rel.plt sections in the output to not only include .rel.plt
> sections from the input but also the .rel.iplt sections and to define
> the hidden symbols __rel_iplt_start and __rel_iplt_end around
> .rel.iplt as otherwise we get undefined references to these when
> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> ---
>  sparc.ld |   16 ++++++++++++++--
>  1 files changed, 14 insertions(+), 2 deletions(-)
>
> diff --git a/sparc.ld b/sparc.ld
> index 56efe34..cec17c9 100644
> --- a/sparc.ld
> +++ b/sparc.ld
> @@ -37,8 +37,20 @@ SECTIONS
>   .rela.fini     : { *(.rela.fini)      }
>   .rel.bss       : { *(.rel.bss)                }
>   .rela.bss      : { *(.rela.bss)               }
> -  .rel.plt       : { *(.rel.plt)                }
> -  .rela.plt      : { *(.rela.plt)               }
> +  .rel.plt      :
> +  {
> +    *(.rel.plt)
> +    PROVIDE (__rel_iplt_start = .);
> +    *(.rel.iplt)
> +    PROVIDE (__rel_iplt_end = .);
> +  }
> +  .rela.plt       :
> +  {
> +    *(.rela.plt)
> +    PROVIDE (__rela_iplt_start = .);
> +    *(.rela.iplt)
> +    PROVIDE (__rela_iplt_end = .);
> +  }
>   .init          : { *(.init)   } =0x47ff041f
>   .text      :
>   {
> --
> 1.7.7.3
>
>
> --
> Aurelien Jarno                          GPG: 1024D/F1BCDB73
> aurelien@aurel32.net                 http://www.aurel32.net
>

Re: [Qemu-devel] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Michael Tokarev]

This patch was submitted more than a year ago (at Jan-2012).
Is it still needed?  If yes, why it hasn't been applied?

It still applies cleanly to the current git, with the exception
of s|^|ldscripts/| - sparc.ld moved from the top directory to
ldscripts/.

(Ref: http://patchwork.ozlabs.org/patch/135267 )

Thanks,

/mjt


10.01.2012 11:38, Aurelien Jarno wrote:
> sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections
> 
> Fix .rel.plt sections in the output to not only include .rel.plt
> sections from the input but also the .rel.iplt sections and to define
> the hidden symbols __rel_iplt_start and __rel_iplt_end around
> .rel.iplt as otherwise we get undefined references to these when
> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
> 
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> 
> ---
> sparc.ld |   16 ++++++++++++++--
>  1 files changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/sparc.ld b/sparc.ld
> index 56efe34..cec17c9 100644
> --- a/sparc.ld
> +++ b/sparc.ld
> @@ -37,8 +37,20 @@ SECTIONS
>    .rela.fini     : { *(.rela.fini)      }
>    .rel.bss       : { *(.rel.bss)                }
>    .rela.bss      : { *(.rela.bss)               }
> -  .rel.plt       : { *(.rel.plt)                }
> -  .rela.plt      : { *(.rela.plt)               }
> +  .rel.plt      :
> +  {
> +    *(.rel.plt)
> +    PROVIDE (__rel_iplt_start = .);
> +    *(.rel.iplt)
> +    PROVIDE (__rel_iplt_end = .);
> +  }
> +  .rela.plt       :
> +  {
> +    *(.rela.plt)
> +    PROVIDE (__rela_iplt_start = .);
> +    *(.rela.iplt)
> +    PROVIDE (__rela_iplt_end = .);
> +  }
>    .init          : { *(.init)   } =0x47ff041f
>    .text      :
>    {
> 

Re: [Qemu-devel] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Michael Tokarev]

Ping #2?

06.04.2013 10:08, Michael Tokarev wrote:
> This patch was submitted more than a year ago (at Jan-2012).
> Is it still needed?  If yes, why it hasn't been applied?
> 
> It still applies cleanly to the current git, with the exception
> of s|^|ldscripts/| - sparc.ld moved from the top directory to
> ldscripts/.
> 
> (Ref: http://patchwork.ozlabs.org/patch/135267 )
> 
> Thanks,
> 
> /mjt
> 
> 
> 10.01.2012 11:38, Aurelien Jarno wrote:
>> sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections
>>
>> Fix .rel.plt sections in the output to not only include .rel.plt
>> sections from the input but also the .rel.iplt sections and to define
>> the hidden symbols __rel_iplt_start and __rel_iplt_end around
>> .rel.iplt as otherwise we get undefined references to these when
>> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>>
>> Cc: Blue Swirl <blauwirbel@gmail.com>
>> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
>>
>> ---
>> sparc.ld |   16 ++++++++++++++--
>>  1 files changed, 14 insertions(+), 2 deletions(-)
>>
>> diff --git a/sparc.ld b/sparc.ld
>> index 56efe34..cec17c9 100644
>> --- a/sparc.ld
>> +++ b/sparc.ld
>> @@ -37,8 +37,20 @@ SECTIONS
>>    .rela.fini     : { *(.rela.fini)      }
>>    .rel.bss       : { *(.rel.bss)                }
>>    .rela.bss      : { *(.rela.bss)               }
>> -  .rel.plt       : { *(.rel.plt)                }
>> -  .rela.plt      : { *(.rela.plt)               }
>> +  .rel.plt      :
>> +  {
>> +    *(.rel.plt)
>> +    PROVIDE (__rel_iplt_start = .);
>> +    *(.rel.iplt)
>> +    PROVIDE (__rel_iplt_end = .);
>> +  }
>> +  .rela.plt       :
>> +  {
>> +    *(.rela.plt)
>> +    PROVIDE (__rela_iplt_start = .);
>> +    *(.rela.iplt)
>> +    PROVIDE (__rela_iplt_end = .);
>> +  }
>>    .init          : { *(.init)   } =0x47ff041f
>>    .text      :
>>    {
>>
> 
> 

Re: [Qemu-devel] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Blue Swirl]

On Wed, May 8, 2013 at 7:18 AM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> Ping #2?
>
> 06.04.2013 10:08, Michael Tokarev wrote:
>> This patch was submitted more than a year ago (at Jan-2012).
>> Is it still needed?  If yes, why it hasn't been applied?

Well, is it still needed?

>>
>> It still applies cleanly to the current git, with the exception
>> of s|^|ldscripts/| - sparc.ld moved from the top directory to
>> ldscripts/.
>>
>> (Ref: http://patchwork.ozlabs.org/patch/135267 )
>>
>> Thanks,
>>
>> /mjt
>>
>>
>> 10.01.2012 11:38, Aurelien Jarno wrote:
>>> sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections
>>>
>>> Fix .rel.plt sections in the output to not only include .rel.plt
>>> sections from the input but also the .rel.iplt sections and to define
>>> the hidden symbols __rel_iplt_start and __rel_iplt_end around
>>> .rel.iplt as otherwise we get undefined references to these when
>>> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>>>
>>> Cc: Blue Swirl <blauwirbel@gmail.com>
>>> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
>>>
>>> ---
>>> sparc.ld |   16 ++++++++++++++--
>>>  1 files changed, 14 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/sparc.ld b/sparc.ld
>>> index 56efe34..cec17c9 100644
>>> --- a/sparc.ld
>>> +++ b/sparc.ld
>>> @@ -37,8 +37,20 @@ SECTIONS
>>>    .rela.fini     : { *(.rela.fini)      }
>>>    .rel.bss       : { *(.rel.bss)                }
>>>    .rela.bss      : { *(.rela.bss)               }
>>> -  .rel.plt       : { *(.rel.plt)                }
>>> -  .rela.plt      : { *(.rela.plt)               }
>>> +  .rel.plt      :
>>> +  {
>>> +    *(.rel.plt)
>>> +    PROVIDE (__rel_iplt_start = .);
>>> +    *(.rel.iplt)
>>> +    PROVIDE (__rel_iplt_end = .);
>>> +  }
>>> +  .rela.plt       :
>>> +  {
>>> +    *(.rela.plt)
>>> +    PROVIDE (__rela_iplt_start = .);
>>> +    *(.rela.iplt)
>>> +    PROVIDE (__rela_iplt_end = .);
>>> +  }
>>>    .init          : { *(.init)   } =0x47ff041f
>>>    .text      :
>>>    {
>>>
>>
>>
>

Re: [Qemu-devel] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Aurelien Jarno]

On Thu, May 09, 2013 at 05:56:03PM +0000, Blue Swirl wrote:
> On Wed, May 8, 2013 at 7:18 AM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> > Ping #2?
> >
> > 06.04.2013 10:08, Michael Tokarev wrote:
> >> This patch was submitted more than a year ago (at Jan-2012).
> >> Is it still needed?  If yes, why it hasn't been applied?
> 
> Well, is it still needed?

Without it, trying to build static qemu-user on sparc leads to the
following error:

|   LINK  mips-linux-user/qemu-mips
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libglib-2.0.a(gutils.o): In function `g_get_any_init_do':
| (.text+0x440): warning: Using 'getpwuid' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libglib-2.0.a(gutils.o): In function `g_get_any_init_do':
| (.text+0x430): warning: Using 'setpwent' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libglib-2.0.a(gutils.o): In function `g_get_any_init_do':
| (.text+0x448): warning: Using 'endpwent' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libglib-2.0.a(gutils.o): In function `g_get_any_init_do':
| (.text+0x188): warning: Using 'getpwnam_r' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libglib-2.0.a(gutils.o): In function `g_get_any_init_do':
| (.text+0x1c8): warning: Using 'getpwuid_r' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
| (.text+0x144): undefined reference to `__rela_iplt_start'
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
| (.text+0x148): undefined reference to `__rela_iplt_end'
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
| (.text+0x14c): undefined reference to `__rela_iplt_start'
| /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
| (.text+0x150): undefined reference to `__rela_iplt_end'
| collect2: ld returned 1 exit status
| make[1]: *** [qemu-mips] Error 1
| make: *** [subdir-mips-linux-user] Error 2

So I think it is still needed, unless you have another fix in mind.

> >>
> >> It still applies cleanly to the current git, with the exception
> >> of s|^|ldscripts/| - sparc.ld moved from the top directory to
> >> ldscripts/.
> >>
> >> (Ref: http://patchwork.ozlabs.org/patch/135267 )
> >>
> >> Thanks,
> >>
> >> /mjt
> >>
> >>
> >> 10.01.2012 11:38, Aurelien Jarno wrote:
> >>> sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections
> >>>
> >>> Fix .rel.plt sections in the output to not only include .rel.plt
> >>> sections from the input but also the .rel.iplt sections and to define
> >>> the hidden symbols __rel_iplt_start and __rel_iplt_end around
> >>> .rel.iplt as otherwise we get undefined references to these when
> >>> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
> >>>
> >>> Cc: Blue Swirl <blauwirbel@gmail.com>
> >>> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
> >>>
> >>> ---
> >>> sparc.ld |   16 ++++++++++++++--
> >>>  1 files changed, 14 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/sparc.ld b/sparc.ld
> >>> index 56efe34..cec17c9 100644
> >>> --- a/sparc.ld
> >>> +++ b/sparc.ld
> >>> @@ -37,8 +37,20 @@ SECTIONS
> >>>    .rela.fini     : { *(.rela.fini)      }
> >>>    .rel.bss       : { *(.rel.bss)                }
> >>>    .rela.bss      : { *(.rela.bss)               }
> >>> -  .rel.plt       : { *(.rel.plt)                }
> >>> -  .rela.plt      : { *(.rela.plt)               }
> >>> +  .rel.plt      :
> >>> +  {
> >>> +    *(.rel.plt)
> >>> +    PROVIDE (__rel_iplt_start = .);
> >>> +    *(.rel.iplt)
> >>> +    PROVIDE (__rel_iplt_end = .);
> >>> +  }
> >>> +  .rela.plt       :
> >>> +  {
> >>> +    *(.rela.plt)
> >>> +    PROVIDE (__rela_iplt_start = .);
> >>> +    *(.rela.iplt)
> >>> +    PROVIDE (__rela_iplt_end = .);
> >>> +  }
> >>>    .init          : { *(.init)   } =0x47ff041f
> >>>    .text      :
> >>>    {
> >>>
> >>
> >>
> >
> 
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

Re: [Qemu-devel] sparc-linux-user: Fix missing symbols in .rel/.rela.plt sections

[Michael Tokarev]

19.05.2013 16:58, Aurelien Jarno wrote:
> On Thu, May 09, 2013 at 05:56:03PM +0000, Blue Swirl wrote:
>> On Wed, May 8, 2013 at 7:18 AM, Michael Tokarev <mjt@tls.msk.ru> wrote:
>>> Ping #2?
>>>
>>> 06.04.2013 10:08, Michael Tokarev wrote:
>>>> This patch was submitted more than a year ago (at Jan-2012).
>>>> Is it still needed?  If yes, why it hasn't been applied?
>>
>> Well, is it still needed?
> 
> Without it, trying to build static qemu-user on sparc leads to the
> following error:
...
> | /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
> | (.text+0x148): undefined reference to `__rela_iplt_end'
> | /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
> | (.text+0x14c): undefined reference to `__rela_iplt_start'
> | /usr/lib/gcc/sparc-linux-gnu/4.6/../../../sparc-linux-gnu/libc.a(elf-init.o): In function `__libc_csu_irel':
> | (.text+0x150): undefined reference to `__rela_iplt_end'
> | collect2: ld returned 1 exit status
> | make[1]: *** [qemu-mips] Error 1
> | make: *** [subdir-mips-linux-user] Error 2
> 
> So I think it is still needed, unless you have another fix in mind.

If we apply this for 1.5, it will be the first release of debian
package without extra fixes.  Hopefully anyway :)

(Ofcourse we may add more fixes later but... ;)

/mjt