Re: [Intel PMC TGPIO Driver 1/5] drivers/ptp: Add Enhanced handling of reserve fields

From: Jacob Keller
Date: Fri Jan 31 2020 - 12:02:22 EST


On 12/11/2019 1:48 PM, christopher.s.hall@xxxxxxxxx wrote:
> From: Christopher Hall <christopher.s.hall@xxxxxxxxx>
>
> Add functions that parameterize checking and zeroing of reserve fields in
> ioctl arguments. Eliminates need to change this code when repurposing
> reserve fields.
>

One thing to note that may be problematic with this handling: a later
usage of a reserved field must still be reported as zero for callers of
the old ioctl variants. Reserved fields were only handled properly in
the "2" variants of the ioctls.

This change might make it harder to notice and remember to ensure that
previously unused fields get zero'd for the old ioctls.

Richard, thoughts?

Thanks,
Jake