netdev - Re: [PATCH net-next] drivers/net: add 6WIND SHULTI support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 27 Apr 2016 12:34:01 -0400 (EDT)
From:	David Miller <davem@...emloft.net>
To:	nicolas.dichtel@...nd.com
Cc:	netdev@...r.kernel.org, jiri@...nulli.us
Subject: Re: [PATCH net-next] drivers/net: add 6WIND SHULTI support

From: Nicolas Dichtel <nicolas.dichtel@...nd.com>
Date: Wed, 27 Apr 2016 11:37:18 +0200

> +/* rtnl linkinfo attributes */
> +enum {
> +	IFLA_SHULTI_UNSPEC,
> +	IFLA_SHULTI_SWDEV_ID,
> +	IFLA_SHULTI_SWDEV_PORTID,
> +	IFLA_SHULTI_PHYS_PORT_NAME,
> +
> +	__IFLA_SHULTI_MAX
> +#define IFLA_SHULTI_MAX	(__IFLA_SHULTI_MAX - 1)
> +};
> +
 ...
> +enum {
> +	SHULTI_A_UNSPEC,
> +	SHULTI_A_NL_PORTIDS,
> +	SHULTI_A_SWDEV_PORTID,
> +	SHULTI_A_RX_QUEUES,
> +	SHULTI_A_ERRCODE,
> +	SHULTI_A_IFINDEX,
> +	SHULTI_A_LINK_STATUS,
> +	SHULTI_A_PACKET,
> +	SHULTI_A_STATS,
> +	SHULTI_A_VLAN_VID,
> +	SHULTI_A_DRVINFO,
> +	SHULTI_A_SETTINGS,
> +	SHULTI_A_RX_MODES,
> +	SHULTI_A_UC_ADDR,
> +	SHULTI_A_MC_ADDR,
> +	SHULTI_A_STP_STATE,
> +	SHULTI_A_BR_STATE_LEARNING,
> +	SHULTI_A_BR_AGEING_TIME,
> +	SHULTI_A_BR_FDB,
> +
> +	__SHULTI_A_MAX
> +#define SHULTI_A_MAX	(__SHULTI_A_MAX - 1)
> +};
> +

If our existing netlink facilities for switchdevs are not sufficient to properly
report and configure the state of your device, extend it rather than making custom
netlink stuff.