From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net-next 0/1] Introduction of the tc tests Date: Mon, 19 Jun 2017 22:40:27 -0700 Message-ID: <20170619224027.133c3578@plumbers-lap.home.lan> References: <1497648155-25521-1-git-send-email-lucasb@mojatatu.com> <20170619.234819.333574326143678552.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: xiyou.wangcong@gmail.com, lucasb@mojatatu.com, netdev@vger.kernel.org, jiri@resnulli.us, mrv@mojatatu.com, jhs@mojatatu.com To: David Miller Return-path: Received: from mail-pf0-f179.google.com ([209.85.192.179]:35324 "EHLO mail-pf0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750992AbdFTFkk (ORCPT ); Tue, 20 Jun 2017 01:40:40 -0400 Received: by mail-pf0-f179.google.com with SMTP id c73so4353750pfk.2 for ; Mon, 19 Jun 2017 22:40:39 -0700 (PDT) In-Reply-To: <20170619.234819.333574326143678552.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 19 Jun 2017 23:48:19 -0400 (EDT) David Miller wrote: > From: Cong Wang > Date: Mon, 19 Jun 2017 16:37:29 -0700 > > > Hi, > > > > On Fri, Jun 16, 2017 at 2:22 PM, Lucas Bates wrote: > >> Apologies for sending this as one big patch. I've been sitting on this a little > >> too long, but it's ready and I wanted to get it out. > >> > >> There are a limited number of tests to start - I plan to add more on a regular > >> basis. > >> > >> Lucas Bates (1): > >> selftests: Introduce tc testsuite > > > > Nice work! > > > > Is there any particular reason you want to put these tests in kernel tree > > especially tools/testing/selftests/ ? > > Yeah, it would be absolutely terrible if we had more tests in the > kernel selftests area for networking. > > More seriously, we need more, not less, tests in the kernel networking > selftests directory. > > It doesn't belong in iproute2 because we want a place to put things > that automatically get tested when someone makes kernel changes and > can be integrated into the kernel development workflow. > > I want as many tests as possible under there, so I'm really surprised > that you're asking "why" tests are being added there. The "Occum's razor" for deciding where tests belong should be does the test need to change to respond to kernel change? Don't want to have iproute2 tests that have if (kernel_version > ...)