Re: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Henrik Bjoernlund <henrik.bjoernlund@xxxxxxxxxxxxx>
Subject: Re: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
From: Jakub Kicinski <kuba@xxxxxxxxxx>
Date: Thu, 15 Oct 2020 10:34:31 -0700
Cc: netdev@xxxxxxxxxxxxxxx, bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, UNGLinuxDriver@xxxxxxxxxxxxx, idosch@xxxxxxxxxxxx, jiri@xxxxxxxxxxxx, nikolay@xxxxxxxxxx, roopa@xxxxxxxxxx, Horatiu Vultur <horatiu.vultur@xxxxxxxxxxxxx>, davem@xxxxxxxxxxxxx
In-reply-to: <20201015115418.2711454-8-henrik.bjoernlund@microchip.com>
References: <20201015115418.2711454-1-henrik.bjoernlund@microchip.com> <20201015115418.2711454-8-henrik.bjoernlund@microchip.com>

On Thu, 15 Oct 2020 11:54:15 +0000 Henrik Bjoernlund wrote:
> +	[IFLA_BRIDGE_CFM_MEP_CONFIG_MDLEVEL]	 = {
> +	.type = NLA_U32, .validation_type = NLA_VALIDATE_MAX, .max = 7 },

	NLA_POLICY_MAX(NLA_U32, 7)

Also why did you keep the validation in the code in patch 4?

Follow-Ups:
- Re: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
  - From: Henrik Bjoernlund via Bridge

References:
- [PATCH net-next v5 00/10] net: bridge: cfm: Add support for Connectivity Fault Management(CFM)
  - From: Henrik Bjoernlund via Bridge
- [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
  - From: Henrik Bjoernlund via Bridge

Prev by Date: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
Next by Date: Re: [RFC PATCH] net: bridge: call br_multicast_del_port before the port leaves
Previous by thread: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
Next by thread: Re: [PATCH net-next v6 07/10] bridge: cfm: Netlink SET configuration Interface.
Index(es):
- Date
- Thread

[Index of Archives] [Netdev] [AoE Tools] [Linux Wireless] [Kernel Newbies] [Security] [Linux for Hams] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux RAID] [Linux Admin] [Samba] [Video 4 Linux]