mbox series

[v2,0/2,next] UAPI: net/ethtool: Avoid thousands of -Wflex-array-member-not-at-end warnings

Message ID cover.1730238285.git.gustavoars@kernel.org (mailing list archive)
Headers show
Series UAPI: net/ethtool: Avoid thousands of -Wflex-array-member-not-at-end warnings | expand

Message

Gustavo A. R. Silva Oct. 29, 2024, 9:55 p.m. UTC
Small patch series aimed at fixing thousands of -Wflex-array-member-not-at-end
warnings by creating a new tagged struct within a flexible structure. We then
use this new struct type to fix problematic middle-flex-array declarations in
multiple composite structs, as well as to update the type of some variables in
various functions.

Changes in v2:
 - Update changelog text in patch 2/2 to better reflect the changes
   made. (Jakub)
 - Adjust variable declarations to follow the reverse xmas tree
   convention. (Jakub)

v1:
 - Link: https://lore.kernel.org/linux-hardening/cover.1729536776.git.gustavoars@kernel.org

Gustavo A. R. Silva (2):
  UAPI: ethtool: Use __struct_group() in struct ethtool_link_settings
  net: ethtool: Avoid thousands of -Wflex-array-member-not-at-end
    warnings

 .../net/ethernet/broadcom/bnxt/bnxt_ethtool.c |  6 ++--
 .../ethernet/chelsio/cxgb4/cxgb4_ethtool.c    |  4 +--
 .../ethernet/chelsio/cxgb4vf/cxgb4vf_main.c   |  2 +-
 .../net/ethernet/cisco/enic/enic_ethtool.c    |  2 +-
 .../net/ethernet/qlogic/qede/qede_ethtool.c   |  4 +--
 include/linux/ethtool.h                       |  2 +-
 include/uapi/linux/ethtool.h                  | 33 ++++++++++---------
 net/ethtool/ioctl.c                           |  2 +-
 net/ethtool/linkinfo.c                        |  8 ++---
 net/ethtool/linkmodes.c                       | 18 ++++++----
 10 files changed, 44 insertions(+), 37 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Nov. 3, 2024, 7:20 p.m. UTC | #1
Hello:

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

On Tue, 29 Oct 2024 15:55:02 -0600 you wrote:
> Small patch series aimed at fixing thousands of -Wflex-array-member-not-at-end
> warnings by creating a new tagged struct within a flexible structure. We then
> use this new struct type to fix problematic middle-flex-array declarations in
> multiple composite structs, as well as to update the type of some variables in
> various functions.
> 
> Changes in v2:
>  - Update changelog text in patch 2/2 to better reflect the changes
>    made. (Jakub)
>  - Adjust variable declarations to follow the reverse xmas tree
>    convention. (Jakub)
> 
> [...]

Here is the summary with links:
  - [v2,1/2,next] UAPI: ethtool: Use __struct_group() in struct ethtool_link_settings
    https://git.kernel.org/netdev/net-next/c/43d3487035e9
  - [v2,2/2,next] net: ethtool: Avoid thousands of -Wflex-array-member-not-at-end warnings
    https://git.kernel.org/netdev/net-next/c/3bd9b9abdf15

You are awesome, thank you!