This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: Compress debug sections for Linux/x86 by default


> On Jan 8, 2015, at 1:35 PM, Joseph Myers <joseph@codesourcery.com> wrote:
> 
> On Thu, 8 Jan 2015, H.J. Lu wrote:
> 
>> On Thu, Jan 8, 2015 at 8:18 AM, Thomas Schwinge <thomas@codesourcery.com> wrote:
>>> Hi!
>>> 
>>> On Mon, 15 Dec 2014 11:04:01 -0800, "H.J. Lu" <hjl.tools@gmail.com> wrote:
>>>> On Mon, Dec 15, 2014 at 10:47 AM, Joseph Myers <joseph@codesourcery.com> wrote:
>>>>> On Sun, 14 Dec 2014, H.J. Lu wrote:
>>>>> 
>>>>>> I checked in this patch to compress debug sections for Linux/x86 by
>>>>>> default.
>>>>> 
>>>>> I don't think it makes sense for something like this to be
>>>>> architecture-dependent.  If desired it should be done for GNU/Linux on all
>>>>> architectures (and probably for GNU/Hurd, GNU/kFreeBSD etc. - all systems
>>>>> with GNU userspace), if not for all ELF targets.
>>>> 
>>>> Sure, someone can contribute a patch.
>>> 
>>> I don't like this attitude very much.  We should strive to be a bit more
>>> cooperative within our own community, at least.  As Joseph says, there is
>>> no reason to restrict this change -- which, as Joel has pointed out, has
>>> not been discussed/reviewed/approved before -- to x86 GNU/Linux only.
>> 
>> It is next to impossible to get agreements for all targets.
>> It is better to start from somewhere.  Each target maintainer
>> should make his/her own decision.
> 
> My point is that agreement for all targets is not what we want anyway.  
> We should make a decision for the GNU system (or for ELF targets in 
> general) and then apply that to all targets that are part of the GNU 
> system (i.e. all with GNU userspace) (or for ELF targets in general).  If 
> an OS maintainer thinks it's bad for their OS, don't enable it there, but 
> enabling / disabling on a per-architecture (as opposed to per-OS) basis 
> doesn't make sense without a compelling justification.  And we don't have 
> many OS maintainers.

I agree.

At the very most, if you are really concerned about lack of agreement, having the feature in Linux generally may be a valid first step.  But only Linux and then just on ONE target platform doesnât make any sense.  There is nothing here that is target specific as far as I can tell, so a limitation to x86 is something I do not understand at all.

	paul

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]