struct ofpact has enums that are packed in case of __GNUC__. This packing does not occur for visual studio. For 'struct ofpact_nest', we are currently expecting that "struct ofpact actions[]" has an offset of 8 bytes. This condition won't be true in compilers where enums are not packed.
It is good enough if struct ofpact actions[] starts at an offset which is a multiple of OFPACT_ALIGNTO. Signed-off-by: Gurucharan Shetty <gshe...@nicira.com> --- lib/ofp-actions.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ofp-actions.h b/lib/ofp-actions.h index 0f6bf70..5fd3185 100644 --- a/lib/ofp-actions.h +++ b/lib/ofp-actions.h @@ -439,7 +439,7 @@ struct ofpact_nest { uint8_t pad[OFPACT_ALIGN(sizeof(struct ofpact)) - sizeof(struct ofpact)]; struct ofpact actions[]; }; -BUILD_ASSERT_DECL(offsetof(struct ofpact_nest, actions) == OFPACT_ALIGNTO); +BUILD_ASSERT_DECL(offsetof(struct ofpact_nest, actions) % OFPACT_ALIGNTO == 0); static inline size_t ofpact_nest_get_action_len(const struct ofpact_nest *on) -- 1.7.9.5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev