From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [patch net-next 09/20] net: sched: convert actions array into rcu list Date: Tue, 1 Aug 2017 06:59:24 +0200 Message-ID: <20170801045924.GB1939@nanopsycho> References: <20170728144042.6380-1-jiri@resnulli.us> <20170728144042.6380-10-jiri@resnulli.us> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Linux Kernel Network Developers , David Miller , Jamal Hadi Salim , Daniel Borkmann , mlxsw@mellanox.com To: Cong Wang Return-path: Received: from mail-wm0-f66.google.com ([74.125.82.66]:33498 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751368AbdHAE70 (ORCPT ); Tue, 1 Aug 2017 00:59:26 -0400 Received: by mail-wm0-f66.google.com with SMTP id q189so1284914wmd.0 for ; Mon, 31 Jul 2017 21:59:26 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Mon, Jul 31, 2017 at 11:07:13PM CEST, xiyou.wangcong@gmail.com wrote: >On Fri, Jul 28, 2017 at 7:40 AM, Jiri Pirko wrote: >> From: Jiri Pirko >> >> Currently the actions are stored in array with array size. To traverse >> this array in fastpath, tcf_tree_lock is taken to protect it. Convert >> the array into a singly linked list, similar to the filter chains style >> and allow traversal protected by rcu. > >Did you read commit 22dc13c837c33207548c8ee5116 ? > >An action can't be shared by multiple filters if you put them >in a list (no matter singly or double), this is why I use pointers. Allright. Will check it out.