From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Subject: Re: [PATCH net-next v3 0/2] socket statistics for ss Date: Wed, 9 May 2018 11:53:50 -0700 Message-ID: References: <20180507184333.32688-1-sthemmin@microsoft.com> <20180509082209.38fa4f43@xeon-e3> <20180509103144.195e7494@xeon-e3> <57fc8849-d1a4-4748-8780-bca3b0c7ca47@gmail.com> <20180509114456.0303a026@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: davem@davemloft.net, gerrit@erg.abdn.ac.uk, kuznet@ms2.inr.ac.ru, yoshfuji@linux-ipv6.org, netdev@vger.kernel.org, dccp@vger.kernel.org, Stephen Hemminger To: Stephen Hemminger , Eric Dumazet Return-path: Received: from mail-pg0-f65.google.com ([74.125.83.65]:45286 "EHLO mail-pg0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935523AbeEISxw (ORCPT ); Wed, 9 May 2018 14:53:52 -0400 In-Reply-To: <20180509114456.0303a026@xeon-e3> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 05/09/2018 11:44 AM, Stephen Hemminger wrote: > On Wed, 9 May 2018 10:53:58 -0700 > Eric Dumazet wrote: > >> On 05/09/2018 10:31 AM, Stephen Hemminger wrote: >>> On Wed, 9 May 2018 10:18:23 -0700 >>> Eric Dumazet wrote: >>> >>>> On 05/09/2018 08:22 AM, Stephen Hemminger wrote: >>>> >>>>> I am not sure if these patches are worth applying. >>>>> The 'ss -s' command has had missing values since 2.4 kernel. >>>>> And the first complaints came in only this year. >>>>> >>>>> Another alternative would be just to remove these fields from ss -s >>>>> output and move on. >>>>> >>>> >>>> Anyway your patches are not netns ready, so lets remove these fields from ss. >>>> >>>> Or you have to spend _much_ more time on writing and testing the kernel part. >>>> >>>> Thanks. >>> >>> The patches only expose the existing TCP socket accounting infrastructure. >>> Several other pieces that sockstat has are not netns aware. >>> That is a completely different problem. >> >> >> Adding a new field counting 'bounds ports' without being netns ready is a total mistake, >> as it is useless by current standards. >> >> The first thing that users will do is add proper netns support, with extra complexity in the kernel. >> >> So, instead of pushing some incomplete feature, trying to fool ourselves with a sentiment of 'small cost' >> that will later need another 100 lines of code in the kernel, please give us the complete picture. >> >> I am just saying, you can of course ignore my feedback. > > The current TCP hashinfo should be moved into netns. The current method of scanning and matching > by net namespace is a scalability issue now. It is not the plan yet, and we have no scalability issue. Before switching to netns hash table, this would need rhashtable conversions but so far this has not been done. - Time to create/delete netns is critical. - Adding few Mbytes of overhead per netns is a nogo, Please do not change subject, this is adding noise to this particular thread. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Dumazet Date: Wed, 09 May 2018 18:53:50 +0000 Subject: Re: [PATCH net-next v3 0/2] socket statistics for ss Message-Id: List-Id: References: <20180507184333.32688-1-sthemmin@microsoft.com> In-Reply-To: <20180507184333.32688-1-sthemmin@microsoft.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: dccp@vger.kernel.org On 05/09/2018 11:44 AM, Stephen Hemminger wrote: > On Wed, 9 May 2018 10:53:58 -0700 > Eric Dumazet wrote: > >> On 05/09/2018 10:31 AM, Stephen Hemminger wrote: >>> On Wed, 9 May 2018 10:18:23 -0700 >>> Eric Dumazet wrote: >>> >>>> On 05/09/2018 08:22 AM, Stephen Hemminger wrote: >>>> >>>>> I am not sure if these patches are worth applying. >>>>> The 'ss -s' command has had missing values since 2.4 kernel. >>>>> And the first complaints came in only this year. >>>>> >>>>> Another alternative would be just to remove these fields from ss -s >>>>> output and move on. >>>>> >>>> >>>> Anyway your patches are not netns ready, so lets remove these fields from ss. >>>> >>>> Or you have to spend _much_ more time on writing and testing the kernel part. >>>> >>>> Thanks. >>> >>> The patches only expose the existing TCP socket accounting infrastructure. >>> Several other pieces that sockstat has are not netns aware. >>> That is a completely different problem. >> >> >> Adding a new field counting 'bounds ports' without being netns ready is a total mistake, >> as it is useless by current standards. >> >> The first thing that users will do is add proper netns support, with extra complexity in the kernel. >> >> So, instead of pushing some incomplete feature, trying to fool ourselves with a sentiment of 'small cost' >> that will later need another 100 lines of code in the kernel, please give us the complete picture. >> >> I am just saying, you can of course ignore my feedback. > > The current TCP hashinfo should be moved into netns. The current method of scanning and matching > by net namespace is a scalability issue now. It is not the plan yet, and we have no scalability issue. Before switching to netns hash table, this would need rhashtable conversions but so far this has not been done. - Time to create/delete netns is critical. - Adding few Mbytes of overhead per netns is a nogo, Please do not change subject, this is adding noise to this particular thread.