Commit 0a3e6939 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Jakub Kicinski

net/smc: Use static_assert() to check struct sizes

Commit 9748dbc9 ("net/smc: Avoid -Wflex-array-member-not-at-end
warnings") introduced tagged `struct smc_clc_v2_extension_fixed` and
`struct smc_clc_smcd_v2_extension_fixed`. We want to ensure that when
new members need to be added to the flexible structures, they are
always included within these tagged structs.

So, we use `static_assert()` to ensure that the memory layout for
both the flexible structure and the tagged struct is the same after
any changes.
Signed-off-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: default avatarJan Karcher <jaka@linux.ibm.com>
Link: https://patch.msgid.link/ZrVBuiqFHAORpFxE@cuteSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 46dd90fe
...@@ -145,6 +145,8 @@ struct smc_clc_v2_extension { ...@@ -145,6 +145,8 @@ struct smc_clc_v2_extension {
); );
u8 user_eids[][SMC_MAX_EID_LEN]; u8 user_eids[][SMC_MAX_EID_LEN];
}; };
static_assert(offsetof(struct smc_clc_v2_extension, user_eids) == sizeof(struct smc_clc_v2_extension_fixed),
"struct member likely outside of struct_group_tagged()");
struct smc_clc_msg_proposal_prefix { /* prefix part of clc proposal message*/ struct smc_clc_msg_proposal_prefix { /* prefix part of clc proposal message*/
__be32 outgoing_subnet; /* subnet mask */ __be32 outgoing_subnet; /* subnet mask */
...@@ -169,6 +171,8 @@ struct smc_clc_smcd_v2_extension { ...@@ -169,6 +171,8 @@ struct smc_clc_smcd_v2_extension {
); );
struct smc_clc_smcd_gid_chid gidchid[]; struct smc_clc_smcd_gid_chid gidchid[];
}; };
static_assert(offsetof(struct smc_clc_smcd_v2_extension, gidchid) == sizeof(struct smc_clc_smcd_v2_extension_fixed),
"struct member likely outside of struct_group_tagged()");
struct smc_clc_msg_proposal { /* clc proposal message sent by Linux */ struct smc_clc_msg_proposal { /* clc proposal message sent by Linux */
struct smc_clc_msg_hdr hdr; struct smc_clc_msg_hdr hdr;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment