From: [email protected]
Date: Wed, 8 Jul 2015 16:16:39 -0700
> @@ -653,6 +654,9 @@ struct sk_buff {
> #ifdef CONFIG_NETWORK_SECMARK
> __u32 secmark;
> #endif
> +#ifdef CONFIG_NET_SWITCHDEV
> + __u32 offload_fwd_mark;
> +#endif
May I humbly suggest:
union {
#ifdef CONFIG_NETWORK_SECMARK
__u32 secmark;
#endif
#ifdef CONFIG_NET_SWITCHDEV
__u32 offload_fwd_mark;
#endif
};
After some studying, I'm largely convinced that these two values are
used in non-conflicting contexts.
Let me know if you can find an exception to this.
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
