This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCHv2] PowerPC - Add a faster way to read the Time Baseregister
On Fri, Feb 24, 2012 at 11:29:43AM -0500, Carlos O'Donell wrote:
> 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.
Sure!
>
> 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.
--
Tulio Magno