2024-04-24 12:19:26

by Asbjørn Sloth Tønnesen

[permalink] [raw]
Subject: [PATCH net-next v2 0/4] net: sparx5: flower: validate control flags

This series adds flower control flags validation to the
sparx5 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 3 (requested by Jiri)
* Convert a missed extack usage (noticed by Daniel)
* Added cover letter (requested by Simon and Daniel)

v1: https://lore.kernel.org/netdev/[email protected]/

Asbjørn Sloth Tønnesen (4):
net: sparx5: flower: only do lookup if fragment flags are set
net: sparx5: flower: add extack to
sparx5_tc_flower_handler_control_usage()
net: sparx5: flower: remove goto in
sparx5_tc_flower_handler_control_usage()
net: sparx5: flower: check for unsupported control flags

.../microchip/sparx5/sparx5_tc_flower.c | 20 +++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)

--
2.43.0



2024-04-24 13:41:56

by Daniel Machon

[permalink] [raw]
Subject: Re: [PATCH net-next v2 0/4] net: sparx5: flower: validate control flags

> This series adds flower control flags validation to the
> sparx5 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 3 (requested by Jiri)
> * Convert a missed extack usage (noticed by Daniel)
> * Added cover letter (requested by Simon and Daniel)
>
> v1: https://lore.kernel.org/netdev/[email protected]/
>
> Asbj?rn Sloth T?nnesen (4):
> net: sparx5: flower: only do lookup if fragment flags are set
> net: sparx5: flower: add extack to
> sparx5_tc_flower_handler_control_usage()
> net: sparx5: flower: remove goto in
> sparx5_tc_flower_handler_control_usage()
> net: sparx5: flower: check for unsupported control flags
>
> .../microchip/sparx5/sparx5_tc_flower.c | 20 +++++++++++--------
> 1 file changed, 12 insertions(+), 8 deletions(-)
>
> --
> 2.43.0
>

For the series:

Reviewed-by: Daniel Machon <[email protected]>
Tested-by: Daniel Machon <[email protected]>



2024-04-26 02:40:51

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH net-next v2 0/4] net: sparx5: flower: validate control flags

Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <[email protected]>:

On Wed, 24 Apr 2024 12:16:21 +0000 you wrote:
> This series adds flower control flags validation to the
> sparx5 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.
>
> ---
>
> [...]

Here is the summary with links:
- [net-next,v2,1/4] net: sparx5: flower: only do lookup if fragment flags are set
https://git.kernel.org/netdev/net-next/c/bcf303c62c98
- [net-next,v2,2/4] net: sparx5: flower: add extack to sparx5_tc_flower_handler_control_usage()
https://git.kernel.org/netdev/net-next/c/8cd1b6c0bf31
- [net-next,v2,3/4] net: sparx5: flower: remove goto in sparx5_tc_flower_handler_control_usage()
https://git.kernel.org/netdev/net-next/c/b92eb1ac13f0
- [net-next,v2,4/4] net: sparx5: flower: check for unsupported control flags
https://git.kernel.org/netdev/net-next/c/8ef631e9c995

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html