This is the mail archive of the mailing list for the glibc 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 2/4] Mark __internal_statvfs[64] with attribute_hidden [BZ #18822]

On 18/08/2017 10:19, Florian Weimer wrote:
> On 08/18/2017 03:15 PM, H.J. Lu wrote:
>> On Fri, Aug 18, 2017 at 5:55 AM, Florian Weimer <> wrote:
>>> On 08/17/2017 10:58 PM, H.J. Lu wrote:
>>>> Mark __internal_statvfs[64] with attribute_hidden to allow direct access
>>>> to them within and libc.a without using GOT nor PLT.
>>>> Tested on i686 and x86-64.  OK for master?
>>> This is a bit odd because the include/ headers are supposed to be
>>> generic, and the declarations you add are Linux-specific.  In the past,
>>> we would have added a Linux-specific header under
>>> sysdeps/unix/sysv/linux for that.
>>> I don't have a strong opinion either way.
>> Header files under include override both generic and non-generic header
>> files.  __KERNEL_STRICT_NAMES is another example.
> Right.  I'm sure your patch works as intended (at least on most
> architectures).  It's just a matter of policy to what extent we want to
> uphold the fiction that upstream glibc supports more than just Linux.

I also think it should be on a Linux only header, mostly for organization.

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