This is the mail archive of the libc-alpha@sourceware.org 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: [PATCHv2] PowerPC - Add a faster way to read the Time Base register


On Thu, Feb 23, 2012 at 12:07 PM, Steve Munroe <sjmunroe@us.ibm.com> wrote:
>> I would rather we create a whole new header, platform.h, and ship that
>> as the location for platform macros that follow no standard.
>
> Ok, I can live with that, assume that the GLIBC community is will to host
> and install it in /usr/include.

Yes, exactly.

> Where you thinking ./sys/platform.h? Also need to work on the make magic to
> select the correct platform specific version of platform.h for install.

Yes, I was thinking exactly that.

The first cut of the patch could provide a default empty platform.h,
and use the normal sysdeps mechanism to override it in power32/power64.

Does that sounds like a way forward?

Tulio, Could you please:
* File a bugzilla bug for this issue, component=libc
* Change your patch to put the macro in sys/platform.h
* Adjust Makefile's to install sys/platform.h
* Post new patch for review.

Does anyone else have any comments here?

I see Dave Miller already commented that he agrees this is a better idea
than using user.h.

Cheers,
Carlos.


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