This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2] Add x86 32 bit vDSO time function support
- From: Nathan Lynch <Nathan_Lynch at codesourcery dot com>
- To: Adhemerval Zanella <azanella at linux dot vnet dot ibm dot com>
- Cc: <libc-alpha at sourceware dot org>
- Date: Wed, 5 Nov 2014 10:22:27 -0600
- Subject: Re: [PATCH v2] Add x86 32 bit vDSO time function support
- Authentication-results: sourceware.org; auth=none
- References: <5436D48C dot 2090509 at linux dot vnet dot ibm dot com> <5436F50C dot 9070002 at codesourcery dot com> <5457E5D5 dot 6000103 at linux dot vnet dot ibm dot com> <54581516 dot 9010908 at codesourcery dot com> <545A1D38 dot 8030700 at linux dot vnet dot ibm dot com>
On 11/05/2014 06:51 AM, Adhemerval Zanella wrote:
> On 03-11-2014 21:51, Nathan Lynch wrote:
>>> i386 does not define HAVE_CLOCK_GETTIME_VSYSCALL and thus:
>>>
>>> sysdeps/unix/sysv/linux/clock_gettime.c:
>>>
>>> 26: # define INTERNAL_VSYSCALL INTERNAL_SYSCALL
>>>
>>> and then if INTERNAL_GETTIME is not defined, it will as:
>>>
>>> 37 #ifndef INTERNAL_GETTIME
>>> 38 # define INTERNAL_GETTIME(id, tp) \
>>> 39 INTERNAL_VSYSCALL (clock_gettime, err, 2, id, tp)
>>> 40 #endif
>>>
>>> And without proper set the PTR_DEMANGLE is not called either.
>> I think my point might be better expressed as: why is INTERNAL_GETTIME
>> defined in linux/clock_gettime.c at all? It is not used in the
>> clock_gettime implementation.
>>
>> The only use in glibc of a macro by that name is in
>> linux/timespec_get.c, and the arch can override it in its own
>> timespec_get.c (as x86_64 currently does).
>>
> I see your point and I removed its definition now for x86. Below it's the revised
> patchset.
I have no further concerns, thanks.