[PATCH 2/2] ethtool: add optional netlink support

[David Bauer]

Hi Robert,

On 5/16/21 6:01 PM, Robert Marko wrote:
> Netlink support is required for stuff like cable testing,
> so offer it as an option.
> 
> Signed-off-by: Robert Marko <robimarko at gmail.com>
> ---
>  package/network/utils/ethtool/Makefile | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/package/network/utils/ethtool/Makefile b/package/network/utils/ethtool/Makefile
> index d645cf9bbb..8cccf838a0 100644
> --- a/package/network/utils/ethtool/Makefile
> +++ b/package/network/utils/ethtool/Makefile
> @@ -23,7 +23,7 @@ PKG_FIXUP:=autoreconf
>  PKG_INSTALL:=1
>  PKG_BUILD_PARALLEL:=1
>  
> -PKG_CONFIG_DEPENDS:=CONFIG_ETHTOOL_PRETTY_DUMP
> +PKG_CONFIG_DEPENDS:=CONFIG_ETHTOOL_PRETTY_DUMP CONFIG_ETHTOOL_NETLINK
>  
>  include $(INCLUDE_DIR)/package.mk
>  
> @@ -32,6 +32,7 @@ define Package/ethtool
>    CATEGORY:=Network
>    TITLE:=Display or change ethernet card settings
>    URL:=http://www.kernel.org/pub/software/network/ethtool/
> +  DEPENDS:=+ETHTOOL_NETLINK:libmnl
>  endef
>  
>  define Package/ethtool/description
> @@ -43,9 +44,17 @@ define Package/ethtool/config
>  	config ETHTOOL_PRETTY_DUMP
>  		depends on PACKAGE_ethtool
>  		bool "Enable pretty printing"
> +
> +	config ETHTOOL_NETLINK
> +		depends on PACKAGE_ethtool
> +		bool "Enable netlink interface"

I did submit a similar patch a while ago which i didn't yet came across to apply to master.

IMHO, adding netlink support as an additional build variant is superior, as it allows users
to obtain the virtual cable tester via OPKG, which they can not with this approach.

I'm aware this complicates the configuration in menuconfig. FWIW, I'd like to redo the config
options and provide a ethtool-full package in the future, dropping the pretty print build option.

Apart from that, ETHTOOL_NETLINK needs to be enabled for kernel 5.10 in order to actually work.

What do you think?

Best
David

>  endef
>  
> +ifeq ($(CONFIG_ETHTOOL_NETLINK),y)
> +CONFIGURE_ARGS += --enable-netlink
> +else
>  CONFIGURE_ARGS += --disable-netlink
> +endif
>  
>  ifeq ($(CONFIG_ETHTOOL_PRETTY_DUMP),y)
>  CONFIGURE_ARGS += --enable-pretty-dump
>