From: sfel...@gmail.com
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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to