On Mon, Feb 25, 2019 at 7:45 AM Vlad Buslov <vla...@mellanox.com> wrote:
>
> Function tc_dump_chain() obtains and releases block->lock on each iteration
> of its inner loop that dumps all chains on block. Outputting chain template
> info is fast operation so locking/unlocking mutex multiple times is an
> overhead when lock is highly contested. Modify tc_dump_chain() to only
> obtain block->lock once and dump all chains without releasing it.
>
> Signed-off-by: Vlad Buslov <vla...@mellanox.com>
> Suggested-by: Cong Wang <xiyou.wangc...@gmail.com>

Thanks for the followup!

Isn't it similar for __tcf_get_next_proto() in tcf_chain_dump()?
And for tc_dump_tfilter()?

Reply via email to