This is the mail archive of the libc-alpha@sources.redhat.com 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: make existing mips files multi-ABI


Alexandre Oliva <aoliva at redhat dot com> writes:

> On Mar 14, 2003, Andreas Jaeger <aj at suse dot de> wrote:
>
>>> +	SETUP_GPX($0)
>>> +	SETUP_GPX64($25,$0)
>
>> Is this change correct?  I didn't find the definition of SETUP_GP/GPX
>> directly.
>
> Yup, they're added to sys/asm.h in one of the pending patches.
>
>> I don't like the XXX here...
>>> +    long int pad[4];	/* XXX */
>
>> and here.  What's the problem?
>
> I don't remember.  I wrote this a very long time ago.  I'll have to
> look more closely into this.  
>
>>> -    unsigned long int st_rdev;	/* Device number, if device.  */
>>> -    long int st_pad2[3];
>>> +    unsigned int st_rdev;	/* Device number, if device.  */
>
>> And also here?  That would be following Posix.
>
> This may very well be unnecessary.  I may have tweaked this to look
> like the kernel data structures before knowing better.

Please use __dev_t if possible.


> How about letting the rest in, while I investigate what (if anything)
> needs changing in these headers?

Ok - but let's wait for Uli's comments first.
Andreas
-- 
 Andreas Jaeger
  SuSE Labs aj at suse dot de
   private aj at arthur dot inka dot de
    http://www.suse.de/~aj


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