mbox series

[net-next,v2,0/3] net: lan966x: flower: validate control flags

Message ID 20240424125347.461995-1-ast@fiberby.net (mailing list archive)
Headers show
Series net: lan966x: flower: validate control flags | expand

Message

Asbjørn Sloth Tønnesen April 24, 2024, 12:53 p.m. UTC
This series adds flower control flags validation to the
lan966x driver, and changes it from assuming that it handles
all control flags, to instead reject rules if they have
masked any unknown/unsupported control flags.

---
Changelog:

v2:
* Split first patch into 2 (requested by Jiri)
* Added cover letter (requested by Simon)

v1: https://lore.kernel.org/netdev/20240423102720.228728-1-ast@fiberby.net/

Asbjørn Sloth Tønnesen (3):
  net: lan966x: flower: add extack to
    lan966x_tc_flower_handler_control_usage()
  net: lan966x: flower: rename goto in
    lan966x_tc_flower_handler_control_usage()
  net: lan966x: flower: check for unsupported control flags

 .../ethernet/microchip/lan966x/lan966x_tc_flower.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

Comments

Jiri Pirko April 24, 2024, 2:49 p.m. UTC | #1
Wed, Apr 24, 2024 at 02:53:37PM CEST, ast@fiberby.net wrote:
>This series adds flower control flags validation to the
>lan966x driver, and changes it from assuming that it handles
>all control flags, to instead reject rules if they have
>masked any unknown/unsupported control flags.
>
>---
>Changelog:
>
>v2:
>* Split first patch into 2 (requested by Jiri)
>* Added cover letter (requested by Simon)

set-
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Horatiu Vultur April 24, 2024, 6:54 p.m. UTC | #2
The 04/24/2024 12:53, Asbjørn Sloth Tønnesen wrote:
> 
> This series adds flower control flags validation to the
> lan966x driver, and changes it from assuming that it handles
> all control flags, to instead reject rules if they have
> masked any unknown/unsupported control flags.

For the entire series:
Reviewed-by: Horatiu Vultur <horatiu.vultur@microchip.com>
> 
> ---
> Changelog:
> 
> v2:
> * Split first patch into 2 (requested by Jiri)
> * Added cover letter (requested by Simon)
> 
> v1: https://lore.kernel.org/netdev/20240423102720.228728-1-ast@fiberby.net/
> 
> Asbjørn Sloth Tønnesen (3):
>   net: lan966x: flower: add extack to
>     lan966x_tc_flower_handler_control_usage()
>   net: lan966x: flower: rename goto in
>     lan966x_tc_flower_handler_control_usage()
>   net: lan966x: flower: check for unsupported control flags
> 
>  .../ethernet/microchip/lan966x/lan966x_tc_flower.c | 14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> --
> 2.43.0
>
patchwork-bot+netdevbpf@kernel.org April 26, 2024, 2:40 a.m. UTC | #3
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 24 Apr 2024 12:53:37 +0000 you wrote:
> This series adds flower control flags validation to the
> lan966x driver, and changes it from assuming that it handles
> all control flags, to instead reject rules if they have
> masked any unknown/unsupported control flags.
> 
> ---
> Changelog:
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/3] net: lan966x: flower: add extack to lan966x_tc_flower_handler_control_usage()
    https://git.kernel.org/netdev/net-next/c/505ccf890c21
  - [net-next,v2,2/3] net: lan966x: flower: rename goto in lan966x_tc_flower_handler_control_usage()
    https://git.kernel.org/netdev/net-next/c/12b8e129c409
  - [net-next,v2,3/3] net: lan966x: flower: check for unsupported control flags
    https://git.kernel.org/netdev/net-next/c/8c65e27b42fc

You are awesome, thank you!