Skip to content

Commit

Permalink
net: sched: protect chain->explicitly_created with block->lock
Browse files Browse the repository at this point in the history
In order to remove dependency on rtnl lock, protect
tcf_chain->explicitly_created flag with block->lock. Consolidate code that
checks and resets 'explicitly_created' flag into __tcf_chain_put() to
execute it atomically with rest of code that puts chain reference.

Signed-off-by: Vlad Buslov <vladbu@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Vlad Buslov authored and David S. Miller committed Feb 12, 2019
1 parent c266f64 commit 91052fa
Showing 1 changed file with 13 additions and 6 deletions.
19 changes: 13 additions & 6 deletions net/sched/cls_api.c
Original file line number Diff line number Diff line change
Expand Up @@ -370,13 +370,22 @@ EXPORT_SYMBOL(tcf_chain_get_by_act);

static void tc_chain_tmplt_del(struct tcf_chain *chain);

static void __tcf_chain_put(struct tcf_chain *chain, bool by_act)
static void __tcf_chain_put(struct tcf_chain *chain, bool by_act,
bool explicitly_created)
{
struct tcf_block *block = chain->block;
bool is_last, free_block = false;
unsigned int refcnt;

mutex_lock(&block->lock);
if (explicitly_created) {
if (!chain->explicitly_created) {
mutex_unlock(&block->lock);
return;
}
chain->explicitly_created = false;
}

if (by_act)
chain->action_refcnt--;

Expand All @@ -402,19 +411,18 @@ static void __tcf_chain_put(struct tcf_chain *chain, bool by_act)

static void tcf_chain_put(struct tcf_chain *chain)
{
__tcf_chain_put(chain, false);
__tcf_chain_put(chain, false, false);
}

void tcf_chain_put_by_act(struct tcf_chain *chain)
{
__tcf_chain_put(chain, true);
__tcf_chain_put(chain, true, false);
}
EXPORT_SYMBOL(tcf_chain_put_by_act);

static void tcf_chain_put_explicitly_created(struct tcf_chain *chain)
{
if (chain->explicitly_created)
tcf_chain_put(chain);
__tcf_chain_put(chain, false, true);
}

static void tcf_chain_flush(struct tcf_chain *chain)
Expand Down Expand Up @@ -2305,7 +2313,6 @@ static int tc_ctl_chain(struct sk_buff *skb, struct nlmsghdr *n,
* to the chain previously taken during addition.
*/
tcf_chain_put_explicitly_created(chain);
chain->explicitly_created = false;
break;
case RTM_GETCHAIN:
err = tc_chain_notify(chain, skb, n->nlmsg_seq,
Expand Down

0 comments on commit 91052fa

Please sign in to comment.