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 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


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