lkml.org 
[lkml]   [2019]   [Jun]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v6 01/19] kernel: Standardize vdso_datapage
From
Date
On 31/05/2019 09:16, Arnd Bergmann wrote:
> On Thu, May 30, 2019 at 4:15 PM Vincenzo Frascino
> <vincenzo.frascino@arm.com> wrote:
>
>> + * vdso_data will be accessed by 64 bit and compat code at the same time
>> + * so we should be careful before modifying this structure.
>> + */
>> +struct vdso_data {
>> + u32 seq;
>> +
>> + s32 clock_mode;
>> + u64 cycle_last;
>> + u64 mask;
>> + u32 mult;
>> + u32 shift;
>> +
>> + struct vdso_timestamp basetime[VDSO_BASES];
>> +
>> + s32 tz_minuteswest;
>> + s32 tz_dsttime;
>> + u32 hrtimer_res;
>> +};
>
> The structure contains four padding bytes at the end, which is
> something we try to avoid, at least if this ends up being used as
> an ABI. Maybe add "u32 __unused" at the end?
>

Agreed, I will fix this in v7.

> Arnd
>

--
Regards,
Vincenzo

\
 
 \ /
  Last update: 2019-06-04 14:06    [W:0.067 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site