lkml.org 
[lkml]   [2018]   [May]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 00/14] Modify action API for implementing lockless actions
    On Mon, May 14, 2018 at 7:27 AM, Vlad Buslov <vladbu@mellanox.com> wrote:
    > Currently, all netlink protocol handlers for updating rules, actions and
    > qdiscs are protected with single global rtnl lock which removes any
    > possibility for parallelism. This patch set is a first step to remove
    > rtnl lock dependency from TC rules update path. It updates act API to
    > use atomic operations, rcu and spinlocks for fine-grained locking. It
    > also extend API with functions that are needed to update existing
    > actions for parallel execution.

    Can you give a summary here for what and how it is achieved?

    You said this is the first step, what do you want to achieve in this
    very first step? And how do you achieve it? Do you break the RTNL
    lock down to, for a quick example, a per-device lock? Or perhaps you
    completely remove it because of what reason?

    I go through all the descriptions of your 14 patches (but not any code),
    I still have no clue how you successfully avoid RTNL. Please don't
    let me read into your code to understand that, there must be some
    high-level justification on how it works. Without it, I don't event want
    to read into the code.

    Thanks.

    \
     
     \ /
      Last update: 2018-05-25 01:36    [W:3.145 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site