Re: [PATCH 1/4] ns: prepare time namespace for clone3()

From: Cyrill Gorcunov
Date: Wed Mar 18 2020 - 06:57:54 EST


On Tue, Mar 17, 2020 at 09:30:41AM +0100, Adrian Reber wrote:
...
> +/*
> + * This structure is used to set the time namespace offset
> + * via /proc as well as via clone3().
> + */
> +struct set_timens_offset {
> + int clockid;
> + struct timespec64 val;
> +};
> +

I'm sorry, I didn't follow this series much so can't comment right now.
Still this structure made me a bit wonder -- the @val seems to be 8 byte
aligned and I guess we have a useless pad between these members. If so
can we swap them? Or it is already part of api?