ethtool: Document the general convention for VLAs in kernel space
authorBen Hutchings <ben@decadent.org.uk>
Wed, 12 Feb 2014 22:14:53 +0000 (22:14 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 12 Feb 2014 22:33:03 +0000 (17:33 -0500)
Various ethtool command structures are declared with zero-length array
at the end which are intended to be variable-length in userland
(relying on lack of compiler bounds checking).  However, in the kernel
the structure and array are always allocated and passed to driver
operations separately.  Make that explicit.

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/ethtool.h

index c8e3e7e39c6bf26ffbfe534cdfa83f59ad561372..0a114d05f68d35bd275924a7290a6c3b75ef17fd 100644 (file)
@@ -183,6 +183,9 @@ static inline u32 ethtool_rxfh_indir_default(u32 index, u32 n_rx_rings)
  * hold the RTNL lock.
  *
  * See the structures used by these operations for further documentation.
+ * Note that for all operations using a structure ending with a zero-
+ * length array, the array is allocated separately in the kernel and
+ * is passed to the driver as an additional parameter.
  *
  * See &struct net_device and &struct net_device_ops for documentation
  * of the generic netdev features interface.