> Yes, it passed all tests with -m32, -mx32, -m64 on Linux/x86-64.
> Installed on trunk. Is this OK to backport to 4.8 branch after
> a few days?

Actually there are two issues with your change:

Using time_t for tv_nsec looks actually wrong, the definition on (my)
linux is:

struct timespec {
        __kernel_time_t tv_sec;                 /* seconds */
        long            tv_nsec;                /* nanoseconds */
};

As you can see, the types for tv_sec and tv_nsec are different.

So the change to timespec looks wrong to me.

Also changing tv_nsec type breaks s-osinte-solaris-posix.ads which would
need a similar change, although not if we change the above.

Can you please follow up on that?

In particular, what the definition of timespec on x32 linux?

Shouldn't tv_nsec be back to a long as it was before? Looks to me this
way.

Arno

Reply via email to