On 17-05-17 05:07 AM, Jiri Pirko wrote:
From: Jiri Pirko <j...@mellanox.com>

Introduce struct tcf_chain object and set of helpers around it. Wraps up
insertion, deletion and search in the filter chain.

Signed-off-by: Jiri Pirko <j...@mellanox.com>
---

[..]
+
+static void
+tcf_chain_filter_chain_ptr_set(struct tcf_chain *chain,
+                              struct tcf_proto __rcu **p_filter_chain)
+

What are the rules for this? Common coding style is:
static void tcf_chain_filter_chain_ptr_set(struct tcf_chain *chain,
                                           struct tcf_proto ..

I am sure that struct tcf_proto __rcu **p_filter_chain would
probably hit boundary of 80 chars - but it would look cleaner
to be consistent.

Otherwise:
Acked-by: Jamal Hadi Salim <j...@mojatatu.com>

cheers,
jamal

Reply via email to