DORMANTno replies REVIEWED: 3 (2M)

Revision v2 of 4 in this series; 2 review trailers.

Revisions (4)
  1. v1 [diff vs current]
  2. v2 current
  3. v3 [diff vs current]
  4. v4 [diff vs current]

[PATCH net-next v2 0/2] Completed requested changes

From: Manish Kurup <hidden>
Date: 2017-10-27 20:47:23

Hi everyone,

Apologies for the resend (newbie mistakes), but I made the following
corrections to the cover letter and patches:
* Added version and comment info to subjects of patches and all the changes
  (below). I used v2, since my earlier mails had no versions.
* Added ACKs from Jamal and Jiri to all patches and this cover letter

I've made all requested changes to the act_vlan TC action, tested them, and
here's the new changeset. Thanks a lot!

Here're the changes:
v2: Fixed all helper functions to use RCU (rtnl_dereference) - Eric, Jamal
v2: Fixed indentation, extra line nits - Jamal, Jiri
v2: Moved rcu_head to the end of the struct - Jiri
v2: Re-formatted locals to reverse-christmas-tree - Jiri
v2: Removed mismatched spin_lock() - Cong
v2: Removed spin_lock_bh() in tcf_vlan_init, rtnl_dereference() should
    suffice - Cong, Jiri

Thanks for all the comments!
 
Acked-by: Jamal Hadi Salim <jhs@mojatatu.com>
Acked-by: Jiri Pirko <redacted>
Signed-off-by: Manish Kurup <redacted>

Manish Kurup (2):
  net sched act_vlan: Change stats update to use per-core stats
  net sched act_vlan: VLAN action rewrite to use RCU lock/unlock and
    update

 include/net/tc_act/tc_vlan.h | 46 ++++++++++++++++++++++------
 net/sched/act_vlan.c         | 71 ++++++++++++++++++++++++++++----------------
 2 files changed, 82 insertions(+), 35 deletions(-)

-- 
2.7.4
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help