On Thu, Mar 3, 2016 at 11:51 AM, Amir Vadai <a...@vadai.me> wrote: > On Thu, Mar 03, 2016 at 09:45:28AM -0800, Cong Wang wrote: >> >> So why? > The struct will not be used, and without exposing it, the compiler will > complain on code like I have in patch 9/10 ("net/mlx5e: Support offload > cls_flower with drop action"): > > static int parse_tc_actions(struct mlx5e_priv *priv, struct tcf_exts *exts, > u32 *action, u32 *flow_tag)
Why not make this a nop when CONFIG_NET_CLS_ACT is not set?