[PATCH v4 4/8] kernel_timespec.h: Add a kernel_timespec_t struct

[Alistair Francis]

On Mon, Mar 9, 2020 at 6:49 PM Dmitry V. Levin <ldv at altlinux.org> wrote:
>
> On Mon, Mar 09, 2020 at 05:43:00PM -0700, Alistair Francis wrote:
> > Add a kernel_timespec_t struct based on the definition in the Linux
> > kernel locatied at include/uapi/linux/time.h.
> >
> > Signed-off-by: Alistair Francis <alistair.francis at wdc.com>
> > ---
> >  kernel_timespec.h | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/kernel_timespec.h b/kernel_timespec.h
> > index 90133f8d..0d541333 100644
> > --- a/kernel_timespec.h
> > +++ b/kernel_timespec.h
> > @@ -22,4 +22,9 @@ typedef struct {
> >
> >  # endif /* HAVE_ARCH_TIME32_SYSCALLS */
> >
> > +typedef struct {
> > +     kernel_long_t tv_sec;
> > +     long tv_nsec;
> > +} kernel_timespec_t;
> > +
> >  #endif /* !STRACE_KERNEL_TIMESPEC_H */
>
> Sorry but this doesn't match struct __kernel_timespec.

This is supposed to match __kernel_old_timespec, which I just noticed
they added recently.

>
> Anyway, I don't see any need to use this structure in strace itself.
> Maybe we could employ it in tests where we currently use struct
> __kernel_timespec.

We can, but then how do we print the values out?

Alistair

>
>
> --
> ldv
> --
> Strace-devel mailing list
> Strace-devel at lists.strace.io
> https://lists.strace.io/mailman/listinfo/strace-devel