From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from shards.monkeyblade.net ([23.128.96.9]:35054 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388422AbeGXSVe (ORCPT ); Tue, 24 Jul 2018 14:21:34 -0400 Date: Tue, 24 Jul 2018 10:14:05 -0700 (PDT) Message-Id: <20180724.101405.797730329231867648.davem@davemloft.net> Subject: Re: [PATCH RFC/RFT net-next 00/17] net: Convert neighbor tables to per-namespace From: David Miller In-Reply-To: <5021d874-8e99-6eba-f24b-4257c62d4457@gmail.com> References: <28c30574-391c-b4bd-c337-51d3040d901a@gmail.com> <5021d874-8e99-6eba-f24b-4257c62d4457@gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: linux-wpan-owner@vger.kernel.org List-ID: To: dsahern@gmail.com Cc: xiyou.wangcong@gmail.com, netdev@vger.kernel.org, nikita.leshchenko@oracle.com, roopa@cumulusnetworks.com, stephen@networkplumber.org, idosch@mellanox.com, jiri@mellanox.com, saeedm@mellanox.com, alex.aring@gmail.com, linux-wpan@vger.kernel.org, netfilter-devel@vger.kernel.org, linux-kernel@vger.kernel.org From: David Ahern Date: Tue, 24 Jul 2018 09:14:01 -0600 > I get the impression there is no longer a strong resistance against > moving the tables to per namespace, but deciding what is the right > approach to handle backwards compatibility. Correct? Changing the > accounting is inevitably going to be noticeable to some use case(s), but > with sysctl settings it is a simple runtime update once the user knows > to make the change. > > neighbor entries round up to 512 byte allocations, so with the current > gc_thresh defaults (128/512/1024) 512k can be consumed. Using those > limits per namespace seems high which is why I suggested a per-namespace > default of (16/32/64) which amounts to 32k per namespace limit by > default. Open to other suggestions as well. No objection from me about going to per-ns neigh tables. About the defaults, I wonder if we can scale them to the amount of memory given to the ns or something like that? I bet this will better match the intended use of the ns.