Hum. Instead of adding all these _get_foo ops to test dev->features bits, would anyone mind if I created a single _get_features() ethtool sub-ioctl?
Given the current ethtool_ops code, its implementation would in fact be completely transparent to low-level drivers. It need only be implemented in net/core/ethtool.c, without even adding a new hook.
Jeff