> struct ethtool_wolinfo { > __u32 cmd; > __u32 supported; > __u32 wolopts; > __u8 sopass[SOPASS_MAX]; // 6, actually > }; > > we could do > > struct ethtool_wolinfo { > __u32 cmd; > __u32 supported; > __u32 wolopts; > __u8 sopass[SOPASS_MAX]; // 6, actually > __u8 reserved[2]; > }; > > and then the compiler has to properly treat it, since it's no longer > unnamed padding. >
Although, on some architectures, that could actually break the ABI by changing the size, oh well. johannes