On Tue, May 10, 2022 at 06:56:02PM +0530, Abhishek Sahu wrote: > > We can add a directive to enforce an alignment regardless of the field > > size. I believe the feature ioctl header is already going to be eight > > byte aligned, so it's probably not strictly necessary, but Jason seems > > to be adding more of these directives elsewhere, so probably a good > > idea regardless. Thanks, > So, should I change it like > > __u8 low_power_state __attribute__((aligned(8))); > > Or > > __aligned_u64 low_power_state You should be explicit about padding, add a reserved to cover the gap. Jasno
