[PATCHv2] PowerPC - Add a faster way to read the Time Base register

Carlos O'Donell carlos@systemhalted.org
Fri Feb 24 16:29:00 GMT 2012


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.



More information about the Libc-alpha mailing list